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

Unified Diff: test/mjsunit/array-literal-transitions.js

Issue 9050001: Ensure newly allocated empty Arrays are transitioned to FAST_ELEMENT (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: fix existing and add new tests Created 9 years 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
Index: test/mjsunit/array-literal-transitions.js
diff --git a/test/mjsunit/array-literal-transitions.js b/test/mjsunit/array-literal-transitions.js
index f657525eb6f1af773732decd853e5eaed0db0c78..949c8f1d23343d2691791161cd973c1689b9cf28 100644
--- a/test/mjsunit/array-literal-transitions.js
+++ b/test/mjsunit/array-literal-transitions.js
@@ -45,6 +45,12 @@ if (support_smi_only_arrays) {
function get(foo) { return foo; } // Used to generate dynamic values.
function array_literal_test() {
+ var c0 = [1, 2, get(3.5)];
+ assertTrue(%HasFastDoubleElements(c0));
+ assertEquals(3.5, c0[2]);
+ assertEquals(2, c0[1]);
+ assertEquals(1, c0[0]);
+
var a0 = [1, 2, 3];
assertTrue(%HasFastSmiOnlyElements(a0));
var a1 = [get(1), get(2), get(3)];
@@ -55,12 +61,6 @@ function array_literal_test() {
var b1 = [get(1), get(2), get("three")];
assertTrue(%HasFastElements(b1));
- var c0 = [1, 2, get(3.5)];
- assertTrue(%HasFastDoubleElements(c0));
- assertEquals(3.5, c0[2]);
- assertEquals(2, c0[1]);
- assertEquals(1, c0[0]);
-
var c1 = [1, 2, 3.5];
assertTrue(%HasFastDoubleElements(c1));
assertEquals(3.5, c1[2]);

Powered by Google App Engine
This is Rietveld 408576698