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 upICE, possibly related to associated types of associated types? #36325
Comments
steveklabnik
added
the
I-ICE
label
Sep 7, 2016
This comment has been minimized.
This comment has been minimized.
|
I have reduced this to a self-contained file with around 50 LOC (and could probably go further but don't have the time right now): https://gist.github.com/cda11d74c476af7339d18a8bf37976e3 This reduced test case is fine on |
This comment has been minimized.
This comment has been minimized.
|
Compiling with |
This comment has been minimized.
This comment has been minimized.
|
This is a regression from stable to beta. Edit: with the latest test code (see below) this also reproduces on current stable (1.11) but not on 1.10, so a variation of this is also a regression from stable to stable. |
This comment has been minimized.
This comment has been minimized.
|
Actually, with slightly different code, this was introduced between cc @nikomatsakis: From the PR's merged in that period, I guess this was an effect of either #33998 or #33816. |
This comment has been minimized.
This comment has been minimized.
|
Cleaned up and slightly more minified version with (hopefully) meaningful names: https://gist.github.com/a8f9885c2bfd585dd8af9b6973cfd219 |
TimNN
added
the
regression-from-stable-to-stable
label
Sep 15, 2016
nrc
added
the
T-compiler
label
Sep 15, 2016
pnkfelix
added
the
P-high
label
Sep 15, 2016
nrc
assigned
pnkfelix
Sep 15, 2016
This comment has been minimized.
This comment has been minimized.
|
@pnkfelix assigned for initial investigation |
This comment has been minimized.
This comment has been minimized.
|
@pnkfelix any updates? |
This comment has been minimized.
This comment has been minimized.
|
@brson nothing to report yet; i did some preliminary investigation but need to get back to it. |
This comment has been minimized.
This comment has been minimized.
|
#37154 was introduced between the same nightlies |
jonas-schievink
referenced this issue
Oct 14, 2016
Closed
Regression: "no method found" error when calling same method twice, with HRTB impl #37154
brson
added
the
I-nominated
label
Oct 20, 2016
This comment has been minimized.
This comment has been minimized.
|
Nominating for discussion with compiler team. P-high bug, no progress. |
This comment has been minimized.
This comment has been minimized.
|
Maybe #37294 already fixes this |
This comment has been minimized.
This comment has been minimized.
|
@jonas-schievink: A stage 1 build of #37294 does indeed seem to fix this issue as well. |
nikomatsakis
removed
the
I-nominated
label
Oct 20, 2016
nikomatsakis
assigned
nikomatsakis
and unassigned
pnkfelix
Oct 20, 2016
This comment has been minimized.
This comment has been minimized.
|
re-assigning to me since #37294 is my PR |
This comment has been minimized.
This comment has been minimized.
|
Fixed! Not backporting to 1.13 because the port is non-trivial. |
elidupree commentedSep 7, 2016
•
edited by pnkfelix
(edit from @pnkfelix: if you're looking at this bug, make sure to skip ahead to the comments to see where people have posted links to self-contained reductions to reproduce the bug.)
To reproduce: on the latest nightly, clone https://github.com/elidupree/time-steward (commit ac2db9f6765eeb18692d8588162bc076c9f198c1) and run
cargo test, but replace tests/paces.rs:85-88 with:When I did this, I got:
Meta
rustc --version --verbose:rustc 1.13.0-nightly (923bac4 2016-09-06)
binary: rustc
commit-hash: 923bac4
commit-date: 2016-09-06
host: x86_64-unknown-linux-gnu
release: 1.13.0-nightly