[llvm] [Coverage] Rework Decision/Expansion/Branch (PR #78969)

Alan Phipps via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 29 14:34:58 PST 2024


================
@@ -13,18 +13,18 @@
 // CHECK-NEXT:  |
 // CHECK-NEXT:  |  Number of Conditions: 5
 // CHECK-NEXT:  |     Condition C1 --> (9:7)
-// CHECK-NEXT:  |     Condition C2 --> (2:11)
-// CHECK-NEXT:  |     Condition C3 --> (3:11)
-// CHECK-NEXT:  |     Condition C4 --> (3:23)
-// CHECK-NEXT:  |     Condition C5 --> (9:22)
+// CHECK-NEXT:  |     Condition C2 --> (9:22)
+// CHECK-NEXT:  |     Condition C3 --> (2:11)
+// CHECK-NEXT:  |     Condition C4 --> (3:11)
+// CHECK-NEXT:  |     Condition C5 --> (3:23)
----------------
evodius96 wrote:

> I could tweak the order to honor the location of `Expansion`. Your #78819 is better in this point. Is it better for me to rework?

That would be my preference. Honoring the location seems more intuitive to me when reading the output.


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


More information about the llvm-commits mailing list