LGTM with comments addressed.
http://codereview.chromium.org/651031/diff/1/6 File src/compiler.h (right): http://codereview.chromium.org/651031/diff/1/6#newcode64 src/compiler.h:64: Bailout(Register left, Register right) : left_(left), right_(right) {} label_ needs to be initialized here. http://codereview.chromium.org/651031/diff/1/12 File src/ia32/fast-codegen-ia32.h (right): http://codereview.chromium.org/651031/diff/1/12#newcode35 src/ia32/fast-codegen-ia32.h:35: #include "list.h" list.h should be already included from compiler.h, shouldn't it? http://codereview.chromium.org/651031 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
