I think Florian's suggestion is the way to go.

https://chromiumcodereview.appspot.com/10411016/diff/1/src/hydrogen-instructions.cc
File src/hydrogen-instructions.cc (right):

https://chromiumcodereview.appspot.com/10411016/diff/1/src/hydrogen-instructions.cc#newcode880
src/hydrogen-instructions.cc:880:
right()->range()->set_can_be_minus_zero(false);
I don't think this is correct. Consider code like:

function mul(a, b) {
  var x = a * b;
  x | 0;
  return x;
}

your code will remove minus zero from the range of x but this is
incorrect.

https://chromiumcodereview.appspot.com/10411016/

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

Reply via email to