From 8764dbc54c1a85f836ae97f5f70896cbf104faa3 Mon Sep 17 00:00:00 2001 From: Clemens Backes Date: Fri, 22 Nov 2019 11:39:05 +0100 Subject: [PATCH] Make DebugPrint use Object::Print if available If v8_enable_object_print is set to true, we should use Object::Print instead of Brief(Object). R=jkummerow@chromium.org Change-Id: I70583c15834f9332aba7760b5e104136712d4e0d Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1930613 Reviewed-by: Jakob Kummerow Commit-Queue: Clemens Backes Cr-Commit-Position: refs/heads/master@{#65112} --- src/runtime/runtime-test.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/runtime/runtime-test.cc b/src/runtime/runtime-test.cc index 6c4ea17937..ceaf47d607 100644 --- a/src/runtime/runtime-test.cc +++ b/src/runtime/runtime-test.cc @@ -751,7 +751,7 @@ RUNTIME_FUNCTION(Runtime_DebugPrint) { Object object = maybe_object.GetHeapObjectOrSmi(); bool weak = maybe_object.IsWeak(); -#ifdef DEBUG +#ifdef OBJECT_PRINT if (object.IsString() && !isolate->context().is_null()) { DCHECK(!weak); // If we have a string, assume it's a code "marker"