Skip to content

Add codegen tests for fshr and clmul#153450

Open
pvdrz wants to merge 2 commits intorust-lang:mainfrom
ferrocene:pvdrz/llvm-codegen-tests
Open

Add codegen tests for fshr and clmul#153450
pvdrz wants to merge 2 commits intorust-lang:mainfrom
ferrocene:pvdrz/llvm-codegen-tests

Conversation

@pvdrz
Copy link
Contributor

@pvdrz pvdrz commented Mar 5, 2026

  • Add fshr test in codegen-llvm
  • Add clmul test in codegen-llvm

@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 Mar 5, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 5, 2026

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Fallback group: @Mark-Simulacrum, @jieyouxu
  • @Mark-Simulacrum, @jieyouxu expanded to Mark-Simulacrum, jieyouxu
  • Random selection from Mark-Simulacrum, jieyouxu

@@ -0,0 +1,31 @@
//@ compile-flags: -C opt-level=1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why =1?

//@ compile-flags: -C opt-level=1

#![crate_type = "lib"]
#![feature(core_intrinsics)]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason not to call our wrapper around it instead?

#![crate_type = "lib"]
#![feature(core_intrinsics)]

use std::intrinsics::rotate_right;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Similarly, why isn't this best done via the wrappers?

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 9, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 9, 2026

Reminder, once the PR becomes ready for a review, use @rustbot ready.

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. 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.

3 participants