[PATCH] D81498: [Matrix] Preserve volatile when loading loads/stores.

[LuoYuanke via Phabricator via llvm-commits]

LuoYuanke added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:1220
           MatrixTy A =
-              loadMatrix(APtr, LShape, Builder.getInt64(I), Builder.getInt64(K),
-                         {TileR, TileM}, EltType, Builder);
+              loadMatrix(APtr, false, LShape, Builder.getInt64(I),
+                         Builder.getInt64(K), {TileR, TileM}, EltType, Builder);
----------------
Why we set volatile as false? Is it possible that the original load instruction is volatile load?
Do we allow fusion if the load and store instruction is volatile?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81498/new/

https://reviews.llvm.org/D81498