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.

Sign up for GitHub

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

Jump to bottom

[BUG] Wrong use of architecture macros #266

Open
armhobbit opened this issue Jun 21, 2023 · 1 comment
Open

[BUG] Wrong use of architecture macros #266

armhobbit opened this issue Jun 21, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@armhobbit
Copy link

Describe the bug

Section 5.4.3 of acle-2022Q4-3.pdf appears to be confused. The document states

The common subset of the A, R and M profiles is indicated by
__ARM_ARCH == 7 && !defined (__ARM_ARCH_PROFILE)

Firstly, I believe the test for the common subset would be just

__ARM_ARCH == 7

and, secondly, all v7 cores should have a profile defined, so the test as shown would not pass for any compilation target.
@armhobbit armhobbit added the bug Something isn't working label Jun 21, 2023
@vhscampos
Copy link
Member

Hi, thanks for your issue report.

If possible, we encourage you to contribute with a Pull Request that addresses this issue. We will be happy to review it.

@vhscampos vhscampos changed the title [BUG] [BUG] Wrong use of architecture macros Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vhscampos @armhobbit