Reland: VM [KERNEL] Avoid emitting :expr_temp and capturing :iterator

runtime/vm/kernel_to_il.h

 // Copyright (c) 2016, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #ifndef RUNTIME_VM_KERNEL_TO_IL_H_
 #define RUNTIME_VM_KERNEL_TO_IL_H_
 
 #if !defined(DART_PRECOMPILED_RUNTIME)
 
 #include "vm/growable_array.h"
@@ skipping 622 matching lines @@
         parsed_function_(parsed_function),
         node_(node),
         translation_helper_(Thread::Current()),
         zone_(translation_helper_.zone()),
         type_translator_(&translation_helper_,
                          &active_class_,
                          /*finalize=*/true),
         current_function_scope_(NULL),
         scope_(NULL),
         depth_(0),
-        name_index_(0) {}
+        name_index_(0),
+        needs_expr_temp_(false) {}
 
   virtual ~ScopeBuilder() {}
 
   ScopeBuildingResult* BuildScopes();
 
   virtual void VisitName(Name* node) { /* NOP */
   }
 
   virtual void VisitThisExpression(ThisExpression* node);
   virtual void VisitTypeParameterType(TypeParameterType* node);
   virtual void VisitVariableGet(VariableGet* node);
   virtual void VisitVariableSet(VariableSet* node);
+  virtual void VisitConditionalExpression(ConditionalExpression* node);
+  virtual void VisitLogicalExpression(LogicalExpression* node);
   virtual void VisitFunctionExpression(FunctionExpression* node);
   virtual void VisitLet(Let* node);
   virtual void VisitBlock(Block* node);
   virtual void VisitVariableDeclaration(VariableDeclaration* node);
   virtual void VisitFunctionDeclaration(FunctionDeclaration* node);
   virtual void VisitWhileStatement(WhileStatement* node);
   virtual void VisitDoStatement(DoStatement* node);
   virtual void VisitForStatement(ForStatement* node);
   virtual void VisitForInStatement(ForInStatement* node);
   virtual void VisitSwitchStatement(SwitchStatement* node);
@@ skipping 66 matching lines @@
   TranslationHelper translation_helper_;
   Zone* zone_;
   DartTypeTranslator type_translator_;
 
   FunctionNode* current_function_node_;
   LocalScope* current_function_scope_;
   LocalScope* scope_;
   DepthState depth_;
 
   intptr_t name_index_;
+
+  bool needs_expr_temp_;
 };
 
 class FlowGraphBuilder : public ExpressionVisitor, public StatementVisitor {
  public:
   FlowGraphBuilder(TreeNode* node,
                    ParsedFunction* parsed_function,
                    const ZoneGrowableArray<const ICData*>& ic_data_array,
                    InlineExitCollector* exit_collector,
                    intptr_t osr_id,
                    intptr_t first_block_id = 1);
@@ skipping 378 matching lines @@
 namespace kernel {
 
 RawObject* EvaluateMetadata(TreeNode* const kernel_node);
 RawObject* BuildParameterDescriptor(TreeNode* const kernel_node);
 
 }  // namespace kernel
 }  // namespace dart
 
 #endif  // !defined(DART_PRECOMPILED_RUNTIME)
 #endif  // RUNTIME_VM_KERNEL_TO_IL_H_