Skip to content

Mark has_global_allocator query as eval_always #84260

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

Merged
merged 1 commit into from
Apr 17, 2021

Conversation

Aaron1011
Copy link
Member

Fixes #84252

This query reads from untracked global state in CStore.

Fixes rust-lang#84252

This query reads from untracked global state in `CStore`.
@rust-highfive
Copy link
Contributor

r? @davidtwco

(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 Apr 16, 2021
@Spaceface16518
Copy link

@Aaron1011 that was fast 😮

@Aaron1011
Copy link
Member Author

@Spaceface16518: This issue is very similar to other ones which have been exposed recently - the fix tends to be the same 😄

@Spaceface16518
Copy link

So is my code now valid or a compiler error?

@Aaron1011
Copy link
Member Author

@Spaceface16518: Your code was perfectly fine - the ICE was due to a compiler bug related to incremental compilation. Applying a #[cfg] to a #[global_allocator] item is perfectly correct.

For now, you can work around this bug by running cargo clean. You'll also need to clear out the Github Actions cache, as it's the cached build directory that's responsible for the crash.

@Spaceface16518
Copy link

Oh I see, that makes sense. Thank you, I'll do that.

@cjgillot
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 17, 2021

📌 Commit 169a221 has been approved by cjgillot

@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 Apr 17, 2021
@bors
Copy link
Collaborator

bors commented Apr 17, 2021

⌛ Testing commit 169a221 with merge cd9b305...

@bors
Copy link
Collaborator

bors commented Apr 17, 2021

☀️ Test successful - checks-actions
Approved by: cjgillot
Pushing cd9b305 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Apr 17, 2021
@bors bors merged commit cd9b305 into rust-lang:master Apr 17, 2021
@rustbot rustbot added this to the 1.53.0 milestone Apr 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. 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.

ICE: found unstable fingerprints for has_global_allocator(): false
7 participants