[clang] [llvm] [MC/DC][Coverage] Loosen the limit of NumConds from 6 (PR #82448)

NAKAMURA Takumi via llvm-commits llvm-commits at lists.llvm.org
Thu May 16 05:11:53 PDT 2024


================
@@ -983,7 +979,7 @@ void CodeGenPGO::mapRegionCounters(const Decl *D) {
   // for most embedded applications. Setting a maximum value prevents the
   // bitmap footprint from growing too large without the user's knowledge. In
   // the future, this value could be adjusted with a command-line option.
-  unsigned MCDCMaxConditions = (CGM.getCodeGenOpts().MCDCCoverage) ? 6 : 0;
+  unsigned MCDCMaxConditions = (CGM.getCodeGenOpts().MCDCCoverage) ? 32767 : 0;
----------------
chapuni wrote:

@evodius96 I've introduced `-fmcdc-max-conditions=32767`.
> against unintended memory footprint expansion
I think `-fmcdc-max-test-vectors=64` can restrict max bitmap size as the current implementation.

@ornata I have no idea how to determine practical limit for that. So I use `SHRT_MAX`.

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


More information about the llvm-commits mailing list