-
Notifications
You must be signed in to change notification settings - Fork 194
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
Document a new SME support routine to query the current value of VG. #263
Document a new SME support routine to query the current value of VG. #263
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but I think it'd be worth mentioning this change in the change history table.
ARM-software/abi-aa#263 - Renamed get_runtime_vl to __arm_get_current_vg - Also return VG if currently in streaming-mode - Added static to get_aarch64_cpu_features - Added aarch64.h and included in sme-abi-init.c
Adds a calling convention for calls to the __arm_get_current_vg support routine, which preserves X1-X15, X19-X29, SP, Z0-Z31 & P0-P15. See ARM-software/abi-aa#263
Adds a calling convention for calls to the __arm_get_current_vg support routine, which preserves X1-X15, X19-X29, SP, Z0-Z31 & P0-P15. See ARM-software/abi-aa#263
Adds a calling convention for calls to the `__arm_get_current_vg` support routine, which preserves X1-X15, X19-X29, SP, Z0-Z31 & P0-P15. See ARM-software/abi-aa#263
**(Beta)** | ||
|
||
Platforms that support SME must provide a subroutine to query the current | ||
value of VG, with the subroutine having the following properties: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change to:
and the subroutine must have the following properties:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
addressed by #278
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No changes required
No description provided.