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

Unified Diff: tools/dm_flags.py

Issue 992203004: Fix bug in Valgrind flags config. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: nanobench too Created 5 years, 9 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 | « tools/dm_flags.json ('k') | tools/nanobench_flags.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/dm_flags.py
diff --git a/tools/dm_flags.py b/tools/dm_flags.py
index 23612428f307b270d26554c20eec78f12576089c..62c93e40c95808b6501b7a2ae870065e8e4e4f1e 100755
--- a/tools/dm_flags.py
+++ b/tools/dm_flags.py
@@ -61,9 +61,9 @@ def get_args(bot):
# PDF + .webp -> jumps depending on uninitialized memory. skia:3505
if 'Valgrind' in bot:
blacklist.extend('pdf _ .webp'.split(' '))
- if 'GPU' in bot:
+ if '_GPU' in bot:
borenet 2015/03/10 14:50:04 Maybe this should be Valgrind_GPU
mtklein 2015/03/10 14:58:02 Done.
args.append('--nocpu')
- elif 'CPU' in bot:
+ elif '_CPU' in bot:
args.append('--nogpu')
if blacklist:
@@ -107,7 +107,7 @@ def self_test():
'Test-Android-Nexus7-Tegra3-Arm7-Release',
'Test-Android-Xoom-Tegra2-Arm7-Release',
'Test-ChromeOS-Alex-GMA3150-x86-Debug',
- 'Test-Ubuntu12-ShuttleA-GTX550Ti-x86_64-Release-Valgrind_CPU',
+ 'Test-Ubuntu14-GCE-NoGPU-x86_64-Release-Valgrind_CPU',
'Test-Ubuntu12-ShuttleA-GTX550Ti-x86_64-Release-Valgrind_GPU',
'Test-Win7-ShuttleA-HD2000-x86-Debug-ANGLE',
]
« no previous file with comments | « tools/dm_flags.json ('k') | tools/nanobench_flags.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698