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

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

Created:
5 years, 3 months ago by Andrew MacDonald
Modified:
5 years, 3 months ago
Reviewers:
pcc
CC:
webrtc-reviews_webrtc.org, Raymond Toy
Base URL:
https://chromium.googlesource.com/external/webrtc/deps/third_party/openmax.git@master
Target Ref:
refs/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. Patch from: https://webrtc-codereview.appspot.com/52269004/ TBR=pcc@google.com BUG=chromium:469376 Committed: https://chromium.googlesource.com/external/webrtc/deps/third_party/openmax/+/2eb98d819bed3082071a09268b886bc4496c6fb5

Patch Set 1 #

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

Messages

Total messages: 6 (2 generated)
Andrew MacDonald
5 years, 3 months ago (2015-08-26 21:56:07 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1318803004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1318803004/1
5 years, 3 months ago (2015-08-26 21:56:23 UTC) #3
commit-bot: I haz the power
CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true and NOPRESUBMIT=true in order for ...
5 years, 3 months ago (2015-08-26 21:56:24 UTC) #5
Andrew MacDonald
5 years, 3 months ago (2015-08-26 21:57:20 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
2eb98d819bed3082071a09268b886bc4496c6fb5 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698