Fix for issue 545: don't reuse this VariableProxy.

src/compiler.cc

 // Copyright 2009 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 38 matching lines @@
   CodeGenSelector()
       : has_supported_syntax_(true),
         context_(Expression::kUninitialized) {
   }
 
   CodeGenTag Select(FunctionLiteral* fun);
 
  private:
   // Visit an expression in a given expression context.
   void ProcessExpression(Expression* expr, Expression::Context context) {
+    ASSERT_EQ(Expression::kUninitialized, expr->context());

[Kevin Millikin (Chromium), 2009/12/08 09:35:54]

As we discussed, change this to

ASSERT(expr->context() == Expression::kUninitialzed ||
       expr->context() == context);

     Expression::Context saved = context_;
     context_ = context;
     Visit(expr);
     expr->set_context(context);
     context_ = saved;
   }
 
   void VisitDeclarations(ZoneList<Declaration*>* decls);
   void VisitStatements(ZoneList<Statement*>* stmts);
 
@@ skipping 1050 matching lines @@
 
 void CodeGenSelector::VisitThisFunction(ThisFunction* expr) {
   BAILOUT("ThisFunction");
 }
 
 #undef BAILOUT
 #undef CHECK_BAILOUT
 
 
 } }  // namespace v8::internal