[Mlir-commits] [mlir] [MLIR][Timing] Fix timing report of nested pass pipeline and negative Rest field when using -mlir-timing (Try to close #169443) (PR #169615)

[Mehdi Amini]

================
@@ -95,6 +97,24 @@
 // NESTED_PIPELINE-NEXT: Rest
 // NESTED_PIPELINE-NEXT: Total
 
+// DYNAMIC-PIPELINE: Execution time report
+// DYNAMIC-PIPELINE: Total Execution Time:
+// DYNAMIC-PIPELINE: Name
+// DYNAMIC-PIPELINE-NEXT: Parser
+// DYNAMIC-PIPELINE-NEXT: 'func.func' Pipeline
+// DYNAMIC-PIPELINE-NEXT:   CSE
+// DYNAMIC-PIPELINE-NEXT:     (A) DominanceInfo
+// DYNAMIC-PIPELINE-NEXT:   Canonicalizer
+// DYNAMIC-PIPELINE-NEXT:   CSE
+// DYNAMIC-PIPELINE-NEXT:     (A) DominanceInfo
+// DYNAMIC-PIPELINE-NEXT: Inliner
+// DYNAMIC-PIPELINE-NEXT:   (A) CallGraph
+// DYNAMIC-PIPELINE-NEXT:   'func.func' Pipeline
+// DYNAMIC-PIPELINE-NEXT:     Canonicalizer
+// DYNAMIC-PIPELINE-NEXT: Output
+// DYNAMIC-PIPELINE-NEXT: Rest
+// DYNAMIC-PIPELINE-NEXT: Total
----------------
joker-eph wrote:

This test is passing at HEAD apparently, so may not be testing what you think it is?

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