summaryrefslogtreecommitdiff
path: root/src/shader_recompiler/backend/glsl/emit_glsl_atomic.cpp
diff options
context:
space:
mode:
authorGravatar ameerj2021-05-30 19:13:22 -0400
committerGravatar ameerj2021-07-22 21:51:37 -0400
commit9f3ffb996b0d02ca64b492d22ff158e8f3659257 (patch)
tree48993eaf320484cf042071a81a1a6b1dcc829eb9 /src/shader_recompiler/backend/glsl/emit_glsl_atomic.cpp
parentglsl: Rework variable allocator to allow for variable reuse (diff)
downloadyuzu-9f3ffb996b0d02ca64b492d22ff158e8f3659257.tar.gz
yuzu-9f3ffb996b0d02ca64b492d22ff158e8f3659257.tar.xz
yuzu-9f3ffb996b0d02ca64b492d22ff158e8f3659257.zip
glsl: Rework var alloc to not assign unused results
Diffstat (limited to 'src/shader_recompiler/backend/glsl/emit_glsl_atomic.cpp')
-rw-r--r--src/shader_recompiler/backend/glsl/emit_glsl_atomic.cpp9
1 files changed, 4 insertions, 5 deletions
diff --git a/src/shader_recompiler/backend/glsl/emit_glsl_atomic.cpp b/src/shader_recompiler/backend/glsl/emit_glsl_atomic.cpp
index 918f90058..db4c60002 100644
--- a/src/shader_recompiler/backend/glsl/emit_glsl_atomic.cpp
+++ b/src/shader_recompiler/backend/glsl/emit_glsl_atomic.cpp
@@ -11,8 +11,7 @@
11 11
12namespace Shader::Backend::GLSL { 12namespace Shader::Backend::GLSL {
13namespace { 13namespace {
14static constexpr std::string_view cas_loop{R"({}; 14static constexpr std::string_view cas_loop{R"(for (;;){{
15for (;;){{
16 uint old_value={}; 15 uint old_value={};
17 {}=atomicCompSwap({},old_value,{}({},{})); 16 {}=atomicCompSwap({},old_value,{}({},{}));
18 if ({}==old_value){{break;}} 17 if ({}==old_value){{break;}}
@@ -22,14 +21,14 @@ void SharedCasFunction(EmitContext& ctx, IR::Inst& inst, std::string_view offset
22 std::string_view value, std::string_view function) { 21 std::string_view value, std::string_view function) {
23 const auto ret{ctx.var_alloc.Define(inst, GlslVarType::U32)}; 22 const auto ret{ctx.var_alloc.Define(inst, GlslVarType::U32)};
24 const std::string smem{fmt::format("smem[{}/4]", offset)}; 23 const std::string smem{fmt::format("smem[{}/4]", offset)};
25 ctx.Add(cas_loop.data(), ret, smem, ret, smem, function, smem, value, ret); 24 ctx.Add(cas_loop.data(), smem, ret, smem, function, smem, value, ret);
26} 25}
27 26
28void SsboCasFunction(EmitContext& ctx, IR::Inst& inst, const IR::Value& binding, 27void SsboCasFunction(EmitContext& ctx, IR::Inst& inst, const IR::Value& binding,
29 const IR::Value& offset, std::string_view value, std::string_view function) { 28 const IR::Value& offset, std::string_view value, std::string_view function) {
30 const auto ret{ctx.var_alloc.Define(inst, GlslVarType::U32)}; 29 const auto ret{ctx.var_alloc.Define(inst, GlslVarType::U32)};
31 const std::string ssbo{fmt::format("ssbo{}[{}]", binding.U32(), offset.U32())}; 30 const std::string ssbo{fmt::format("ssbo{}[{}]", binding.U32(), offset.U32())};
32 ctx.Add(cas_loop.data(), ret, ssbo, ret, ssbo, function, ssbo, value, ret); 31 ctx.Add(cas_loop.data(), ssbo, ret, ssbo, function, ssbo, value, ret);
33} 32}
34 33
35void SsboCasFunctionF32(EmitContext& ctx, IR::Inst& inst, const IR::Value& binding, 34void SsboCasFunctionF32(EmitContext& ctx, IR::Inst& inst, const IR::Value& binding,
@@ -37,7 +36,7 @@ void SsboCasFunctionF32(EmitContext& ctx, IR::Inst& inst, const IR::Value& bindi
37 std::string_view function) { 36 std::string_view function) {
38 const std::string ssbo{fmt::format("ssbo{}[{}]", binding.U32(), offset.U32())}; 37 const std::string ssbo{fmt::format("ssbo{}[{}]", binding.U32(), offset.U32())};
39 const auto ret{ctx.var_alloc.Define(inst, GlslVarType::U32)}; 38 const auto ret{ctx.var_alloc.Define(inst, GlslVarType::U32)};
40 ctx.Add(cas_loop.data(), ret, ssbo, ret, ssbo, function, ssbo, value, ret); 39 ctx.Add(cas_loop.data(), ssbo, ret, ssbo, function, ssbo, value, ret);
41 ctx.AddF32("{}=uintBitsToFloat({});", inst, ret); 40 ctx.AddF32("{}=uintBitsToFloat({});", inst, ret);
42} 41}
43} // namespace 42} // namespace