diff options
author | Rob Clark <[email protected]> | 2018-06-01 14:07:15 -0400 |
---|---|---|
committer | Rob Clark <[email protected]> | 2018-07-18 10:10:44 -0400 |
commit | 8dfc9e22c12ff840510d876b09d7f7163256bb17 (patch) | |
tree | 2f4e00e1bc0029bbb4da8c7095fbcf099914812a /src/compiler | |
parent | 09f240eb5ff6c24471bd55770771b7755ad07bc0 (diff) |
nir: add lowering for gl_HelperInvocation
v2: reword comment about lower_helper_invocations to be more clear
that it might not work on all hardware
v3: add special variant of load_sample_id which does not imply per-
sample shading
Signed-off-by: Rob Clark <[email protected]>
Diffstat (limited to 'src/compiler')
-rw-r--r-- | src/compiler/nir/nir.h | 14 | ||||
-rw-r--r-- | src/compiler/nir/nir_intrinsics.py | 3 | ||||
-rw-r--r-- | src/compiler/nir/nir_lower_system_values.c | 17 |
3 files changed, 34 insertions, 0 deletions
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h index d3e63be091f..3bfe7d7f7bf 100644 --- a/src/compiler/nir/nir.h +++ b/src/compiler/nir/nir.h @@ -2006,6 +2006,20 @@ typedef struct nir_shader_compiler_options { */ bool lower_base_vertex; + /** + * If enabled, gl_HelperInvocation will be lowered as: + * + * !((1 << sample_id) & sample_mask_in)) + * + * This depends on some possibly hw implementation details, which may + * not be true for all hw. In particular that the FS is only executed + * for covered samples or for helper invocations. So, do not blindly + * enable this option. + * + * Note: See also issue #22 in ARB_shader_image_load_store + */ + bool lower_helper_invocation; + bool lower_cs_local_index_from_id; bool lower_device_index_to_zero; diff --git a/src/compiler/nir/nir_intrinsics.py b/src/compiler/nir/nir_intrinsics.py index eaa40e95acc..d688a57181b 100644 --- a/src/compiler/nir/nir_intrinsics.py +++ b/src/compiler/nir/nir_intrinsics.py @@ -424,6 +424,9 @@ system_value("instance_id", 1) system_value("base_instance", 1) system_value("draw_id", 1) system_value("sample_id", 1) +# sample_id_no_per_sample is like sample_id but does not imply per- +# sample shading. See the lower_helper_invocation option. +system_value("sample_id_no_per_sample", 1) system_value("sample_pos", 2) system_value("sample_mask_in", 1) system_value("primitive_id", 1) diff --git a/src/compiler/nir/nir_lower_system_values.c b/src/compiler/nir/nir_lower_system_values.c index da04895d66c..2820dcd1b3c 100644 --- a/src/compiler/nir/nir_lower_system_values.c +++ b/src/compiler/nir/nir_lower_system_values.c @@ -144,6 +144,23 @@ convert_block(nir_block *block, nir_builder *b) nir_load_first_vertex(b)); break; + case SYSTEM_VALUE_HELPER_INVOCATION: + if (b->shader->options->lower_helper_invocation) { + nir_ssa_def *tmp; + + tmp = nir_ishl(b, + nir_imm_int(b, 1), + nir_load_sample_id_no_per_sample(b)); + + tmp = nir_iand(b, + nir_load_sample_mask_in(b), + tmp); + + sysval = nir_inot(b, nir_i2b(b, tmp)); + } + + break; + case SYSTEM_VALUE_INSTANCE_INDEX: sysval = nir_iadd(b, nir_load_instance_id(b), |