Skip to content

Conversation

@emesare
Copy link
Member

@emesare emesare commented Dec 5, 2025

The partial register id is the "main" id so we return that, whereas the full register id is not.

@bdash did you have any more thoughts surrounding the usage of this API?

The partial register id is the "main" id so we return that, whereas the full register id is not.
@emesare emesare self-assigned this Dec 5, 2025
@emesare emesare added the Component: Rust API Issue needs changes to the Rust API label Dec 5, 2025
@bdash
Copy link
Contributor

bdash commented Dec 8, 2025

I'm not sure about this API. It does not seem obvious to a client what id will return for a partial register. Maybe a doc comment fixes that, or perhaps there's a clearer name.

I'm not even 100% sure whether get_ssa_register_uses and friends are doing the right thing with how they handle partial registers at the moment.

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

Labels

Component: Rust API Issue needs changes to the Rust API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants