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

Move built-in syntax extensions to a separate crate #30300

Merged
merged 4 commits into from
Dec 16, 2015

Conversation

sanxiyn
Copy link
Member

@sanxiyn sanxiyn commented Dec 10, 2015

This reduces iteration time (make rustc-stage1) for moved syntax extensions from 11 minutes to 3 minutes on my machine.

Because of the signature change, this is a [breaking-change] for people directly calling expand_crate. I think it is rare: from GitHub search, only case I found is glassful.

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@sanxiyn
Copy link
Member Author

sanxiyn commented Dec 11, 2015

Fixed all tests.

@bors
Copy link
Contributor

bors commented Dec 14, 2015

☔ The latest upstream changes (presumably #29735) made this pull request unmergeable. Please resolve the merge conflicts.

@nikomatsakis
Copy link
Contributor

This looks good to me, though it needs a rebase.

@sanxiyn
Copy link
Member Author

sanxiyn commented Dec 15, 2015

Rebased and marked the crate unstable.

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Dec 15, 2015

📌 Commit 0883f10 has been approved by nikomatsakis

@bors
Copy link
Contributor

bors commented Dec 16, 2015

⌛ Testing commit 0883f10 with merge ce7bc51...

bors added a commit that referenced this pull request Dec 16, 2015
This reduces iteration time (`make rustc-stage1`) for moved syntax extensions from 11 minutes to 3 minutes on my machine.

Because of the signature change, this is a [breaking-change] for people directly calling `expand_crate`. I think it is rare: from GitHub search, only case I found is [glassful](https://github.com/kmcallister/glassful).
@bors bors merged commit 0883f10 into rust-lang:master Dec 16, 2015
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.

4 participants