forked from OSchip/llvm-project
[sanitizers] [PowerPC] Intercept __tls_get_addr_opt.
On PowerPC, if binutils and glibc are new enough, the linker uses an optimized code sequence to implement __tls_get_addr call stub, which will end up calling __tls_get_addr_opt instead of __tls_get_addr. Thus, we need to intercept it in addition to __tls_get_addr. This symbol is actually an alias of __tls_get_addr - its only purpose is that its presence in glibc triggers the optimization in linker. This means we can make our own intercepting symbol an alias as well. This patch will make the linker attempt optimization even on older glibc's (since it sees a defined __tls_get_addr_opt symbol in msan) - however, this is only a very minor performance problem (the linker generated code will never recognize a filled static TLS descriptor, always burning a few cycles), not a correctness problem. This fixes MSan's dtls_test.c, allowing us to finally enable MSan on PowerPC64. llvm-svn: 273250
This commit is contained in:
parent
879b0978f4
commit
613c252517
|
@ -4435,6 +4435,14 @@ INTERCEPTOR(void *, __tls_get_addr, void *arg) {
|
|||
}
|
||||
return res;
|
||||
}
|
||||
#if SANITIZER_PPC
|
||||
// On PowerPC, we also need to intercept __tls_get_addr_opt, which has
|
||||
// mostly the same semantics as __tls_get_addr, but its presence enables
|
||||
// some optimizations in linker (which are safe to ignore here).
|
||||
extern "C" __attribute__((alias("__interceptor___tls_get_addr"),
|
||||
visibility("default")))
|
||||
void *__tls_get_addr_opt(void *arg);
|
||||
#endif
|
||||
#else // SANITIZER_S390
|
||||
// On s390, we have to intercept two functions here:
|
||||
// - __tls_get_addr_internal, which is a glibc-internal function that is like
|
||||
|
|
Loading…
Reference in New Issue