moved BinaryExpression's data into IRNode
This is another step in the process of merging the various IRNodes' data
into the base class.
Change-Id: Ide39c240e6178e23bb6fe317dd56addf2ffefcbb
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/317102
Commit-Queue: John Stiles <johnstiles@google.com>
Reviewed-by: John Stiles <johnstiles@google.com>
diff --git a/src/sksl/SkSLCPPCodeGenerator.cpp b/src/sksl/SkSLCPPCodeGenerator.cpp
index 3eacb80..a62374e 100644
--- a/src/sksl/SkSLCPPCodeGenerator.cpp
+++ b/src/sksl/SkSLCPPCodeGenerator.cpp
@@ -70,42 +70,45 @@
void CPPCodeGenerator::writeBinaryExpression(const BinaryExpression& b,
Precedence parentPrecedence) {
- if (b.fOperator == Token::Kind::TK_PERCENT) {
+ const Expression& left = b.left();
+ const Expression& right = b.right();
+ Token::Kind op = b.getOperator();
+ if (op == Token::Kind::TK_PERCENT) {
// need to use "%%" instead of "%" b/c the code will be inside of a printf
- Precedence precedence = GetBinaryPrecedence(b.fOperator);
+ Precedence precedence = GetBinaryPrecedence(op);
if (precedence >= parentPrecedence) {
this->write("(");
}
- this->writeExpression(*b.fLeft, precedence);
+ this->writeExpression(left, precedence);
this->write(" %% ");
- this->writeExpression(*b.fRight, precedence);
+ this->writeExpression(right, precedence);
if (precedence >= parentPrecedence) {
this->write(")");
}
- } else if (b.fLeft->kind() == Expression::Kind::kNullLiteral ||
- b.fRight->kind() == Expression::Kind::kNullLiteral) {
+ } else if (left.kind() == Expression::Kind::kNullLiteral ||
+ right.kind() == Expression::Kind::kNullLiteral) {
const Variable* var;
- if (b.fLeft->kind() != Expression::Kind::kNullLiteral) {
- var = &b.fLeft->as<VariableReference>().fVariable;
+ if (left.kind() != Expression::Kind::kNullLiteral) {
+ var = &left.as<VariableReference>().fVariable;
} else {
- var = &b.fRight->as<VariableReference>().fVariable;
+ var = &right.as<VariableReference>().fVariable;
}
SkASSERT(var->type().typeKind() == Type::TypeKind::kNullable &&
var->type().componentType() == *fContext.fFragmentProcessor_Type);
this->write("%s");
- const char* op = "";
- switch (b.fOperator) {
+ const char* prefix = "";
+ switch (op) {
case Token::Kind::TK_EQEQ:
- op = "!";
+ prefix = "!";
break;
case Token::Kind::TK_NEQ:
- op = "";
+ prefix = "";
break;
default:
SkASSERT(false);
}
int childIndex = this->getChildFPIndex(*var);
- fFormatArgs.push_back(String(op) + "_outer.childProcessor(" + to_string(childIndex) +
+ fFormatArgs.push_back(String(prefix) + "_outer.childProcessor(" + to_string(childIndex) +
") ? \"true\" : \"false\"");
} else {
INHERITED::writeBinaryExpression(b, parentPrecedence);