[Mesa-dev] [PATCH 1/2] mesa: Add GL and GLSL plumbing for ARB_post_depth_coverage for i965 (gen9+).
Timothy Arceri
timothy.arceri at collabora.com
Thu Dec 1 03:16:02 UTC 2016
On Tue, 2016-11-22 at 23:53 +0200, Plamena Manolova wrote:
> This extension allows the fragment shader to control whether values
> in
> gl_SampleMaskIn[] reflect the coverage after application of the early
> depth and stencil tests.
>
> Signed-off-by: Plamena Manolova <plamena.manolova at intel.com>
> ---
> src/compiler/glsl/ast.h | 5 +++++
> src/compiler/glsl/ast_to_hir.cpp | 5 +++++
> src/compiler/glsl/ast_type.cpp | 8 +++++++-
> src/compiler/glsl/glsl_parser.yy | 11 +++++++++++
> src/compiler/glsl/glsl_parser_extras.cpp | 4 ++++
> src/compiler/glsl/glsl_parser_extras.h | 4 ++++
> src/compiler/glsl/linker.cpp | 4 ++++
> src/compiler/shader_info.h | 1 +
> src/mesa/main/extensions_table.h | 1 +
> src/mesa/main/mtypes.h | 2 ++
> src/mesa/main/shaderapi.c | 1 +
> 11 files changed, 45 insertions(+), 1 deletion(-)
>
> diff --git a/src/compiler/glsl/ast.h b/src/compiler/glsl/ast.h
> index 55f9a6c..ad19493 100644
> --- a/src/compiler/glsl/ast.h
> +++ b/src/compiler/glsl/ast.h
> @@ -606,6 +606,11 @@ struct ast_type_qualifier {
> /** \{ */
> unsigned blend_support:1; /**< Are there any blend_support_
> qualifiers */
> /** \} */
> +
> + /**
> + * Flag set if GL_ARB_post_depth_coverage layout qualifier
> is used.
> + */
> + unsigned post_depth_coverage:1;
> }
> /** \brief Set of flags, accessed by name. */
> q;
> diff --git a/src/compiler/glsl/ast_to_hir.cpp
> b/src/compiler/glsl/ast_to_hir.cpp
> index 9b8678c..c31da86 100644
> --- a/src/compiler/glsl/ast_to_hir.cpp
> +++ b/src/compiler/glsl/ast_to_hir.cpp
> @@ -3632,6 +3632,11 @@ apply_layout_qualifier_to_variable(const
> struct ast_type_qualifier *qual,
> _mesa_glsl_error(loc, state, "early_fragment_tests layout
> qualifier only "
> "valid in fragment shader input layout
> declaration.");
> }
> +
> + if (qual->flags.q.post_depth_coverage) {
> + _mesa_glsl_error(loc, state, "post_depth_coverage layout
> qualifier only "
> + "valid in fragment shader input layout
> declaration.");
> + }
> }
>
> static void
> diff --git a/src/compiler/glsl/ast_type.cpp
> b/src/compiler/glsl/ast_type.cpp
> index 2856f18..1905721 100644
> --- a/src/compiler/glsl/ast_type.cpp
> +++ b/src/compiler/glsl/ast_type.cpp
> @@ -489,6 +489,7 @@ ast_type_qualifier::merge_in_qualifier(YYLTYPE
> *loc,
> break;
> case MESA_SHADER_FRAGMENT:
> valid_in_mask.flags.q.early_fragment_tests = 1;
> + valid_in_mask.flags.q.post_depth_coverage = 1;
> break;
> case MESA_SHADER_COMPUTE:
> create_cs_ast |=
> @@ -540,6 +541,10 @@ ast_type_qualifier::merge_in_qualifier(YYLTYPE
> *loc,
> state->fs_early_fragment_tests = true;
> }
>
> + if (q.flags.q.post_depth_coverage) {
> + state->fs_post_depth_coverage = true;
> + }
> +
> if (this->flags.q.vertex_spacing) {
> if (q.flags.q.vertex_spacing &&
> this->vertex_spacing != q.vertex_spacing) {
> @@ -671,7 +676,8 @@ ast_type_qualifier::validate_flags(YYLTYPE *loc,
> bad.flags.q.point_mode ? " point_mode" : "",
> bad.flags.q.vertices ? " vertices" : "",
> bad.flags.q.subroutine ? " subroutine" : "",
> - bad.flags.q.subroutine_def ? " subroutine_def" :
> "");
> + bad.flags.q.subroutine_def ? " subroutine_def" :
> "",
> + bad.flags.q.post_depth_coverage ? "
> post_depth_coverage" : "");
> return false;
> }
>
> diff --git a/src/compiler/glsl/glsl_parser.yy
> b/src/compiler/glsl/glsl_parser.yy
> index a48dc68..a53f476 100644
> --- a/src/compiler/glsl/glsl_parser.yy
> +++ b/src/compiler/glsl/glsl_parser.yy
> @@ -1373,6 +1373,17 @@ layout_qualifier_id:
>
> $$.flags.q.early_fragment_tests = 1;
> }
> +
> + if (!$$.flags.i &&
> + match_layout_qualifier($1, "post_depth_coverage",
> state) == 0) {
> + if (state->stage != MESA_SHADER_FRAGMENT) {
> + _mesa_glsl_error(& @1, state,
> + "post_depth_coverage layout
> qualifier only "
> + "valid in fragment shaders");
> + }
> +
> + $$.flags.q.post_depth_coverage = 1;
> + }
> }
>
> /* Layout qualifiers for tessellation evaluation shaders. */
> diff --git a/src/compiler/glsl/glsl_parser_extras.cpp
> b/src/compiler/glsl/glsl_parser_extras.cpp
> index 85a2e94..bc252a0 100644
> --- a/src/compiler/glsl/glsl_parser_extras.cpp
> +++ b/src/compiler/glsl/glsl_parser_extras.cpp
> @@ -295,6 +295,7 @@
> _mesa_glsl_parse_state::_mesa_glsl_parse_state(struct gl_context
> *_ctx,
> this->in_qualifier = new(this) ast_type_qualifier();
> this->out_qualifier = new(this) ast_type_qualifier();
> this->fs_early_fragment_tests = false;
> + this->fs_post_depth_coverage = false;
> this->fs_blend_support = 0;
> memset(this->atomic_counter_offsets, 0,
> sizeof(this->atomic_counter_offsets));
> @@ -608,6 +609,7 @@ static const _mesa_glsl_extension
> _mesa_glsl_supported_extensions[] = {
> EXT(ARB_fragment_layer_viewport),
> EXT(ARB_gpu_shader5),
> EXT(ARB_gpu_shader_fp64),
> + EXT(ARB_post_depth_coverage),
> EXT(ARB_sample_shading),
> EXT(ARB_separate_shader_objects),
> EXT(ARB_shader_atomic_counter_ops),
> @@ -1692,6 +1694,7 @@ set_shader_inout_layout(struct gl_shader
> *shader,
> assert(!state->fs_pixel_center_integer);
> assert(!state->fs_origin_upper_left);
> assert(!state->fs_early_fragment_tests);
> + assert(!state->fs_post_depth_coverage);
> }
>
> for (unsigned i = 0; i < MAX_FEEDBACK_BUFFERS; i++) {
> @@ -1812,6 +1815,7 @@ set_shader_inout_layout(struct gl_shader
> *shader,
> shader->info.ARB_fragment_coord_conventions_enable =
> state->ARB_fragment_coord_conventions_enable;
> shader->info.EarlyFragmentTests = state-
> >fs_early_fragment_tests;
> + shader->info.PostDepthCoverage = state-
> >fs_post_depth_coverage;
> shader->info.BlendSupport = state->fs_blend_support;
> break;
>
> diff --git a/src/compiler/glsl/glsl_parser_extras.h
> b/src/compiler/glsl/glsl_parser_extras.h
> index d757c1d..4277d43 100644
> --- a/src/compiler/glsl/glsl_parser_extras.h
> +++ b/src/compiler/glsl/glsl_parser_extras.h
> @@ -610,6 +610,8 @@ struct _mesa_glsl_parse_state {
> bool ARB_gpu_shader5_warn;
> bool ARB_gpu_shader_fp64_enable;
> bool ARB_gpu_shader_fp64_warn;
> + bool ARB_post_depth_coverage_enable;
> + bool ARB_post_depth_coverage_warn;
> bool ARB_sample_shading_enable;
> bool ARB_sample_shading_warn;
> bool ARB_separate_shader_objects_enable;
> @@ -786,6 +788,8 @@ struct _mesa_glsl_parse_state {
>
> bool fs_early_fragment_tests;
>
> + bool fs_post_depth_coverage;
> +
> unsigned fs_blend_support;
>
> /**
> diff --git a/src/compiler/glsl/linker.cpp
> b/src/compiler/glsl/linker.cpp
> index 1a00a90..d127d0c 100644
> --- a/src/compiler/glsl/linker.cpp
> +++ b/src/compiler/glsl/linker.cpp
> @@ -1887,6 +1887,10 @@ link_fs_inout_layout_qualifiers(struct
> gl_shader_program *prog,
>
> linked_shader->info.EarlyFragmentTests |=
> shader->info.EarlyFragmentTests;
> +
> + linked_shader->info.PostDepthCoverage |=
> + shader->info.PostDepthCoverage;
> +
You should be able to just set this directly in shader_info after
recent changes. For example:
linked_shader->Program->info.fs.post_depth_coverage |=
shader->info.PostDepthCoverage;
> linked_shader->info.BlendSupport |= shader->info.BlendSupport;
> }
> }
> diff --git a/src/compiler/shader_info.h b/src/compiler/shader_info.h
> index 7ea5d9c..b2830e0 100644
> --- a/src/compiler/shader_info.h
> +++ b/src/compiler/shader_info.h
> @@ -116,6 +116,7 @@ typedef struct shader_info {
> * ARB_shader_image_load_store.
> */
> bool early_fragment_tests;
> + bool post_depth_coverage;
>
> /** gl_FragDepth layout for ARB_conservative_depth. */
> enum gl_frag_depth_layout depth_layout;
> diff --git a/src/mesa/main/extensions_table.h
> b/src/mesa/main/extensions_table.h
> index d3ec551..f2d3a5b 100644
> --- a/src/mesa/main/extensions_table.h
> +++ b/src/mesa/main/extensions_table.h
> @@ -93,6 +93,7 @@ EXT(ARB_pipeline_statistics_query ,
> ARB_pipeline_statistics_query
> EXT(ARB_pixel_buffer_object ,
> EXT_pixel_buffer_object , GLL, GLC, x , x , 2004)
> EXT(ARB_point_parameters ,
> EXT_point_parameters , GLL, x , x , x , 1997)
> EXT(ARB_point_sprite ,
> ARB_point_sprite , GLL, GLC, x , x , 2003)
> +EXT(ARB_post_depth_coverage ,
> ARB_post_depth_coverage , x , 32, x , x, 2015)
> EXT(ARB_program_interface_query ,
> dummy_true , GLL, GLC, x , x , 2012)
> EXT(ARB_provoking_vertex ,
> EXT_provoking_vertex , GLL, GLC, x , x , 2009)
> EXT(ARB_query_buffer_object ,
> ARB_query_buffer_object , GLL, GLC, x , x , 2013)
> diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
> index 26b1965..9d50304 100644
> --- a/src/mesa/main/mtypes.h
> +++ b/src/mesa/main/mtypes.h
> @@ -2174,6 +2174,7 @@ struct gl_shader_info
> {
> bool uses_gl_fragcoord;
> bool redeclares_gl_fragcoord;
> + bool PostDepthCoverage;
> bool ARB_fragment_coord_conventions_enable;
>
> /**
> @@ -3815,6 +3816,7 @@ struct gl_extensions
> GLboolean ARB_occlusion_query2;
> GLboolean ARB_pipeline_statistics_query;
> GLboolean ARB_point_sprite;
> + GLboolean ARB_post_depth_coverage;
> GLboolean ARB_query_buffer_object;
> GLboolean ARB_robust_buffer_access_behavior;
> GLboolean ARB_sample_shading;
> diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
> index 83ee0d4..33e4334 100644
> --- a/src/mesa/main/shaderapi.c
> +++ b/src/mesa/main/shaderapi.c
> @@ -2194,6 +2194,7 @@ _mesa_copy_linked_program_data(const struct
> gl_shader_program *src,
> case MESA_SHADER_FRAGMENT: {
> dst->info.fs.depth_layout = src->FragDepthLayout;
> dst->info.fs.early_fragment_tests = dst_sh-
> >info.EarlyFragmentTests;
> + dst->info.fs.post_depth_coverage = dst_sh-
> >info.PostDepthCoverage;
You won't need this is you follow my suggestion above.
> break;
> }
> case MESA_SHADER_COMPUTE: {
More information about the mesa-dev
mailing list