Done. Extended and fixed version ready for next round of review... :-)
https://chromiumcodereview.appspot.com/10836133/diff/4006/test/mjsunit/compiler/inline-accessors.js File test/mjsunit/compiler/inline-accessors.js (right): https://chromiumcodereview.appspot.com/10836133/diff/4006/test/mjsunit/compiler/inline-accessors.js#newcode80 test/mjsunit/compiler/inline-accessors.js:80: assertEquals(obj, this); On 2012/08/09 12:20:55, Michael Starzinger wrote:
I would really use assertSame() here instead, because you want
identity
comparison and not a deepEquals() of the object body.
Done. https://chromiumcodereview.appspot.com/10836133/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
