Commit c4bbd378 by Nicolas Capens Committed by Nicolas Capens

Remove typeinfo from GLES exported symbols

This typeinfo was added for UBSan failures reported in crbug.com/737384, but it didn't address other 'Incorrect-function-pointer-type' failures for crbug.com/746914. We've suppressed sanitization for these functions with the NO_SANITIZE_FUNCTION macro, refactored interfaces used across library boundaries into purely abstract ones for crbug.com/732667, and use '-mllvm -asan-use-private-alias=1' to address b/128551743. It's unclear whether this typeinfo still serves ay purpose We shouldn't have to export typeinfo for objects used only inside the shared library, to avoid ODR violations. Remove it entirely for now. If this causes a regression and we still have to export any typeinfo, we should do so more selectively. For example to only export typeinfo for objects in the egl namespace: _ZTS*egl*; _ZTI*egl*; Alternatively, or in addition, we can try using '-mllvm -asan-use-private-alias=1' for more build systems. Bug: b/155441530 Change-Id: Ia966c40dfe45817f356d11725910afef1bb94d6e Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/44688 Presubmit-Ready: Nicolas Capens <nicolascapens@google.com> Tested-by: 's avatarNicolas Capens <nicolascapens@google.com> Kokoro-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: 's avatarAlexis Hétu <sugoi@google.com> Reviewed-by: 's avatarAntonio Maiorano <amaiorano@google.com>
parent b798cfa3
...@@ -60,10 +60,6 @@ global: ...@@ -60,10 +60,6 @@ global:
# Table of function pointers to disambiguate between libraries # Table of function pointers to disambiguate between libraries
libEGL_swiftshader; libEGL_swiftshader;
# Type-strings and type-infos required by sanitizers
_ZTS*;
_ZTI*;
local: local:
*; *;
}; };
...@@ -298,10 +298,6 @@ global: ...@@ -298,10 +298,6 @@ global:
# Table of function pointers to disambiguate between libraries # Table of function pointers to disambiguate between libraries
libGLESv2_swiftshader; libGLESv2_swiftshader;
# Type-strings and type-infos required by sanitizers
_ZTS*;
_ZTI*;
local: local:
*; *;
}; };
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment