From: Mike Klein <mtklein@chromium.org>
Date: Tue, 27 Sep 2016 20:46:29 +0000 (-0400)
Subject: Hide monobench target on debug builds.
X-Git-Tag: accepted/tizen/5.0/unified/20181102.025319~106^2~153
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a343017fc2a1dd3d7aba50f60f2064b787748bb2;p=platform%2Fupstream%2FlibSkiaSharp.git

Hide monobench target on debug builds.

Unlike nanobench this tool has no purpose when built in Debug mode.
Just don't let it happen.

BUG=skia:

GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2718

Change-Id: Iaa7b8c44d46024485d4f5ce3d9c3e33d865b99d7
Reviewed-on: https://skia-review.googlesource.com/2718
Reviewed-by: Herb Derby <herb@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
---

diff --git a/BUILD.gn b/BUILD.gn
index c1cc64c..d7cdec6 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -849,15 +849,17 @@ if (skia_enable_tools) {
     testonly = true
   }
 
-  executable("monobench") {
-    sources = [
-      "tools/monobench.cpp",
-    ]
-    deps = [
-      ":bench",
-      ":skia",
-    ]
-    testonly = true
+  if (!is_debug) {  # I've benchmarked debug code once too many times...
+    executable("monobench") {
+      sources = [
+        "tools/monobench.cpp",
+      ]
+      deps = [
+        ":bench",
+        ":skia",
+      ]
+      testonly = true
+    }
   }
 
   executable("nanobench") {