<br><br><div class="gmail_quote">On Thu, Feb 9, 2012 at 1:19 AM, Duncan Sands <span dir="ltr"><<a href="mailto:baldrick@free.fr" target="_blank">baldrick@free.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hi Craig,<div><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    I see that you removed all clang support for these builtins, rather than having<br>
    clang turn the builtin into a shufflevector instruction (which is what I thought<br>
    you did).  Is that wise?  You just reduced GCC compatibility, when you could<br>
    have kept it by expanding the builtins into generic IR.<br>
<br>
    Ciao, Duncan.<br>
<br>
<br>
That's correct, but what I did is no different than what is already done for<br>
pshufd, shufps/pd, punpckl*, punpckh*, unpcklps/pd, unpckhps/pd, pshuflw,<br>
pshufhw, etc.<br>
</blockquote>
<br></div>
was it really a good idea to remove them?  You could have had clang support them<br>
at little cost, maintaining GCC compatibility.<br>
<br></blockquote><div><br>I only removed builtins for a couple avx/avx2 instruction, vpermil* and vperm2(f/i)128. pshufd and the others already didn't exist before I got here.<br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


Ciao, Duncan.<br>
</blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>