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

2020-07-24 Thread Kuba (Brecka) Mracek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG33d9c4109ac2: [tsan] Allow TSan in the Clang driver for Apple Silicon Macs (authored by kubamracek). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84082/new/

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

2020-07-21 Thread Kuba (Brecka) Mracek via Phabricator via cfe-commits
kubamracek added a comment. Let's handle simulators separately. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84082/new/ https://reviews.llvm.org/D84082 ___ cfe-commits mailing list cfe-commits@lists.l

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

2020-07-20 Thread Julian Lettner via Phabricator via cfe-commits
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 bre

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

2020-07-20 Thread Julian Lettner via Phabricator via cfe-commits
yln added a comment. Simulators on Apple Silicon are not yet supported? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84082/new/ https://reviews.llvm.org/D84082 ___ cfe-commits mailing list cfe-commits

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

2020-07-17 Thread Kuba (Brecka) Mracek via Phabricator via cfe-commits
kubamracek created this revision. kubamracek added reviewers: dcoughlin, delcypher, yln, arphaman, steven_wu. kubamracek added a project: Sanitizers. Herald added subscribers: cfe-commits, Charusso, dexonsmith. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://reviews.l