Skip to content

Conversation

@ionut-arm
Copy link
Contributor

Make assumptions clearer about the impact of the target platform and how it influences compiler output.

cc @voltur01


[^Armclang]: ARM Developer, [Arm Compiler armclang Reference Guide -mbranch-protection](https://developer.arm.com/documentation/101754/latest/armclang-Reference/armclang-Command-line-Options/-mbranch-protection).

[^ArmclangExample]: ARM Developer, [Examples for the armclang -mbranch-protection command-line option](https://developer.arm.com/documentation/101754/latest/armclang-Reference/armclang-Command-line-Options/Examples-for-the-armclang--mbranch-protection-command-line-option).
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we add the the current version of the Arm Compiler for Embedded Reference Guide to this reference and the Armclang one which seems to have slipped by without a version or date? Dating the references can be useful for future readers in case the contents of the referred document change.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure! I've pinned the link to the latest version, added that as part of the link description, and added a date as well :) Hadn't considered that, good spot

Make assumptions clearer about the impact of the target platform and how
it influences compiler output.

Signed-off-by: Ionut Mihalcea <[email protected]>
@ionut-arm ionut-arm force-pushed the pac-bti-clarifications branch from 0fdfec5 to 766d934 Compare March 28, 2025 09:58
Copy link
Contributor

@thomasnyman thomasnyman left a comment

Choose a reason for hiding this comment

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

lgtm

@thomasnyman thomasnyman merged commit 61adf79 into ossf:main Mar 28, 2025
2 checks passed
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.

2 participants