[llvm] don't set TracePCGuard as the default (PR #106464)

Andrew Kelley via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 28 15:51:42 PDT 2024


andrewrk wrote:

Alternately:

```diff
--- a/llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp
+++ b/llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp
@@ -197,7 +197,7 @@ SanitizerCoverageOptions OverrideFromCL(SanitizerCoverageOptions Options) {
   Options.TraceStores |= ClStoreTracing;
   if (!Options.TracePCGuard && !Options.TracePC &&
       !Options.Inline8bitCounters && !Options.StackDepth &&
-      !Options.InlineBoolFlag && !Options.TraceLoads && !Options.TraceStores)
+      !Options.InlineBoolFlag && !Options.TraceLoads && !Options.TraceStores && !Options.TraceCmp)
     Options.TracePCGuard = true; // TracePCGuard is default.
   Options.CollectControlFlow |= ClCollectCF;
   return Options;
```

https://github.com/llvm/llvm-project/pull/106464


More information about the llvm-commits mailing list