<div dir="ltr">Thanks! ;)</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 5 February 2013 14:06, Arnold Schwaighofer <span dir="ltr"><<a href="mailto:aschwaighofer@apple.com" target="_blank">aschwaighofer@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">r174371<div><br><div><div><div class="h5"><div>On Feb 5, 2013, at 7:33 AM, Arnold <<a href="mailto:aschwaighofer@apple.com" target="_blank">aschwaighofer@apple.com</a>> wrote:</div>
<br></div></div><blockquote type="cite"><div><div class="h5"><div dir="auto"><div>No not yet. I am going to today.<br></div><div><br>On Feb 5, 2013, at 5:21 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>> wrote:<br>
<br></div><blockquote type="cite"><div dir="ltr">Hi Arnold,<div><br></div><div>I can't find this change in the main tree, have you committed it?</div><div><br></div><div>cheers,</div><div>--renato</div></div><div class="gmail_extra">

<br><br><div class="gmail_quote">On 1 February 2013 20:02, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div style="word-wrap:break-word"><div>I don't have any preferences. LGTM. </div><div><div><br><div><div>On Feb 1, 2013, at 11:49 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>> wrote:</div>

<br><blockquote type="cite"><div dir="ltr">Hi Arnold,<div><br></div><div>Looks a lot better. The x86 function has multiple tables first, than the IF blocks, which is slightly different style, if anything. I don't mind either way, maybe Nadav has some preference.</div>


<div><br></div><div>If not, LGTM.</div><div><br></div><div>Thanks!</div><div><br></div><div>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 1 February 2013 19:05, Arnold Schwaighofer <span dir="ltr"><<a href="mailto:aschwaighofer@apple.com" target="_blank">aschwaighofer@apple.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I created four tables:<br>
<br>
- NEONVectorConversionTbl for vector<br>
- NEONFloatConversionTbl for scalar floating point<br>
- NEONIntegerConversionTbl for scalar int to floating point<br>
- ARMIntegerConversionTbl for scalar int<br>
<br>
Thanks,<br>
Arnold<br>
<div><br>
On Feb 1, 2013, at 10:52 AM, Arnold Schwaighofer <<a href="mailto:aschwaighofer@apple.com" target="_blank">aschwaighofer@apple.com</a>> wrote:<br>
> On Feb 1, 2013, at 10:03 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>> wrote:<br>
>><br>
</div><div>>> However, the table is called NEONConversionTbl and there's a test below (if ST->hasNEON), so it'd be better if the non-vector instructions could be added to a separate table.<br>

>><br>
>> Possibly adding an Ty->isVector() && ST->hasNEON() or some combination that allows for both vectorized and scalar codes to execute correctly on all possible combinations, but doesn't iterate through the wrong table when the type is obviously not there.<br>



><br>
> Yes, you are absolutely right. Will fix this. I was so focused on the table that I forgot about the surrounding code. Also better for performance (makes for shorter linear searches).<br>
<br>
<br>
</div></blockquote></div><br></div>
</blockquote></div><br></div></div></div>
</blockquote></div><br></div>
</blockquote></div></div></div>
_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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>
</blockquote></div><br></div></div>
</blockquote></div><br></div>