[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