Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Add a few ICEs and move some fixed ones around #564

Merged
merged 17 commits into from
Dec 20, 2020

Conversation

fanninpm
Copy link
Contributor

@fanninpm fanninpm commented Dec 17, 2020

My previous PR (#562) contained a file that broke the CI. This manually fixes that. EDIT: This is no longer necessary thanks to #565.

The link to each ICE is in the corresponding commit message.

fixed/79569.sh Outdated Show resolved Hide resolved
ices/52893.rs Outdated Show resolved Hide resolved
fixed/79569.sh Outdated Show resolved Hide resolved
fixed/79569.sh Outdated Show resolved Hide resolved
@camelid
Copy link
Member

camelid commented Dec 18, 2020

@fanninpm Could you update the PR description with the rest of the ICEs that you added?

EDIT: Never mind, I didn't notice those were only the fixed ICEs :)

@JohnTitor
Copy link
Member

I'd like to check if #565 fixes the autofix issue so I'm holding this off a bit.
And could you stop adding more ICEs here, please? The current amount makes review hard :/
Also, you can add ICEs via comment like #560 instead of adding manually.

@fanninpm
Copy link
Contributor Author

Also, you can add ICEs via comment like #560 instead of adding manually.

By my understanding, I wouldn't have permission to do that in that manner.

ices/67684.sh Outdated Show resolved Hide resolved
ices/79018.sh Outdated Show resolved Hide resolved
@JohnTitor
Copy link
Member

By my understanding, I wouldn't have permission to do that in that manner.

Ah, seems so :/

Almost looks fine, r=me with the above comments are addressed and moving fixed ICEs are dropped.

@JohnTitor JohnTitor merged commit 09dee98 into rust-lang:master Dec 20, 2020
@fanninpm fanninpm deleted the add-ices-2020-12-16 branch December 20, 2020 18:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants