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

EvaluationCache overwrites its entries #50507

Open
Zoxc opened this issue May 7, 2018 · 1 comment
Open

EvaluationCache overwrites its entries #50507

Zoxc opened this issue May 7, 2018 · 1 comment
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@Zoxc
Copy link
Contributor

Zoxc commented May 7, 2018

When the insertion to EvaluationCache is changed to use HashMapExt::insert_same. It triggers an the assertion which checks that the old value is the same as the new one. This is because the DepNodeIndex changed.

Backtrace
Log output

cc @arielb1 @nikomatsakis

@XAMPPRocky XAMPPRocky added C-enhancement Category: An issue proposing an enhancement or a PR with one. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Oct 2, 2018
@sasurau4
Copy link
Contributor

sasurau4 commented Jan 28, 2021

@nikomatsakis
I looked into this issue.
In #49834, the build seems failed at build stage0 core and left FIXME comment.
When I changed the code same to #73792 and turn on parallel-compiler = true in config.toml, I met following tests failed but couldn't reproduce the build error same as #49834.

failures:
    [ui] ui/associated-types/defaults-cyclic-fail-1.rs
    [ui] ui/associated-types/defaults-cyclic-fail-2.rs
    [ui] ui/privacy/privacy2.rs
    [ui] ui/privacy/privacy3.rs
    [ui] ui/type_length_limit.rs

The cause of test fails are snapshot diffs mentioned at #75760, so this issue seems solved.
Any other information or investigation needed? I'll work on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
3 participants