maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

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.

llvm-svn: 205059
This commit is contained in:
David Blaikie 2014-03-28 21:48:31 +00:00
parent 02f2188bb9
commit dca7c7c5f1
2 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
llvm/lib/MC/MCContext.cpp
View File

@ -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.

10
llvm/test/MC/ELF/compression.s
View File

@ -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