Skip to content

[AArch64][FMV] Add a non-comprehensive test for ACLE FMV #87

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

Merged
merged 4 commits into from
Feb 2, 2024

Conversation

jroelofs
Copy link
Contributor

@jroelofs jroelofs commented Feb 1, 2024

This covers the ones we currently have enabled via apple.inc in compiler-rt, as well as a couple that are not. Each test uses Function Multi Versioning to check for the presence or absence of each given feature, and subsequently exercises that feature if it is presumed to exist. For features that are not thought to exist by the FMV infra, it forks and attempts to try the feature anyway, expecting the resulting thread to die, thus affirming the inverse.

Copy link
Contributor

@ilinpv ilinpv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for compiler-rt FMV tests! We are going to add more on runtime detection implementing __builtin_cpu_supports aarch64 part https://reviews.llvm.org/D153153 on top of recently merged llvm/llvm-project#68919

@ilinpv ilinpv requested a review from labrinea February 1, 2024 17:53
@jroelofs jroelofs force-pushed the jroelofs/fmv-functional-test branch from 31ab960 to 719c9ed Compare February 1, 2024 20:47
@jroelofs jroelofs force-pushed the jroelofs/fmv-functional-test branch from 719c9ed to b0941c3 Compare February 1, 2024 20:52
Copy link
Contributor

@ilinpv ilinpv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to go with Linux part disabled - we'll take it over and address separately, thank you!

@jroelofs jroelofs merged commit 85e64d5 into llvm:main Feb 2, 2024
@jroelofs jroelofs deleted the jroelofs/fmv-functional-test branch February 2, 2024 22:13
ilinpv added a commit to ilinpv/llvm-test-suite that referenced this pull request Feb 21, 2024
ilinpv added a commit that referenced this pull request Feb 23, 2024
* [AArch64][FMV] Enable tests for ACLE FMV on Linux.

Enabling Linux part of #87

* Update SingleSource/UnitTests/AArch64/acle-fmv-features.c

Co-authored-by: Jon Roelofs <[email protected]>

---------

Co-authored-by: Jon Roelofs <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants