diff options
| author | glider@chromium.org <glider@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-18 22:54:44 +0000 |
|---|---|---|
| committer | glider@chromium.org <glider@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-18 22:54:44 +0000 |
| commit | 8b22522ce5c76e1e6ee60775d59018ddb3de5fbb (patch) | |
| tree | 6b7b64d486993d3ffdefa3cefc70b8976c6908ba /build | |
| parent | cad72014d56088c22d415cef6802abb3d093bdb6 (diff) | |
| download | chromium_src-8b22522ce5c76e1e6ee60775d59018ddb3de5fbb.zip chromium_src-8b22522ce5c76e1e6ee60775d59018ddb3de5fbb.tar.gz chromium_src-8b22522ce5c76e1e6ee60775d59018ddb3de5fbb.tar.bz2 | |
Disable source fortification under Clang on Linux.
Right now source fortification isn't supported by Clang (http://llvm.org/bugs/show_bug.cgi?id=16821)
It may also prevent the sanitizer tools from catching some bugs.
BUG=162783,341809
R=hans@chromium.org
Review URL: https://codereview.chromium.org/138913015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@251868 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'build')
| -rw-r--r-- | build/common.gypi | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/build/common.gypi b/build/common.gypi index 9cbe372..02d600d 100644 --- a/build/common.gypi +++ b/build/common.gypi @@ -2898,7 +2898,11 @@ ['win_use_allocator_shim==0', { 'defines': ['NO_TCMALLOC'], }], - ['os_posix==1', { + # _FORTIFY_SOURCE isn't really supported by Clang now, see + # http://llvm.org/bugs/show_bug.cgi?id=16821. + # TODO(glider): once the bug is fixed, disable source fortification + # under the sanitizer tools only. + ['os_posix==1 and (OS!="linux" or clang!=1)', { 'target_conditions': [ ['chromium_code==1', { # Non-chromium code is not guaranteed to compile cleanly |