[PATCH] D25234: [LLD][ARM] Garbage collection support for .ARM.exidx sections
Rui Ueyama via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 4 13:29:19 PDT 2016
ruiu added inline comments.
> InputFiles.cpp:259
> }
> + // .ARM.exidx sections have a reverse dependency on the InputSection they
> + // have a SHF_LINK_ORDER dependency, this is identified by the sh_link.
Can you make this a separate function, say, initializeARMLinkOrderSection, and call it from ObjectFile::parse?
Please do this only when Config->GcSections is true.
> InputSection.h:232
> + // InputSection that is dependent on us (reverse dependency for GC)
> + InputSectionBase<ELFT> *dependentSection = nullptr;
> +
Variable names should start with uppercase letters.
Is it guaranteed that a section has at most one corresponding .ARM.exidx section?
> arm-exidx-gc.s:13
> +
> + .syntax unified
> + .section .text.func1, "ax",%progbits
I don't know much about ARM assembly, but do you actually need this amount of assembly code to test your feature?
https://reviews.llvm.org/D25234
More information about the llvm-commits
mailing list