diff options
| author | Fernando Sahmkow <fsahmkow27@gmail.com> | 2020-01-04 14:40:57 -0400 |
|---|---|---|
| committer | Fernando Sahmkow <fsahmkow27@gmail.com> | 2020-01-04 14:40:57 -0400 |
| commit | 3dd6b55851978440f39487a6ad06b30b792b3b36 (patch) | |
| tree | d5019053dc20d420181443ff90207aab2ff28938 /src/video_core/renderer_vulkan | |
| parent | b3371ed09e6866e235141119f9eecc2bb962dc8d (diff) | |
Shader_IR: Address Feedback
Diffstat (limited to 'src/video_core/renderer_vulkan')
| -rw-r--r-- | src/video_core/renderer_vulkan/vk_shader_decompiler.cpp | 18 |
1 files changed, 4 insertions, 14 deletions
diff --git a/src/video_core/renderer_vulkan/vk_shader_decompiler.cpp b/src/video_core/renderer_vulkan/vk_shader_decompiler.cpp index 50feeb003..8fe852ce8 100644 --- a/src/video_core/renderer_vulkan/vk_shader_decompiler.cpp +++ b/src/video_core/renderer_vulkan/vk_shader_decompiler.cpp @@ -954,10 +954,8 @@ private: Expression Visit(const Node& node) { if (const auto operation = std::get_if<OperationNode>(&*node)) { - auto amend_index = operation->GetAmendIndex(); - if (amend_index) { - const Node& amend_node = ir.GetAmendNode(*amend_index); - [[maybe_unused]] const Type type = Visit(amend_node).type; + if (const auto amend_index = operation->GetAmendIndex()) { + [[maybe_unused]] const Type type = Visit(ir.GetAmendNode(*amend_index)).type; ASSERT(type == Type::Void); } const auto operation_index = static_cast<std::size_t>(operation->GetCode()); @@ -1148,10 +1146,8 @@ private: } if (const auto conditional = std::get_if<ConditionalNode>(&*node)) { - auto amend_index = conditional->GetAmendIndex(); - if (amend_index) { - const Node& amend_node = ir.GetAmendNode(*amend_index); - [[maybe_unused]] const Type type = Visit(amend_node).type; + if (const auto amend_index = conditional->GetAmendIndex()) { + [[maybe_unused]] const Type type = Visit(ir.GetAmendNode(*amend_index)).type; ASSERT(type == Type::Void); } // It's invalid to call conditional on nested nodes, use an operation instead @@ -1176,12 +1172,6 @@ private: } if (const auto comment = std::get_if<CommentNode>(&*node)) { - auto amend_index = comment->GetAmendIndex(); - if (amend_index) { - const Node& amend_node = ir.GetAmendNode(*amend_index); - [[maybe_unused]] const Type type = Visit(amend_node).type; - ASSERT(type == Type::Void); - } Name(OpUndef(t_void), comment->GetText()); return {}; } |
