I don't like the name 'fast compiler', but I guess it LGTM.


http://codereview.chromium.org/549158/diff/6001/7002
File src/compiler.cc (right):

http://codereview.chromium.org/549158/diff/6001/7002#newcode97
src/compiler.cc:97: ASSERT(!FLAG_always_full_compiler ||
!FLAG_always_fast_compiler);
I think CHECK would be more appropriate.

http://codereview.chromium.org/549158/diff/6001/7008
File src/fast-codegen.cc (right):

http://codereview.chromium.org/549158/diff/6001/7008#newcode108
src/fast-codegen.cc:108: // Supported.
A good start!

http://codereview.chromium.org/549158

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

Reply via email to