[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