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

[Kamil Rytarowski via Phabricator via llvm-commits]

krytarowski marked an inline comment as done.
krytarowski 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)) ||                        \
----------------
arichardson wrote:
> 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).
There is also o32. I don't know CHERI specifics and it is not supported today on NetBSD. (And there is no LSan for FreeBSD either).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69765