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

Issue 2722063002: zlib: inflate using wider loads and stores

Created:
3 years, 9 months ago by Simon Hosie
Modified:
3 years, 4 months ago
Reviewers:
cavalcantii1, cblume
CC:
msarett1, f(malita), cblume
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

zlib: inflate using wider loads and stores In inflate_fast() the output pointer always has plenty of room to write. This means that so long as the target is capable, wide un-aligned loads and stores can be used to transfer several bytes at once. When the reference distance is too short simply unroll the data a little to increase the distance. BUG=697280 Change-Id: I59854eb25d2b1e43561c8a2afaf9175bf10cf674

Patch Set 1 #

Patch Set 2 : zlib: inflate using wider loads and stores #

Patch Set 3 : zlib: inflate using wider loads and stores #

Patch Set 4 : zlib: inflate using wider loads and stores #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2175 lines, -0 lines) Patch
M third_party/zlib/BUILD.gn View 1 2 3 2 chunks +18 lines, -0 lines 0 comments Download
A third_party/zlib/contrib/arm/chunkcopy.h View 1 2 3 1 chunk +279 lines, -0 lines 0 comments Download
A third_party/zlib/contrib/arm/inffast.c View 1 2 1 chunk +307 lines, -0 lines 0 comments Download
A third_party/zlib/contrib/arm/inflate.c View 1 2 1 chunk +1571 lines, -0 lines 0 comments Download

Messages

Total messages: 22 (17 generated)
cavalcantii1
This optimization was upstreamed to zlib-ng at: https://github.com/Dead2/zlib-ng/commit/71d71ad4ebdd443f9bf1ff17d0b564b5c344a237
3 years, 8 months ago (2017-03-28 22:16:21 UTC) #7
cavalcantii1
3 years, 8 months ago (2017-03-28 22:16:58 UTC) #8
cavalcantii1
3 years, 8 months ago (2017-03-28 22:17:54 UTC) #9
cblume
3 years, 4 months ago (2017-08-15 23:56:54 UTC) #21
cavalcantii1
3 years, 4 months ago (2017-08-22 18:59:27 UTC) #22

Powered by Google App Engine
This is Rietveld 408576698