Skip to content

mark const fn constructors as rustc_const_stable_indirect#586

Merged
Amanieu merged 2 commits into
rust-lang:masterfrom
RalfJung:rustc_const_stable_indirect
Nov 13, 2024
Merged

mark const fn constructors as rustc_const_stable_indirect#586
Amanieu merged 2 commits into
rust-lang:masterfrom
RalfJung:rustc_const_stable_indirect

Conversation

@RalfJung

@RalfJung RalfJung commented Nov 13, 2024

Copy link
Copy Markdown
Member

This is a promise that they can be implemented entirely using stable const features (and the compiler checks this when -Zforce-unstable-if-unmarked is set).

Part of rust-lang/rust#102575.

@Amanieu Amanieu added this pull request to the merge queue Nov 13, 2024
@Amanieu

Amanieu commented Nov 13, 2024

Copy link
Copy Markdown
Member

I assume you need a new hashbrown release with this?

Merged via the queue into rust-lang:master with commit 1baa359 Nov 13, 2024
@RalfJung

RalfJung commented Nov 13, 2024 via email

Copy link
Copy Markdown
Member Author

@cuviper

cuviper commented Nov 14, 2024

Copy link
Copy Markdown
Member

I think you won't be able to bootstrap that until the new attribute reaches beta for stage0.

@RalfJung

Copy link
Copy Markdown
Member Author

Ah, true, good point.

There's a beta branching in about a week so that's not too long of a wait. :) But I should probably double-check that this will actually work...

bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 15, 2024
…r=compiler-errors

check_consts: fix error requesting feature gate when that gate is not actually needed

When working on rust-lang/hashbrown#586 I noticed that the compiler asks for the `rustc_private` feature to be enabled if one forgets to set `rustc_const_stable_indirect` on a function -- but enabling `rustc_private` would not actually help. This fixes the diagnostics.

r? `@compiler-errors`
@RalfJung RalfJung deleted the rustc_const_stable_indirect branch November 16, 2024 14:37
@RalfJung

Copy link
Copy Markdown
Member Author

I checked that if I do make these not(bootstrap), and add my local hashbrown as a path dependency to std, then everything builds in stage 1. So it should therefore also work with stage 0 after a bootstrap bump.

@RalfJung

Copy link
Copy Markdown
Member Author

The bootstrap bump should happen any day now. @Amanieu would be nice to get a release. :)

@Amanieu

Amanieu commented Nov 25, 2024

Copy link
Copy Markdown
Member

Done

RalfJung added a commit to RalfJung/rust that referenced this pull request Nov 30, 2024
bump hashbrown version

This pulls in rust-lang/hashbrown#586, in preparation for rust-lang#102575.

Cc `@Amanieu`
RalfJung added a commit to RalfJung/rust that referenced this pull request Nov 30, 2024
bump hashbrown version

This pulls in rust-lang/hashbrown#586, in preparation for rust-lang#102575.

Cc ``@Amanieu``
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Nov 30, 2024
Rollup merge of rust-lang#133670 - RalfJung:hashbrown, r=Amanieu

bump hashbrown version

This pulls in rust-lang/hashbrown#586, in preparation for rust-lang#102575.

Cc ``@Amanieu``
github-actions Bot pushed a commit to tautschnig/verify-rust-std that referenced this pull request Mar 11, 2025
bump hashbrown version

This pulls in rust-lang/hashbrown#586, in preparation for rust-lang#102575.

Cc ``@Amanieu``
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants