I added additional tests to math-min-max.js to test mixed representations.
https://chromiumcodereview.appspot.com/9372021/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):
https://chromiumcodereview.appspot.com/9372021/diff/1/src/hydrogen.cc#newcode5324
src/hydrogen.cc:5324: left, Representation::Double(), false, true);
On 2012/02/20 12:18:39, fschneider wrote:
Please break like this and comment the boolean paramters:
HChange(left,
Representation::Double(),
false,
true);
Done.
https://chromiumcodereview.appspot.com/9372021/diff/1/src/hydrogen.cc#newcode5330
src/hydrogen.cc:5330: right, Representation::Double(), false, true);
On 2012/02/20 12:18:39, fschneider wrote:
Also here: Short same-line comment for the boolean paramters.
Done.
https://chromiumcodereview.appspot.com/9372021/
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
