Re: [Mesa-dev] [PATCH 2/2] i915c: Use the actual MIN instruction.

2015-02-10 Thread Matt Turner
Both look like what I'd expect, FWIW. Thanks Ken! Reviewed-by: Matt Turner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 2/2] i915c: Use the actual MIN instruction.

2015-02-10 Thread Ian Romanick
On 02/10/2015 03:36 AM, Kenneth Graunke wrote: > Matt Turner noticed that the hardware has always had a MIN > instruction, but the driver always used MAX+MOV for no > apparent reason. And it has been right there in the docs the whole time too. WTF? > This should cut an instruction, and a tempora

[Mesa-dev] [PATCH 2/2] i915c: Use the actual MIN instruction.

2015-02-10 Thread Kenneth Graunke
Matt Turner noticed that the hardware has always had a MIN instruction, but the driver always used MAX+MOV for no apparent reason. This should cut an instruction, and a temporary, allowing more programs to run in hardware. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i915/i915_fragpr