-
Notifications
You must be signed in to change notification settings - Fork 13k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP: remove compiler plugin API for syntax transformers #64665
Conversation
This role is full filled by proc-macros now
r? @cramertj (rust_highfive has picked a reviewer for you, use r? to override) |
This doesn't remove the fundamental possibility to add syntactic plugins, only helper functions, so r=me after updating the docs/examples/etc. |
Hm, I am confused by this statement. |
Well, that's what I meant. |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Hm, #64675 proposes to remove the whole compiler plugin infrastructure, so I think its best to just wait for that. |
Seems we are not removing them as soon as I'd like so meanwhile we should remove some stuff. :) |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
Ping from triage: @matklad any updates on this? |
☔ The latest upstream changes (presumably #65495) made this pull request unmergeable. Please resolve the merge conflicts. |
@matklad visiting for triage -- this would be good to rebase and land, I think. Ideally we'd entirely remove syntactic extension plugins -- I believe all or almost all use cases are replaceable via proc macro today (though possibly in an unstable way, but plugins are unstable anyway) -- so we should be able to make that move. Servo, Clippy, etc. don't appear to use it. |
Ping from triage: @matklad any updates on this? |
Heh, I think empirical evidence shows that I am swamped with rust-analyzer task at the moment :( Let's close this for now: I plan to get back to this, but I can't give any ETA at the moment. |
cc: https://rust-lang.zulipchat.com/#narrow/stream/131828-t-compiler/topic/Removing.20.60Register.3A.3Aregister_macro.60.20compiler.20extension.20point
Things to check: