Skip to content

Conversation

@artagnon
Copy link
Contributor

@artagnon artagnon commented Nov 7, 2025

-- 8< --
Various tests need to be updated. Pending verification of IR diff.

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

⚠️ C/C++ code formatter, clang-format found issues in your code. ⚠️

You can test this locally with the following command:
git-clang-format --diff origin/main HEAD --extensions cpp -- llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp --diff_from_common_commit

⚠️
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing origin/main to the base branch/commit you want to compare against.
⚠️

View the diff from clang-format here.
diff --git a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
index b6e8e2a4d..e684fa2cf 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
@@ -1419,10 +1419,9 @@ static void narrowToSingleScalarRecipes(VPlan &Plan) {
                                             true /*IsSingleScalar*/,
                                             nullptr /*Mask*/, Metadata);
         Clone->insertBefore(&R);
-        unsigned ExtractOpc =
-            vputils::isUniformAcrossVFsAndUFs(OperandToCheck)
-                ? VPInstruction::ExtractLastElement
-                : VPInstruction::ExtractLastLanePerPart;
+        unsigned ExtractOpc = vputils::isUniformAcrossVFsAndUFs(OperandToCheck)
+                                  ? VPInstruction::ExtractLastElement
+                                  : VPInstruction::ExtractLastLanePerPart;
         auto *Ext = new VPInstruction(ExtractOpc, {Clone->getOperand(0)});
         Ext->insertBefore(Clone);
         Clone->setOperand(0, Ext);

@artagnon
Copy link
Contributor Author

artagnon commented Nov 7, 2025

Huge thinko!

@artagnon artagnon closed this Nov 7, 2025
@artagnon artagnon deleted the vplan-narrow-wide-store branch November 7, 2025 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant