Reviewers: Jakob,

Message:
PTAL.

Description:
Enable --smi-only-arrays flag in test case for no-snapshot build.

TEST=element-kind.js


Please review this at http://codereview.chromium.org/8077008/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M test/mjsunit/element-kind.js


Index: test/mjsunit/element-kind.js
diff --git a/test/mjsunit/element-kind.js b/test/mjsunit/element-kind.js
index 8c5121058d0b0e104852bcad9c207e238ba1cd50..d61e26a1738440c49ef7090f59a8ce8bf2239593 100644
--- a/test/mjsunit/element-kind.js
+++ b/test/mjsunit/element-kind.js
@@ -25,11 +25,23 @@
 // (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
-// Test element kind of objects
+// Flags: --allow-natives-syntax --smi-only-arrays
+// Test element kind of objects.
+// Since --smi-only-arrays affects builtins, its default setting at compile
+// time sticks if built with snapshot.  If --smi-only-arrays is deactivated
+// by default, only a no-snapshot build actually has smi-only arrays enabled
+// in this test case.  Depending on whether smi-only arrays are actually
+// enabled, this test takes the appropriate code path to check smi-only arrays.
+

 support_smi_only_arrays = %HasFastSmiOnlyElements([]);

+if (support_smi_only_arrays) {
+  print("Tests include smi-only arrays.");
+} else {
+  print("Tests do NOT include smi-only arrays.");
+}
+
 var element_kind = {
   fast_smi_only_elements            :  0,
   fast_elements                     :  1,


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to