Revert 169592: Now that the libvpx roll is in, this should work.
- Revert 169589: Caused a bunch of chromeos-chrome-25.0.1336.0_alpha-r1: /var/cache/chromeos-cache/distfiles/target/chrome-src/src/third_party/libvpx/source/libvpx/vp8/encoder/x86/quantize_sse2.asm:58: error: undefined symbol `vp8_block_coeff' (first use) Likely because third_party/libvpx/libvpx.gyp still uses the old form. - Use new -fsanitize=address instead of deprecated -faddress-sanitizer BUG=159580,162783 Review URL: https://codereview.chromium.org/11416192 TBR=thakis@chromium.org Review URL: https://codereview.chromium.org/11411191 TBR=thakis@chromium.org Review URL: https://codereview.chromium.org/11308229 git-svn-id: http://src.chromium.org/svn/trunk/src/build@169670 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
This commit is contained in:
Родитель
54e0576113
Коммит
5cb0f17456
17
common.gypi
17
common.gypi
|
@ -2700,13 +2700,12 @@
|
|||
'target_conditions': [
|
||||
['_toolset=="target"', {
|
||||
'cflags': [
|
||||
'-faddress-sanitizer',
|
||||
'-fsanitize=address',
|
||||
'-fno-omit-frame-pointer',
|
||||
# See http://crbug.com/159580
|
||||
'-w',
|
||||
'-w', # http://crbug.com/162783
|
||||
],
|
||||
'ldflags': [
|
||||
'-faddress-sanitizer',
|
||||
'-fsanitize=address',
|
||||
],
|
||||
'defines': [
|
||||
'ADDRESS_SANITIZER',
|
||||
|
@ -3164,9 +3163,8 @@
|
|||
# Due to issues in Clang build system, using ASan on 32-bit
|
||||
# binaries on x86_64 host is problematic.
|
||||
# TODO(eugenis): re-enable.
|
||||
'-faddress-sanitizer',
|
||||
# See http://crbug.com/159580
|
||||
'-w',
|
||||
'-fsanitize=address',
|
||||
'-w', # http://crbug.com/162783
|
||||
],
|
||||
'ldflags!': [
|
||||
'-faddress-sanitizer',
|
||||
|
@ -3287,9 +3285,8 @@
|
|||
['asan==1', {
|
||||
'xcode_settings': {
|
||||
'OTHER_CFLAGS': [
|
||||
'-faddress-sanitizer',
|
||||
# See http://crbug.com/159580
|
||||
'-w',
|
||||
'-fsanitize=address',
|
||||
'-w', # http://crbug.com/162783
|
||||
],
|
||||
},
|
||||
'defines': [
|
||||
|
|
Загрузка…
Ссылка в новой задаче