[llvm] a5cf82c - Remove SDNPSideEffect from ARMcallseq_start and ARMcallseq_end (NFC) (#153248)

[via llvm-commits]

Author: AZero13
Date: 2025-08-24T20:53:40+01:00
New Revision: a5cf82c645245c4f6ed3dcc2178b8abfeed0f7d5

URL: https://github.com/llvm/llvm-project/commit/a5cf82c645245c4f6ed3dcc2178b8abfeed0f7d5
DIFF: https://github.com/llvm/llvm-project/commit/a5cf82c645245c4f6ed3dcc2178b8abfeed0f7d5.diff

LOG: 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.

Added: 
    

Modified: 
    llvm/lib/Target/ARM/ARMInstrInfo.td

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/ARM/ARMInstrInfo.td b/llvm/lib/Target/ARM/ARMInstrInfo.td
index 934ec52c6f1e4..4345f7a4a0ebc 100644
--- a/llvm/lib/Target/ARM/ARMInstrInfo.td
+++ b/llvm/lib/Target/ARM/ARMInstrInfo.td
@@ -164,10 +164,9 @@ def ARMWrapperPIC    : SDNode<"ARMISD::WrapperPIC",  SDTIntUnaryOp>;
 def ARMWrapperJT     : SDNode<"ARMISD::WrapperJT",   SDTIntUnaryOp>;
 
 def ARMcallseq_start : SDNode<"ISD::CALLSEQ_START", SDT_ARMCallSeqStart,
-                              [SDNPHasChain, SDNPSideEffect, SDNPOutGlue]>;
+                              [SDNPHasChain, SDNPOutGlue]>;
 def ARMcallseq_end   : SDNode<"ISD::CALLSEQ_END",   SDT_ARMCallSeqEnd,
-                              [SDNPHasChain, SDNPSideEffect,
-                               SDNPOptInGlue, SDNPOutGlue]>;
+                              [SDNPHasChain, SDNPOptInGlue, SDNPOutGlue]>;
 def ARMcopystructbyval : SDNode<"ARMISD::COPY_STRUCT_BYVAL" ,
                                 SDT_ARMStructByVal,
                                 [SDNPHasChain, SDNPInGlue, SDNPOutGlue,