[llvm] [AArch64][GlobalISel] Add support for post-indexed loads/stores. (PR #69532)

Jon Roelofs via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 19 09:45:14 PDT 2023


================
@@ -23615,6 +23617,23 @@ bool AArch64TargetLowering::mayBeEmittedAsTailCall(const CallInst *CI) const {
   return CI->isTailCall();
 }
 
+bool AArch64TargetLowering::isIndexingLegal(MachineInstr &MI, Register Base,
+                                            Register Offset, bool IsPre,
+                                            MachineRegisterInfo &MRI) const {
+  // HACK
+  if (IsPre)
+    return false; // Until we implement.
+
+  auto CstOffset = getIConstantVRegVal(Offset, MRI);
+  if (!CstOffset || CstOffset->isZero())
----------------
jroelofs wrote:

Why isn't `0` a legal immediate offset here?

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


More information about the llvm-commits mailing list