Skip to content

Commit d8ef156

Browse files
authored
DAG: Remove verifyReturnAddressArgumentIsConstant (#147240)
The intrinsic argument is already marked with immarg so non-constant values are rejected by the IR verifier.
1 parent 19afd27 commit d8ef156

File tree

15 files changed

+0
-53
lines changed

15 files changed

+0
-53
lines changed

llvm/include/llvm/CodeGen/TargetLowering.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5083,9 +5083,6 @@ class LLVM_ABI TargetLowering : public TargetLoweringBase {
50835083
return nullptr;
50845084
}
50855085

5086-
bool verifyReturnAddressArgumentIsConstant(SDValue Op,
5087-
SelectionDAG &DAG) const;
5088-
50895086
//===--------------------------------------------------------------------===//
50905087
// Inline Asm Support hooks
50915088
//

llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -7412,17 +7412,6 @@ TargetLowering::prepareSREMEqFold(EVT SETCCVT, SDValue REMNode,
74127412
return Blended;
74137413
}
74147414

7415-
bool TargetLowering::
7416-
verifyReturnAddressArgumentIsConstant(SDValue Op, SelectionDAG &DAG) const {
7417-
if (!isa<ConstantSDNode>(Op.getOperand(0))) {
7418-
DAG.getContext()->emitError("argument to '__builtin_return_address' must "
7419-
"be a constant integer");
7420-
return true;
7421-
}
7422-
7423-
return false;
7424-
}
7425-
74267415
SDValue TargetLowering::getSqrtInputTest(SDValue Op, SelectionDAG &DAG,
74277416
const DenormalMode &Mode) const {
74287417
SDLoc DL(Op);

llvm/lib/Target/ARM/ARMISelLowering.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6187,9 +6187,6 @@ SDValue ARMTargetLowering::LowerRETURNADDR(SDValue Op, SelectionDAG &DAG) const{
61876187
MachineFrameInfo &MFI = MF.getFrameInfo();
61886188
MFI.setReturnAddressIsTaken(true);
61896189

6190-
if (verifyReturnAddressArgumentIsConstant(Op, DAG))
6191-
return SDValue();
6192-
61936190
EVT VT = Op.getValueType();
61946191
SDLoc dl(Op);
61956192
unsigned Depth = Op.getConstantOperandVal(0);

llvm/lib/Target/CSKY/CSKYISelLowering.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1207,9 +1207,6 @@ SDValue CSKYTargetLowering::LowerRETURNADDR(SDValue Op,
12071207
MachineFrameInfo &MFI = MF.getFrameInfo();
12081208
MFI.setReturnAddressIsTaken(true);
12091209

1210-
if (verifyReturnAddressArgumentIsConstant(Op, DAG))
1211-
return SDValue();
1212-
12131210
EVT VT = Op.getValueType();
12141211
SDLoc dl(Op);
12151212
unsigned Depth = Op.getConstantOperandVal(0);

llvm/lib/Target/Hexagon/HexagonISelLowering.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1183,9 +1183,6 @@ HexagonTargetLowering::LowerRETURNADDR(SDValue Op, SelectionDAG &DAG) const {
11831183
MachineFrameInfo &MFI = MF.getFrameInfo();
11841184
MFI.setReturnAddressIsTaken(true);
11851185

1186-
if (verifyReturnAddressArgumentIsConstant(Op, DAG))
1187-
return SDValue();
1188-
11891186
EVT VT = Op.getValueType();
11901187
SDLoc dl(Op);
11911188
unsigned Depth = Op.getConstantOperandVal(0);

llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2607,9 +2607,6 @@ SDValue LoongArchTargetLowering::lowerFRAMEADDR(SDValue Op,
26072607

26082608
SDValue LoongArchTargetLowering::lowerRETURNADDR(SDValue Op,
26092609
SelectionDAG &DAG) const {
2610-
if (verifyReturnAddressArgumentIsConstant(Op, DAG))
2611-
return SDValue();
2612-
26132610
// Currently only support lowering return address for current frame.
26142611
if (Op.getConstantOperandVal(0) != 0) {
26152612
DAG.getContext()->emitError(

llvm/lib/Target/MSP430/MSP430ISelLowering.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1142,9 +1142,6 @@ SDValue MSP430TargetLowering::LowerRETURNADDR(SDValue Op,
11421142
MachineFrameInfo &MFI = DAG.getMachineFunction().getFrameInfo();
11431143
MFI.setReturnAddressIsTaken(true);
11441144

1145-
if (verifyReturnAddressArgumentIsConstant(Op, DAG))
1146-
return SDValue();
1147-
11481145
unsigned Depth = Op.getConstantOperandVal(0);
11491146
SDLoc dl(Op);
11501147
EVT PtrVT = Op.getValueType();

llvm/lib/Target/Mips/MipsISelLowering.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2698,9 +2698,6 @@ lowerFRAMEADDR(SDValue Op, SelectionDAG &DAG) const {
26982698

26992699
SDValue MipsTargetLowering::lowerRETURNADDR(SDValue Op,
27002700
SelectionDAG &DAG) const {
2701-
if (verifyReturnAddressArgumentIsConstant(Op, DAG))
2702-
return SDValue();
2703-
27042701
// check the depth
27052702
if (Op.getConstantOperandVal(0) != 0) {
27062703
DAG.getContext()->emitError(

llvm/lib/Target/PowerPC/PPCISelLowering.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17961,9 +17961,6 @@ SDValue PPCTargetLowering::LowerRETURNADDR(SDValue Op,
1796117961
MachineFrameInfo &MFI = MF.getFrameInfo();
1796217962
MFI.setReturnAddressIsTaken(true);
1796317963

17964-
if (verifyReturnAddressArgumentIsConstant(Op, DAG))
17965-
return SDValue();
17966-
1796717964
SDLoc dl(Op);
1796817965
unsigned Depth = Op.getConstantOperandVal(0);
1796917966

llvm/lib/Target/RISCV/RISCVISelLowering.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9339,9 +9339,6 @@ SDValue RISCVTargetLowering::lowerRETURNADDR(SDValue Op,
93399339
MVT XLenVT = Subtarget.getXLenVT();
93409340
int XLenInBytes = Subtarget.getXLen() / 8;
93419341

9342-
if (verifyReturnAddressArgumentIsConstant(Op, DAG))
9343-
return SDValue();
9344-
93459342
EVT VT = Op.getValueType();
93469343
SDLoc DL(Op);
93479344
unsigned Depth = Op.getConstantOperandVal(0);

0 commit comments

Comments
 (0)