Skip to content
This repository was archived by the owner on Jun 1, 2026. It is now read-only.

qptr: add an immediate offset to Load/Store ops. #41

Closed
eddyb wants to merge 11 commits into
mainfrom
qptr-imm-offsets
Closed

qptr: add an immediate offset to Load/Store ops. #41
eddyb wants to merge 11 commits into
mainfrom
qptr-imm-offsets

Conversation

@eddyb
Copy link
Copy Markdown
Contributor

@eddyb eddyb commented Jun 26, 2023

While separate offseting works fine in general, having an immediate offset inside loads and stores (as this PR adds) can make the simplest case (i.e. no array indexing or buffers - especially local variables) easier.

There's two usecases that motivated this change:

TODO: changelog, maybe a comparison example (kinda hard tho).

@eddyb eddyb force-pushed the qptr-imm-offsets branch from aa73fbc to ac8eafb Compare July 2, 2023 08:33
@eddyb eddyb force-pushed the qptr-imm-offsets branch from 231e84c to 83ffe71 Compare October 1, 2023 18:13
@eddyb eddyb force-pushed the qptr-imm-offsets branch 4 times, most recently from bc53e10 to a2cd69c Compare October 29, 2023 13:04
@eddyb eddyb force-pushed the qptr-imm-offsets branch 6 times, most recently from dd5857b to 74f388d Compare November 8, 2023 05:38
@eddyb eddyb force-pushed the qptr-imm-offsets branch 2 times, most recently from d1f1f1f to b03c9d6 Compare January 31, 2024 14:19
@eddyb
Copy link
Copy Markdown
Contributor Author

eddyb commented Apr 27, 2026

@eddyb eddyb closed this Apr 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant