[llvm] [clang] [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:21:47 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),
----------------
evodius96 wrote:

You're correct.  I was thinking it might be useful to support ExpandedFileIDs in the future, but if that's the case, we can add it.  At the moment, I don't see when it would be useful anyway.

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


More information about the cfe-commits mailing list