Improved test coverage. Landing.
https://chromiumcodereview.appspot.com/10908194/diff/3001/test/mjsunit/regress/regress-2261.js File test/mjsunit/regress/regress-2261.js (right): https://chromiumcodereview.appspot.com/10908194/diff/3001/test/mjsunit/regress/regress-2261.js#newcode32 test/mjsunit/regress/regress-2261.js:32: On 2012/09/12 07:14:22, Sven Panne wrote:
As was dicussed already offline, it would be good to have test cases
which
involve an arguments adaptor (both with too few/to many arguments),
even if it
is not easily possible to do without extensive copy-n-paste. I would
very much
prefer a broader test coverage for such a tricky area over
copy-n-paste-free
code. :-)
Probably these tests should live in mjsunit/compiler then.
Done. Added a generic test case to mjsunit/compiler/inline-arguments that tests all combinations of arguments adapters for two-level inlining. https://chromiumcodereview.appspot.com/10908194/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
