<div dir="ltr"><div><div><div><div>Thanks! The XFAIL jab was just a joke attempt. The C intrinsic tests are absolutely needed.<br><br></div>I've added a comment to:<br><a href="https://llvm.org/bugs/show_bug.cgi?id=26815">https://llvm.org/bugs/show_bug.cgi?id=26815</a><br><br></div>To give more context on a slightly different concern - the mapping of intrinsics to asm at -O0.<br><br></div>For the sake of progress on this particular patch, are you ok if I make a change similar to:<br><a href="http://reviews.llvm.org/rL256395" target="_blank">http://reviews.llvm.org/rL256395</a><br>?<br></div>I can post that patch for review before committing to make sure it hasn't done something awful to ARM64.<br><div><div><div><br></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 2, 2016 at 9:50 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 2 March 2016 at 15:54, Sanjay Patel <<a href="mailto:spatel@rotateright.com">spatel@rotateright.com</a>> wrote:<br>
> Thanks for replying. I was going to ping the AArch64 experts about this.<br>
> There were 7 tests that broke with my patch in:<br>
> clang/test/CodeGen/aarch64-neon-misc.c<br>
<br>
</span>Right, this makes things easier, since we only need to change one test for now.<br>
<span class=""><br>
<br>
> This is at least the 3rd time I've had a clang test fail on me after making<br>
> an LLVM optimizer change. So I admit that I wanted to just XFAIL the whole<br>
> thing, but I wouldn't do that. :)<br>
<br>
</span>Please, don't. Bad valid tests are better than no tests. If they were<br>
useless, however, I'd just remove.<br>
<span class=""><br>
<br>
> So unless something has changed, I think we need to split each test in<br>
> aarch64-neon-misc.c (and other files like it) into pieces:<br>
> 1. A clang test for the C-level intrinsics -> LLVM intrinsics and/or native IR.<br>
> 2. A codegen test for LLVM intrinsic/IR -> asm.<br>
<br>
</span>I absolutely agree, but I don't want this to become a witch hunt. I<br>
think we (ARM folks) really need to get together and come up with a<br>
plan.<br>
<br>
That's what PR26815 is all about. I've copied some people, feel free<br>
to copy even more. We need to prioritise this once and for all.<br>
<br>
cheers,<br>
--renato<br>
</blockquote></div><br></div>