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

Issue 10919237: Update TSan binaries for Linux, Mac OS 10.6 and 10.7 to r4397. (Closed)

Created:
8 years, 3 months ago by Alexander Potapenko
Modified:
8 years, 3 months ago
Reviewers:
Timur Iskhodzhanov
CC:
chromium-reviews, Dale Curtis
Visibility:
Public.

Description

Update TSan binaries for Linux, Mac OS 10.6 and 10.7 to r4397. TBR=timurrrr Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=156279

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M README.chromium View 1 1 chunk +3 lines, -3 lines 0 comments Download
M binaries/linux_x64/bin/valgrind-tsan.sh View Binary file 0 comments Download
M binaries/linux_x86/bin/valgrind-tsan.sh View Binary file 0 comments Download
M binaries/mac_10.6/bin/valgrind-tsan.sh View Binary file 0 comments Download
A binaries/mac_10.7/bin/valgrind-tsan.sh View Binary file 0 comments Download
M scripts/build-tsan-for-chromium.sh View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Alexander Potapenko
Timur, TBR Dale, FYI
8 years, 3 months ago (2012-09-12 11:32:14 UTC) #1
Timur Iskhodzhanov
LGTM, thanks! Can you please make sure that the new TSan works on Precise now? ...
8 years, 3 months ago (2012-09-12 11:33:35 UTC) #2
DaleCurtis
8 years, 3 months ago (2012-09-12 11:34:23 UTC) #3
On 2012/09/12 11:33:35, Timur Iskhodzhanov wrote:
> LGTM, thanks!
> Can you please make sure that the new TSan works on Precise now?
> The old one had problems with reading DWARF or something.

Thanks! Timur: This is in response to me running into that issue yesterday, the
newer TSAN version worked for me.

Powered by Google App Engine
This is Rietveld 408576698