[PATCH] D105629: [TSan] Add SystemZ support

[Ulrich Weigand via Phabricator via cfe-commits]

uweigand added a comment.

See inline comments ... otherwise the SystemZ platform-specific parts look good to me.



================
Comment at: compiler-rt/lib/tsan/rtl/tsan_platform_posix.cpp:123
+#if defined(__s390x__)
+  ProtectRange(HiAppMemEnd(), 0xfffffffffffff000ull);
+#endif
----------------
Did you test this on older kernels without 5-level page table support?   I believe the allocation / mprotect may fail on those ...


================
Comment at: compiler-rt/lib/tsan/rtl/tsan_rtl_s390x.S:22
+  CFI_REL_OFFSET(%r2, R2_REL_OFFSET)
+  CFI_REL_OFFSET(%r3, R3_REL_OFFSET)
+  stmg %r14, %r15, R14_REL_OFFSET(%r15)
----------------
Do we need CFI for r2/r3 ?  Those are call-clobbered any cannot be unwound normally anyway ...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105629