Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upType inference regression in rustc 1.18 #42545
Comments
frewsxcv
added
the
regression-from-stable-to-stable
label
Jun 8, 2017
nikomatsakis
modified the milestone:
t-com
Jun 8, 2017
nikomatsakis
added
I-nominated
T-compiler
labels
Jun 8, 2017
This comment has been minimized.
This comment has been minimized.
|
For posterity, this is also broken on the latest nightly as well
|
This comment has been minimized.
This comment has been minimized.
|
triage: P-high -- regression |
rust-highfive
added
P-high
and removed
I-nominated
labels
Jun 8, 2017
nikomatsakis
self-assigned this
Jun 8, 2017
This comment has been minimized.
This comment has been minimized.
|
Bisected to 141e8a6, #41716 -- cc @nikomatsakis. |
This comment has been minimized.
This comment has been minimized.
|
Well, I can certainly reproduce it =) |
This comment has been minimized.
This comment has been minimized.
|
This may be a dup of #42552 |
This comment has been minimized.
This comment has been minimized.
|
My fix for #42552 also fixes this. |
nikomatsakis
added a commit
to nikomatsakis/rust
that referenced
this issue
Jun 14, 2017
bors
closed this
in
9fec409
Jun 17, 2017
This comment has been minimized.
This comment has been minimized.
|
Is the fix going to make it into 1.19? A 1.18.1? |
This comment has been minimized.
This comment has been minimized.
|
The PR (#42659) is beta-nominated, so if it's accepted the fix will be backported and make it into 1.19. However, I doubt we'd release 1.18.1 -- it's possible, though. |
brson
added a commit
to brson/rust
that referenced
this issue
Jun 22, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
khuey commentedJun 8, 2017
https://github.com/khuey/rustc-1.18-regression
I'm not sure that this is a minimal testcase, but it's small enough to poke around at. With rustc 1.17 this compiles fine. With 1.18 I get