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

Issue 1294293003: Re-enable LLVM LTO on Neon targets. (Closed)

Created:
5 years, 4 months ago by Andrew MacDonald
Modified:
5 years, 4 months ago
Reviewers:
pcc
CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com, qiang.lu, niklas.enbom, yujie_mao (webrtc)
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Re-enable LLVM LTO on Neon targets. LTO was disabled due to a GCC compiler bug that does not affect LLVM. This fixes the build in the cfi_vptr==1 configuration, which requires LLVM LTO. TBR=pcc@google.com BUG=chromium:469376 patch from issue 1295663002 at patchset 1 (http://crrev.com/1295663002#ps1) Committed: https://chromium.googlesource.com/external/webrtc/+/7b3de4b7d0d30eed77f986cd0efb7c4deb4d1bb1

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M webrtc/build/arm_neon.gypi View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Andrew MacDonald
Committed patchset #1 (id:1) manually as 7b3de4b7d0d30eed77f986cd0efb7c4deb4d1bb1 (presubmit successful).
5 years, 4 months ago (2015-08-18 04:06:05 UTC) #1
Andrew MacDonald
5 years, 4 months ago (2015-08-18 04:06:18 UTC) #2
Message was sent while issue was closed.

          

Powered by Google App Engine
This is Rietveld 408576698