It's being tested now. But it's only trying for the presence of the new expansion, not that the old expansion isn't there. I'll add CHECK-NOT for the old expansion and move the comment there.<div><br></div>
<div>Is that what you're saying?</div><div><br></div><div> Filipe<span></span><br><div><br>On Monday, May 12, 2014, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 12 May 2014 22:58, Filipe Cabecinhas <<a href="javascript:;" onclick="_e(event, 'cvml', 'me@filcab.net')">me@filcab.net</a>> wrote:<br>
> Ah, so I didn't fully understand then. Sorry about that.<br>
><br>
> But right now the only test that tests those functions is builtins-x86.c,<br>
> but that test never runs FileCheck. Shall I put the comment there?<br>
<br>
Well, when we implement this optimization it will probably be tested<br>
in test/CodeGen/avx-builtins.c, right?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div></div>