Re: [Mesa-dev] [PATCH] spirv: Implement IsInf using an integer comparison

2017-03-19 Thread Dave Airlie
On 17 March 2017 at 14:18, Jason Ekstrand wrote: > Since we already do fabs on the one source, we're guaranteed to get > positive infinity if we get any infinity at all. Since +inf only has > one IEEE 754 representation, we can use an integer comparison and avoid > all of

Re: [Mesa-dev] [PATCH] spirv: Implement IsInf using an integer comparison

2017-03-17 Thread tournier.elie
Reviewed-by: Elie Tournier On 17 March 2017 at 04:18, Jason Ekstrand wrote: > Since we already do fabs on the one source, we're guaranteed to get > positive infinity if we get any infinity at all. Since +inf only has > one IEEE 754

[Mesa-dev] [PATCH] spirv: Implement IsInf using an integer comparison

2017-03-16 Thread Jason Ekstrand
Since we already do fabs on the one source, we're guaranteed to get positive infinity if we get any infinity at all. Since +inf only has one IEEE 754 representation, we can use an integer comparison and avoid all of the ordered/unordered issues. Cc: Dave Airlie ---