<div dir="ltr">







<p class="">Committed revision 215209.</p></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 5, 2014 at 9:06 AM, Josh Klontz <span dir="ltr"><<a href="mailto:josh.klontz@gmail.com" target="_blank">josh.klontz@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Pinging Evan</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">
On Mon, Aug 4, 2014 at 8:51 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">This LGTM.  You'll have to ping the code owner to get it into<br>
the 3.5 branch though.<br>
<div><div><br>
> On 2014-Aug-04, at 17:40, Josh Klontz <<a href="mailto:josh.klontz@gmail.com" target="_blank">josh.klontz@gmail.com</a>> wrote:<br>
><br>
> Duncan,<br>
><br>
> 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.).<br>
><br>
> I agree it's odd that `lli` lowers libc & libm functions through a different interface than the native backend.<br>
><br>
> v/r,<br>
> Josh<br>
><br>
><br>
> On Mon, Aug 4, 2014 at 5:57 PM, Duncan P. N. Exon Smith <<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>> wrote:<br>
> 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>
><br>
> > On 2014-Aug-04, at 14:38, Josh Klontz <<a href="mailto:josh.klontz@gmail.com" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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" 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>
> ><br>
> ><br>
> > <patch.diff><br>
><br>
><br>
</div></div>> <patch.diff><br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>