Revision: 17888
Author:   [email protected]
Date:     Tue Nov 19 17:29:43 2013 UTC
Log:      Turn ASSERTs to CHECKs in test-heap.cc

This fixes unused variable compile failures in release builds introduced in r17887.

[email protected]

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

Modified:
 /branches/bleeding_edge/test/cctest/test-heap.cc

=======================================
--- /branches/bleeding_edge/test/cctest/test-heap.cc Tue Nov 19 16:57:49 2013 UTC +++ /branches/bleeding_edge/test/cctest/test-heap.cc Tue Nov 19 17:29:43 2013 UTC
@@ -3584,7 +3584,7 @@

     // 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<AllocationSite>::cast(
         global_handles->Create(
             AllocationSite::cast(heap->allocation_sites_list())));
@@ -3592,15 +3592,15 @@
     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<JSFunction> bar_handle =
         v8::Utils::OpenHandle(
             *v8::Handle<v8::Function>::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 @@
   // 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)));
 }

--
--
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