Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(515)

Unified Diff: test/mjsunit/compiler/inline-arguments.js

Issue 11028027: Revert trunk to bleeding_edge at r12484 (Closed) Base URL: https://v8.googlecode.com/svn/trunk
Patch Set: Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/mjsunit/bugs/bug-2337.js ('k') | test/mjsunit/compiler/regress-gvn.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/compiler/inline-arguments.js
diff --git a/test/mjsunit/compiler/inline-arguments.js b/test/mjsunit/compiler/inline-arguments.js
index df1bd2214d7e97f078f034bf599917236ba1259b..572340ab6b9f2fbfa64c47a3f210ddf184683bda 100644
--- a/test/mjsunit/compiler/inline-arguments.js
+++ b/test/mjsunit/compiler/inline-arguments.js
@@ -25,7 +25,7 @@
// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-// Flags: --allow-natives-syntax --max-opt-count=100
+// Flags: --allow-natives-syntax
function A() {
}
@@ -157,8 +157,9 @@ function test_toarr(toarr) {
test_toarr(toarr1);
test_toarr(toarr2);
-
// Test that arguments access from inlined function uses correct values.
+// TODO(mstarzinger): Tests disabled, see bug 2261
+/*
(function () {
function inner(x, y) {
"use strict";
@@ -203,66 +204,4 @@ test_toarr(toarr2);
%OptimizeFunctionOnNextCall(outer);
assertEquals(2, outer(1, 2));
})();
-
-
-// Test inlining and deoptimization of functions accessing and modifying
-// the arguments object in strict mode with mismatched arguments count.
-(function () {
- "use strict";
- function test(outerCount, middleCount, innerCount) {
- var forceDeopt = { deopt:false };
- function inner(x,y) {
- x = 0; y = 0;
- forceDeopt.deopt;
- assertSame(innerCount, arguments.length);
- for (var i = 0; i < arguments.length; i++) {
- assertSame(30 + i, arguments[i]);
- }
- }
-
- function middle(x,y) {
- x = 0; y = 0;
- if (innerCount == 1) inner(30);
- if (innerCount == 2) inner(30, 31);
- if (innerCount == 3) inner(30, 31, 32);
- assertSame(middleCount, arguments.length);
- for (var i = 0; i < arguments.length; i++) {
- assertSame(20 + i, arguments[i]);
- }
- }
-
- function outer(x,y) {
- x = 0; y = 0;
- if (middleCount == 1) middle(20);
- if (middleCount == 2) middle(20, 21);
- if (middleCount == 3) middle(20, 21, 22);
- assertSame(outerCount, arguments.length);
- for (var i = 0; i < arguments.length; i++) {
- assertSame(10 + i, arguments[i]);
- }
- }
-
- for (var step = 0; step < 4; step++) {
- if (outerCount == 1) outer(10);
- if (outerCount == 2) outer(10, 11);
- if (outerCount == 3) outer(10, 11, 12);
- if (step == 1) %OptimizeFunctionOnNextCall(outer);
- if (step == 2) delete forceDeopt.deopt;
- }
-
- %DeoptimizeFunction(outer);
- %DeoptimizeFunction(middle);
- %DeoptimizeFunction(inner);
- %ClearFunctionTypeFeedback(outer);
- %ClearFunctionTypeFeedback(middle);
- %ClearFunctionTypeFeedback(inner);
- }
-
- for (var a = 1; a <= 3; a++) {
- for (var b = 1; b <= 3; b++) {
- for (var c = 1; c <= 3; c++) {
- test(a,b,c);
- }
- }
- }
-})();
+*/
« no previous file with comments | « test/mjsunit/bugs/bug-2337.js ('k') | test/mjsunit/compiler/regress-gvn.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698