diff options
| -rw-r--r-- | llvm/lib/Target/AArch64/AArch64FalkorHWPFFix.cpp | 7 | ||||
| -rw-r--r-- | llvm/test/CodeGen/AArch64/falkor-hwpf-fix.mir | 23 |
2 files changed, 29 insertions, 1 deletions
diff --git a/llvm/lib/Target/AArch64/AArch64FalkorHWPFFix.cpp b/llvm/lib/Target/AArch64/AArch64FalkorHWPFFix.cpp index 6df8df0e65a..3acacf7ce58 100644 --- a/llvm/lib/Target/AArch64/AArch64FalkorHWPFFix.cpp +++ b/llvm/lib/Target/AArch64/AArch64FalkorHWPFFix.cpp @@ -638,9 +638,14 @@ static Optional<LoadInfo> getLoadInfo(const MachineInstr &MI) { break; } + // Loads from the stack pointer don't get prefetched. + unsigned BaseReg = MI.getOperand(BaseRegIdx).getReg(); + if (BaseReg == AArch64::SP || BaseReg == AArch64::WSP) + return None; + LoadInfo LI; LI.DestReg = DestRegIdx == -1 ? 0 : MI.getOperand(DestRegIdx).getReg(); - LI.BaseReg = MI.getOperand(BaseRegIdx).getReg(); + LI.BaseReg = BaseReg; LI.BaseRegIdx = BaseRegIdx; LI.OffsetOpnd = OffsetIdx == -1 ? nullptr : &MI.getOperand(OffsetIdx); LI.IsPrePost = IsPrePost; diff --git a/llvm/test/CodeGen/AArch64/falkor-hwpf-fix.mir b/llvm/test/CodeGen/AArch64/falkor-hwpf-fix.mir index 70da36cdb89..38622ae0e49 100644 --- a/llvm/test/CodeGen/AArch64/falkor-hwpf-fix.mir +++ b/llvm/test/CodeGen/AArch64/falkor-hwpf-fix.mir @@ -330,3 +330,26 @@ body: | bb.1: RET_ReallyLR ... +--- +# Check that we treat sp based loads as non-prefetching. + +# CHECK-LABEL: name: hwpf_spbase +# CHECK-NOT: ORRXrs %xzr +# CHECK: LDRWui %x15 +# CHECK: LDRWui %sp +name: hwpf_spbase +tracksRegLiveness: true +body: | + bb.0: + liveins: %w0, %x15 + + %w1 = LDRWui %x15, 0 :: ("aarch64-strided-access" load 4) + %w17 = LDRWui %sp, 0 + + %w0 = SUBWri %w0, 1, 0 + %wzr = SUBSWri %w0, 0, 0, implicit-def %nzcv + Bcc 9, %bb.0, implicit %nzcv + + bb.1: + RET_ReallyLR +... |
