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

macros: fix ICE on certain sequence repetitions #39730

Merged
merged 1 commit into from
Feb 15, 2017

Conversation

jseyfried
Copy link
Contributor

Fixes #39709.
r? @nrc

@jseyfried jseyfried added beta-nominated Nominated for backporting to the compiler in the beta channel. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Feb 11, 2017
@jseyfried
Copy link
Contributor Author

This is a minimal reversion back to stable's behavior for backporting, details in #39709 (comment).

@nrc
Copy link
Member

nrc commented Feb 14, 2017

@bors: r+

@bors
Copy link
Contributor

bors commented Feb 14, 2017

📌 Commit b3d7399 has been approved by nrc

frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 14, 2017
macros: fix ICE on certain sequence repetitions

Fixes rust-lang#39709.
r? @nrc
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 14, 2017
macros: fix ICE on certain sequence repetitions

Fixes rust-lang#39709.
r? @nrc
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 14, 2017
macros: fix ICE on certain sequence repetitions

Fixes rust-lang#39709.
r? @nrc
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Feb 14, 2017
macros: fix ICE on certain sequence repetitions

Fixes rust-lang#39709.
r? @nrc
bors added a commit that referenced this pull request Feb 14, 2017
Rollup of 8 pull requests

- Successful merges: #39659, #39730, #39754, #39772, #39785, #39788, #39790, #39813
- Failed merges:
@bors bors merged commit b3d7399 into rust-lang:master Feb 15, 2017
@jseyfried jseyfried deleted the fix_empty_seq_rep_ice branch February 16, 2017 11:03
@brson brson added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Feb 23, 2017
@brson
Copy link
Contributor

brson commented Feb 23, 2017

Accepted for beta. Picks cleanly. ICEs bad.

@brson brson mentioned this pull request Feb 23, 2017
@alexcrichton alexcrichton removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Feb 23, 2017
bors added a commit that referenced this pull request Mar 2, 2017
[beta] next

- #39913
- #39730
- #39674
- #39602
- #39586
- #39471
- #39980
- #40020
- #40135

@nikomatsakis [this commit](3787d33) did not pick cleanly. You might peek at it.

I took the liberty of accepting all the nominations myself, but the [packed struct alignment](#39586) PR is quite large. It did pick fine though and there's a comment there suggesting it works on beta cc @rust-lang/compiler.

cc @alexcrichton
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants