Skip to content

[ATfE] Add required strictly aligned variants for Armv8.1-M Main#298

Merged
vhscampos merged 1 commit intoarm:arm-softwarefrom
vhscampos:armv8.1m
Apr 25, 2025
Merged

[ATfE] Add required strictly aligned variants for Armv8.1-M Main#298
vhscampos merged 1 commit intoarm:arm-softwarefrom
vhscampos:armv8.1m

Conversation

@vhscampos
Copy link
Contributor

Two non-PACBTI variants needed a strictly aligned counterpart:

  • armv8.1m.main_hard_nofp_mve
  • armv8.1m.main_hard_nofp_mve_exn_rtti

because no suitable Armv7-M variant exists for these two as a fallback option. The combination of hard FP ABI and no FPU is not covered by our Armv7-M variants.

On the PACBTI side, all the existing variants needed their strictly aligned counterparts. This is because it's not safe to fallback to Armv7-M variants in this case, as these are not built for PACBTI.

This patch introduces new strictly aligned variants for Armv8.1-M Main. Two of them are for non-PACBTI + hard FP ABI + no FPU, while the rest is for all the PACBTI variants.

Two non-PACBTI variants needed a strictly aligned counterpart:

 - armv8.1m.main_hard_nofp_mve
 - armv8.1m.main_hard_nofp_mve_exn_rtti

because no suitable Armv7-M variant exists for these two as a fallback
option. The combination of hard FP ABI and no FPU is not covered by our
Armv7-M variants.

On the PACBTI side, all the existing variants needed their strictly
aligned counterparts. This is because it's not safe to fallback to
Armv7-M variants in this case, as these are not built for PACBTI.

This patch introduces new strictly aligned variants for Armv8.1-M Main.
Two of them are for non-PACBTI + hard FP ABI + no FPU, while the rest is
for all the PACBTI variants.
Copy link
Contributor

@simpal01 simpal01 left a comment

Choose a reason for hiding this comment

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

LGTM

@vhscampos vhscampos merged commit 1ffe681 into arm:arm-software Apr 25, 2025
@vhscampos vhscampos deleted the armv8.1m branch April 25, 2025 11:28
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