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

Alan Phipps via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 27 14:54:20 PST 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;
----------------
evodius96 wrote:

I know you will do this later, but I also wanted to echo the desire for a means to control the condition limit to warn the user against unintended memory footprint expansion, most useful in embedded development contexts.  It may not otherwise be obvious to most users what is contributing to larger memory usage. We could also override the default in downstream toolchains.

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


More information about the cfe-commits mailing list