[llvm] [clang] [clang][CoverageMapping] Refactor when setting MC/DC True/False (PR #78202)

NAKAMURA Takumi via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 16 05:11:11 PST 2024


https://github.com/chapuni requested changes to this pull request.

I've found this change fails with the expression;
`((a && (b || c) || (d && e)) && f)`.

I guess there might be a confusion in `AndRHS` and `OrRHS`. I haven't understood your code completely, though.

I've pushed my refactoring in https://github.com/chapuni/llvm-project/commits/mcdc/77873covgen/
Feel free to follow and use it. Hope it helps.


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


More information about the cfe-commits mailing list