Skip to content

Conversation

@lenary
Copy link
Contributor

@lenary lenary commented Oct 17, 2025

These have some effect on the ABI of code that use them, and need to be kept stable long-term, so we should document the sizes they are.

These have some effect on the ABI of code that use them, and need to be
kept stable long-term, so we should document the sizes they are.
@kito-cheng
Copy link
Collaborator

I am little hesitate to document that down in the ABI spec, do you mind give few more background about why we need to document that? and the follow up question is: should we also document all other [u]int*_t from the C language?

@lenary
Copy link
Contributor Author

lenary commented Oct 25, 2025

Here is more background on how this leaks into the ABI:

I have actually opened riscv-non-isa/riscv-c-api-doc#129 to not document the other int_fastN_t and int_leastN_t in the ABI as they currently don't always match between toolchains, and I don't think we want to make them ABI stable. Sorry for not cross-linking to that change originally.

Copy link
Collaborator

@kito-cheng kito-cheng left a comment

Choose a reason for hiding this comment

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

We've discussed this topic in last psABI meeting, and we agree defined that in psABI will help RISC-V has more stable ABI even the C spec isn't mandatory it.

@kito-cheng kito-cheng merged commit 98de20a into riscv-non-isa:master Nov 19, 2025
5 checks passed
@lenary lenary deleted the pr/intmax-t branch November 19, 2025 08:47
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