[lld] r317306 - [ELF] - Teach LLD to strip .zdebug_* sections when --strip-debug/--strip-all is specified.

[George Rimar via llvm-commits]

Author: grimar
Date: Fri Nov  3 01:13:16 2017
New Revision: 317306

URL: http://llvm.org/viewvc/llvm-project?rev=317306&view=rev
Log:
[ELF] - Teach LLD to strip .zdebug_* sections when --strip-debug/--strip-all is specified.

Currently we do not strip .zdebug_*, what looks wrong.
Also this simplifies the testcase we have for this options.

Differential revision: https://reviews.llvm.org/D39552

Modified:
    lld/trunk/ELF/InputFiles.cpp
    lld/trunk/test/ELF/strip-debug.s

Modified: lld/trunk/ELF/InputFiles.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/InputFiles.cpp?rev=317306&r1=317305&r2=317306&view=diff
==============================================================================
--- lld/trunk/ELF/InputFiles.cpp (original)
+++ lld/trunk/ELF/InputFiles.cpp Fri Nov  3 01:13:16 2017
@@ -537,7 +537,8 @@ InputSectionBase *ObjFile<ELFT>::createI
     return &InputSection::Discarded;
   }
 
-  if (Config->Strip != StripPolicy::None && Name.startswith(".debug"))
+  if (Config->Strip != StripPolicy::None &&
+      (Name.startswith(".debug") || Name.startswith(".zdebug")))
     return &InputSection::Discarded;
 
   // The linkonce feature is a sort of proto-comdat. Some glibc i386 object

Modified: lld/trunk/test/ELF/strip-debug.s
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/test/ELF/strip-debug.s?rev=317306&r1=317305&r2=317306&view=diff
==============================================================================
--- lld/trunk/test/ELF/strip-debug.s (original)
+++ lld/trunk/test/ELF/strip-debug.s Fri Nov  3 01:13:16 2017
@@ -1,25 +1,14 @@
 # REQUIRES: x86
-
-# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux -g %s -o %t
-# RUN: ld.lld %t -o %t2
-# RUN: llvm-readobj -sections -symbols %t2 | FileCheck -check-prefix=DEFAULT %s
+# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o %t
 # RUN: ld.lld %t -o %t2 --strip-debug
-# RUN: llvm-readobj -sections -symbols %t2 | FileCheck -check-prefix=STRIP %s
+# RUN: llvm-readobj -sections %t2 | FileCheck %s
 # RUN: ld.lld %t -o %t2 -S
-# RUN: llvm-readobj -sections -symbols %t2 | FileCheck -check-prefix=STRIP %s
+# RUN: llvm-readobj -sections %t2 | FileCheck %s
 # RUN: ld.lld %t -o %t2 --strip-all
-# RUN: llvm-readobj -sections -symbols %t2 | FileCheck -check-prefix=STRIP %s
-
-# DEFAULT: Name: .debug_info
-# DEFAULT: Name: .debug_abbrev
-# DEFAULT: Name: .debug_aranges
-# DEFAULT: Name: .debug_line
+# RUN: llvm-readobj -sections %t2 | FileCheck %s
 
-# STRIP-NOT: Name: .debug_info
-# STRIP-NOT: Name: .debug_abbrev
-# STRIP-NOT: Name: .debug_aranges
-# STRIP-NOT: Name: .debug_line
+# CHECK-NOT: Foo
+# CHECK-NOT: Bar
 
-.globl _start
-_start:
-  ret
+.section .debug_Foo,"", at progbits
+.section .zdebug_Bar,"", at progbits