[Beignet] [PATCH 6/7] replace mad with llvm intrinsic.

2015-03-09 Thread xionghu . luo
From: Luo Xionghu translate native mad to llvm.fma. Signed-off-by: Luo Xionghu --- backend/src/libocl/tmpl/ocl_math.tmpl.cl | 2 +- backend/src/llvm/llvm_gen_backend.cpp | 9 - backend/src/llvm/llvm_gen_ocl_function.hxx | 1 - 3 files changed, 1 insertion(+), 11 deletions(-) di

Re: [Beignet] [PATCH 6/7] replace mad with llvm intrinsic.

2015-03-10 Thread Matt Turner
On Mon, Mar 9, 2015 at 10:59 PM, wrote: > From: Luo Xionghu > > translate native mad to llvm.fma. I'm not sure that it matters for this patch, but do we know if Gen's MAD instruction is a fused-multiply-add? That is, does it not do an intermediate rounding step after the multiply? _

Re: [Beignet] [PATCH 6/7] replace mad with llvm intrinsic.

2015-03-10 Thread Song, Ruiling
> -Original Message- > From: Beignet [mailto:beignet-boun...@lists.freedesktop.org] On Behalf Of > Matt Turner > Sent: Wednesday, March 11, 2015 1:49 AM > To: Luo, Xionghu > Cc: beignet@lists.freedesktop.org > Subject: Re: [Beignet] [PATCH 6/7] replace mad with ll

Re: [Beignet] [PATCH 6/7] replace mad with llvm intrinsic.

2015-03-10 Thread Matt Turner
On Tue, Mar 10, 2015 at 6:55 PM, Song, Ruiling wrote: >> I'm not sure that it matters for this patch, but do we know if Gen's MAD >> instruction is a fused-multiply-add? That is, does it not do an intermediate >> rounding step after the multiply? > I also have such kind of concern, so I did a simp

Re: [Beignet] [PATCH 6/7] replace mad with llvm intrinsic.

2015-03-10 Thread Song, Ruiling
> -Original Message- > From: Matt Turner [mailto:matts...@gmail.com] > Sent: Wednesday, March 11, 2015 10:20 AM > To: Song, Ruiling > Cc: Luo, Xionghu; beignet@lists.freedesktop.org > Subject: Re: [Beignet] [PATCH 6/7] replace mad with llvm intrinsic. > > On Tue,

Re: [Beignet] [PATCH 6/7] replace mad with llvm intrinsic.

2015-03-10 Thread Song, Ruiling
replace mad with llvm intrinsic. > > > > > -Original Message- > > From: Matt Turner [mailto:matts...@gmail.com] > > Sent: Wednesday, March 11, 2015 10:20 AM > > To: Song, Ruiling > > Cc: Luo, Xionghu; beignet@lists.freedesktop.org > > Subject: Re:

Re: [Beignet] [PATCH 6/7] replace mad with llvm intrinsic.

2015-03-10 Thread Song, Ruiling
> -Original Message- > From: Matt Turner [mailto:matts...@gmail.com] > Sent: Wednesday, March 11, 2015 10:20 AM > To: Song, Ruiling > Cc: Luo, Xionghu; beignet@lists.freedesktop.org > Subject: Re: [Beignet] [PATCH 6/7] replace mad with llvm intrinsic. > > On Tue,