Skip to content

Commit c935f6e

Browse files
author
Krzysztof Parzyszek
committed
[Hexagon] Punt on registers without reaching defs in addr mode opt
This fixes #52636.
1 parent ce8022f commit c935f6e

File tree

2 files changed

+48
-1
lines changed

2 files changed

+48
-1
lines changed

llvm/lib/Target/Hexagon/HexagonOptAddrMode.cpp

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -313,12 +313,18 @@ bool HexagonOptAddrMode::isSafeToExtLR(NodeAddr<StmtNode *> SN,
313313
return false;
314314
}
315315

316+
// If the register is undefined (for example if it's a reserved register),
317+
// it may still be possible to extend the range, but it's safer to be
318+
// conservative and just punt.
319+
if (LRExtRegRD == 0)
320+
return false;
321+
316322
MachineInstr *UseMI = NodeAddr<StmtNode *>(IA).Addr->getCode();
317323
NodeAddr<DefNode *> LRExtRegDN = DFG->addr<DefNode *>(LRExtRegRD);
318324
// Reaching Def to LRExtReg can't be a phi.
319325
if ((LRExtRegDN.Addr->getFlags() & NodeAttrs::PhiRef) &&
320326
MI->getParent() != UseMI->getParent())
321-
return false;
327+
return false;
322328
}
323329
return true;
324330
}
Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
; RUN: llc -march=hexagon < %s | FileCheck %s
2+
3+
; Check that this doesn't crash.
4+
; CHECK: call f1
5+
6+
target datalayout = "e-m:e-p:32:32:32-a:0-n16:32-i64:64:64-i32:32:32-i16:16:16-i1:8:8-f32:32:32-f64:64:64-v32:32:32-v64:64:64-v512:512:512-v1024:1024:1024-v2048:2048:2048"
7+
target triple = "hexagon"
8+
9+
%s.0 = type { %s.1 }
10+
%s.1 = type { %s.2 }
11+
%s.2 = type { i32, i32 }
12+
13+
define dso_local i32 @f0(i1 zeroext %a0) local_unnamed_addr #0 {
14+
b0:
15+
%v0 = tail call i32 @llvm.read_register.i32(metadata !0)
16+
%v1 = add nsw i32 %v0, 4096
17+
%v2 = inttoptr i32 %v1 to %s.0*
18+
%v3 = getelementptr inbounds %s.0, %s.0* %v2, i32 -1
19+
%v4 = tail call i32 bitcast (i32 (...)* @f1 to i32 (%s.0*)*)(%s.0* noundef nonnull %v3) #2
20+
br i1 %a0, label %b2, label %b1
21+
22+
b1: ; preds = %b0
23+
%v5 = getelementptr inbounds %s.0, %s.0* %v3, i32 0, i32 0, i32 0, i32 0
24+
%v6 = load i32, i32* %v5, align 4
25+
br label %b2
26+
27+
b2: ; preds = %b1, %b0
28+
%v7 = phi i32 [ %v6, %b1 ], [ undef, %b0 ]
29+
ret i32 %v7
30+
}
31+
32+
; Function Attrs: nounwind readonly
33+
declare i32 @llvm.read_register.i32(metadata) #1
34+
35+
declare dso_local i32 @f1(...) local_unnamed_addr #0
36+
37+
attributes #0 = { "target-features"="+reserved-r19,+v60,-long-calls" }
38+
attributes #1 = { nounwind readonly }
39+
attributes #2 = { nounwind }
40+
41+
!0 = !{!"r19"}

0 commit comments

Comments
 (0)