<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, May 17, 2015 at 1:26 PM, Andy Gibbs <span dir="ltr"><<a href="mailto:andyg1001@hotmail.co.uk" target="_blank">andyg1001@hotmail.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Reid,<br>
<br>
I'm forwarding this onto you since you are the original committer for r208733<br>
which I wish to partially revert.  The patch is attached and full(er) details<br>
can be found at the bug report (link below).  Sorry for sending it to two<br>
email addresses, but I don't know which is your active one.<br>
<br>
I think you simply factored out code block too many, where (if I understand<br>
your intentions correctly) you were factoring out the common code for<br>
*ABI*::classifyReturnType but unintentionally included<br>
DefaultABIInfo::classifyArgumentType, breaking ppc32 code generation.<br>
<br>
Please can you review my patch to be sure I haven't done anything stupid<br>
before I commit it.<br></blockquote><div><br></div><div>Yeah, it looks like the refactor went awry. I'm concerned that PPC32 is relying on DefaultABIInfo and that no test caught this. I'll go ahead and apply that, thanks for the patch!</div></div></div></div>