[PATCH] D30889: [ELF] Fix error reporting for eh_frame and other synthetic sections

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 13 06:01:20 PDT 2017


grimar added a comment.

Looks testcase can be simplified a bit.



================
Comment at: test/ELF/linkerscript/eh-frame-reloc-out-of-range.s:6
+# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o %t.o
+# RUN: echo "PHDRS { eh PT_LOAD; text PT_LOAD; }  \
+# RUN:       SECTIONS { . = 0x10000; \
----------------
Do you need PHDRS for this testcase ?


================
Comment at: test/ELF/linkerscript/eh-frame-reloc-out-of-range.s:8
+# RUN:       SECTIONS { . = 0x10000; \
+# RUN:         .eh_frame_hdr : { *(.eh_frame_hdr*) } : eh \
+# RUN:         .eh_frame : { *(.eh_frame) } : eh \
----------------
Do you need to place .eh_frame_hdr explicitly ?


Repository:
  rL LLVM

https://reviews.llvm.org/D30889





More information about the llvm-commits mailing list