Skip to content
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

add #simplext, a super-simple syntax extension system #545

Closed
wants to merge 3 commits into from

Conversation

paulstansifer
Copy link
Contributor

No description provided.

@graydon
Copy link
Contributor

graydon commented Jun 23, 2011

Very exciting! Generally quite happy with this.

Perhaps just call it "macro" or "defmacro" or something? There are no other macro systems now, so it's not like you're colliding with names. When and if you make a better macro system, you can swap the names around.

@graydon
Copy link
Contributor

graydon commented Jun 28, 2011

Integrated, thanks.

@graydon graydon closed this Jun 28, 2011
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this pull request Dec 12, 2017
pdietl pushed a commit to pdietl/rust that referenced this pull request Apr 23, 2020
Document `target_feature` and `cfg_target_feature`.
celinval added a commit to celinval/rust-dev that referenced this pull request Jun 4, 2024
I refactored how we codegen panic statements so it now it terminate the
program per its spec. I have also removed the hack we had to try to get
the assert location.

Since we currently do not support stack unwinding, the panic codegen
will always terminate immediately and it will not try to unwind the stack.
I added an option to RMC to force the compiler to use abort as the panic
strategy and a check to rmc codegen that will fail if the user tries to
override that.

Another note is that we currently do not support `#[track_caller]` and I
have not changed that.

This change fixes rust-lang#67, fixes rust-lang#466, fixes rust-lang#543, and fixes rust-lang#636. This
change also mitigates rust-lang#545.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants