[llvm] [AArch64][GISel] Support SVE with 128-bit min-size for G_LOAD and G_STORE (PR #92130)

Thorsten Schütt via llvm-commits llvm-commits at lists.llvm.org
Tue May 21 23:37:35 PDT 2024


================
@@ -329,7 +335,23 @@ AArch64LegalizerInfo::AArch64LegalizerInfo(const AArch64Subtarget &ST)
     return ValTy.isPointerVector() && ValTy.getAddressSpace() == 0;
   };
 
-  getActionDefinitionsBuilder(G_LOAD)
+  auto &LoadActions = getActionDefinitionsBuilder(G_LOAD);
+  auto &StoreActions = getActionDefinitionsBuilder(G_STORE);
+
+  if (ST.hasSVE()) {
----------------
tschuett wrote:

For this PR, I vote for the G_ABS pattern and remove the verbose loop. We should open a ticket to discuss making feature-based legalization more economic. The X86 guys have similar issues.

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


More information about the llvm-commits mailing list