[PATCH] D33824: [ELF] - Do not merge relocation sections by name when using --emit-relocs.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 5 22:42:54 PDT 2017


grimar added inline comments.


================
Comment at: test/ELF/emit-relocs-merge.test:1
+# RUN: yaml2obj %s -o %t.o
+# RUN: ld.lld %t.o -o %t --emit-relocs
----------------
ruiu wrote:
> grimar wrote:
> > ruiu wrote:
> > > Can you create this using llvm-mc?
> > I do not think so. Idea here was to create relocation section that does not have name starting from ".rel" or ".rela".
> > I do not think llvm-mc allows to do that.
> But you can test this if you use a linker script, no? You are using` .zed : { *(.foo) *(.bar) }` as an example.
I sure can, but linkerscript case (a bit outdated) is a D33780. I am about to update this patch
to support linkerscript case, add testcase from D33780 with Rafael's comments addressed and remove
this yaml test then.


https://reviews.llvm.org/D33824





More information about the llvm-commits mailing list