<div class="gmail_quote">On 22 June 2011 01:26, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@arm.com">renato.golin@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im">On 22 June 2011 00:19, Nick Lewycky <<a href="mailto:nlewycky@google.com">nlewycky@google.com</a>> wrote:<br>
> Hi Renato! Sorry, but I don't understand your reply at all.<br>
<br>
</div>I think you did, and answered my question at the same time. ;)<br>
<div class="im"><br>
<br>
> There's nothing ARM-specific about this change, which is why it belongs in<br>
> generic CodeGen and shouldn't be in the ARMAsmPrinter. It just so happens<br>
> that <3 x float> on ARM provides a counter-example that would require this<br>
> padding or trigger this assertion.<br>
<br>
</div>I was fooled by your comment on ARM's vector size and thought your<br>
patch was in the wrong place. As you say (and Eli has shown), this is<br>
not platform specific, so the patch is good.<br>
<br>
Sorry about the confusion.<br></blockquote><div><br></div><div>No problem, thanks for the review!</div><div><br></div><div>Nick</div><div><br></div></div>