Skip to content

Fix wrong BITWISE XOR#4220

Open
benediktjohannes wants to merge 2 commits intoNVIDIA:mainfrom
benediktjohannes:patch-6
Open

Fix wrong BITWISE XOR#4220
benediktjohannes wants to merge 2 commits intoNVIDIA:mainfrom
benediktjohannes:patch-6

Conversation

@benediktjohannes
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Benedikt Johannes <benedikt.johannes.hofer@gmail.com>
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Mar 29, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@sacpis
Copy link
Copy Markdown
Collaborator

sacpis commented Mar 30, 2026

/ok to test 58a8470

Command Bot: Processing...

__qpu__ auto operator()(const int n_qubits, CallableKernel &&oracle,
const long target_state) {
int n_iterations = round(0.25 * M_PI * sqrt(2 ^ n_qubits));
int n_iterations = round(0.25 * M_PI * sqrt(std::pow(2, n_qubits)));
Copy link
Copy Markdown
Collaborator

@sacpis sacpis Mar 30, 2026

Choose a reason for hiding this comment

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

Can you please fix the MLIR below? You will need to replace arith.xori.

Copy link
Copy Markdown
Collaborator

@sacpis sacpis Mar 30, 2026

Choose a reason for hiding this comment

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

nit: I don't think you need to specify the namespace std::, as already have included cmath lib.

@sacpis sacpis self-requested a review March 30, 2026 21:02
@sacpis
Copy link
Copy Markdown
Collaborator

sacpis commented Mar 30, 2026

Would like to request to run the test or any relevant file after you make changes.

github-actions bot pushed a commit that referenced this pull request Mar 30, 2026
@github-actions
Copy link
Copy Markdown

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

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.

2 participants