Skip to content

Commit a5cf82c

Browse files
authored
Remove SDNPSideEffect from ARMcallseq_start and ARMcallseq_end (NFC) (#153248)
A call sequence does not have any unmodeled side effects in of itself. ADJCALLSTACKUP and ADJCALLSTACKDOWN do, however, so the attribute should be there.
1 parent d84be8a commit a5cf82c

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

llvm/lib/Target/ARM/ARMInstrInfo.td

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -164,10 +164,9 @@ def ARMWrapperPIC : SDNode<"ARMISD::WrapperPIC", SDTIntUnaryOp>;
164164
def ARMWrapperJT : SDNode<"ARMISD::WrapperJT", SDTIntUnaryOp>;
165165

166166
def ARMcallseq_start : SDNode<"ISD::CALLSEQ_START", SDT_ARMCallSeqStart,
167-
[SDNPHasChain, SDNPSideEffect, SDNPOutGlue]>;
167+
[SDNPHasChain, SDNPOutGlue]>;
168168
def ARMcallseq_end : SDNode<"ISD::CALLSEQ_END", SDT_ARMCallSeqEnd,
169-
[SDNPHasChain, SDNPSideEffect,
170-
SDNPOptInGlue, SDNPOutGlue]>;
169+
[SDNPHasChain, SDNPOptInGlue, SDNPOutGlue]>;
171170
def ARMcopystructbyval : SDNode<"ARMISD::COPY_STRUCT_BYVAL" ,
172171
SDT_ARMStructByVal,
173172
[SDNPHasChain, SDNPInGlue, SDNPOutGlue,

0 commit comments

Comments
 (0)