diff options
| author | bors <bors@rust-lang.org> | 2021-07-17 15:26:27 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2021-07-17 15:26:27 +0000 |
| commit | c78ebb7bdcfc924a20fd069891ffe1364d6814e7 (patch) | |
| tree | 42c1c852e5ca783556e9477361ed6b3fef733aad /compiler/rustc_interface/src | |
| parent | f502bd3abd12111bbfae0974db018c165a977c0e (diff) | |
| parent | efbee50600e63293608811d7ee200f12641f1958 (diff) | |
| download | rust-c78ebb7bdcfc924a20fd069891ffe1364d6814e7.tar.gz rust-c78ebb7bdcfc924a20fd069891ffe1364d6814e7.zip | |
Auto merge of #87123 - RalfJung:miri-provenance-overhaul, r=oli-obk
CTFE/Miri engine Pointer type overhaul This fixes the long-standing problem that we are using `Scalar` as a type to represent pointers that might be integer values (since they point to a ZST). The main problem is that with int-to-ptr casts, there are multiple ways to represent the same pointer as a `Scalar` and it is unclear if "normalization" (i.e., the cast) already happened or not. This leads to ugly methods like `force_mplace_ptr` and `force_op_ptr`. Another problem this solves is that in Miri, it would make a lot more sense to have the `Pointer::offset` field represent the full absolute address (instead of being relative to the `AllocId`). This means we can do ptr-to-int casts without access to any machine state, and it means that the overflow checks on pointer arithmetic are (finally!) accurate. To solve this, the `Pointer` type is made entirely parametric over the provenance, so that we can use `Pointer<AllocId>` inside `Scalar` but use `Pointer<Option<AllocId>>` when accessing memory (where `None` represents the case that we could not figure out an `AllocId`; in that case the `offset` is an absolute address). Moreover, the `Provenance` trait determines if a pointer with a given provenance can be cast to an integer by simply dropping the provenance. I hope this can be read commit-by-commit, but the first commit does the bulk of the work. It introduces some FIXMEs that are resolved later. Fixes https://github.com/rust-lang/miri/issues/841 Miri PR: https://github.com/rust-lang/miri/pull/1851 r? `@oli-obk`
Diffstat (limited to 'compiler/rustc_interface/src')
0 files changed, 0 insertions, 0 deletions
