[llvm] CoverageMappingWriter: Emit `Decision` before `Expansion` (PR #78966)

Jessica Paquette via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 25 20:00:46 PST 2024


================
@@ -167,7 +167,18 @@ void CoverageMappingWriter::write(raw_ostream &OS) {
       return LHS.FileID < RHS.FileID;
     if (LHS.startLoc() != RHS.startLoc())
       return LHS.startLoc() < RHS.startLoc();
-    return LHS.Kind < RHS.Kind;
+
+    // Put `Decision` before `Expansion`.
+    auto getKindKey = [](CounterMappingRegion::RegionKind Kind) {
----------------
ornata wrote:

Would it make sense to make this into a real function, so that its return value can be tested in mock?

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


More information about the llvm-commits mailing list