[PATCH] D34676: [ELF] - Do not crash when LLD synthesizes output sections with BYTE commands and -r

[George Rimar via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL306527: [ELF] - Do not crash when LLD synthesizes output sections with BYTE commands… (authored by grimar).

Changed prior to commit:
  https://reviews.llvm.org/D34676?vs=104123&id=104370#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D34676

Files:
  lld/trunk/ELF/Writer.cpp
  lld/trunk/test/ELF/relocatable-script.s


Index: lld/trunk/ELF/Writer.cpp
===================================================================
--- lld/trunk/ELF/Writer.cpp
+++ lld/trunk/ELF/Writer.cpp
@@ -1014,13 +1014,13 @@
 }
 
 template <class ELFT> void Writer<ELFT>::sortSections() {
+  if (Script->Opt.HasSections)
+    Script->adjustSectionsBeforeSorting();
+
   // Don't sort if using -r. It is not necessary and we want to preserve the
   // relative order for SHF_LINK_ORDER sections.
   if (Config->Relocatable)
-    return;
-
-  if (Script->Opt.HasSections)
-    Script->adjustSectionsBeforeSorting();
+      return;
 
   for (BaseCommand *Base : Script->Opt.Commands)
     if (auto *Cmd = dyn_cast<OutputSectionCommand>(Base))
Index: lld/trunk/test/ELF/relocatable-script.s
===================================================================
--- lld/trunk/test/ELF/relocatable-script.s
+++ lld/trunk/test/ELF/relocatable-script.s
@@ -0,0 +1,7 @@
+# REQUIRES: x86
+# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux -o %t1.o %s
+# RUN: echo "SECTIONS { .foo : { BYTE(0x0) } }" > %t.script
+# RUN: ld.lld -r %t1.o -script %t.script -o %t2.o
+# RUN: llvm-readobj -sections %t2.o | FileCheck %s
+
+# CHECK:  Name: .foo


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34676.104370.patch
Type: text/x-patch
Size: 1192 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170628/d597432e/attachment.bin>