[asan] Restore dead-code-elimination optimization for Fuchsia

[mcgrathr]

Summary

r310244 fixed a bug introduced by r309914 for non-Fuchsia builds.
In doing so it also reversed the intended effect of the change for
Fuchsia builds, which was to allow all the AllocateFromLocalPool
code and its variables to be optimized away entirely.

This change restores that optimization for Fuchsia builds, but
doesn't have the original change's bug because the comparison
arithmetic now takes into account the size of the elements.

[vitalybuka]

lib/asan/asan_malloc_linux.cc

35 ↗

(On Diff #110094)

Maybe just to guard #if !defined(SANITIZER_FUCHSIA)
from line 29 to 64, and create empty implementation in #else section

[mcgrathr]

lib/asan/asan_malloc_linux.cc

35 ↗

(On Diff #110094)

So, you mean exactly what I had originally in https://reviews.llvm.org/D36190?id=109395 before you asked me to change it to avoid #if tests? Really?

I'll change it back if you want. But you actually did convince me that avoiding #if here was better.
If you are concerned about the correctness of this change (which I don't think you should be), then I can add a simple "&& !SANITIZER_FUCHSIA" short-circuit instead of this change.

[vitalybuka]

LGTM

lib/asan/asan_malloc_linux.cc

35 ↗

(On Diff #110094)

I know, and actually now sure.
I don't like #ifs and I don't like multiple branches where they are not needed.

Correctness looks good to me, but all this hassle for optimization which can be omitted by compiler.

So I don't like all three approaches in the same degree :-)
Aleksey WDYT?

[alekseyshl]

lib/asan/asan_malloc_linux.cc

35 ↗

(On Diff #110094)

I guess it's fine, the extra multiplication should not be a problem (it should be compiled into shift anyway). If it is going to cause any issues, we will convert it into actual offset, not an index.

[mcgrathr]

Thanks! Please land it for me.

compiler-rt/trunk/lib/asan/asan_malloc_linux.cc

 using namespace __asan;  // NOLINT
 
 static uptr allocated_for_dlsym;
 static const uptr kDlsymAllocPoolSize = 1024;
 static uptr alloc_memory_for_dlsym[kDlsymAllocPoolSize];
 
 static INLINE bool IsInDlsymAllocPool(const void *ptr) {
   uptr off = (uptr)ptr - (uptr)alloc_memory_for_dlsym;
-  return off < sizeof(alloc_memory_for_dlsym);
+  return off < allocated_for_dlsym * sizeof(alloc_memory_for_dlsym[0]);
 }
 
 static void *AllocateFromLocalPool(uptr size_in_bytes) {
   uptr size_in_words = RoundUpTo(size_in_bytes, kWordSize) / kWordSize;
   void *mem = (void*)&alloc_memory_for_dlsym[allocated_for_dlsym];
   allocated_for_dlsym += size_in_words;
   CHECK_LT(allocated_for_dlsym, kDlsymAllocPoolSize);
   return mem;