KEMBAR78
Legalization error in basis conversion. by schweitzpgi · Pull Request #794 · NVIDIA/cuda-quantum · GitHub
Skip to content

Conversation

@schweitzpgi
Copy link
Collaborator

This allows legalization to pass in the basis conversion pass. The resulting code may not be executable on the final target, but the assumption is that the QIR profile validation pass will handle those cases.

This allows legalization to pass in the basis conversion pass. The
resulting code may not be executable on the final target, but the
assumption is that the QIR profile validation pass will handle those
cases.
@schweitzpgi schweitzpgi added the bug fix To be listed under Bug Fixes in the release notes label Oct 19, 2023
@schweitzpgi schweitzpgi added this to the release 0.5.0 milestone Oct 19, 2023
@github-actions
Copy link

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

github-actions bot pushed a commit that referenced this pull request Oct 19, 2023
@bmhowe23
Copy link
Collaborator

Just for clarification/documentation, is this change intended to fix this sequence of commands?

[root@docker-desktop build (main)]# nvq++ --target quantinuum --emulate examples/cpp/algorithms/amplitude_estimation.cpp
[root@docker-desktop build (main)]# ./a.out
loc(callsite("amplitude_estimation.cpp":69:15 at "amplitude_estimation.cpp":100:5)): error: failed to legalize operation 'math.fpowi'
terminate called after throwing an instance of 'std::runtime_error'
  what():  Remote rest platform Quake lowering failed.
Aborted

@schweitzpgi schweitzpgi enabled auto-merge (squash) October 19, 2023 19:59
@schweitzpgi schweitzpgi merged commit 6643b2a into NVIDIA:main Oct 19, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Oct 19, 2023
@schweitzpgi schweitzpgi deleted the ch-crash.bc branch October 19, 2023 20:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug fix To be listed under Bug Fixes in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants