[test] Test expectations in cctest should use CHECK and not DCHECK.

test/cctest/compiler/test-run-bytecode-graph-builder.cc

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // TODO(jochen): Remove this after the setting is turned on globally.
 #define V8_IMMINENT_DEPRECATION_WARNINGS
 
 #include <utility>
 
 #include "src/compiler/pipeline.h"
@@ skipping 158 matching lines @@
 
 
 template <int N, typename T = Handle<Object>>
 struct ExpectedSnippet {
   const char* code_snippet;
   T return_value_and_parameters[N + 1];
 
   inline T return_value() const { return return_value_and_parameters[0]; }
 
   inline T parameter(int i) const {
-    DCHECK_GE(i, 0);
-    DCHECK_LT(i, N);
+    CHECK_GE(i, 0);
+    CHECK_LT(i, N);
     return return_value_and_parameters[1 + i];
   }
 };
 
 
 TEST(BytecodeGraphBuilderReturnStatements) {
   HandleAndZoneScope scope;
   Isolate* isolate = scope.main_isolate();
   Zone* zone = scope.main_zone();
   Factory* factory = isolate->factory();
@@ skipping 991 matching lines @@
     auto callable = tester.GetCallable<Handle<Object>>("f");
     Handle<Object> return_value =
         callable(snippets[i].parameter(0)).ToHandleChecked();
     CHECK(return_value->SameValue(*snippets[i].return_value()));
   }
 }
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8