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

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


================
@@ -298,7 +298,7 @@ struct CounterMappingRegion {
                        unsigned ExpandedFileID, unsigned LineStart,
                        unsigned ColumnStart, unsigned LineEnd,
                        unsigned ColumnEnd, RegionKind Kind)
-      : MCDCParams(MCDCParams), ExpandedFileID(ExpandedFileID),
+      : MCDCParams(MCDCParams), FileID(FileID), ExpandedFileID(ExpandedFileID),
----------------
chapuni wrote:

I'm okay with piggybacking :)

I think this change will not require test cases and can be committed w/o review even if this will be split out. @ornata how do you think?

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


More information about the llvm-commits mailing list