From: Evan Cheng <evan.cheng@apple.com>
Date: Wed, 24 Aug 2011 22:31:37 +0000 (+0000)
Subject: Some autoconf tests use module level inline asm to test compiler's handling of
X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=5fbe5e783ee0c5ae27c17490a752d7e603e84ed2;p=oota-llvm.git

Some autoconf tests use module level inline asm to test compiler's handling of
.cfi_startproc. e.g. libffi:

 $ cat confopt.c
asm (".cfi_startproc\n\t.cfi_endproc");

int main () { return 0; }

Teach MC / dwarf emission to handle these cfi directives which essentially
create an empty frame.

rdar://10017184


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@138504 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/MC/MCDwarf.cpp b/lib/MC/MCDwarf.cpp
index 06ce502d572..b57fd9a13bd 100644
--- a/lib/MC/MCDwarf.cpp
+++ b/lib/MC/MCDwarf.cpp
@@ -895,7 +895,7 @@ MCSymbol *FrameEmitterImpl::EmitFDE(MCStreamer &streamer,
   const MCObjectFileInfo *MOFI = context.getObjectFileInfo();
   bool verboseAsm = streamer.isVerboseAsm();
 
-  if (!MOFI->isFunctionEHFrameSymbolPrivate() && IsEH) {
+  if (IsEH && frame.Function && !MOFI->isFunctionEHFrameSymbolPrivate()) {
     MCSymbol *EHSym =
       context.GetOrCreateSymbol(frame.Function->getName() + Twine(".eh"));
     streamer.EmitEHSymAttributes(frame.Function, EHSym);
diff --git a/lib/MC/MCStreamer.cpp b/lib/MC/MCStreamer.cpp
index 5295d0f58f3..7e3d7a6886e 100644
--- a/lib/MC/MCStreamer.cpp
+++ b/lib/MC/MCStreamer.cpp
@@ -190,14 +190,14 @@ void MCStreamer::EmitCFIStartProc() {
   MCDwarfFrameInfo *CurFrame = getCurrentFrameInfo();
   if (CurFrame && !CurFrame->End)
     report_fatal_error("Starting a frame before finishing the previous one!");
-  MCDwarfFrameInfo Frame;
 
+  MCDwarfFrameInfo Frame;
   Frame.Function = LastSymbol;
 
   // If the function is externally visible, we need to create a local
   // symbol to avoid relocations.
   StringRef Prefix = getContext().getAsmInfo().getPrivateGlobalPrefix();
-  if (LastSymbol->getName().startswith(Prefix)) {
+  if (LastSymbol && LastSymbol->getName().startswith(Prefix)) {
     Frame.Begin = LastSymbol;
   } else {
     Frame.Begin = getContext().CreateTempSymbol();
diff --git a/test/MC/MachO/debug_frame.s b/test/MC/MachO/debug_frame.s
index 47264ef8799..20bfd8dde2e 100644
--- a/test/MC/MachO/debug_frame.s
+++ b/test/MC/MachO/debug_frame.s
@@ -1,5 +1,11 @@
 // RUN: llvm-mc -triple i386-apple-darwin %s -filetype=obj -o - | macho-dump | FileCheck %s
 
+// Make sure MC can handle file level .cfi_startproc and .cfi_endproc that creates
+// an empty frame.
+// rdar://10017184
+.cfi_startproc
+.cfi_endproc
+
 // Check that we don't produce a relocation for the CIE pointer and therefore
 // we have only one relocation in __debug_frame.
 
@@ -22,17 +28,20 @@ Ltext_end:
 // CHECK:       (('section_name', '__debug_frame\x00\x00\x00')
 // CHECK-NEXT:   ('segment_name', '__DWARF\x00\x00\x00\x00\x00\x00\x00\x00\x00')
 // CHECK-NEXT:   ('address', 8)
-// CHECK-NEXT:   ('size', 36)
+// CHECK-NEXT:   ('size', 52)
 // CHECK-NEXT:   ('offset', 332)
 // CHECK-NEXT:   ('alignment', 2)
-// CHECK-NEXT:   ('reloc_offset', 368)
-// CHECK-NEXT:   ('num_reloc', 1)
+// CHECK-NEXT:   ('reloc_offset', 384)
+// CHECK-NEXT:   ('num_reloc', 2)
 // CHECK-NEXT:   ('flags', 0x2000000)
 // CHECK-NEXT:   ('reserved1', 0)
 // CHECK-NEXT:   ('reserved2', 0)
 // CHECK-NEXT:  ),
 // CHECK-NEXT: ('_relocations', [
 // CHECK-NEXT:   # Relocation 0
+// CHECK-NEXT:   (('word-0', 0x2c),
+// CHECK-NEXT:    ('word-1', 0x4000001)),
+// CHECK-NEXT:   # Relocation 1
 // CHECK-NEXT:   (('word-0', 0x1c),
 // CHECK-NEXT:    ('word-1', 0x4000001)),
 // CHECK-NEXT: ])