From f5d4941716506a24d9492f8724e63bdf84ef8eb5 Mon Sep 17 00:00:00 2001 From: sephiroth99 Date: Wed, 30 Dec 2015 16:09:03 -0500 Subject: [PATCH] shader_translator: Remove always true expression vector_dest() returns an unsigned value, it will always be >= 0. --- src/xenia/gpu/shader_translator.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/xenia/gpu/shader_translator.cc b/src/xenia/gpu/shader_translator.cc index 71d9b419d..689bcbf97 100644 --- a/src/xenia/gpu/shader_translator.cc +++ b/src/xenia/gpu/shader_translator.cc @@ -184,12 +184,12 @@ void ShaderTranslator::GatherBindingInformation( auto& op = *reinterpret_cast(ucode_dwords_ + instr_offset * 3); if (op.has_vector_op() && op.is_export()) { - if (op.vector_dest() >= 0 && op.vector_dest() <= 3) { + if (op.vector_dest() <= 3) { writes_color_targets_[op.vector_dest()] = true; } } if (op.has_scalar_op() && op.is_export()) { - if (op.vector_dest() >= 0 && op.vector_dest() <= 3) { + if (op.vector_dest() <= 3) { writes_color_targets_[op.vector_dest()] = true; } }