[Mesa-dev] [PATCH 2/3] i965/vs: Add a pass to set dependency control fields on instructions.

Kenneth Graunke kenneth at whitecape.org
Sun Mar 31 23:59:19 PDT 2013


On 03/12/2013 12:25 PM, Eric Anholt wrote:
> This is a more aggressive version of the old brw_optimize() path.
> Reduces cycles spent in the vertex shader on minecraft by 18.6% +/- 10.0%
> (n=15).
> ---
>   src/mesa/drivers/dri/i965/brw_vec4.cpp      |  109 +++++++++++++++++++++++++++
>   src/mesa/drivers/dri/i965/brw_vec4.h        |    2 +
>   src/mesa/drivers/dri/i965/brw_vec4_emit.cpp |   15 ++++
>   3 files changed, 126 insertions(+)

Okay, I flipped through the Destination Dependency Control section of 
the ISA docs and your implementation of this optimization pass looks 
good.  Really glad to see our driver finally do this.  Thanks Eric!

Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>


More information about the mesa-dev mailing list