New version with 2 changes:

a) Fixed a bug where string-only ToBoolean code ran into an incorrect deopt.
Moral of the story: Use EmitBranch only in tail positions. :-/

b) When there is no type info for ToBoolean, generated code for handling all cases instead of always-deopt-code. This is essential for some benchmarks like checkers or deltablue. A probably better solution would be to collect type info
from other uses in the same function and use the union of those instead, but
this will be in a separate CL.

http://codereview.chromium.org/7461107/

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

Reply via email to