Skip to content

fix pointerarith_tfunc for Const ptr#60011

Merged
Keno merged 2 commits intomasterfrom
os/fix-pointer-arith
Nov 1, 2025
Merged

fix pointerarith_tfunc for Const ptr#60011
Keno merged 2 commits intomasterfrom
os/fix-pointer-arith

Conversation

@oscardssmith
Copy link
Copy Markdown
Member

fixes #60009

@oscardssmith oscardssmith requested a review from maleadt November 1, 2025 00:07
@oscardssmith oscardssmith added bugfix This change fixes an existing bug backport 1.12 Change should be backported to release-1.12 labels Nov 1, 2025
@adienes adienes added the backport 1.13 Change should be backported to release-1.13 label Nov 1, 2025
Copy link
Copy Markdown
Member

@Keno Keno left a comment

Choose a reason for hiding this comment

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

Ok, but might as well implement this properly and constant fold (can be a follow up though).

@oscardssmith
Copy link
Copy Markdown
Member Author

It already has good effects, so it should constant fold, right?

@vtjnash
Copy link
Copy Markdown
Member

vtjnash commented Nov 1, 2025

Constant folding is handled elsewhere (to avoid needing to duplicate that every time you ever write a tfunc)

@Keno Keno merged commit 97f880a into master Nov 1, 2025
5 of 8 checks passed
@Keno Keno deleted the os/fix-pointer-arith branch November 1, 2025 12:32
KristofferC pushed a commit that referenced this pull request Nov 5, 2025
fixes #60009

(cherry picked from commit 97f880a)
@KristofferC KristofferC mentioned this pull request Nov 5, 2025
17 tasks
KristofferC pushed a commit that referenced this pull request Nov 7, 2025
fixes #60009

(cherry picked from commit 97f880a)
@KristofferC KristofferC mentioned this pull request Nov 7, 2025
35 tasks
@KristofferC KristofferC removed the backport 1.12 Change should be backported to release-1.12 label Nov 21, 2025
@KristofferC KristofferC removed the backport 1.13 Change should be backported to release-1.13 label Nov 28, 2025
kpamnany pushed a commit to RelationalAI/julia that referenced this pull request Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This change fixes an existing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pointer Integer addition ignored when in a function.

5 participants