Reland "Revert "moved BinaryExpression's data into IRNode""
This reverts commit 1d3e0e0054b243b7a62029c29090e7e76a19a805.
Reason for revert: possibly causing https://task-scheduler.skia.org/task/F1DoniJAddPuEkH9ETTE
Original change's description:
> Revert "Revert "moved BinaryExpression's data into IRNode""
>
> This reverts commit b61c3a9a01c44840eaa35b28cae0a4b358727f3c.
>
> Change-Id: I4689e1f4977fab3233ff492cee06fbc301b5c689
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/317386
> Reviewed-by: John Stiles <johnstiles@google.com>
> Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
TBR=brianosman@google.com,ethannicholas@google.com,johnstiles@google.com
# Not skipping CQ checks because this is a reland.
Change-Id: Id0f3f211f09fbf31b626c648ed141fc6154a450c
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/317395
Reviewed-by: Ethan Nicholas <ethannicholas@google.com>
Commit-Queue: Ethan Nicholas <ethannicholas@google.com>
diff --git a/src/sksl/SkSLCPPCodeGenerator.cpp b/src/sksl/SkSLCPPCodeGenerator.cpp
index a62374e..3eacb80 100644
--- a/src/sksl/SkSLCPPCodeGenerator.cpp
+++ b/src/sksl/SkSLCPPCodeGenerator.cpp
@@ -70,45 +70,42 @@
void CPPCodeGenerator::writeBinaryExpression(const BinaryExpression& b,
Precedence parentPrecedence) {
- const Expression& left = b.left();
- const Expression& right = b.right();
- Token::Kind op = b.getOperator();
- if (op == Token::Kind::TK_PERCENT) {
+ if (b.fOperator == Token::Kind::TK_PERCENT) {
// need to use "%%" instead of "%" b/c the code will be inside of a printf
- Precedence precedence = GetBinaryPrecedence(op);
+ Precedence precedence = GetBinaryPrecedence(b.fOperator);
if (precedence >= parentPrecedence) {
this->write("(");
}
- this->writeExpression(left, precedence);
+ this->writeExpression(*b.fLeft, precedence);
this->write(" %% ");
- this->writeExpression(right, precedence);
+ this->writeExpression(*b.fRight, precedence);
if (precedence >= parentPrecedence) {
this->write(")");
}
- } else if (left.kind() == Expression::Kind::kNullLiteral ||
- right.kind() == Expression::Kind::kNullLiteral) {
+ } else if (b.fLeft->kind() == Expression::Kind::kNullLiteral ||
+ b.fRight->kind() == Expression::Kind::kNullLiteral) {
const Variable* var;
- if (left.kind() != Expression::Kind::kNullLiteral) {
- var = &left.as<VariableReference>().fVariable;
+ if (b.fLeft->kind() != Expression::Kind::kNullLiteral) {
+ var = &b.fLeft->as<VariableReference>().fVariable;
} else {
- var = &right.as<VariableReference>().fVariable;
+ var = &b.fRight->as<VariableReference>().fVariable;
}
SkASSERT(var->type().typeKind() == Type::TypeKind::kNullable &&
var->type().componentType() == *fContext.fFragmentProcessor_Type);
this->write("%s");
- const char* prefix = "";
- switch (op) {
+ const char* op = "";
+ switch (b.fOperator) {
case Token::Kind::TK_EQEQ:
- prefix = "!";
+ op = "!";
break;
case Token::Kind::TK_NEQ:
- prefix = "";
+ op = "";
break;
default:
SkASSERT(false);
}
int childIndex = this->getChildFPIndex(*var);
- fFormatArgs.push_back(String(prefix) + "_outer.childProcessor(" + to_string(childIndex) +
+ fFormatArgs.push_back(String(op) + "_outer.childProcessor(" + to_string(childIndex) +
") ? \"true\" : \"false\"");
} else {
INHERITED::writeBinaryExpression(b, parentPrecedence);