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

Issue 2888433002: Roll SwiftShader 30385f0..9ed48ba (Closed)

Created:
3 years, 7 months ago by capn
Modified:
3 years, 7 months ago
Reviewers:
govind1, sugoi1, govind
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3071
Project:
chromium
Visibility:
Public.

Description

Roll SwiftShader 30385f0..9ed48ba https://swiftshader.googlesource.com/SwiftShader.git/+log/30385f0..9ed48ba - Fixes LTO causing hard crash on Linux official builds. - Fixes buffer overflow. BUG=720933 BUG=719291 NOTRY=true NOPRESUBMIT=true TBR=kbr@chromium.org Review-Url: https://codereview.chromium.org/2888433002 Cr-Commit-Position: refs/branch-heads/3071@{#568} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} Committed: https://chromium.googlesource.com/chromium/src/+/b01b9acf5b1e54795099cdd7a62dc07ce01f06e7

Patch Set 1 #

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

Messages

Total messages: 21 (11 generated)
capn
PTAL. Merge approved at https://bugs.chromium.org/p/chromium/issues/detail?id=720933#c11
3 years, 7 months ago (2017-05-15 20:58:54 UTC) #2
govind
lgtm
3 years, 7 months ago (2017-05-15 21:02:29 UTC) #4
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/2888433002/1
3 years, 7 months ago (2017-05-15 21:11:48 UTC) #6
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-15 21:11:49 UTC) #8
sugoi1
On 2017/05/15 21:11:49, commit-bot: I haz the power wrote: > No L-G-T-M from a valid ...
3 years, 7 months ago (2017-05-15 21:41:17 UTC) #9
govind1
lgtm
3 years, 7 months ago (2017-05-15 21:42:14 UTC) #11
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/2888433002/1
3 years, 7 months ago (2017-05-15 21:43:10 UTC) #12
commit-bot: I haz the power
CLs for remote refs other than refs/heads/master must contain NOTRY=true and NOPRESUBMIT=true in order for ...
3 years, 7 months ago (2017-05-15 21:43:12 UTC) #14
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/2888433002/1
3 years, 7 months ago (2017-05-15 21:46:49 UTC) #18
commit-bot: I haz the power
3 years, 7 months ago (2017-05-15 21:49:15 UTC) #21
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b01b9acf5b1e54795099cdd7a62d...

Powered by Google App Engine
This is Rietveld 408576698