[PATCH] D26508: [ELF] - Better diagnostic for relative relocation to an absolute value error.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 11 02:18:54 PST 2016


grimar added a comment.

In https://reviews.llvm.org/D26508#592123, @ruiu wrote:

> What is the previous output, and how does it change with this patch? An example would be helpful to understand.


So previous output would be just something like "relocation R_X86_64_PLT32 cannot refer to absolute symbol answer",
Body->File was empty so it was not possible to print symbol location definition.

I added missing argument to replaceBody (that was an issue of first diff) and also implemented Eugene's
suggestion, now messages seems to be much more helpfull (please see testcase).


https://reviews.llvm.org/D26508





More information about the llvm-commits mailing list