diff --git a/llvm/lib/Target/Hexagon/HexagonBitSimplify.cpp b/llvm/lib/Target/Hexagon/HexagonBitSimplify.cpp index 5fef6083552b..41d4f6c17e89 100644 --- a/llvm/lib/Target/Hexagon/HexagonBitSimplify.cpp +++ b/llvm/lib/Target/Hexagon/HexagonBitSimplify.cpp @@ -2480,6 +2480,7 @@ bool BitSimplification::simplifyExtractLow(MachineInstr *MI, continue; if (RW != SW) SR = (Off/RW == 0) ? Hexagon::isub_lo : Hexagon::isub_hi; + Off = Off % RW; if (!validateReg({R,SR}, ExtOpc, 1)) continue; diff --git a/llvm/test/CodeGen/Hexagon/bit-extract-off.ll b/llvm/test/CodeGen/Hexagon/bit-extract-off.ll new file mode 100644 index 000000000000..183435ab7b23 --- /dev/null +++ b/llvm/test/CodeGen/Hexagon/bit-extract-off.ll @@ -0,0 +1,23 @@ +; RUN: llc -march=hexagon < %s | FileCheck %s +; CHECK: extractu(r1,#31,#0) + +; In the IR this was an extract of 31 bits starting at position 32 in r1:0. +; When mapping it to an extract from r1, the offset was not reset to 0, and +; we had "extractu(r1,#31,#32)". + +target triple = "hexagon" + +define hidden i32 @fred([101 x double]* %a0, i32 %a1, i32* %a2, i32* %a3) #0 { +b4: + br label %b5 + +b5: ; preds = %b5, %b4 + %v6 = call double @fabs(double undef) #1 + store double %v6, double* undef, align 8 + br label %b5 +} + +declare double @fabs(double) #1 + +attributes #0 = { nounwind "target-cpu"="hexagonv60" "target-features"="-hvx,-hvx-double,-long-calls" } +attributes #1 = { nounwind readnone "target-cpu"="hexagonv60" "target-features"="-hvx,-hvx-double,-long-calls" }