-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[AArch64][SVE] Fix for wide adds trying to be generated #118838
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Wide adds would try to be generated when SVE is enabled not SVE2.
@llvm/pr-subscribers-backend-aarch64 Author: James Chesterman (JamesChesterman) ChangesWide adds would try to be generated when SVE is enabled not SVE2. Full diff: https://github.com/llvm/llvm-project/pull/118838.diff 1 Files Affected:
diff --git a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
index 7ab3fc06715ec8..75f0bae84db67e 100644
--- a/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
+++ b/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp
@@ -21814,7 +21814,7 @@ SDValue tryLowerPartialReductionToWideAdd(SDNode *N,
Intrinsic::experimental_vector_partial_reduce_add &&
"Expected a partial reduction node");
- if (!Subtarget->isSVEorStreamingSVEAvailable())
+ if (!Subtarget->hasSVE2() && !Subtarget->isStreamingSVEAvailable())
return SDValue();
SDLoc DL(N);
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's worth adding an sve
RUN line to llvm/test/CodeGen/AArch64/sve-partial-reduce-wide-add.ll to protect the code.
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/30/builds/11717 Here is the relevant piece of the build log for the reference
|
Wide adds would try to be generated when SVE is enabled not SVE2.