[PATCH] D18190: [ELF] - Error out when R_X86_64_PC32/R_X86_64_32 are used against preemptible symbol when linking shared object.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 17 04:25:14 PDT 2016


grimar added a comment.

In http://reviews.llvm.org/D18190#376427, @ruiu wrote:

> Ah, gotcha. So this change would break the existing test for r259831 because it would now fail because of "recompile with -fPIC" error, so you had to update it.
>
> But you don't have to update the binary. Remove "-shared" from llvm33-rela-outside-group.s.


So after investigation and looking on Sean's comment I think my solution was OK. My solution avoids useless stuff like add of "_start" and so on. So I agree with Sean here.
At the same time you was right when saying that I was not convinced about this test changes. Now I am.


http://reviews.llvm.org/D18190





More information about the llvm-commits mailing list