http://codereview.chromium.org/9177001/diff/4001/src/hydrogen-instructions.cc
File src/hydrogen-instructions.cc (right):

http://codereview.chromium.org/9177001/diff/4001/src/hydrogen-instructions.cc#newcode793
src/hydrogen-instructions.cc:793: if (op() == Token::BIT_XOR) return
this;
On 2012/01/11 09:54:03, Sven wrote:
I don't think we need this special case, because 'x ^ 0 == x', just
like 'x | 0
== x'.

Done.

http://codereview.chromium.org/9177001/diff/4001/src/hydrogen-instructions.cc#newcode795
src/hydrogen-instructions.cc:795: int nop_constant = (op() ==
Token::BIT_AND) ? -1 : 0;
On 2012/01/11 09:54:03, Sven wrote:
Use int32_t, not int.

Done.

http://codereview.chromium.org/9177001/

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

Reply via email to