On Thu, Nov 5, 2015 at 9:47 PM, Ryan Houdek <sonicadvan...@gmail.com> wrote:
> This implements support for a very new extension to ES, which allows you
> to do dual source blending just like in desktop OpenGL.
> This works with the piglits that I am writing that just need to be cleaned
> before they are pushed, and it also works fantastically with the
> Dolphin emulator (Only known application that uses this extension)

Well this series looks good to me... perhaps someone can glance at how
the outputs are defined for ES2... both use FRAG_RESULT_DATA0, which
is a little questionable. I'm pretty weak on all the details there.
But it's only for ESSL 1.0, where the other output will be
FRAG_RESULT_COLOR. Probably OK?

Ryan, piglits would be nice. In the meanwhile, this series is

Reviewed-by: Ilia Mirkin <imir...@alum.mit.edu>

>
> Ryan Houdek (8):
>   Add EXT_blend_func_extended XML definitions
>   Add required variables to _mesa_glsl_parse_state for
>     EXT_blend_func_extended
>   Add support for the new builtins that EXT_blend_func_extended
>     provides.
>   Add a parse check to check for the index layout qualifier
>   Enable usage of blend_func_extended blend factors
>   Allow MAX_DUAL_SOURCE_DRAW_BUFFERS to be available to ES
>   Add EXT_blend_func_extended functions to the dispatch_sanity test
>   Enable EXT_blend_func_extended if the driver supports the ARB version
>
>  src/glsl/ast_to_hir.cpp                 | 16 +++++++++
>  src/glsl/builtin_variables.cpp          | 62 
> +++++++++++++++++++++++++++++++++
>  src/glsl/glsl_parser.yy                 |  5 +++
>  src/glsl/glsl_parser_extras.cpp         |  3 ++
>  src/glsl/glsl_parser_extras.h           |  5 +++
>  src/mapi/glapi/gen/EXT_gpu_shader4.xml  |  3 +-
>  src/mapi/glapi/gen/es_EXT.xml           | 38 ++++++++++++++++++++
>  src/mesa/main/blend.c                   |  4 +--
>  src/mesa/main/extensions.c              |  1 +
>  src/mesa/main/get_hash_params.py        |  4 ++-
>  src/mesa/main/tests/dispatch_sanity.cpp |  8 +++++
>  11 files changed, 145 insertions(+), 4 deletions(-)
>
> --
> 2.5.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to