[llvm] [VPlan] Explicitly handle scalar pointer inductions. (PR #83068)

via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 7 07:30:40 PST 2024


================
@@ -509,7 +530,19 @@ void VPInstruction::execute(VPTransformState &State) {
   if (hasFastMathFlags())
     State.Builder.setFastMathFlags(getFastMathFlags());
   for (unsigned Part = 0; Part < State.UF; ++Part) {
-    Value *GeneratedValue = generateInstruction(State, Part);
+    if (generatesScalars()) {
----------------
ayalz wrote:

Would it be clearer to further distinguish between singular and plural, i.e., generates-a-single-scalar and generates-multiple-scalars? The former case should arguably call `State.set(this, P, Part, !IsVector);` where IsVector is set to false, rather than degenerate Lane to iterate over [0,1).

https://github.com/llvm/llvm-project/pull/83068


More information about the llvm-commits mailing list