On Sat, 2017-01-07 at 11:13 -0800, Matt Turner wrote: > On Sat, Jan 7, 2017 at 3:58 AM, Timothy Arceri > <timothy.arc...@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@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev