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

Issue 2854253003: Roll src/third_party/libyuv/ 3b583396b..945ea1b74 (2 commits) (Closed)

Created:
3 years, 7 months ago by fbarchard1
Modified:
3 years, 7 months ago
CC:
chromium-reviews
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Roll src/third_party/libyuv/ 3b583396b..945ea1b74 (2 commits) https://chromium.googlesource.com/libyuv/libyuv.git/+log/3b583396bf13..945ea1b74630 $ git log 3b583396b..945ea1b74 --date=short --no-merges --format='%ad %ae %s' 2017-05-02 fbarchard mips switch sgtu to sltu for clang in ndk r14 2017-04-25 vigneshv Fix mips build on android ndk r14+ The new llvm (3.8+) has a built assembler, but it was missing the sgtu (set greater than unsigned) pseudo op. Its been fixed for 4.0, but the pseudo op just swaps the arguments and uses sltu. In this change the source is switched to use sltu directly. Created with: roll-dep src/third_party/libyuv R=kjellander@chromium.org BUG=libyuv:700 TEST=built under android with mips target Review-Url: https://codereview.chromium.org/2854253003 Cr-Commit-Position: refs/heads/master@{#469177} Committed: https://chromium.googlesource.com/chromium/src/+/2f32140a3bf95b2e2da567f53b5b5bdd8ece1508

Patch Set 1 #

Patch Set 2 : bump version comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 17 (12 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2854253003/20001
3 years, 7 months ago (2017-05-03 18:52:28 UTC) #9
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. CQ run can only be started once the ...
3 years, 7 months ago (2017-05-03 18:52:29 UTC) #11
kjellander_chromium
lgtm
3 years, 7 months ago (2017-05-03 19:31:12 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2854253003/20001
3 years, 7 months ago (2017-05-03 22:46:38 UTC) #14
commit-bot: I haz the power
3 years, 7 months ago (2017-05-03 22:51:45 UTC) #17
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/2f32140a3bf95b2e2da567f53b5b...

Powered by Google App Engine
This is Rietveld 408576698