ARM: Move declaration of supportSplitCSR to be public

[arsenm]

This is an implementation of a public method from the base
class, so it should also be public. Avoids unrelated diff
in a future patch.

[arsenm]

• ARM: Move ABI helpers from Subtarget to TargetMachine #144680
• ARM: Move declaration of supportSplitCSR to be public #144679 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[llvmbot]

@llvm/pr-subscribers-backend-arm

Author: Matt Arsenault (arsenm)

Changes

This is an implementation of a public method from the base
class, so it should also be public. Avoids unrelated diff
in a future patch.

---

Full diff: https://github.com/llvm/llvm-project/pull/144679.diff

1 Files Affected:

• (modified) llvm/lib/Target/ARM/ARMISelLowering.h (+5-5)

diff --git a/llvm/lib/Target/ARM/ARMISelLowering.h b/llvm/lib/Target/ARM/ARMISelLowering.h
index 87710ee29a249..e48967bd7f262 100644
--- a/llvm/lib/Target/ARM/ARMISelLowering.h
+++ b/llvm/lib/Target/ARM/ARMISelLowering.h
@@ -716,6 +716,11 @@ class VectorType;
       return true;
     }
 
+    bool supportSplitCSR(MachineFunction *MF) const override {
+      return MF->getFunction().getCallingConv() == CallingConv::CXX_FAST_TLS &&
+          MF->getFunction().hasFnAttribute(Attribute::NoUnwind);
+    }
+
     bool hasStandaloneRem(EVT VT) const override {
       return HasStandaloneRem;
     }
@@ -914,11 +919,6 @@ class VectorType;
                             SmallVectorImpl<SDValue> &InVals, bool isThisReturn,
                             SDValue ThisVal, bool isCmseNSCall) const;
 
-    bool supportSplitCSR(MachineFunction *MF) const override {
-      return MF->getFunction().getCallingConv() == CallingConv::CXX_FAST_TLS &&
-          MF->getFunction().hasFnAttribute(Attribute::NoUnwind);
-    }
-
     void initializeSplitCSR(MachineBasicBlock *Entry) const override;
     void insertCopiesSplitCSR(
       MachineBasicBlock *Entry,

[github-actions]

✅ With the latest revision this PR passed the C/C++ code formatter.

[efriedma-quic]

LGTM

[arsenm]

Merge activity

• Jun 19, 12:33 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Jun 19, 12:34 AM UTC: Graphite rebased this pull request as part of a merge.
• Jun 19, 12:36 AM UTC: @arsenm merged this pull request with Graphite.