[llvm] [SFrames] Add FDEs for functions with .cfi_startproc (PR #154213)

Indu Bhagat via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 25 22:46:33 PDT 2025


================
@@ -84,7 +150,19 @@ class SFrameEmitterImpl {
 
 void MCSFrameEmitter::emit(MCObjectStreamer &Streamer) {
   MCContext &Context = Streamer.getContext();
+  // If this target doesn't support sframes, return now. Gas doesn't warn in
+  // this case, but if we want to, it should be done at option-parsing time,
+  // rather than here.
----------------
ibhagatgnu wrote:

I would agree that such error handling is ideally done at earlier (and not at emission time).

https://github.com/llvm/llvm-project/pull/154213


More information about the llvm-commits mailing list