<div dir="ltr">Looks good. I don't think there will be any problems, but you may wait for someone with more experience in the ELF class to give an opinion.<div><br></div><div style>--renato</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On 7 January 2013 14:58, Tim Northover <span dir="ltr"><<a href="mailto:Tim.Northover@arm.com" target="_blank">Tim.Northover@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi all,<br>
<br>
The ELF64 specification says that 32 bits of the r_info field are used for the<br>
relocation type; however the ELF support code currently uses "unsigned char"<br>
to store this type. Current targets work because they have <= 256 relocations,<br>
but AArch64 defines more (or, at least, spreads them out more).<br>
<br>
This patch switches the types used so that everything works properly.<br>
<br>
It's part of the change to generic LLVM code that we had to implement while<br>
developing the AArch64 backend. These are the changes that no other target<br>
exercises, so sending them to the list before would have been a little<br>
motive-less, and often untestable.<br>
<br>
This one is reasonably simple, OK to commit?<br>
<br>
Cheers.<br>
<br>
Tim.<br>
<br>
-- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.<br>
_______________________________________________<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>