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

Properly encode AnonConst into crate metadata #72600

Merged
merged 1 commit into from
Jun 20, 2020

Conversation

Aaron1011
Copy link
Member

Fixes #68104

Previous, we were encoding AnonConst as a regular Const, causing us to
treat them differently after being deserialized in another compilation
session.

@rust-highfive
Copy link
Collaborator

r? @varkor

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 26, 2020
@Aaron1011
Copy link
Member Author

r? @eddyb

@rust-highfive rust-highfive assigned eddyb and unassigned varkor May 26, 2020
Fixes rust-lang#68104

Previous, we were encoding AnonConst as a regular Const, causing us to
treat them differently after being deserialized in another compilation
session.
@Aaron1011 Aaron1011 force-pushed the fix/anon-const-encoding branch from edb8c06 to ebe5a91 Compare May 26, 2020 05:48
@varkor varkor added the F-const_generics `#![feature(const_generics)]` label May 26, 2020
@Elinvynia Elinvynia added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 3, 2020
@Elinvynia Elinvynia added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 10, 2020
@Aaron1011
Copy link
Member Author

@eddyb: Are there any changes you'd like me to make?

@varkor
Copy link
Member

varkor commented Jun 19, 2020

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Jun 19, 2020

📌 Commit ebe5a91 has been approved by varkor

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 19, 2020
Manishearth added a commit to Manishearth/rust that referenced this pull request Jun 20, 2020
…r=varkor

Properly encode AnonConst into crate metadata

Fixes rust-lang#68104

Previous, we were encoding AnonConst as a regular Const, causing us to
treat them differently after being deserialized in another compilation
session.
RalfJung added a commit to RalfJung/rust that referenced this pull request Jun 20, 2020
…r=varkor

Properly encode AnonConst into crate metadata

Fixes rust-lang#68104

Previous, we were encoding AnonConst as a regular Const, causing us to
treat them differently after being deserialized in another compilation
session.
RalfJung added a commit to RalfJung/rust that referenced this pull request Jun 20, 2020
…r=varkor

Properly encode AnonConst into crate metadata

Fixes rust-lang#68104

Previous, we were encoding AnonConst as a regular Const, causing us to
treat them differently after being deserialized in another compilation
session.
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 20, 2020
Rollup of 9 pull requests

Successful merges:

 - rust-lang#72600 (Properly encode AnonConst into crate metadata)
 - rust-lang#73055 (remove leftover mentions of `skol` and `int` from the compiler)
 - rust-lang#73058 (Support sanitizers on aarch64-unknown-linux-gnu)
 - rust-lang#73171 (RISC-V Emulated Testing)
 - rust-lang#73404 (Update CFGuard syntax)
 - rust-lang#73444 (ci: disable alt build during try builds)
 - rust-lang#73471 (Prevent attacker from manipulating FPU tag word used in SGX enclave)
 - rust-lang#73539 (Deprecate `Vec::remove_item`)
 - rust-lang#73543 (Clean up E0695 explanation)

Failed merges:

r? @ghost
@bors bors merged commit 5431ef6 into rust-lang:master Jun 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
F-const_generics `#![feature(const_generics)]` S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Specialized impl for a type with const generics in another crate causes a stack overflow in rustc
6 participants