commit 327ed1bf8a34b5e8c0c97ad7b6dd483307b67f6f
author dan sinclair <dsinclair@google.com> Tue Apr 07 19:27:21 2020 +0000
committer dan sinclair <dsinclair@google.com> Tue Apr 07 19:27:21 2020 +0000
tree 315e4f321609127e8d50050eef9a702ed4b4b4fc
parent 0e2576250a186974d030aad3e4f91ef930699435

Add missing break, continue and return tests.

This CL adds tests for break, continue and return without attached
expressions.

Bug: tint:5
Change-Id: I8c4221e787af95f5597db3ea4e42b38962223bf4
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/18980
Reviewed-by: Ryan Harrison <rharrison@chromium.org>

src/type_determiner.cc

diff --git a/src/type_determiner.cc b/src/type_determiner.cc
index 51e8885..abb1c9f 100644
--- a/src/type_determiner.cc
+++ b/src/type_determiner.cc
@@ -41,9 +41,9 @@
 #include "src/ast/type/struct_type.h"
 #include "src/ast/type/vector_type.h"
 #include "src/ast/type_constructor_expression.h"
-#include "src/ast/unary_op_expression.h"
 #include "src/ast/unary_derivative_expression.h"
 #include "src/ast/unary_method_expression.h"
+#include "src/ast/unary_op_expression.h"
 #include "src/ast/unless_statement.h"
 #include "src/ast/variable_decl_statement.h"
 
@@ -381,9 +381,8 @@
 
     } else if (lhs_type->IsMatrix() && rhs_type->IsVector()) {
       auto mat = lhs_type->AsMatrix();
-      expr->set_result_type(
-          ctx_.type_mgr().Get(std::make_unique<ast::type::VectorType>(
-              mat->type(), mat->rows())));
+      expr->set_result_type(ctx_.type_mgr().Get(
+          std::make_unique<ast::type::VectorType>(mat->type(), mat->rows())));
     } else if (lhs_type->IsVector() && rhs_type->IsMatrix()) {
       auto mat = rhs_type->AsMatrix();
       expr->set_result_type(
@@ -487,12 +486,12 @@
 }
 
 bool TypeDeterminer::DetermineUnaryOp(ast::UnaryOpExpression* expr) {
-    // Result type matches the parameter type.
-    if (!DetermineResultType(expr->expr())) {
-      return false;
-    }
-    expr->set_result_type(expr->expr()->result_type());
-    return true;
+  // Result type matches the parameter type.
+  if (!DetermineResultType(expr->expr())) {
+    return false;
+  }
+  expr->set_result_type(expr->expr()->result_type());
+  return true;
 }
 
 }  // namespace tint