LGTM once it passes all tests on the ARM simulator too.


http://codereview.chromium.org/536029/diff/1/3
File src/codegen.cc (right):

http://codereview.chromium.org/536029/diff/1/3#newcode290
src/codegen.cc:290: StoreToSlot(scope_->function()->slot(),
NOT_CONST_INIT);
As we discussed, this won't work on ARM.  There is no StoreToSlot there.

I'd move it into the platform specific CodeGenerator::GenCode functions
(just before calling CodeGenerator::ProcessDeclarations).

http://codereview.chromium.org/536029
-- 
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to