util: Always specify default visibility on exports.

This fixes undefined behaviour with CFI. Bug: chromium:1015810 Bug: angleproject:3162 Change-Id: I58cfb78adabbff05e5b4560dfd70b190411fa26d Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/1869303Reviewed-by: Jamie Madill <jmadill@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org>

util: Always specify default visibility on exports.
2bf23ea8 · Jamie Madill · Commit Bot · 0cb09633 · 2bf23ea8
Commit 2bf23ea8 authored Oct 18, 2019 by Jamie Madill Committed by Commit Bot Oct 19, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 5 deletions

util_export.h util/util_export.h +1 -5

No files found.
--- a/util/util_export.h
+++ b/util/util_export.h
@@ -16,11 +16,7 @@
 #            define ANGLE_UTIL_EXPORT __declspec(dllimport)
 #        endif
 #    elif defined(__GNUC__)
-#        if defined(LIBANGLE_UTIL_IMPLEMENTATION)
-#            define ANGLE_UTIL_EXPORT __attribute__((visibility("default")))
-#        else
-#            define ANGLE_UTIL_EXPORT
-#        endif
+#        define ANGLE_UTIL_EXPORT __attribute__((visibility("default")))
 #    else
 #        define ANGLE_UTIL_EXPORT
 #    endif