Revert 84802 - Disable ICF for Valgrind/ThreadSanitizer Release builds

Revert 84802 - Disable ICF for Valgrind/ThreadSanitizer Release builds

With ICF the linker merges identical functions into one block of assembly,
which leads to
a) wrong symbols for some functions - which is debugger/Valgrind-unfriendly
b) all ThreadSanitizer annotations end up being one function which makes TSan crazy.

Since we run release code (with -O1 -g) on Valgrind bots,
we have to disable ICF if release_valgrind_build!=0.

BUG=82114
TEST=linux_tsan trybot goes green
Review URL: http://codereview.chromium.org/6991009

TBR=timurrrr@chromium.org
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=84804

[Timur Iskhodzhanov, 2011-05-10 17:35:47]

LGTM.
Sorry for breaking this in the first place... :(