[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