Revert "Add weak declarations for ASan global variables"

Blocking roll into Chromium because it breaks Chromium build on Mac
ASan. (Note this doesn't happen until after landing - see bug.)

This reverts commit 2843cbcc714fe111e1083127c048a18002bc10ed.

Bug: 553628939
Change-Id: Ia1fae3ed301088a48aeeecd3f30daafc60808cee
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/77808
Tested-by: Antonio Maiorano <amaiorano@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Commit-Queue: Antonio Maiorano <amaiorano@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>

src/Reactor/LLVMJIT.cpp

diff --git a/src/Reactor/LLVMJIT.cpp b/src/Reactor/LLVMJIT.cpp
index 0162722..6db6ca1 100644
--- a/src/Reactor/LLVMJIT.cpp
+++ b/src/Reactor/LLVMJIT.cpp
@@ -92,18 +92,6 @@
 #	if __has_feature(memory_sanitizer) || __has_feature(address_sanitizer)
 #		include <dlfcn.h>  // dlsym()
 #	endif
-#	if __has_feature(address_sanitizer)
-// Google3 static linking configurations map mocks via linkopt wraps
-// (-Wl,--wrap), which strips unused internal ASan tracking symbols from the
-// dynamic symbol table. This prevents dynamic runtime dlsym(RTLD_DEFAULT)
-// lookups from finding them, leading to materialization failures during ORC JIT
-// shader compilation. Declaring them here as weak externs forces static
-// resolution from the host binary when compiled with ASan, while reverting to
-// nullptr safely in standard or non-ASan configurations.
-extern "C" __attribute__((weak)) char __start_asan_globals;
-extern "C" __attribute__((weak)) char __stop_asan_globals;
-extern "C" __attribute__((weak)) char ___asan_globals_registered;
-#	endif
 #else
 #	define ADDRESS_SANITIZER_INSTRUMENTATION_SUPPORTED false
 #endif
@@ -717,28 +705,6 @@
 				continue;
 			}
 
-#if __has_feature(address_sanitizer)
-			// Under static linking configurations, compiler-builtin ASan symbols
-			// are stripped from the dynamic symbol table, causing dlsym to fail.
-			// We statically resolve them using the weak references declared at
-			// the head of the file.
-			if (unmangled == "__start_asan_globals" &&
-			    &__start_asan_globals != nullptr) {
-				symbols[name] = toSymbol(&__start_asan_globals);
-				continue;
-			}
-			if (unmangled == "__stop_asan_globals" &&
-			    &__stop_asan_globals != nullptr) {
-				symbols[name] = toSymbol(&__stop_asan_globals);
-				continue;
-			}
-			if (unmangled == "___asan_globals_registered" &&
-			    &___asan_globals_registered != nullptr) {
-				symbols[name] = toSymbol(&___asan_globals_registered);
-				continue;
-			}
-#endif
-
 #if __has_feature(memory_sanitizer) || (__has_feature(address_sanitizer) && ADDRESS_SANITIZER_INSTRUMENTATION_SUPPORTED)
 			// Sanitizers use a dynamically linked runtime. Instrumented routines reference some
 			// symbols from this library. Look them up dynamically in the default namespace.