julia icon indicating copy to clipboard operation
julia copied to clipboard

A change list for the compiler

Open mhauru opened this issue 2 months ago • 12 comments

Changes to the compiler in 1.12, while very welcome, caused work for some packages that interface with the compiler. I'm aware of https://github.com/chalk-lab/Mooncake.jl/pull/714 and https://github.com/TuringLang/Libtask.jl/pull/196, there are probably others. This is of course understandable and unavoidable when packages interface with internals that aren't public.

However, as discussed on Slack a few times, it would be very useful for package maintainers if there was even a rough list of changes, and advice on how to adapt to them. To kickstart this, I started a draft with changes I encountered in 1.12. It is very incomplete (reflecting my very superficial understanding of the compiler), and I would be grateful and happy if those who know this stuff better would be up for completing it, merging it, and adding to it as more things happen in 1.13.

mhauru avatar Oct 23 '25 13:10 mhauru

This is great, thank you. I think it should probably go in devdocs?

JeffBezanson avatar Oct 23 '25 20:10 JeffBezanson

Since this is meant to be like HISTORY.md, it would be good to keep it as is. More compiler docs in devdocs can be fixed separately.

yebai avatar Oct 24 '25 10:10 yebai

OK I guess this is fine.

JeffBezanson avatar Oct 24 '25 18:10 JeffBezanson

This is a great idea, thank you for getting it started.

Is the plan to fill in the TODOs next? Or are you looking for help with that? Or should we just merge this and gradually fill in the TODOs if someone has time to work on 'em?

fingolfin avatar Oct 27 '25 22:10 fingolfin

Yep, I'm looking for help for someone to fill in the TODOs, and to add any bits that I'm not aware of. For how to do that, I'm happy with whatever the maintainers prefer: Merge this and open more PRs to improve it, merge PRs into my branch, move my branch to this repo and start a new PR from it, all fine by me.

mhauru avatar Oct 28 '25 09:10 mhauru

@serenity4 kindly filled in the TODOs that I had left in my draft. If anyone wants to expand the change list please feel very free, but if no other changes are imminent I would favour merging this now, and adding things gradually as changes are made.

mhauru avatar Nov 18 '25 16:11 mhauru

Are you committed in updating this file over releases? Also, somewhere it should probably be stated that this is a non-exhaustive / best effort list.

KristofferC avatar Nov 18 '25 16:11 KristofferC

I can not commit to that because I don't work on the compiler or even understand most of the changes to it. The point of this PR is to be a starting point and a nudge, hoping the compiler devs would commit to it, on, as you said, a best-effort basis.

I added the sentence

[This file] is maintained on a best-effort basis and may be incomplete.

mhauru avatar Nov 18 '25 16:11 mhauru

The point of this PR is to be a starting point and a nudge, hoping the compiler devs would commit to it

I envy your optimism...

I also think for this to be useful, it has to go into the manual somehow (like the NEWS file does for the language itself).

KristofferC avatar Nov 18 '25 16:11 KristofferC

I also think for this to be useful, it has to go into the manual somehow (like the NEWS file does for the language itself).

I'm very happy for this to be in the manual if the maintainers prefer that. I do think having this document exist anywhere is preferable to not having it at all though, especially given the quite specialised audience of this information.

mhauru avatar Nov 24 '25 11:11 mhauru

@mhauru Can you make a PR to add it to the devdocs section of the manual so that we can get it merged?

ViralBShah avatar Dec 05 '25 00:12 ViralBShah

I updated this PR to have the same content as before but in doc/src/devdocs/compiler_changes.md. Looks something like this image

mhauru avatar Dec 05 '25 15:12 mhauru

Can we merge this now?

ViralBShah avatar Dec 16 '25 15:12 ViralBShah

Good to go from my side.

mhauru avatar Dec 16 '25 16:12 mhauru