[Mlir-commits] [mlir] [mlir][ArmSME] Make tile allocation order deterministic (PR #229096)

[Federico Bruzzone]

================
@@ -370,13 +371,15 @@ generateOperationNumbering(FunctionOpInterface function) {
   return operationToIndexMap;
 }
 
-/// Gather live ranges for SME tiles from the MLIR liveness analysis.
-DenseMap<Value, LiveRange>
+/// Gather live ranges for SME tiles from the MLIR liveness analysis. Uses a
+/// `MapVector` (rather than a `DenseMap`) so that iteration order is
+/// deterministic (insertion order).
+llvm::MapVector<Value, LiveRange>
----------------
FedericoBruzzone wrote:

Yes, we need to keep `llvm::` because [`mlir/Support/LLVM.h`](https://github.com/llvm/llvm-project/blob/7bd5b596c3d0afff84caebe3be6f0e61190e937b/mlir/include/mlir/Support/LLVM.h) doesn't have an alias for `MapVector`. For `SetVector` we have `using SetVector = llvm::SetVector<...>`. Right?

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