From faaa90d13c3171c76cecf25786dbed5f9d3e9e91 Mon Sep 17 00:00:00 2001 From: "mvstanton@chromium.org" Date: Mon, 8 Jul 2013 09:11:56 +0000 Subject: [PATCH] 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= R=hpayer@chromium.org Review URL: https://codereview.chromium.org/18753005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@15543 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- test/mjsunit/allocation-site-info.js | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/test/mjsunit/allocation-site-info.js b/test/mjsunit/allocation-site-info.js index 86c28aac63..1369a2d2ec 100644 --- a/test/mjsunit/allocation-site-info.js +++ b/test/mjsunit/allocation-site-info.js @@ -133,9 +133,7 @@ if (support_smi_only_arrays) { 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 @@ if (support_smi_only_arrays) { 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();