Re: [Mesa-dev] [PATCH 11/28] util: added float to float16 conversions with RTZ and RTNE

2018-12-10 Thread Matt Turner
On Mon, Dec 10, 2018 at 6:21 AM Samuel Iglesias Gonsálvez wrote: > > Hello Matt, > > On 07/12/2018 03:20, Matt Turner wrote: > > Since this is for an extension that will be BDW+ can we use the > > _cvtss_sh() intrinsic instead? It corresponds to an IVB+ instruction > > and even takes the rounding

Re: [Mesa-dev] [PATCH 11/28] util: added float to float16 conversions with RTZ and RTNE

2018-12-10 Thread Samuel Iglesias Gonsálvez
Hello Matt, On 07/12/2018 03:20, Matt Turner wrote: > Since this is for an extension that will be BDW+ can we use the > _cvtss_sh() intrinsic instead? It corresponds to an IVB+ instruction > and even takes the rounding mode directly as an immediate argument. > If I understand currently this

Re: [Mesa-dev] [PATCH 11/28] util: added float to float16 conversions with RTZ and RTNE

2018-12-07 Thread Roland Scheidegger
Am 07.12.18 um 05:22 schrieb Matt Turner: > On Thu, Dec 6, 2018 at 7:22 PM Roland Scheidegger wrote: >> >> Am 07.12.18 um 03:20 schrieb Matt Turner: >>> Since this is for an extension that will be BDW+ can we use the >>> _cvtss_sh() intrinsic instead? It corresponds to an IVB+ instruction >>> and

Re: [Mesa-dev] [PATCH 11/28] util: added float to float16 conversions with RTZ and RTNE

2018-12-06 Thread Matt Turner
On Thu, Dec 6, 2018 at 7:22 PM Roland Scheidegger wrote: > > Am 07.12.18 um 03:20 schrieb Matt Turner: > > Since this is for an extension that will be BDW+ can we use the > > _cvtss_sh() intrinsic instead? It corresponds to an IVB+ instruction > > and even takes the rounding mode directly as an

Re: [Mesa-dev] [PATCH 11/28] util: added float to float16 conversions with RTZ and RTNE

2018-12-06 Thread Roland Scheidegger
Am 07.12.18 um 03:20 schrieb Matt Turner: > Since this is for an extension that will be BDW+ can we use the > _cvtss_sh() intrinsic instead? It corresponds to an IVB+ instruction > and even takes the rounding mode directly as an immediate argument. Not saying trying to use it isn't a good idea,

Re: [Mesa-dev] [PATCH 11/28] util: added float to float16 conversions with RTZ and RTNE

2018-12-06 Thread Matt Turner
Since this is for an extension that will be BDW+ can we use the _cvtss_sh() intrinsic instead? It corresponds to an IVB+ instruction and even takes the rounding mode directly as an immediate argument. ___ mesa-dev mailing list

[Mesa-dev] [PATCH 11/28] util: added float to float16 conversions with RTZ and RTNE

2018-12-05 Thread Samuel Iglesias Gonsálvez
Signed-off-by: Samuel Iglesias Gonsálvez --- src/util/half_float.c | 74 +++ src/util/half_float.h | 7 2 files changed, 81 insertions(+) diff --git a/src/util/half_float.c b/src/util/half_float.c index 63aec5c5c14..5fdcb20045b 100644 ---