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

Add kl and widekl target features, and the feature gate #134814

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

sayantn
Copy link
Contributor

@sayantn sayantn commented Dec 27, 2024

This is an effort towards #134813. This PR adds the target-features and the feature gate to rustc

@rustbot
Copy link
Collaborator

rustbot commented Dec 27, 2024

r? @BoxyUwU

rustbot has assigned @BoxyUwU.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Dec 27, 2024
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Contributor

bors commented Jan 6, 2025

☔ The latest upstream changes (presumably #134794) made this pull request unmergeable. Please resolve the merge conflicts.

@rustbot rustbot added A-target-feature Area: Enabling/disabling target features like AVX, Neon, etc. O-x86_32 Target: x86 processors, 32 bit (like i686-*) O-x86_64 Target: x86-64 processors (like x86_64-*) labels Jan 8, 2025
@rustbot rustbot assigned jieyouxu and unassigned BoxyUwU Jan 8, 2025
@jieyouxu
Copy link
Member

jieyouxu commented Jan 8, 2025

r? compiler

@rustbot rustbot assigned fmease and unassigned jieyouxu Jan 8, 2025
@rustbot rustbot assigned oli-obk and unassigned fmease Jan 24, 2025
@oli-obk
Copy link
Contributor

oli-obk commented Jan 29, 2025

The tracking issue is a bit sparse on what the plan here is. Please add some intrinsics you'll be adding in the future and maybe show a small code snippet about how these will be used in the future.

I'm not sure where we document individual target features, @veluca93 is there anything like this? Should we start having such documentation?

@veluca93
Copy link
Contributor

AFAIU the stabilized target features are documented (at least) here: https://doc.rust-lang.org/reference/attributes/codegen.html#the-target_feature-attribute

@sayantn
Copy link
Contributor Author

sayantn commented Feb 2, 2025

I cannot add intrinsics until the target feature is added, because stdarch is a separate repo which uses nightly rust.

And for clarity, keylocker is a standard Intel ISA. I would argue that it is not more difficult to interpret the meaning of keylocker compared to things like avx512vbmi2

@oli-obk
Copy link
Contributor

oli-obk commented Feb 2, 2025

Sorry, not trying to hold this up. It's just a bit outside my comfort zone.

What I meant was "add some info to the issue" like just an example intrinsic and an example how a caller could look.

Doesn't need to block this PR tho

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Feb 2, 2025

📌 Commit dc49fdd has been approved by oli-obk

It is now in the queue for this repository.

@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 Feb 2, 2025
@sayantn
Copy link
Contributor Author

sayantn commented Feb 2, 2025

Ah ok, no problems. I would add an example intrinsic in the issue

jieyouxu added a commit to jieyouxu/rust that referenced this pull request Feb 3, 2025
Add `kl` and `widekl` target features, and the feature gate

This is an effort towards rust-lang#134813. This PR adds the target-features and the feature gate to `rustc`

<!--
`@rustbot` label O-x86_64 O-x86_32 A-target-feature
r? compiler
-->
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 3, 2025
Rollup of 8 pull requests

Successful merges:

 - rust-lang#134807 (fix(rustdoc): always use a channel when linking to doc.rust-lang.org)
 - rust-lang#134814 (Add `kl` and `widekl` target features, and the feature gate)
 - rust-lang#135836 (bootstrap: only build `crt{begin,end}.o` when compiling to MUSL)
 - rust-lang#136022 (Port ui/simd tests to use the intrinsic macro)
 - rust-lang#136235 (Pretty print pattern type values with transmute if they don't satisfy their pattern)
 - rust-lang#136309 (set rustc dylib on manually constructed rustc command)
 - rust-lang#136392 (bootstrap: add wrapper macros for `feature = "tracing"`-gated `tracing` macros)
 - rust-lang#136462 (mir_build: Simplify `lower_pattern_range_endpoint`)

r? `@ghost`
`@rustbot` modify labels: rollup
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Feb 3, 2025
Add `kl` and `widekl` target features, and the feature gate

This is an effort towards rust-lang#134813. This PR adds the target-features and the feature gate to `rustc`

<!--
``@rustbot`` label O-x86_64 O-x86_32 A-target-feature
r? compiler
-->
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 4, 2025
…iaskrgr

Rollup of 7 pull requests

Successful merges:

 - rust-lang#134807 (fix(rustdoc): always use a channel when linking to doc.rust-lang.org)
 - rust-lang#134814 (Add `kl` and `widekl` target features, and the feature gate)
 - rust-lang#135836 (bootstrap: only build `crt{begin,end}.o` when compiling to MUSL)
 - rust-lang#136022 (Port ui/simd tests to use the intrinsic macro)
 - rust-lang#136309 (set rustc dylib on manually constructed rustc command)
 - rust-lang#136392 (bootstrap: add wrapper macros for `feature = "tracing"`-gated `tracing` macros)
 - rust-lang#136462 (mir_build: Simplify `lower_pattern_range_endpoint`)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 4, 2025
…iaskrgr

Rollup of 6 pull requests

Successful merges:

 - rust-lang#134807 (fix(rustdoc): always use a channel when linking to doc.rust-lang.org)
 - rust-lang#134814 (Add `kl` and `widekl` target features, and the feature gate)
 - rust-lang#135836 (bootstrap: only build `crt{begin,end}.o` when compiling to MUSL)
 - rust-lang#136022 (Port ui/simd tests to use the intrinsic macro)
 - rust-lang#136309 (set rustc dylib on manually constructed rustc command)
 - rust-lang#136462 (mir_build: Simplify `lower_pattern_range_endpoint`)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit e4eedb5 into rust-lang:master Feb 4, 2025
6 checks passed
@rustbot rustbot added this to the 1.86.0 milestone Feb 4, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Feb 4, 2025
Rollup merge of rust-lang#134814 - sayantn:keylocker, r=oli-obk

Add `kl` and `widekl` target features, and the feature gate

This is an effort towards rust-lang#134813. This PR adds the target-features and the feature gate to `rustc`

<!--
```@rustbot``` label O-x86_64 O-x86_32 A-target-feature
r? compiler
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-target-feature Area: Enabling/disabling target features like AVX, Neon, etc. O-x86_32 Target: x86 processors, 32 bit (like i686-*) O-x86_64 Target: x86-64 processors (like x86_64-*) S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants