[Mesa-dev] [PATCH 10/10] nir: add late opt to turn inot/b2f combos back to bcsel

Timothy Arceri timothy.arceri at collabora.com
Tue Jan 10 09:26:03 UTC 2017


On Sat, 2017-01-07 at 11:13 -0800, Matt Turner wrote:
> On Sat, Jan 7, 2017 at 3:58 AM, Timothy Arceri
> <timothy.arceri at collabora.com> wrote:
> > We turn these from bcsel into inot/b2f combos in order for other
> > optimisation passes to get further, once we have finished turn
> > the ones that remain and are used in more than a single expression
> > back into a bcsel.
> 
> Run-on sentence.
> 
> > 
> > On BDW:
> > 
> > total instructions in shared programs: 13060965 -> 13060297 (-
> > 0.01%)
> > instructions in affected programs: 835701 -> 835033 (-0.08%)
> > helped: 670
> > HURT: 2
> 
> What happened in the two hurt cases?

Adding bcsel back in creates the only use of the constant 1.0 in the
whole shader.

> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list