Reviewers: Kasper Lund,

Message:
There are a lot of changes to comments in cfg.h.

It's a known issue that appending lists of expressions like this is
quadratic in the size of the expression AST (copying the entire
right-hand subtree at each level).  That will be solve by a linked
representation of instruction lists.

Description:
Add support to the CFG builder for non-short-circuited binary
expressions.  Add compilation support in fast mode (optimized for
compilation time and code size).

This breaks one debugger test each on ARM and IA32 (expectedly).

Please review this at http://codereview.chromium.org/160584

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
   M     src/arm/cfg-arm.cc
   M     src/arm/codegen-arm.h
   M     src/arm/codegen-arm.cc
   M     src/cfg.h
   M     src/cfg.cc
   M     src/ia32/cfg-ia32.cc
   M     src/ia32/codegen-ia32.h
   M     src/ia32/codegen-ia32.cc
   M     src/x64/cfg-x64.cc
   M     src/x64/codegen-x64.h
   M     src/x64/codegen-x64.cc



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

Reply via email to