diff options
| author | pcc <pcc@chromium.org> | 2015-08-25 17:03:57 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-08-26 00:04:57 +0000 |
| commit | e510374e7f8b5a9ee03c10d120f582b85f5b3cf8 (patch) | |
| tree | c71e0804ce2d2a77766787cf2d730e33024e3eca /sandbox/linux | |
| parent | dee98ed1c1c86594e60ab41540bfda7c4e0ff5aa (diff) | |
| download | chromium_src-e510374e7f8b5a9ee03c10d120f582b85f5b3cf8.zip chromium_src-e510374e7f8b5a9ee03c10d120f582b85f5b3cf8.tar.gz chromium_src-e510374e7f8b5a9ee03c10d120f582b85f5b3cf8.tar.bz2 | |
build: On Android, disable LTO for specific targets only when targeting GCC.
The set of targets for which LTO is problematic will most likely differ
between GCC and LLVM. We also need different logic for LLVM (specifically,
LTO should not be disabled when building with cfi_vptr==1, as the -flto flag
must be passed together with -fsanitize=cfi*); this unbreaks the build for
those targets on Android when cfi_vptr==1.
Also add a drive-by TODO to use -lto_library flag on Mac.
BUG=469376
R=thakis@chromium.org, fdegans@chromium.org
TBR=jam@chromium.org
Review URL: https://codereview.chromium.org/1289623006
Cr-Commit-Position: refs/heads/master@{#345492}
Diffstat (limited to 'sandbox/linux')
| -rw-r--r-- | sandbox/linux/sandbox_linux.gypi | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sandbox/linux/sandbox_linux.gypi b/sandbox/linux/sandbox_linux.gypi index 1958258..6ab991c 100644 --- a/sandbox/linux/sandbox_linux.gypi +++ b/sandbox/linux/sandbox_linux.gypi @@ -171,7 +171,7 @@ 'includes': [ # Disable LTO due to compiler bug # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57703 - '../../build/android/disable_lto.gypi', + '../../build/android/disable_gcc_lto.gypi', ], 'include_dirs': [ '../..', |