[llvm] [Matrix] Use DenseMap for ShapeMap instead of ValueMap. (PR #118282)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 4 05:22:19 PST 2024


================
@@ -493,10 +480,8 @@ class LowerMatrixIntrinsics {
   /// the result value of the instruction, with the only exceptions being store
   /// instructions and the matrix_column_major_store intrinsics. For those, the
   /// shape information indicates that those instructions should be lowered
-  /// using shape information as well.  A ValueMap is used so that when
-  /// sub-passes like optimizeTransposes performs RAUW the map stays
-  /// up-to-date.
-  ValueMap<Value *, ShapeInfo> ShapeMap;
+  /// using shape information as well.
----------------
fhahn wrote:

Extended the comment, thanks!

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


More information about the llvm-commits mailing list