franklinblanco/yuzu
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

gl_shader_decompiler: Fix decompilation of condition codes

Browse Source 
Use Visit instead of reimplementing it. Fixes unimplemented negations
for condition codes.
This commit is contained in:
ReinUsesLisp 2020-01-17 19:06:27 -03:00
parent 657b3a366e
commit f34e519da3
1 changed files with 5 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
src/video_core/renderer_opengl/gl_shader_decompiler.cpp
View File

@ -2313,7 +2313,7 @@ public:
explicit ExprDecompiler(GLSLDecompiler& decomp) : decomp{decomp} {} explicit ExprDecompiler(GLSLDecompiler& decomp) : decomp{decomp} {}
void operator()(const ExprAnd& expr) { void operator()(const ExprAnd& expr) {
inner += "( "; inner += '(';
std::visit(*this, *expr.operand1); std::visit(*this, *expr.operand1);
inner += " && "; inner += " && ";
std::visit(*this, *expr.operand2); std::visit(*this, *expr.operand2);
@ -2321,7 +2321,7 @@ public:
} }
void operator()(const ExprOr& expr) { void operator()(const ExprOr& expr) {
inner += "( "; inner += '(';
std::visit(*this, *expr.operand1); std::visit(*this, *expr.operand1);
inner += " || "; inner += " || ";
std::visit(*this, *expr.operand2); std::visit(*this, *expr.operand2);
@ -2339,28 +2339,7 @@ public:
} }
void operator()(const ExprCondCode& expr) { void operator()(const ExprCondCode& expr) {
const Node cc = decomp.ir.GetConditionCode(expr.cc); inner += decomp.Visit(decomp.ir.GetConditionCode(expr.cc)).AsBool();
std::string target;
if (const auto pred = std::get_if<PredicateNode>(&*cc)) {
const auto index = pred->GetIndex();
switch (index) {
case Tegra::Shader::Pred::NeverExecute:
target = "false";
break;
case Tegra::Shader::Pred::UnusedIndex:
target = "true";
break;
default:
target = decomp.GetPredicate(index);
break;
}
} else if (const auto flag = std::get_if<InternalFlagNode>(&*cc)) {
target = decomp.GetInternalFlag(flag->GetFlag());
} else {
UNREACHABLE();
}
inner += target;
} }
void operator()(const ExprVar& expr) { void operator()(const ExprVar& expr) {
@ -2372,8 +2351,7 @@ public:
} }
void operator()(VideoCommon::Shader::ExprGprEqual& expr) { void operator()(VideoCommon::Shader::ExprGprEqual& expr) {
inner += inner += fmt::format("(ftou({}) == {})", decomp.GetRegister(expr.gpr), expr.value);
"( ftou(" + decomp.GetRegister(expr.gpr) + ") == " + std::to_string(expr.value) + ')';
} }
const std::string& GetResult() const { const std::string& GetResult() const {
@ -2381,8 +2359,8 @@ public:
} }
private: private:
std::string inner;
GLSLDecompiler& decomp; GLSLDecompiler& decomp;
std::string inner;
}; };
class ASTDecompiler { class ASTDecompiler {