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

Issue 175026: Remove the GDB-related code from memcheck_analyze.py to use it from tsan_analyze.py in the future.

Created:
11 years, 3 months ago by not_the_right_glider
Modified:
4 years, 10 months ago
Reviewers:
dank
CC:
chromium-reviews_googlegroups.com, not_the_right_glider, Nirnimesh, Pam (message me for reviews), dank, the_wrong_timurrrr
Visibility:
Public.

Description

This CL moves the GDB-related code to a helper module making it usable by tsan_analyze.py as well as memcheck_analyze.py The corresponding patch for tsan_analyze.py is ready (available on demand), but at the moment we're adding Valgrind-style XML logging to ThreadSanitizer, so tsan_analyze.py may be reworked completely after that.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+91 lines, -76 lines) Patch
A tools/valgrind/gdb_helper.py View 1 chunk +88 lines, -0 lines 0 comments Download
M tools/valgrind/memcheck_analyze.py View 3 chunks +3 lines, -76 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698