<div dir="ltr">Duncan,<div><br></div><div>Attached is an updated patch with f64 tests added. f128 was not added as libffi does not seem to support this type (LLVM ERROR: Type could not be mapped for use with libffi.).</div>








<div><br></div><div>I agree it's odd that `lli` lowers libc & libm functions through a different interface than the native backend.</div><div><br></div><div>v/r,</div><div>Josh</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Mon, Aug 4, 2014 at 5:57 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Huh, I'm surprised that IntrinsicLowering is only used in the interpreter.<br>
I don't know `lli` too well, so anyone else should feel free to chime in<br>
here.<br>
<br>
I did notice that your test only checks the f32 versions -- I think it<br>
should exercise the full matrix.<br>
<div><div class="h5"><br>
> On 2014-Aug-04, at 14:38, Josh Klontz <<a href="mailto:josh.klontz@gmail.com">josh.klontz@gmail.com</a>> wrote:<br>
><br>
> Duncan,<br>
><br>
> Attached is an updated patch with a test case. Note, this required extending lit to recognize `LLVM_ENABLE_FFI`.<br>
><br>
> v/r,<br>
> Josh<br>
><br>
><br>
> On Mon, Aug 4, 2014 at 2:02 PM, Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com">dexonsmith@apple.com</a>> wrote:<br>
> Testcase?<br>
><br>
> > On 2014-Jul-30, at 05:00, Josh Klontz <<a href="mailto:josh.klontz@gmail.com">josh.klontz@gmail.com</a>> wrote:<br>
> ><br>
> > Ping. CC'ing code owner and release manager.<br>
> > -Josh<br>
> ><br>
> ><br>
> > On Wed, Jul 23, 2014 at 12:01 PM, Josh Klontz <<a href="mailto:josh.klontz@gmail.com">josh.klontz@gmail.com</a>> wrote:<br>
> > This was discovered when trying to run lli on a function with an llvm.cos.f32 intrinsic. Hopefully this can get applied to the 3.5 branch as well. Thanks!<br>
> > -Josh<br>
> ><br>
> > _______________________________________________<br>
> > llvm-commits mailing list<br>
> > <a href="mailto:llvm-commits@cs.uiuc.edu">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>
><br>
><br>
</div></div>> <patch.diff><br>
<br>
</blockquote></div><br></div>