[PATCH] D15712: [ELF] - implemented --eh-frame-hdr command line option.
George Rimar via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 14 02:34:47 PST 2016
grimar added inline comments.
================
Comment at: ELF/OutputSections.cpp:1051
@@ +1050,3 @@
+ typedef typename ELFFile<ELFT>::uintX_t uintX_t;
+ unsigned format = Enc & 0x0f;
+ switch (format) {
----------------
ruiu wrote:
> format -> Format
Replaced with
```
switch (Enc & 0x0f) {
```
================
Comment at: ELF/OutputSections.h:481
@@ -443,2 +480,3 @@
+ static EhFrameHdr<ELFT> *EhFrameHdr;
static GnuHashTableSection<ELFT> *GnuHashTab;
static GotPltSection<ELFT> *GotPlt;
----------------
I renamed class EhFrameHdr to EhFrameHeader because of compile error under linux here.
Repository:
rL LLVM
http://reviews.llvm.org/D15712
More information about the llvm-commits
mailing list