Skip to content

Commit b1017a4

Browse files
authored
Use getSignedTargetConstant for offset (llvm#141149)
This is to fix an assertion failure with PeepholePPC64. The load/store offset can be negative. A reduced case from one of our failures is added as well.
1 parent c7ccfc6 commit b1017a4

File tree

2 files changed

+36
-2
lines changed

2 files changed

+36
-2
lines changed

llvm/lib/Target/PowerPC/PPCISelDAGToDAG.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7858,8 +7858,8 @@ void PPCDAGToDAGISel::PeepholePPC64() {
78587858
if (!isInt<16>(Offset))
78597859
continue;
78607860

7861-
ImmOpnd = CurDAG->getTargetConstant(Offset, SDLoc(ImmOpnd),
7862-
ImmOpnd.getValueType());
7861+
ImmOpnd = CurDAG->getSignedTargetConstant(Offset, SDLoc(ImmOpnd),
7862+
ImmOpnd.getValueType());
78637863
} else if (Offset != 0) {
78647864
// This optimization is performed for non-TOC-based local-[exec|dynamic]
78657865
// accesses.
Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py UTC_ARGS: --version 5
2+
; RUN: llc -verify-machineinstrs < %s -mcpu=ppc -mtriple powerpc-ibm-aix-xcoff \
3+
; RUN: -ppc-asm-full-reg-names | FileCheck %s
4+
5+
%_type = type <{ [90112 x i8] }>
6+
7+
@g = external global %_type
8+
9+
define void @foo(ptr %p) {
10+
; CHECK-LABEL: foo:
11+
; CHECK: stfd f31, 0(r31)
12+
; CHECK-NEXT: lwz r3, -32(r30)
13+
; CHECK-NEXT: lwz r4, -28(r30)
14+
; CHECK-NEXT: cmplwi r3, 0
15+
; CHECK-NEXT: cmpwi cr1, r3, 0
16+
; CHECK-NEXT: crandc 4*cr5+lt, 4*cr1+gt, eq
17+
; CHECK-NEXT: cmpwi cr1, r4, 0
18+
entry:
19+
%0 = load double, ptr getelementptr inbounds nuw (i8, ptr @g, i32 83272), align 8
20+
%1 = call i32 @bar(ptr getelementptr inbounds nuw (i8, ptr @g, i32 83272))
21+
%2 = call i32 @bar(ptr getelementptr inbounds nuw (i8, ptr @g, i32 83240))
22+
store double %0, ptr %p, align 8
23+
%3 = load i64, ptr getelementptr inbounds nuw (i8, ptr @g, i32 83240), align 8
24+
%4 = icmp slt i64 %3, 1
25+
br i1 %4, label %then, label %else
26+
27+
then: ; preds = %entry
28+
ret void
29+
30+
else: ; preds = %entry
31+
ret void
32+
}
33+
34+
declare signext i32 @bar(ptr)

0 commit comments

Comments
 (0)