[PATCH] D69765: [compiler-rt] Support more CPUs in LSan Allocator Address Space

[Alexander Richardson via Phabricator via llvm-commits]

arichardson added inline comments.


================
Comment at: compiler-rt/lib/lsan/lsan_allocator.h:55
+    defined(__sh3__) || defined(__vax__) ||                           \
+    (defined(__mips64) && !defined(_LP64)) ||                         \
+    (defined(__sparc__) && !defined(_LP64)) ||                        \
----------------
Maybe change this to `&& !defined(__mips_n32)`?
For our CHERI CPU we use n64 but don't have _LP64 defined (since pointers are 128 bits).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69765/new/

https://reviews.llvm.org/D69765