From a343017fc2a1dd3d7aba50f60f2064b787748bb2 Mon Sep 17 00:00:00 2001
From: Mike Klein <mtklein@chromium.org>
Date: Tue, 27 Sep 2016 16:46:29 -0400
Subject: [PATCH] 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>
---
 BUILD.gn | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

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") {
-- 
2.7.4