[PATCH] Add missing 64-bit relocation types to the x86-64 machine code

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed Oct 30 11:29:35 PDT 2013


LGTM

On 30 October 2013 14:23, Tom Roeder <tmroeder at google.com> wrote:
> Thanks for the comments. Here's a version of the patch that merges the test
> into test/MC/ELF/relocation.s, where all the related relocations seem to be
> getting tested.
>
>
> On Wed, Oct 30, 2013 at 10:27 AM, Rafael EspĂ­ndola
> <rafael.espindola at gmail.com> wrote:
>>
>> If it is easy to merge this test into some other file, please do so.
>>
>> By using "llvm-readobj -r" and making f global you can make the CHECK
>> lines more specific by including the symbol name (f).
>>
>>
>>
>> On 29 October 2013 19:42, Tom Roeder <tmroeder at google.com> wrote:
>> > This is a small patch to add some (but not all) of the x86-64
>> > relocations
>> > that are not currently supported in the ELF object writer. Please let me
>> > know what you think.
>> >
>> > Tom
>> >
>> > _______________________________________________
>> > llvm-commits mailing list
>> > llvm-commits at cs.uiuc.edu
>> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>> >
>
>




More information about the llvm-commits mailing list