Debug Compression: Avoid compression debug_frame for now

Turns out debug_frame does use multiple fragments, so it doesn't
compress correctly with the current approach. Disable compressing it for
now while I figure out what's the best solution for it.

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

diff --git a/lib/MC/MCContext.cpp b/lib/MC/MCContext.cpp
index b228d18ba7f06d1cd262b5f2d642ee685d08e38c..fdedec1031abf6803bf98aa93a5bf12ac95129a2 100644 (file)
--- a/lib/MC/MCContext.cpp
+++ b/lib/MC/MCContext.cpp
@@ -253,7 +253,7 @@ getELFSection(StringRef Section, unsigned Type, unsigned Flags,
 
   SmallString<32> ZDebugName;
   if (MAI->compressDebugSections() && Section.startswith(".debug_") &&
-      Section != ".debug_frame")
+      Section != ".debug_frame" && Section != ".debug_line")
     Section = (".z" + Section.drop_front(1)).toStringRef(ZDebugName);
 
   // Do the lookup, if we have a hit, return it.

diff --git a/test/MC/ELF/compression.s b/test/MC/ELF/compression.s
index b5ba2624f8ac9fa73bd897018a9f515ece7f2841..305a84e1babbc665924a77903a5088323c340db3 100644 (file)
--- a/test/MC/ELF/compression.s
+++ b/test/MC/ELF/compression.s
@@ -2,14 +2,22 @@
 
 // REQUIRES: zlib
 
-// CHECK: Contents of section .zdebug_line:
+// CHECK: Contents of section .debug_line:
+// FIXME: Figure out how to handle debug_line that currently uses multiple section fragments
+// CHECK-NOT: ZLIB
+
+// CHECK: Contents of section .zdebug_abbrev:
 // Check for the 'ZLIB' file magic at the start of the section
 // CHECK-NEXT: ZLIB
+
 // We shouldn't compress the debug_frame section, since it can be relaxed
 // CHECK: Contents of section .debug_frame
 // CHECK-NOT: ZLIB
 
        .section        .debug_line,"",@progbits
+
+       .section        .debug_abbrev,"",@progbits
+       .byte   1                       # Abbreviation Code
        .text
 foo:
        .cfi_startproc