[PATCH] D69067: [LV] Record GEP widening decisions in recipe (NFCI)

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 5 02:02:43 PST 2019


fhahn accepted this revision.
fhahn added a comment.

LGTM, thanks for working on this!



================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:767
+    unsigned I = 0;
+    for (auto &U : GEP->indices())
+      IsIndexLoopInvariant[I++] = OrigLoop->isLoopInvariant(U.get());
----------------
nit: you could also use enumerate(GEP->indices()) and get rid of the extra counter I, which might be a bit more explicit


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

https://reviews.llvm.org/D69067





More information about the llvm-commits mailing list