Skip to content

Commit

Permalink
[llvm-objcopy] Add -strip-non-alloc option to remove all non-allocate…
Browse files Browse the repository at this point in the history
…d sections

This change adds a new flag not present in GNU objcopy that we call
--strip-non-alloc.

Differential Revision: https://reviews.llvm.org/D39926

llvm-svn: 318168
  • Loading branch information
jakehehrlich committed Nov 14, 2017
1 parent 0b2f73f commit d56725a
Show file tree
Hide file tree
Showing 2 changed files with 37 additions and 0 deletions.
26 changes: 26 additions & 0 deletions llvm/test/tools/llvm-objcopy/strip-non-alloc.test
@@ -0,0 +1,26 @@
# RUN: yaml2obj %s > %t
# RUN: llvm-objcopy --strip-non-alloc %t %t2
# RUN: llvm-readobj -file-headers -sections %t2 | FileCheck %s

!ELF
FileHeader:
Class: ELFCLASS64
Data: ELFDATA2LSB
Type: ET_REL
Machine: EM_X86_64
Sections:
- Name: .bss
Type: SHT_NOBITS
Flags: [ SHF_ALLOC ]
- Name: .text
Type: SHT_PROGBITS
Flags: [ SHF_ALLOC, SHF_EXECINSTR ]
- Name: .blarg
Type: SHT_PROGBITS
Flags: [ ]

# CHECK: SectionHeaderCount: 4

# CHECK: Name: .bss
# CHECK: Name: .text
# CHECK: Name: .shstrtab
11 changes: 11 additions & 0 deletions llvm/tools/llvm-objcopy/llvm-objcopy.cpp
Expand Up @@ -87,6 +87,8 @@ static cl::opt<bool> StripDebug("strip-debug",
cl::desc("Removes all debug information"));
static cl::opt<bool> StripSections("strip-sections",
cl::desc("Remove all section headers"));
static cl::opt<bool> StripNonAlloc("strip-non-alloc",
cl::desc("Remove all non-allocated sections"));
static cl::opt<bool>
StripDWO("strip-dwo", cl::desc("remove all DWARF .dwo sections from file"));
static cl::opt<bool> ExtractDWO(
Expand Down Expand Up @@ -206,6 +208,15 @@ void CopyBinary(const ELFObjectFile<ELFT> &ObjFile) {
};
}

if (StripNonAlloc)
RemovePred = [RemovePred, &Obj](const SectionBase &Sec) {
if (RemovePred(Sec))
return true;
if (&Sec == Obj->getSectionHeaderStrTab())
return false;
return (Sec.Flags & SHF_ALLOC) == 0;
};

Obj->removeSections(RemovePred);
Obj->finalize();
WriteObjectFile(*Obj, OutputFilename.getValue());
Expand Down

0 comments on commit d56725a

Please sign in to comment.