coffee/libyuv
1
0
Fork 0
mirror of https://chromium.googlesource.com/libyuv/libyuv synced 2025-12-06 08:46:47 +08:00
Code Issues Packages Projects Releases Wiki Activity

Revert "Do not enable libyuv_use_sme for is_android"

Browse Source 
This reverts commit 51e2e12b9b59452b1ad16c33a88bbcdd085b5450.

Reason for revert: The llvm bug fix
https://github.com/llvm/llvm-project/pull/102979 has been rolled into
Chrome in https://chromium-review.googlesource.com/5921462.

Original change's description:
> Do not enable libyuv_use_sme for is_android
>
> Revert the changes to libyuv.gni in commit dfa279f.
>
> The linker error "undefined symbol: __getauxval" referenced by
> sme-abi-init.c:26 on Android, previously reported in
> https://libyuv.g-issues.chromium.org/issues/359006069#comment2, has not
> been fixed yet. See
> https://chromium-review.googlesource.com/c/chromium/src/+/5918245?tab=checks.
>
> Change-Id: I94bd243e2863b9c316909f63f757fd95ec55dc18
> Reviewed-on: https://chromium-review.googlesource.com/c/libyuv/libyuv/+/5917455
> Reviewed-by: Frank Barchard <fbarchard@chromium.org>

Bug: 359006069
Change-Id: Ic801c1bcb65894fdfe718ba6454669c8623a2e15
Reviewed-on: https://chromium-review.googlesource.com/c/libyuv/libyuv/+/5935026
Reviewed-by: Frank Barchard <fbarchard@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Reviewed-by: George Steed <george.steed@arm.com>
This commit is contained in:
Wan-Teh Chang 2024-10-15 16:10:08 +00:00 committed by Frank Barchard
parent a8e59d2074
commit 6ac7c8f251
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
libyuv.gni
View File

@ -20,11 +20,11 @@ declare_args() {
(current_cpu == "arm" && (arm_use_neon || arm_optionally_use_neon)) (current_cpu == "arm" && (arm_use_neon || arm_optionally_use_neon))
libyuv_use_sve = current_cpu == "arm64" libyuv_use_sve = current_cpu == "arm64"
# Restrict to is_linux to work around undefined symbol linker errors on # Restrict to (is_linux || is_android) to work around undefined symbol linker
# Android, Fuchsia, macOS, and compilation errors on Windows. # errors on Fuchsia, macOS, and compilation errors on Windows.
# TODO: bug 359006069 - Remove the is_linux restriction after the linker and # TODO: bug 359006069 - Remove the restriction after the linker and
# compilation errors are fixed. # compilation errors are fixed.
libyuv_use_sme = current_cpu == "arm64" && is_linux libyuv_use_sme = current_cpu == "arm64" && (is_linux || is_android)
libyuv_use_msa = libyuv_use_msa =
(current_cpu == "mips64el" || current_cpu == "mipsel") && mips_use_msa (current_cpu == "mips64el" || current_cpu == "mipsel") && mips_use_msa
libyuv_use_mmi = libyuv_use_mmi =