[Mlir-commits] [mlir] 008999e - [MLIR] Fix crash when using --mlir-timing (#109463)

[llvmlistbot at llvm.org]

Author: Ian Wood
Date: 2024-09-20T17:33:45-07:00
New Revision: 008999ee8c3259ad1d7280eb4350e59ce57eb585

URL: https://github.com/llvm/llvm-project/commit/008999ee8c3259ad1d7280eb4350e59ce57eb585
DIFF: https://github.com/llvm/llvm-project/commit/008999ee8c3259ad1d7280eb4350e59ce57eb585.diff

LOG: [MLIR] Fix crash when using --mlir-timing (#109463)

Added: 
    

Modified: 
    mlir/lib/Pass/PassTiming.cpp

Removed: 
    


################################################################################
diff  --git a/mlir/lib/Pass/PassTiming.cpp b/mlir/lib/Pass/PassTiming.cpp
index 9554c5dd9cb19c..45852455aa7370 100644
--- a/mlir/lib/Pass/PassTiming.cpp
+++ b/mlir/lib/Pass/PassTiming.cpp
@@ -35,9 +35,6 @@ struct PassTiming : public PassInstrumentation {
   /// parent thread into which the new thread should be nested.
   DenseMap<PipelineParentInfo, unsigned> parentTimerIndices;
 
-  /// A stack of the currently active timing scopes per thread.
-  DenseMap<uint64_t, SmallVector<TimingScope, 4>> activeThreadTimers;
-
   /// The timing manager owned by this instrumentation (in case timing was
   /// enabled by the user on the pass manager without providing an external
   /// timing manager). This *must* appear before the `ownedTimingScope` to
@@ -46,6 +43,9 @@ struct PassTiming : public PassInstrumentation {
   std::unique_ptr<TimingManager> ownedTimingManager;
   TimingScope ownedTimingScope;
 
+  /// A stack of the currently active timing scopes per thread.
+  DenseMap<uint64_t, SmallVector<TimingScope, 4>> activeThreadTimers;
+
   /// The root timing scope into which timing is reported.
   TimingScope &rootScope;