-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Fix ICE on unsound_collection_transmute
#4975
Conversation
Thanks for moving the tests, I also wanted to do this recently. Maybe we should put the |
It's reasonable, moved into |
@bors r+ |
📌 Commit bf67fcf has been approved by |
Fix ICE on `unsound_collection_transmute` Fixes #4968 Check if `Ty`s are normalizable. It might show hidden false negative, I'm not sure. Also, the regression tests are placed on two dirs, so move them to `/crashes`. I think it will be easier to find the right place. changelog: Fix ICE on `unsound_collection_transmute`
💔 Test failed - status-appveyor |
Huh, never saw this before.
Maybe move the auxiliary file into |
Oh, it seems so, moved it. |
@bors r+ thanks! |
📌 Commit c6aeda7 has been approved by |
Fix ICE on `unsound_collection_transmute` Fixes #4968 Check if `Ty`s are normalizable. It might show hidden false negative, I'm not sure. Also, the regression tests are placed on two dirs, so move them to `/crashes`. I think it will be easier to find the right place. changelog: Fix ICE on `unsound_collection_transmute`
☀️ Test successful - checks-travis, status-appveyor |
Fixes #4968
Check if
Ty
s are normalizable. It might show hidden false negative, I'm not sure.Also, the regression tests are placed on two dirs, so move them to
/crashes
. I think it will be easier to find the right place.changelog: Fix ICE on
unsound_collection_transmute