[clang] [llvm] [clang][CoverageMapping] Refactor when setting MC/DC True/False (PR #78202)
Alan Phipps via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 16 12:27:16 PST 2024
evodius96 wrote:
> I've found this change fails with the expression; `((a && (b || c) || (d && e)) && f)`.
Thank you for pointing this out. I think you figured out the point I was struggling to get right -- when to actually "pop" the child Decision from the stack, and also to separate the "pop" operation from the calculation of the total conditions. I was constrained by the design. I like your cleanup, including the removal of the NestLevel stack, so I integrated most of your refactor.
https://github.com/llvm/llvm-project/pull/78202
More information about the cfe-commits
mailing list