[PATCH] D84082: [tsan] Allow TSan in the Clang driver for Apple Silicon Macs

Julian Lettner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 20 09:52:20 PDT 2020


yln added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Darwin.cpp:2734
 
-  if (isTargetMacOS()) {
-    if (IsX86_64)
-      Res |= SanitizerKind::Thread;
+  if ((IsX86_64 || IsAArch64) && isTargetMacOS()) {
+    Res |= SanitizerKind::Thread;
----------------
This breaks the symmetry with the if below.  Is this intentional?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84082





More information about the llvm-commits mailing list