Enable inlining of functions that reference context slots.

src/ast.cc

 // Copyright 2011 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 511 matching lines @@
 }
 
 
 bool Conditional::IsInlineable() const {
   return condition()->IsInlineable() && then_expression()->IsInlineable() &&
       else_expression()->IsInlineable();
 }
 
 
 bool VariableProxy::IsInlineable() const {
-  return var()->IsUnallocated() || var()->IsStackAllocated();
+  return !var()->IsLookupSlot();

[Kevin Millikin (Chromium), 2011/09/15 09:45:01]

I'd prefer to keep this a whitelist, listing the things that we can support. 
It's too risky that someone will add a new kind of thing that becomes inlinable
without supporting it.

 }
 
 
 bool Assignment::IsInlineable() const {
   return target()->IsInlineable() && value()->IsInlineable();
 }
 
 
 bool Property::IsInlineable() const {
   return obj()->IsInlineable() && key()->IsInlineable();
@@ skipping 626 matching lines @@
                        int pos)
     : label_(label),
       statements_(statements),
       position_(pos),
       compare_type_(NONE),
       compare_id_(AstNode::GetNextId(isolate)),
       entry_id_(AstNode::GetNextId(isolate)) {
 }
 
 } }  // namespace v8::internal