[PATCH] D143938: [VPlan] Compute costs for plans directly after construction (WIP).

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 13 12:33:08 PST 2023


tschuett added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h:388
+
+  bool isMoreProfitable(const VectorizationFactor &A,
+                        const VectorizationFactor &B) const;
----------------
Could you add a comment `/// Returns true when Factor A is more profitable than Factor B` ? I was unclear about the direction A v.s B.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143938



More information about the llvm-commits mailing list