drive-by:

https://chromiumcodereview.appspot.com/10543094/diff/6001/src/ia32/lithium-codegen-ia32.cc
File src/ia32/lithium-codegen-ia32.cc (right):

https://chromiumcodereview.appspot.com/10543094/diff/6001/src/ia32/lithium-codegen-ia32.cc#newcode3883
src/ia32/lithium-codegen-ia32.cc:3883: #if DEBUG
We usually use FLAG_debug_code to emit assertion in generated code so
that the generated code in debug and release mode is the same by
default.

https://chromiumcodereview.appspot.com/10543094/diff/6001/src/ia32/lithium-codegen-ia32.cc#newcode3888
src/ia32/lithium-codegen-ia32.cc:3888: __ bind(&ok);
You could replace this block of instructions with:

__ AbortIfNotSmi(ToRegister(input));

https://chromiumcodereview.appspot.com/10543094/

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

Reply via email to