<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 style>If not, LGTM.</div><div><br></div><div style>Thanks!</div><div style><br></div><div style>--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 class="im HOEnZb"><br>
On Feb 1, 2013, at 10:52 AM, Arnold Schwaighofer <<a href="mailto:aschwaighofer@apple.com">aschwaighofer@apple.com</a>> wrote:<br>
> On Feb 1, 2013, at 10:03 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:<br>
>><br>
</div><div class="HOEnZb"><div class="h5">>> 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></div></blockquote></div><br></div>