Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Function Multi Versioning: fix crashes, adjust names, priorities and mangling, support Android API 23-29, rcpc3 and mops features. #1947

Closed
ilinpv opened this issue Oct 14, 2023 · 4 comments
Labels

Comments

@ilinpv
Copy link

ilinpv commented Oct 14, 2023

Description

Improve Function Multi Versioning support in NDK.

Upstream bug

llvm/llvm-project#65669
llvm/llvm-project#81043

Commit to cherry-pick

LLVM commits to cherry-pick
llvm/llvm-project@30b0fdf
llvm/llvm-project@8ec50d6
llvm/llvm-project@b04fe22
llvm/llvm-project@e5fe3d2
llvm/llvm-project@1be0d9d
llvm/llvm-project@179ba12
llvm/llvm-project@f214933
llvm/llvm-project@99d7433
llvm/llvm-project@b42b7c8
llvm/llvm-project@e81ef46

Affected versions

r26

Canary version

No response

Host OS

Linux, Windows

Host OS version

Ubuntu 22.04 LTS

Affected ABIs

arm64-v8a

@ilinpv ilinpv added the bug label Oct 14, 2023
@ilinpv ilinpv changed the title [Bug]: Function Multi Versioning. Fix crashes, adjust names and mangling, support Android API 23-29 and rcpc3 feauture. [Bug]: Function Multi Versioning. Fix crashes, adjust names and mangling, support Android API 23-29 and rcpc3 feature. Oct 14, 2023
@ilinpv
Copy link
Author

ilinpv commented Oct 14, 2023

Preparing patch of cherry-picks to llvm-r487747 for next NDK respin.

@DanAlbert
Copy link
Member

These aren't regressions, are they? FMV wasn't in r25 at all, I thought.

@pirama-arumuga-nainar
Copy link
Collaborator

IIUC, it's a new feature in r26 toolchain that's broken at launch. One exception is that llvm/llvm-project@b04fe22 seems like a feature rather than a bug fix to me.

@DanAlbert
Copy link
Member

I'm inclined to say that we should not be backporting changes for experimental clang features that were not ready yet when we branched the toolchain. If these had been filed before we reached the release candidate that'd probably be a different story, but it's hard to justify risking the stability of the release with fixes for unfinished features. We did that last year with r25 and aiui that was the cause of #1862. It's critical that point releases do not introduce regresssions, so unless the cherry-pick is fixing a regression from the previous major release, it's not typically a candidate.

@DanAlbert DanAlbert moved this to Needs prebuilt update in NDK r27 Jan 16, 2024
@github-project-automation github-project-automation bot moved this from Needs prebuilt update to Merged in NDK r27 Mar 12, 2024
@ilinpv ilinpv changed the title [Bug]: Function Multi Versioning. Fix crashes, adjust names and mangling, support Android API 23-29 and rcpc3 feature. [Bug]: Function Multi Versioning: fix crashes, adjust names, priorities and mangling, support Android API 23-29, rcpc3 and mops features. May 29, 2024
@github-project-automation github-project-automation bot moved this to Prebuilts submitted in LLVM Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Prebuilts submitted
Status: Merged
Development

No branches or pull requests

3 participants