Re: [Mesa-dev] [PATCH 14/22] nir: Replace fmin(b2f(a), b) with a bcsel

2018-02-27 Thread Samuel Iglesias Gonsálvez
Reviewed-by: Samuel Iglesias Gonsálvez On 24/02/18 00:55, Ian Romanick wrote: > From: Ian Romanick > > All of the affected shaders are HDR mappers from Serious Sam 3. > > All Gen7+ platforms had similar results. (Skylake shown) > total instructions in shared programs: 14516285 -> 14516273 (<.01

[Mesa-dev] [PATCH 14/22] nir: Replace fmin(b2f(a), b) with a bcsel

2018-02-23 Thread Ian Romanick
From: Ian Romanick All of the affected shaders are HDR mappers from Serious Sam 3. All Gen7+ platforms had similar results. (Skylake shown) total instructions in shared programs: 14516285 -> 14516273 (<.01%) instructions in affected programs: 348 -> 336 (-3.45%) helped: 12 HURT: 0 helped stats (