Skip to content

Define pointer metadata#2286

Merged
traviscross merged 1 commit into
masterfrom
TC/define-pointer-metadata
Jun 16, 2026
Merged

Define pointer metadata#2286
traviscross merged 1 commit into
masterfrom
TC/define-pointer-metadata

Conversation

@traviscross

Copy link
Copy Markdown
Contributor

The pointer-to-pointer cast rules and the wide-pointer validity rule both speak of the metadata of a pointer, but we hadn't explicitly defined the term (even though we had defined the contents of that metadata). Let's do that and link to it.


I'm breaking this out from #2282 so that we can merge the prerequisites before considering the new lang guarantees.

cc @ehuss @RalfJung @Mark-Simulacrum

@ehuss ehuss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ehuss ehuss removed the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Jun 16, 2026
The pointer-to-pointer cast rules and the wide-pointer validity rule
both speak of the *metadata* of a pointer, but we hadn't explicitly
defined the term (even though we had defined the contents of that
metadata).  Let's do that and link to it.
@traviscross traviscross force-pushed the TC/define-pointer-metadata branch from 91e9963 to e9ebd56 Compare June 16, 2026 18:52
@rustbot

rustbot commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@traviscross traviscross enabled auto-merge June 16, 2026 18:55
@traviscross traviscross added this pull request to the merge queue Jun 16, 2026
Merged via the queue into master with commit be573a7 Jun 16, 2026
6 checks passed
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.

3 participants