Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(369)

Unified Diff: gyp/bench.gyp

Issue 347443002: Add nanobench, a fresh look at our microbenchmark harness. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: printf -> SkDebugf Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « bench/nanobench.cpp ('k') | gyp/bench.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gyp/bench.gyp
diff --git a/gyp/bench.gyp b/gyp/bench.gyp
index 9089bf8bc07d04a728ef183747a2623bb6504573..cbbf27f9a7017a5251fc4895952049da84a078b8 100644
--- a/gyp/bench.gyp
+++ b/gyp/bench.gyp
@@ -9,13 +9,10 @@
'target_name': 'bench',
'type': 'executable',
'dependencies': [
- 'etc1.gyp:libetc1',
'flags.gyp:flags',
'jsoncpp.gyp:jsoncpp',
'skia_lib.gyp:skia_lib',
djsollen 2014/06/25 19:52:40 if skia_lib is pulled in by the bench gypi then we
mtklein 2014/06/25 20:28:08 Let me hold off on this for another CL. I actuall
'tools.gyp:crash_handler',
- 'tools.gyp:resources',
- 'tools.gyp:sk_tool_utils',
'tools.gyp:timer',
],
'sources': [
@@ -27,28 +24,27 @@
'../bench/benchmain.cpp',
],
'conditions': [
- ['skia_gpu == 1',
- {
- 'include_dirs' : [
- '../src/gpu',
- ],
- 'dependencies': [
- 'gputest.gyp:skgputest',
- ],
- },
- ],
- ['skia_android_framework == 1',
- {
- 'libraries': [
- '-lskia',
- ],
- },
- ],
+ ['skia_android_framework == 1', {
+ 'libraries': [ '-lskia' ],
+ }],
],
'includes': [
'bench.gypi',
'gmslides.gypi',
],
},
+ {
+ 'target_name': 'nanobench',
+ 'type': 'executable',
+ 'sources': [
+ '../bench/nanobench.cpp',
tfarina 2014/06/25 20:06:25 NanoBenchMain.cpp? otherwise can it be either nan
mtklein 2014/06/25 20:28:08 I prefer nanobench.cpp. It compiles into a binary
tfarina 2014/06/25 21:08:04 I don't understand why you call naming a file nano
+ ],
+ 'includes': [ 'bench.gypi' ],
+ 'dependencies': [
+ 'flags.gyp:flags',
+ 'tools.gyp:crash_handler',
+ 'tools.gyp:timer',
+ ],
+ },
],
}
« no previous file with comments | « bench/nanobench.cpp ('k') | gyp/bench.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698