ShuriZma
/
xenia-canary
mirror of https://github.com/xenia-canary/xenia-canary.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[D3D12] DXBC: Fix 0*anything check with negated operands

This commit is contained in:
Triang3l 2019-01-12 18:52:06 +03:00
parent b8d1bb740b
commit 27a8e4259e
1 changed files with 16 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/xenia/gpu/dxbc_shader_translator.cc
View File

@ -1942,12 +1942,15 @@ uint32_t DxbcShaderTranslator::DxbcSourceOperandLength(
// totally invalid operand replaced by a literal.
return 5;
}
// Apply both the operand negation and the usage negation (for subtraction)
// and absolute from both sources.
// Apply overrides (for instance, for subtraction). Xenos operand modifiers
// are ignored when forcing absolute value (though negated absolute can still
// be forced in this case).
if (!absolute) {
if (operand.is_negated) {
negate = !negate;
}
absolute |= operand.is_absolute_value;
}
// Modifier extension - neg/abs or non-uniform binding index.
if (negate || absolute) {
++length;
@ -1981,12 +1984,15 @@ void DxbcShaderTranslator::UseDxbcSourceOperand(
(swizzle << D3D10_SB_OPERAND_4_COMPONENT_SWIZZLE_SHIFT);
}
// Apply both the operand negation and the usage negation (for subtraction)
// and absolute value from both sources.
// Apply overrides (for instance, for subtraction). Xenos operand modifiers
// are ignored when forcing absolute value (though negated absolute can still
// be forced in this case).
if (!absolute) {
if (operand.is_negated) {
negate = !negate;
}
absolute |= operand.is_absolute_value;
}
// Build OperandToken1 for modifiers (negate, absolute, minimum precision,
// non-uniform binding index) - if it has any, it will be non-zero.
// NOTE: AMD GPUs or drivers do NOT support non-uniform constant buffer