[PATCH][CodeGen] Add missing intrinsic lowering for sin, cos and ceil
Duncan P. N. Exon Smith
dexonsmith at apple.com
Mon Aug 4 14:57:33 PDT 2014
Huh, I'm surprised that IntrinsicLowering is only used in the interpreter.
I don't know `lli` too well, so anyone else should feel free to chime in
here.
I did notice that your test only checks the f32 versions -- I think it
should exercise the full matrix.
> On 2014-Aug-04, at 14:38, Josh Klontz <josh.klontz at gmail.com> wrote:
>
> Duncan,
>
> Attached is an updated patch with a test case. Note, this required extending lit to recognize `LLVM_ENABLE_FFI`.
>
> v/r,
> Josh
>
>
> On Mon, Aug 4, 2014 at 2:02 PM, Duncan P. N. Exon Smith <dexonsmith at apple.com> wrote:
> Testcase?
>
> > On 2014-Jul-30, at 05:00, Josh Klontz <josh.klontz at gmail.com> wrote:
> >
> > Ping. CC'ing code owner and release manager.
> > -Josh
> >
> >
> > On Wed, Jul 23, 2014 at 12:01 PM, Josh Klontz <josh.klontz at gmail.com> wrote:
> > 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!
> > -Josh
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
> <patch.diff>
More information about the llvm-commits
mailing list