Skip to content

Use cfg_os_version_min in std#136868

Draft
madsmtm wants to merge 4 commits intorust-lang:mainfrom
madsmtm:cfg_os_version_min_std
Draft

Use cfg_os_version_min in std#136868
madsmtm wants to merge 4 commits intorust-lang:mainfrom
madsmtm:cfg_os_version_min_std

Conversation

@madsmtm
Copy link
Copy Markdown
Contributor

@madsmtm madsmtm commented Feb 11, 2025

Use cfg(os_version_min) to avoid weak linkage when we statically know that a symbol is available.

Builds upon #136867 (tracking issue #136866), hence why this is still a draft. See the diff in the last commit.

NOTE: This partly exposes implementation details of cfg(os_version_min) to -Zbuild-std, namely the ability to unconditionally select the newer implementations when passing *_DEPLOYMENT_TARGET. This should only be a performance improvement (or a regression, if we decide to remove the feature in the future).

r? rust-lang/libs
@rustbot label O-apple O-unix O-android

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants