On 22/11/16 21:53, 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.manol...@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;
           }

I wonder if the following check should include state-> ARB_post_depth_coverage_enable as condition before matching the layout qualifier.
Maybe Ilia can confirm?

Thanks!

+
+         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;
+
        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;
        break;
     }
     case MESA_SHADER_COMPUTE: {


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

Reply via email to