Reviewers: Mads Ager, Description: Changed fuzzer to randomly remove half of the arguments when running on functions that takes five or more arguments. Original version runs for a long time when there is 5 arguments in a runtime function (45+ seconds). The fuzzer can be run with all arguments on all functions regardless of the number of arguments by setting RUN_WITH_ALL_ARGUMENT_ENTRIES to true in fuzz-natives.js
Please review this at http://codereview.chromium.org/598011 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M test/mjsunit/fuzz-natives.js Index: test/mjsunit/fuzz-natives.js =================================================================== --- test/mjsunit/fuzz-natives.js (revision 3816) +++ test/mjsunit/fuzz-natives.js (working copy) @@ -27,6 +27,9 @@ // Flags: --allow-natives-syntax +var RUN_WITH_ALL_ARGUMENT_ENTRIES = false; +var kOnManyArgumentsRemove = 5; + function makeArguments() { var result = [ ]; result.push(17); @@ -74,13 +77,23 @@ var func = makeFunction(name, argc); while (hasMore) { var argPool = makeArguments(); + // When we have 5 or more arguments we lower the amount of tests cases + // by randomly removing kOnManyArgumentsRemove entries + var numArguments = RUN_WITH_ALL_ARGUMENT_ENTRIES ? + kArgObjects : kArgObjects-kOnManyArgumentsRemove; + if (argc >= 5 && !RUN_WITH_ALL_ARGUMENT_ENTRIES) { + for (var i = 0; i < kOnManyArgumentsRemove; i++) { + var rand = Math.floor(Math.random() * (kArgObjects - i)); + argPool.splice(rand,1); + } + } var current = type; var hasMore = false; var argList = [ ]; for (var i = 0; i < argc; i++) { - var index = current % kArgObjects; - current = (current / kArgObjects) << 0; - if (index != (kArgObjects - 1)) + var index = current % numArguments; + current = (current / numArguments) << 0; + if (index != (numArguments - 1)) hasMore = true; argList.push(argPool[index]); } -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
