[PATCH] D133019: [VPlan] Only generate single instr for loads uniform across all parts.

[Philip Reames via Phabricator via llvm-commits]

reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM w/changes applied.



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:9659
                                       State);
+      if (!UI->getType()->isVoidTy())
+        for (unsigned Part = 1; Part < State.UF; ++Part)
----------------
A load can't be of void type.  


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:9663
+                    VPIteration(Part, 0));
+    } else {
+      // Uniform within VL means we need to generate lane 0 only for each
----------------
Please use early turn instead of else clause.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133019