Hide monobench target on debug builds.
authorMike Klein <mtklein@chromium.org>
Tue, 27 Sep 2016 20:46:29 +0000 (16:46 -0400)
committerSkia Commit-Bot <skia-commit-bot@chromium.org>
Wed, 28 Sep 2016 15:11:37 +0000 (15:11 +0000)
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

index c1cc64c..d7cdec6 100644 (file)
--- 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") {