I tested the patch and it does indeed fix the original bug (passing all local tests as well). Please go ahead and submit.<div><br></div><div>Thanks,</div><div>Steve<br><br><div class="gmail_quote">On Thu, Apr 5, 2012 at 8:07 AM, James Molloy <span dir="ltr"><<a href="mailto:james.molloy@arm.com">james.molloy@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 lang="EN-GB" link="blue" vlink="purple">

<div>

<p class="MsoNormal">Hi,<u></u><u></u></p>

<p class="MsoNormal"><u></u> <u></u></p>

<p class="MsoNormal">The attached patch fixes PR12274, an assertion failure
firing when an ARM NEON pattern is hit.<u></u><u></u></p>

<p class="MsoNormal"><u></u> <u></u></p>

<p class="MsoNormal">The bug is with code I submitted myself as part of the fix
for PR11836 and is trivial, so I don’t know if I could have just
committed without pre-review. Perhaps someone could advise about that for the
future?<u></u><u></u></p>

<p class="MsoNormal"><u></u> <u></u></p>

<p class="MsoNormal">The problem is the pattern creating an EXTRACT_SUBREG node
from a Qreg asking for a qsub_0, which I thought would be fine but apparently
is not (I didn’t realise a no-op/identity extract_subreg like that would
fail). My original testcases also failed to cover it <span style="font-family:Wingdings">L</span><u></u><u></u></p>

<p class="MsoNormal"><u></u> <u></u></p>

<p class="MsoNormal">OK to commit?<u></u><u></u></p>

<p class="MsoNormal"><u></u> <u></u></p>

<p class="MsoNormal">Cheers,<u></u><u></u></p>

<p class="MsoNormal"><u></u> <u></u></p>

<p class="MsoNormal">James<u></u><u></u></p>

</div>

</div>

<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>