Revision: 15543
Author:   [email protected]
Date:     Mon Jul  8 02:11:56 2013
Log:      Allocation-site-info test, removed TODOs.

Some code was commented out earlier as a todo. Now the code can be reenabled,
because allocation site feedback is working there again.

BUG=
[email protected]

Review URL: https://codereview.chromium.org/18753005
http://code.google.com/p/v8/source/detail?r=15543

Modified:
 /branches/bleeding_edge/test/mjsunit/allocation-site-info.js

=======================================
--- /branches/bleeding_edge/test/mjsunit/allocation-site-info.js Fri Jun 28 06:16:14 2013 +++ /branches/bleeding_edge/test/mjsunit/allocation-site-info.js Mon Jul 8 02:11:56 2013
@@ -133,9 +133,7 @@
   obj = fastliteralcase(get_standard_literal(), 1.5);
   assertKind(elements_kind.fast_double, obj);
   obj = fastliteralcase(get_standard_literal(), 2);
-  // TODO(hpayer): bring the following assert back as soon as allocation
-  // sites work again for fast literals
-  //assertKind(elements_kind.fast_double, obj);
+  assertKind(elements_kind.fast_double, obj);

   // The test below is in a loop because arrays that live
   // at global scope without the chance of being recreated
@@ -175,9 +173,7 @@
   obj = fastliteralcase_smifast("carter");
   assertKind(elements_kind.fast, obj);
   obj = fastliteralcase_smifast(2);
-  // TODO(hpayer): bring the following assert back as soon as allocation
-  // sites work again for fast literals
-  //assertKind(elements_kind.fast, obj);
+  assertKind(elements_kind.fast, obj);

   function newarraycase_smidouble(value) {
     var a = new Array();

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to