LGTM

http://codereview.chromium.org/598016/diff/1/3
File test/mjsunit/compiler/simple-bailouts.js (right):

http://codereview.chromium.org/598016/diff/1/3#newcode86
test/mjsunit/compiler/simple-bailouts.js:86: this.z = // (14 | 1.1) ==
15
I'd align the // vertically in this function as well.

http://codereview.chromium.org/598016/diff/1/3#newcode92
test/mjsunit/compiler/simple-bailouts.js:92: | a;  // 1.0
// 1.1

http://codereview.chromium.org/598016

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

Reply via email to