[llvm] [llvm][lit] Add tsan feature when enabled (PR #94573)
Alexander Richardson via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 7 17:20:10 PDT 2024
================
@@ -131,6 +131,8 @@ def __init__(self, lit_config, config):
features.add("msan")
if "undefined" in sanitizers:
features.add("ubsan")
+ if "thread" in sanitizers:
----------------
arichardson wrote:
How much are these features used? I think the naming is somewhat ambiguous: it could mean "LLVM built with sanitizer X" (which is the meaning of that flag) but I could also read it as "default to building with sanitizer x enabled" or even "<sanitizer> is supported on the current target". Maybe an explicit `built-with-<sanitizer>` or `llvm-instrumented-with-<sanitizer>` would be better?
https://github.com/llvm/llvm-project/pull/94573
More information about the llvm-commits
mailing list