[llvm] [AArch64][GISel] Support SVE with 128-bit min-size for G_LOAD and G_STORE (PR #92130)
David Green via llvm-commits
llvm-commits at lists.llvm.org
Tue May 14 10:29:50 PDT 2024
================
@@ -26375,12 +26375,20 @@ bool AArch64TargetLowering::shouldLocalize(
return TargetLoweringBase::shouldLocalize(MI, TTI);
}
+static bool isScalableTySupported(const unsigned Op) {
+ return Op == Instruction::Load || Op == Instruction::Store;
+}
+
bool AArch64TargetLowering::fallBackToDAGISel(const Instruction &Inst) const {
- if (Inst.getType()->isScalableTy())
- return true;
+ const auto ScalableTySupported = isScalableTySupported(Inst.getOpcode());
+
+ // Fallback for scalable vectors
+ if (Inst.getType()->isScalableTy() && !ScalableTySupported) {
----------------
davemgreen wrote:
Hi - Please keep the command line option from #72976 at the moment. It will help having a debug option we can use for testing, but not be limited to implement every single SVE optimization before global-isel becomes production ready.
This doesn't need to be dependant on the operations we currently "support", the normal fallback mechanism for unsupported operations should handle that.
https://github.com/llvm/llvm-project/pull/92130
More information about the llvm-commits
mailing list