On Tuesday, May 29, 2018 5:02:51 PM PDT Jason Ekstrand wrote:
> This is much cleaner than everything that wants a default value poking
> at the bits of p->current directly.
> ---
>  src/intel/compiler/brw_eu.c             | 29 ++++++++++
>  src/intel/compiler/brw_eu.h             |  3 +
>  src/intel/compiler/brw_eu_emit.c        | 98 
> +++++++++++++++------------------
>  src/intel/compiler/brw_fs_generator.cpp |  4 +-
>  4 files changed, 79 insertions(+), 55 deletions(-)

This is much nicer!

Please add
Cc: mesa-sta...@lists.freedesktop.org
to this patch too because it's required for the 2-3, which you CC'd to
stable.  That will help the release managers out a bit, I believe.

Series is:
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
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to