[PATCH] D110479: [LV] Record memory widening decisions (NFCI)

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 25 12:46:51 PDT 2021


fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:8781
+      CM.getWideningDecision(I, Range.Start);
+  bool Reverse = (Decision == LoopVectorizationCostModel::CM_Widen_Reverse);
+  bool Consecutive =
----------------
nit: no `()` required.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:1517
 
+  bool Consecutive;
+  bool Reverse;
----------------
Should we document both fields?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110479



More information about the llvm-commits mailing list