KEMBAR78
Add support for some math functions inside kernels by schweitzpgi · Pull Request #2989 · NVIDIA/cuda-quantum · GitHub
Skip to content

Conversation

@schweitzpgi
Copy link
Collaborator

@schweitzpgi schweitzpgi commented Jun 3, 2025

Some of the math functions on the list only appear in MLIR after LLVM 16. However, we can add support for the rest to the C++ bridge. This patch adds support for sin, cos, tan, exp, log, ceil, and floor.

See issue #2942

Some of the math functions on the list only appear in MLIR after LLVM 16.
However, we can add support for the rest to the C++ bridge. This patch
adds support for sin, cos, tan, exp, log, ceil, and floor.

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>
github-actions bot pushed a commit that referenced this pull request Jun 3, 2025
@github-actions
Copy link

github-actions bot commented Jun 3, 2025

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

Copy link
Collaborator

@sacpis sacpis left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @schweitzpgi.

@schweitzpgi schweitzpgi enabled auto-merge (squash) June 4, 2025 17:47
github-actions bot pushed a commit that referenced this pull request Jun 4, 2025
@github-actions
Copy link

github-actions bot commented Jun 4, 2025

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

@github-actions
Copy link

github-actions bot commented Jun 4, 2025

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

github-actions bot pushed a commit that referenced this pull request Jun 4, 2025
github-actions bot pushed a commit that referenced this pull request Jun 4, 2025
@github-actions
Copy link

github-actions bot commented Jun 4, 2025

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

@schweitzpgi schweitzpgi merged commit 164fbb5 into NVIDIA:main Jun 5, 2025
193 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 5, 2025
github-actions bot pushed a commit that referenced this pull request Jun 5, 2025
@github-actions
Copy link

github-actions bot commented Jun 5, 2025

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

@schweitzpgi schweitzpgi deleted the ch-i2942 branch June 5, 2025 01:46
annagrin pushed a commit to annagrin/cuda-quantum that referenced this pull request Jun 17, 2025
…42. (NVIDIA#2989)

Some of the math functions on the list only appear in MLIR after LLVM 16.
However, we can add support for the rest to the C++ bridge. This patch
adds support for sin, cos, tan, exp, log, ceil, and floor.

Signed-off-by: Eric Schweitz <eschweitz@nvidia.com>
Signed-off-by: Anna Gringauze <agringauze@nvidia.com>
@bettinaheim bettinaheim changed the title [C++ bridge] Add support for some of the functions listed in issue 2942. Add support for some math functions inside kernels Jul 22, 2025
@bettinaheim bettinaheim added the release notes Changes need to be captured in the release notes label Jul 22, 2025
@bettinaheim bettinaheim added this to the release 0.12.0 milestone Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes Changes need to be captured in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants