<div dir="ltr">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.</div><div class="gmail_extra"><br><br>

<div class="gmail_quote">On Wed, Oct 30, 2013 at 10:27 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

If it is easy to merge this test into some other file, please do so.<br>
<br>
By using "llvm-readobj -r" and making f global you can make the CHECK<br>
lines more specific by including the symbol name (f).<br>
<div><div class="h5"><br>
<br>
<br>
On 29 October 2013 19:42, Tom Roeder <<a href="mailto:tmroeder@google.com">tmroeder@google.com</a>> wrote:<br>
> This is a small patch to add some (but not all) of the x86-64 relocations<br>
> that are not currently supported in the ELF object writer. Please let me<br>
> know what you think.<br>
><br>
> Tom<br>
><br>
</div></div>> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
</blockquote></div><br></div>