[PATCH] D43025: [tsan] Add support for linux/powerpc64 in buildgo.sh

[Carlos Eduardo Seo via Phabricator via llvm-commits]

cseo marked 6 inline comments as done.
cseo added inline comments.


================
Comment at: lib/tsan/rtl/tsan_platform.h:468
+  static const uptr kLoAppMemEnd   = 0x010000000000ull;
+  static const uptr kHiAppMemBeg   = 0x7e8000000000ull;
+  static const uptr kHiAppMemEnd   = 0x800000000000ull; // 47 bits
----------------
dvyukov wrote:
> What lives in kHiAppMemBeg-kHiAppMemEnd range?
ld.so and shared libraries.

Actually, let me ask you about this, since it seems I'm understanding wrongly how this should work. Is it necessary to map these memory areas for shared libraries? Or just executable/heap/shadow/meta are enough for the Go sanitizer?


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D43025