Skip to content
Commit 0e6b3ca1 authored by Adenilson Cavalcanti's avatar Adenilson Cavalcanti Committed by Copybara-Service
Browse files

Revert "[zlib] Adding PMULL based CRC-32 for aarch64"

This reverts commit a0771caebe87477558454cc6d793562e3afe74ac.

Reason for revert: Missing call to CPU features detection

Original change's description:
> [zlib] Adding PMULL based CRC-32 for aarch64
>
> Using the polynomial multiplication instruction allows to improve a bit
> more the decompression speed of gzipped content in zlib.
>
> The average gains are not massive though (around +4.9%), but is more expressive
> for some relevant content (e.g. +10% for HTMLx4 & JPEG, +6% for JS) in more
> recent chip designs.
>
> Since Chrome is distributed as a 64bit binary (i.e. aarch64) only for
> higher end devices, it should be safe from a performance perspective.
>
> For lower spec devices, the same serial crc32 code using the crypto
> extensions is used, therefore no change on behavior for 32bit.
>
> Bug: 873725
> Change-Id: I33b9b345b82b5256ec922324984f9a371949bbe6
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2422723


> Reviewed-by: default avatarChris Blume <cblume@chromium.org>
> Commit-Queue: Adenilson Cavalcanti <cavalcantii@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1006184}

Bug: 873725
Change-Id: I7b6a2bfb60a96bc640b59187c5dd63590d09599e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3659852


Reviewed-by: default avatarNoel Gordon <noel@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Noel Gordon <noel@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1006331}
NOKEYCHECK=True
GitOrigin-RevId: 42a820acf433146b284bb195b0c3507244cf1c30
parent 80b28c99
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment