Skip to content
Commit c2c06719 authored by Andrew Grieve's avatar Andrew Grieve Committed by Chromium LUCI CQ
Browse files

Revert "[clang][MLGO] enable ml inliner on linux, when targeting android"

This reverts commit 0cd2cf4d.

Reason for revert: Suspect cause randomness in android-binary-size

Original change's description:
> [clang][MLGO] enable ml inliner on linux, when targeting android
>
> This follows up https://crrev.com/c/3829793. Quoting from it:
>
> Benchmarks (on an earlier tree):
> Android GO speedometer2:
> https://pinpoint-dot-chromeperf.appspot.com/job/14d399416a0000
> (summary: 0.6% (regr) to -2.0% (improvement))
>
> Pixel2 speedometer2:
> https://pinpoint-dot-chromeperf.appspot.com/job/12f3871c6a0000
> (summary: 2.3% (regr) to -1.1% (improvement))
>
> Pixel4 speedometer2:
> https://pinpoint-dot-chromeperf.appspot.com/job/14fd83d86a0000
> (summary: 1.8% (regr) to -2.8% (improvement))
>
> Size reductions (on an earlier tree):
> arm32: -375KB
> arm64: -1.35MB
>
>
> Bug: 1241834
> Change-Id: I4576bf9022ee084bec424d073a2a1e68e56fec6f
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3852308


> Reviewed-by: default avatarHans Wennborg <hans@chromium.org>
> Commit-Queue: Hans Wennborg <hans@chromium.org>
> Reviewed-by: default avatarAndrew Grieve <agrieve@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1045024}

Bug: 1241834, 1361952
Change-Id: I51bf2c2876e8db0b91348344345be3ce6c42dd20
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3887360
Auto-Submit: Andrew Grieve <agrieve@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#1045336}
parent 690f500c
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment