fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-07 17:58:26 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

spirv: replace discard with demote for incorrect HLSL->SPIR-V translations

Browse source 
Fixes artifacts on decals in Path of Exile.

Signed-off-by: Rhys Perry <pendingchaos02@gmail.com>
Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/3610
Cc: mesa-stable
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7062>
(cherry picked from commit 037d9fb278)
This commit is contained in:
Rhys Perry 2020-10-07 20:07:42 +01:00 • committed by Dylan Baker
parent 732b28c8b1
commit 8c2ad9f9cb
4 changed files with 31 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
.pick_status.json
View file

@ -5332,7 +5332,7 @@
"description": "spirv: replace discard with demote for incorrect HLSL->SPIR-V translations",
"nominated": true,
"nomination_type": 0,
"resolution": 0,
"resolution": 1,
"master_sha": null,
"because_sha": null
},

21
src/compiler/spirv/spirv_to_nir.c
View file

@ -4086,6 +4086,8 @@ vtn_handle_preamble_instruction(struct vtn_builder *b, SpvOp opcode,
(count > 3) ? vtn_value(b, w[3], vtn_value_type_string)->str : "";
vtn_info("Parsing SPIR-V from %s %u source file %s", lang, version, file);
b->source_lang = w[1];
break;
}
@ -4340,6 +4342,7 @@ vtn_handle_preamble_instruction(struct vtn_builder *b, SpvOp opcode,
case SpvCapabilityDemoteToHelperInvocationEXT:
spv_check_supported(demote_to_helper_invocation, cap);
b->uses_demote_to_helper_invocation = true;
break;
case SpvCapabilityShaderClockKHR:
@ -5423,7 +5426,7 @@ vtn_create_builder(const uint32_t *words, size_t word_count,
goto fail;
}
uint16_t generator_id = words[2] >> 16;
b->generator_id = words[2] >> 16;
uint16_t generator_version = words[2];
/* In GLSLang commit 8297936dd6eb3, their handling of barrier() was fixed
@ -5432,7 +5435,7 @@ vtn_create_builder(const uint32_t *words, size_t word_count,
* GLSLang fix caused them to bump to generator version 3.
*/
b->wa_glslang_cs_barrier =
(generator_id == vtn_generator_glslang_reference_front_end &&
(b->generator_id == vtn_generator_glslang_reference_front_end &&
generator_version < 3);
/* words[2] == generator magic */
@ -5545,6 +5548,20 @@ spirv_to_nir(const uint32_t *words, size_t word_count,
words = vtn_foreach_instruction(b, words, word_end,
vtn_handle_preamble_instruction);
/* DirectXShaderCompiler and glslang/shaderc both create OpKill from HLSL's
* discard/clip, which uses demote semantics. DirectXShaderCompiler will use
* demote if the extension is enabled, so we disable this workaround in that
* case.
*
* Related glslang issue: https://github.com/KhronosGroup/glslang/issues/2416
*/
bool glslang = b->generator_id == vtn_generator_glslang_reference_front_end ||
b->generator_id == vtn_generator_shaderc_over_glslang;
bool dxsc = b->generator_id == vtn_generator_spiregg;
b->convert_discard_to_demote = ((dxsc && !b->uses_demote_to_helper_invocation) ||
(glslang && b->source_lang == SpvSourceLanguageHLSL)) &&
options->caps.demote_to_helper_invocation;
if (b->entry_point == NULL) {
vtn_fail("Entry point not found");
ralloc_free(b);

4
src/compiler/spirv/vtn_cfg.c
View file

@ -939,8 +939,10 @@ vtn_emit_branch(struct vtn_builder *b, enum vtn_branch_type branch_type,
nir_jump(&b->nb, nir_jump_return);
break;
case vtn_branch_type_discard: {
nir_intrinsic_op op =
b->convert_discard_to_demote ? nir_intrinsic_demote : nir_intrinsic_discard;
nir_intrinsic_instr *discard =
nir_intrinsic_instr_create(b->nb.shader, nir_intrinsic_discard);
nir_intrinsic_instr_create(b->nb.shader, op);
nir_builder_instr_insert(&b->nb, &discard->instr);
break;
}

8
src/compiler/spirv/vtn_private.h
View file

@ -666,9 +666,17 @@ struct vtn_builder {
unsigned value_id_bound;
struct vtn_value *values;
/* Information on the origin of the SPIR-V */
enum vtn_generator generator_id;
SpvSourceLanguage source_lang;
/* True if we need to fix up CS OpControlBarrier */
bool wa_glslang_cs_barrier;
/* Workaround discard bugs in HLSL -> SPIR-V compilers */
bool uses_demote_to_helper_invocation;
bool convert_discard_to_demote;
gl_shader_stage entry_point_stage;
const char *entry_point_name;
struct vtn_value *entry_point;