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

Expand ABI support by replacing opaque items #58

Closed
3 tasks
celinval opened this issue Dec 19, 2023 · 0 comments · Fixed by rust-lang/rust#121824
Closed
3 tasks

Expand ABI support by replacing opaque items #58

celinval opened this issue Dec 19, 2023 · 0 comments · Fixed by rust-lang/rust#121824
Assignees

Comments

@celinval
Copy link
Contributor

celinval commented Dec 19, 2023

We are adding some basic support to function ABI and type layout in this PR: rust-lang/rust#119094, however, some information was left as Opaque for now.

We still need to expand the following:

Tasks

@celinval celinval self-assigned this Feb 22, 2024
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Mar 1, 2024
Implement missing ABI structures in StableMIR

Add implementations for Scalar, Primitive and WrappingRange for StableMIR.

FYI, I thought about reusing the `rustc_abi` module, since it is designed to not necessarily depend on the `rustc` internals, but the maintenance burden to maintain this crate in crates.io doesn't seem worth it at this point.

Fixes rust-lang/project-stable-mir#58
rust-timer added a commit to rust-lang-ci/rust that referenced this issue Mar 2, 2024
Rollup merge of rust-lang#121824 - celinval:smir-scalar, r=oli-obk

Implement missing ABI structures in StableMIR

Add implementations for Scalar, Primitive and WrappingRange for StableMIR.

FYI, I thought about reusing the `rustc_abi` module, since it is designed to not necessarily depend on the `rustc` internals, but the maintenance burden to maintain this crate in crates.io doesn't seem worth it at this point.

Fixes rust-lang/project-stable-mir#58
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 a pull request may close this issue.

1 participant