From a5f910eb2ba8877f8bbfdba57a4d3f8012ef8736 Mon Sep 17 00:00:00 2001 From: "danno@chromium.org" Date: Tue, 19 Nov 2013 17:29:43 +0000 Subject: [PATCH] Turn ASSERTs to CHECKs in test-heap.cc This fixes unused variable compile failures in release builds introduced in r17887. R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/76413004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17888 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- test/cctest/test-heap.cc | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/test/cctest/test-heap.cc b/test/cctest/test-heap.cc index 1c81995..448c589 100644 --- a/test/cctest/test-heap.cc +++ b/test/cctest/test-heap.cc @@ -3584,7 +3584,7 @@ TEST(EnsureAllocationSiteDependentCodesProcessed) { // One allocation site should have been created. int new_count = AllocationSitesCount(heap); - ASSERT(new_count == (count + 1)); + CHECK_EQ(new_count, (count + 1)); site = Handle::cast( global_handles->Create( AllocationSite::cast(heap->allocation_sites_list()))); @@ -3592,15 +3592,15 @@ TEST(EnsureAllocationSiteDependentCodesProcessed) { CompileRun("%OptimizeFunctionOnNextCall(bar); bar();"); DependentCode::GroupStartIndexes starts(site->dependent_code()); - ASSERT(starts.number_of_entries() >= 1); + CHECK_GE(starts.number_of_entries(), 1); int index = starts.at(DependentCode::kAllocationSiteTransitionChangedGroup); - ASSERT(site->dependent_code()->is_code_at(index)); + CHECK(site->dependent_code()->is_code_at(index)); Code* function_bar = site->dependent_code()->code_at(index); Handle bar_handle = v8::Utils::OpenHandle( *v8::Handle::Cast( CcTest::global()->Get(v8_str("bar")))); - ASSERT(bar_handle->code() == function_bar); + CHECK_EQ(bar_handle->code(), function_bar); } // Now make sure that a gc should get rid of the function, even though we @@ -3613,5 +3613,5 @@ TEST(EnsureAllocationSiteDependentCodesProcessed) { // longer referred to by dependent_code(). DependentCode::GroupStartIndexes starts(site->dependent_code()); int index = starts.at(DependentCode::kAllocationSiteTransitionChangedGroup); - ASSERT(!(site->dependent_code()->is_code_at(index))); + CHECK(!(site->dependent_code()->is_code_at(index))); } -- 2.7.4