[PATCH] D133880: [msan] Change logic of ClInstrumentationWithCallThreshold

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added inline comments.


================
Comment at: llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp:1156
+    // Constants likely will be eliminated by folloup passes.
+    if (!isa<Constant>(V))
+      --SplitBlockBudget;
----------------
kstoimenov wrote:
> To avoid negative integer underflow. 
Total count is useful for debugging
maybe this way?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133880