[PATCH] D86377: [tsan] Use large address space mapping on Apple Silicon Macs

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 10 22:17:00 PST 2021


MaskRay added inline comments.


================
Comment at: compiler-rt/lib/tsan/rtl/tsan_platform_posix.cpp:102
 
-#if defined(__aarch64__) && defined(__APPLE__)
+#if defined(__aarch64__) && !HAS_48_BIT_ADDRESS_SPACE
   ProtectRange(HeapMemEnd(), ShadowBeg());
----------------
This change is not correct for aarch64-*-linux.

I've fixed it in 32b684acc4490fbddce39d501e577cb029728e41


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86377



More information about the llvm-commits mailing list