[Beignet] [PATCH] enable FP_CONTRACT on as default, and implemented with MAD

Song, Ruiling ruiling.song at intel.com
Wed Mar 2 08:22:07 UTC 2016


> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> Guo Yejun
> Sent: Monday, February 29, 2016 8:08 AM
> To: beignet at lists.freedesktop.org
> Cc: Guo, Yejun <yejun.guo at intel.com>
> Subject: [Beignet] [PATCH] enable FP_CONTRACT on as default, and
> implemented with MAD
> 
> According to OpenCL spec, FP_CONTRACT is on as default, while LLVM/clang
> just enabled it at http://reviews.llvm.org/D14200 at Nov 2015. So we still
> need set this option explicitly now.
> 
> The implementation is hardware MAD instruction whose accuracy is enough
> for FP_CONTRACT.
> 
> Passed test:  contractions of conformance test
> 
> Signed-off-by: Guo Yejun <yejun.guo at intel.com>

The solution looks good. LGTM

Thanks!
Ruiling


More information about the Beignet mailing list