[PATCH] D13728: ELF2: Merge .{text,rodata,data,bss}.* sections.

[Rafael Ávila de Espíndola via llvm-commits]

rafael accepted this revision.
rafael added a comment.
This revision is now accepted and ready to land.

Thanks!

I always compile with -fno-unique-section-names, which is why I didn't notice this.


================
Comment at: ELF/Writer.cpp:371
@@ +370,3 @@
+template <class ELFT>
+static StringRef getOutputSection(InputSection<ELFT> *Sec) {
+  StringRef S = Sec->getSectionName();
----------------
Pass in the name and you can drop the template.

================
Comment at: test/elf2/section-name.s:28
@@ +27,3 @@
+// CHECK:     Name: .foo
+// CHECK:     Name: .foo
+// CHECK:     Name: .bss
----------------
foo.a?



http://reviews.llvm.org/D13728