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

fix ARM-M class definition #59

Merged
merged 1 commit into from
May 22, 2022
Merged

fix ARM-M class definition #59

merged 1 commit into from
May 22, 2022

Conversation

ikcalB
Copy link
Contributor

@ikcalB ikcalB commented May 21, 2022

without the proper parameter for Elf.Abi, reset_architecture produces a
warning. This warnin is not only inconvenient, but does stop the debugger from
attaching automatically.

according to i.e. https://community.arm.com/support-forums/f/architectures-and-processors-forum/51867/how-can-i-know-if-an-elf-file-is-for-cortex-a-or-cortex-m
there is no different ABI code form ARM platforms. CPU Type would be needed to
make a proper distinction.

without the proper parameter for Elf.Abi, `reset_architecture` produces a
warning. This warnin is not only inconvenient, but does stop the debugger from
attaching automatically.

according to i.e. https://community.arm.com/support-forums/f/architectures-and-processors-forum/51867/how-can-i-know-if-an-elf-file-is-for-cortex-a-or-cortex-m
there is no different ABI code form ARM platforms. CPU Type would be needed to
make a proper distinction.
@ikcalB ikcalB marked this pull request as ready for review May 21, 2022 15:54
@hugsy hugsy merged commit 2730d78 into hugsy:master May 22, 2022
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