Skip to content

[only tests] Add tests for current read/write unaligned behaviour#158341

Open
scottmcm wants to merge 1 commit into
rust-lang:mainfrom
scottmcm:unaligned-pinning
Open

[only tests] Add tests for current read/write unaligned behaviour#158341
scottmcm wants to merge 1 commit into
rust-lang:mainfrom
scottmcm:unaligned-pinning

Conversation

@scottmcm

Copy link
Copy Markdown
Member

This adds some tests to record what we're currently doing for read_unaligned and write_unaligned.

I had part of this in #158202, but I'm going to close that in favour of a different approach (#158291 being the first part of that) so figured I'd check in tests showing current behaviour to help make the fix PRs clearer.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jun 24, 2026
@rustbot

rustbot commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

r? @mejrs

rustbot has assigned @mejrs.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 73 candidates
  • Random selection from 20 candidates

@rust-log-analyzer

This comment has been minimized.

@scottmcm scottmcm force-pushed the unaligned-pinning branch from 177fe9d to d6f80bf Compare June 24, 2026 05:00
@mejrs

mejrs commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

I'm not qualified to review this sort of stuff

r? @folkertdev perhaps?

@rustbot rustbot assigned folkertdev and unassigned mejrs Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants