commit db8b8672e2b7659a6ab890393cdd512049b07e80
author Florian Weikert <fwe@google.com> Mon Nov 09 13:26:24 2015 +0000
committer Damien Martin-Guillerez <dmarting@google.com> Tue Nov 10 10:19:34 2015 +0000
tree c35fac75865a01f584cf3e69857055edfcb582b6
parent b487ac69185fd0080461a3c8795589fe4532f4bb

Compile assignments to byte code and throw errors.

Add EvalExceptions for cases in which the interpreter would throw them
during evaluation of the function definition.

--
MOS_MIGRATED_REVID=107376021

src/main/java/com/google/devtools/build/lib/syntax/AssignmentStatement.java

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/AssignmentStatement.java b/src/main/java/com/google/devtools/build/lib/syntax/AssignmentStatement.java
index f731ed5..e3dc7d1 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/AssignmentStatement.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/AssignmentStatement.java
@@ -14,6 +14,13 @@
 
 package com.google.devtools.build.lib.syntax;
 
+import com.google.common.base.Optional;
+import com.google.devtools.build.lib.syntax.compiler.DebugInfo;
+import com.google.devtools.build.lib.syntax.compiler.LoopLabels;
+import com.google.devtools.build.lib.syntax.compiler.VariableScope;
+
+import net.bytebuddy.implementation.bytecode.ByteCodeAppender;
+
 /**
  * Syntax node for an assignment statement.
  */
@@ -66,4 +73,13 @@
     expression.validate(env);
     lvalue.validate(env, getLocation());
   }
+
+  @Override
+  ByteCodeAppender compile(
+      VariableScope scope, Optional<LoopLabels> loopLabels, DebugInfo debugInfo)
+          throws EvalException {
+    return new ByteCodeAppender.Compound(
+        expression.compile(scope, debugInfo),
+        lvalue.compileAssignment(this, debugInfo.add(this), scope));
+  }
 }