<div dir="ltr">Hi Tim,<div><br></div><div style>Doesn't look wrong, but could be like the others, repeating a big but having a section just for the long double. I mean, all of them repeat themselves a bit anyway, so unless you're ready to join all of them, I think joining two vaguely similar won't add much value to the code. ;)</div>
<div style><br></div><div style>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 7 January 2013 13:57, 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>
This patch adds support for printing fp128 values to assembly, necessary for<br>
AArch64 since the ABI defines "long double" to be IEEE-754 quad precision.<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>
I think 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>