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

[SME][compiler-rt] Guard .variant_pcs under HAS_ASM_SME macro. #68991

Merged
merged 2 commits into from
Oct 14, 2023

Conversation

sdesmalen-arm
Copy link
Collaborator

This also adds the .variant_pcs directive to some functions from which it was previously missing.

This also adds the .variant_pcs directive to some functions from
which it was previously missing.
Copy link
Collaborator

@paulwalker-arm paulwalker-arm left a comment

Choose a reason for hiding this comment

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

If .variant_pcs is optional, why do we need it at all?

This is mostly needed because the .variant_pcs directive is required to
build the SME ABI routines, but takes it one step further by requiring
SME assembler support from the compiler. This removes the need for the
macros that use .inst <encoding> otherwise.
@sdesmalen-arm
Copy link
Collaborator Author

If .variant_pcs is optional, why do we need it at all?

AIUI .variant_pcs is not optional. I've removed the support to build these routines when SME ASM support is not available, which also gets rid of some of the macros and cleans up the code a bit further.

@sdesmalen-arm sdesmalen-arm merged commit 311bc66 into llvm:main Oct 14, 2023
3 checks passed
@sdesmalen-arm sdesmalen-arm deleted the fix-compiler-rt-sme branch February 23, 2024 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants