search

Gilnaa / memoffset

offsetof for Rust
MIT License
224 stars 29 forks source link

use MaybeUninit::as_ptr instead of transmute #46

Closed RalfJung closed 4 years ago

RalfJung commented 4 years ago

With https://github.com/rust-lang/rust/pull/75250 having landed, we can finally get rid of the transmute here.

RalfJung commented 4 years ago

Ah, we have to wait until Miri gets an update to work with the newer rustc that has as_ptr.

Gilnaa commented 4 years ago

Cool. Does that mean waiting for the next nightly or the next release?

RalfJung commented 4 years ago

It means waiting for https://github.com/rust-lang/rust/issues/75274 to be fixed, which is currently blocked on https://github.com/rust-lang/rust/pull/75339.

RalfJung commented 4 years ago

Miri got that update, so CI should pass now. Cycling the PR.