On Friday, February 27, 2015 10:22:08 AM Ben Widawsky wrote:
> I'm not really sure of the origins of the existing flag names. Modern docs 
> have
> some slightly different names. Having the correct names makes it easier to
> determine if existing PIPE_CONTROL flag settings are correct, as well as 
> making
> adding new PIPE_CONTROLs easier.
> 
> This originally came up while I was trying to implement workarounds and 
> spotted
> some things called, "flush" which should have been called "invalidate."
> 
> Signed-off-by: Ben Widawsky <b...@bwidawsk.net>
> ---
>  src/mesa/drivers/dri/i965/brw_program.c       | 10 +++++-----
>  src/mesa/drivers/dri/i965/gen6_vs_state.c     |  2 +-
>  src/mesa/drivers/dri/i965/gen8_depth_state.c  |  2 +-
>  src/mesa/drivers/dri/i965/intel_batchbuffer.c |  8 ++++----
>  src/mesa/drivers/dri/i965/intel_reg.h         |  6 +++---
>  5 files changed, 14 insertions(+), 14 deletions(-)

Thanks, this is much nicer.

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to