[PATCH] D19656: [ELF] - keep alive all non-text sections referenced by .eh_frame
George Rimar via llvm-commits
llvm-commits at lists.llvm.org
Mon May 2 07:00:26 PDT 2016
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
More information about the llvm-commits
mailing list