rust-lang / stdarch

Rust's standard library vendor-specific APIs and run-time feature detection
https://doc.rust-lang.org/stable/core/arch/
Apache License 2.0
599 stars 260 forks source link

Stabilize AArch64 SHA3 intrinsics #1552

Closed tarcieri closed 3 months ago

tarcieri commented 4 months ago

See also: rust-lang/rust#117225

rustbot commented 4 months ago

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Amanieu (or someone else) some time within the next two weeks.

Amanieu commented 4 months ago

These intrinsics are tested against clang and are known to produce the same results.

@rfcbot fcp merge

Amanieu commented 4 months ago

@rfcbot fcp merge

rfcbot commented 4 months ago

Team member @Amanieu has proposed to merge this. The next step is review by the rest of the tagged team members:

No concerns currently listed.

Once a majority of reviewers approve (and at most 2 approvals are outstanding), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for info about what commands tagged team members can give me.

dtolnay commented 4 months ago

@rfcbot reviewed

BurntSushi commented 4 months ago

The checkboxes are greyed out for me (I can't tick my box), but I approve.

Amanieu commented 4 months ago

I've ticked your box.

rfcbot commented 4 months ago

:bell: This is now entering its final comment period, as per the review above. :bell:

psst @Amanieu, I wasn't able to add the final-comment-period label, please do so.

tarcieri commented 4 months ago

Merging master caused some build failures which seem to be unrelated:

https://github.com/rust-lang/stdarch/actions/runs/8491609619/job/23263755908?pr=1552#step:14:8910

   Compiling intrinsic-test v0.1.0 (/checkout/crates/intrinsic-test)
error: field `reg` is never read
  --> crates/intrinsic-test/src/json_parser.rs:21:9
   |
19 |     Register {
   |     -------- field in this variant
20 |         #[serde(rename = "register")]
21 |         reg: String,
   |         ^^^
   |
   = note: `-D dead-code` implied by `-D warnings`
   = help: to override `-D warnings` add `#[allow(dead_code)]`

error: could not compile `intrinsic-test` (bin "intrinsic-test") due to 1 previous error
Error: Process completed with exit code 101.
Amanieu commented 4 months ago

This seems to be fallout from https://github.com/rust-lang/rust/pull/119552 which was recently merged. Just add #[allow(dead_code)] on that field to ignore the lint.

rfcbot commented 3 months ago

The final comment period, with a disposition to merge, as per the review above, is now complete.

As the automated representative of the governance process, I would like to thank the author for their work and everyone else who contributed.

This will be merged soon.

psst @Amanieu, I wasn't able to add the finished-final-comment-period label, please do so.

bors commented 3 months ago

:umbrella: The latest upstream changes (presumably 24068c78d625f8ddc8ffdabdb3aeb2dc644fac1e) made this pull request unmergeable. Please resolve the merge conflicts.

Amanieu commented 3 months ago

Can you squash and rebase?

tarcieri commented 3 months ago

@Amanieu done