fabi/Xenia-Canary
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

shader_translator: Remove always true expression

Browse Source 
vector_dest() returns an unsigned value, it will always be >= 0.
This commit is contained in:
sephiroth99
2015-12-30 16:09:03 -05:00
parent f3fc50c8db
commit f5d4941716
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/xenia/gpu/shader_translator.cc
View File

@@ -184,12 +184,12 @@ void ShaderTranslator::GatherBindingInformation(
auto& op = *reinterpret_cast<const AluInstruction*>(ucode_dwords_ + auto& op = *reinterpret_cast<const AluInstruction*>(ucode_dwords_ +
instr_offset * 3); instr_offset * 3);
if (op.has_vector_op() && op.is_export()) { 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; writes_color_targets_[op.vector_dest()] = true;
} }
} }
if (op.has_scalar_op() && op.is_export()) { 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; writes_color_targets_[op.vector_dest()] = true;
} }
} }