[PATCH] D73600: [Matrix] Add remark propagation along the inlined-at chain.

[Florian Hahn via Phabricator via llvm-commits]

fhahn marked 5 inline comments as done.
fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:960
 
+    /// Set of values to consider as matrix operations.
+    const SmallSetVector<Value *, 32> &ExprSet;
----------------
anemet wrote:
> I am assuming these values that falls into a given DISubprogram?  More explanation would be good here and probably a bit more specific name than ExprSet.
Yes, I've renamed it to ExprsInSubprogram and updated the comment.


================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:1225
+    SmallVector<Value *, 4>
+    getExpressionLeaves(const SmallSetVector<Value *, 32> &Ops) {
       SmallVector<Value *, 4> Leaves;
----------------
anemet wrote:
> Explain Ops in the comment.
Also renamed Ops to ExprsInSubprogram.


================
Comment at: llvm/lib/Transforms/Scalar/LowerMatrixIntrinsics.cpp:1292
+      // to the containing function.
+      MapVector<DISubprogram *, SmallVector<Value *, 8>> Mapping;
+      for (auto &KV : Inst2ColumnMatrix) {
----------------
anemet wrote:
> Mapping is usually not a good name ;)
Done, renamed to Subprog2Exprs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73600