[PATCH] D19656: [ELF] - keep alive all non-text sections referenced by .eh_frame

[George Rimar via llvm-commits]

grimar added inline comments.

================
Comment at: ELF/MarkLive.cpp:78
@@ +77,3 @@
+static void run(ELFFile<ELFT> &Obj, InputSectionBase<ELFT> *Sec,
+                Elf_Shdr *RelSec, Func Fn) {
+  typedef typename ELFT::Rel Elf_Rel;
----------------
rafael wrote:
> Please use std::function<void(ResolvedReloc<ELFT>)> to mach the declaration of forEachSuccessor.
Done.

================
Comment at: ELF/MarkLive.cpp:79
@@ +78,3 @@
+                Elf_Shdr *RelSec, Func Fn) {
+  typedef typename ELFT::Rel Elf_Rel;
+  typedef typename ELFT::Rela Elf_Rela;
----------------
rafael wrote:
> These now have a single use each, please inline.
Done.

================
Comment at: ELF/MarkLive.cpp:94
@@ -78,5 +93,3 @@
                              std::function<void(ResolvedReloc<ELFT>)> Fn) {
-  typedef typename ELFT::Rel Elf_Rel;
-  typedef typename ELFT::Rela Elf_Rela;
   typedef typename ELFT::Shdr Elf_Shdr;
 
----------------
rafael wrote:
> Single use typedef.
It was single use, I removed typedef (like was in run()).

================
Comment at: test/ELF/eh-frame-gc.s:9
@@ +8,3 @@
+# CHECK: Sections [
+# CHECK: Name: .data
+
----------------
rafael wrote:
> Give the section a more unique name like test_personality_section.
Done.


Repository:
  rL LLVM

http://reviews.llvm.org/D19656