LGTM (with a nit).
https://chromiumcodereview.appspot.com/10828049/diff/1/src/hydrogen.cc File src/hydrogen.cc (right): https://chromiumcodereview.appspot.com/10828049/diff/1/src/hydrogen.cc#newcode5073 src/hydrogen.cc:5073: AddInstruction(new(zone()) HCheckNonSmi(object)); Can we move this down right in front of the block where instructions are actually emitted? Because this first block is just checking some optimistic assumptions but not generating any instructions. https://chromiumcodereview.appspot.com/10828049/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
