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

ilp32e issue #269

Open
kito-cheng opened this issue Apr 21, 2022 · 2 comments
Open

ilp32e issue #269

kito-cheng opened this issue Apr 21, 2022 · 2 comments

Comments

@kito-cheng
Copy link
Collaborator

Review comment come from @anderslindgren-iar and Anders Berg(IAR) https://lists.riscv.org/g/tech-toolchain-runtime/message/344

ILP32E Calling Convention

Anders: This will be changed!
Anders: The usage of the D ISA must/will be possible and the stack alignment can be altered by request (as part of the "deviations"). There will also be a special ABI for RV32E that is more suited for the upcoming code size ISA with PUSH and POP instructions.

@jrtc27
Copy link
Collaborator

jrtc27 commented Apr 21, 2022

  1. It's marked as not stable
  2. It's documenting the existing ABI implemented in GCC; we don't have a time machine to change it

@kito-cheng
Copy link
Collaborator Author

Last psABI call we've discussed the possibility solution for ilp32e with D ext.

Here is two way to resolve this issue:

  1. Fixing ilp32e to make it possible work with D-ext correct, e.g. change the stack alignment to 64 bits.
  2. Define ilp32ef and ilp32ed AND forbid D use with ilp32e calling convention

The target of fixing this issue would be next release, we need PoC and more experiment data to making decision.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants