[NVVM] Expose nvvm version detection in cuda.bindings.utils. by abhilash1910 · Pull Request #1837 · NVIDIA/cuda-python

abhilash1910 · 2026-03-31T13:09:39Z

Description

Fixes issue #1457 . Provides an api call to cuda.bindings.utils to check the version of nvvm.
@leofang @rwgk pinging for review.

copy-pr-bot · 2026-03-31T13:09:44Z

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.

abhilash1910 · 2026-03-31T13:10:10Z

pre-commit.ci autofix

rwgk · 2026-04-01T04:12:18Z

@@ -0,0 +1,93 @@
+# SPDX-FileCopyrightText: Copyright (c) 2026-2027 NVIDIA CORPORATION & AFFILIATES. All rights reserved.


Remove -2027

abhilash1910 · 2026-04-06T12:42:53Z

pre-commit.ci autofix

abhilash1910 · 2026-04-07T07:01:30Z

pre-commit.ci autofix

rwgk · 2026-04-07T15:22:57Z

/ok to test c13c860

rwgk · 2026-04-07T16:02:01Z

/ok to test e2afa85

rwgk · 2026-04-07T17:11:09Z

This doesn't work with the cuda-bindings from the 12.9.x backport branch, e.g.:

https://github.com/NVIDIA/cuda-python/actions/runs/24091305075/job/70279241852?pr=1837

tests/test_program.py:76: in _check_nvvm_arch
    from cuda.bindings.utils import check_nvvm_compiler_options
E   ImportError: cannot import name 'check_nvvm_compiler_options' from 'cuda.bindings.utils' (/opt/hostedtoolcache/Python/3.10.20/x64/lib/python3.10/site-packages/cuda/bindings/utils/__init__.py)

@leofang wrote under #1457:

We should expose it to cuda.bindings.utils, similar to the PTX version helpers.

That makes using it from cuda_core/tests/ a little tricky: it'll only be in future cuda-bindings releases.

@leofang how would you handle this situation?

rwgk · 2026-04-07T20:26:38Z

@abhilash1910 — Offline we decided (roughly): let's move the (or some) tests to cuda_bindings/, and if we want to keep certain tests in cuda_core/, skip them if cuda.bindings.utils.check_nvvm_compiler_options is not available. Could you please try how that shakes out? Please ping me on chat if you have questions.

abhilash1910 · 2026-04-16T10:36:04Z

pre-commit.ci autofix

rwgk

A few pretty minor suggestions (docstring fixes and test code only).

I verified that all existing test coverage is preserved.

abhilash1910 · 2026-04-17T06:55:16Z

pre-commit.ci autofix

abhilash1910 · 2026-04-17T09:11:35Z

/ok to test 9ad12b1

rwgk · 2026-04-17T13:44:50Z

/ok to test 9ad12b1

rwgk · 2026-04-17T15:35:07Z

/ok to test 9ad12b1

copy-pr-bot · 2026-04-17T15:35:11Z

/ok to test 9ad12b1

@rwgk, there was an error processing your request: E2

See the following link for more information: https://docs.gha-runners.nvidia.com/cpr/e/2/

rwgk · 2026-04-17T15:54:48Z

/ok to test d821db1

github-actions · 2026-04-20T00:26:00Z

Doc Preview CI
Preview removed because the pull request was closed or merged.

…1837) * refactor cuda bindings utils for nvvm * [pre-commit.ci] auto code formatting * refresh * renaming * exceptions * refresh * [pre-commit.ci] auto code formatting * refresh * [pre-commit.ci] auto code formatting * refresh * refresh * refresh * [pre-commit.ci] auto code formatting * refresh * [pre-commit.ci] auto code formatting * refresh --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

refactor cuda bindings utils for nvvm

65fb587

[pre-commit.ci] auto code formatting

10943c8

abhilash1910 changed the title ~~[NVVM]Expose nvvm version detection in cuda.bindings.utils.~~ [NVVM][Fix] Expose nvvm version detection in cuda.bindings.utils. Mar 31, 2026

refresh

a7ef552

abhilash1910 changed the title ~~[NVVM][Fix] Expose nvvm version detection in cuda.bindings.utils.~~ [NVVM] Expose nvvm version detection in cuda.bindings.utils. Mar 31, 2026

rwgk assigned abhilash1910 Mar 31, 2026

rwgk added the cuda.bindings Everything related to the cuda.bindings module label Mar 31, 2026

rwgk added this to the cuda.bindings backlog milestone Mar 31, 2026

rwgk added enhancement Any code-related improvements P1 Medium priority - Should do labels Mar 31, 2026

rwgk reviewed Apr 1, 2026

View reviewed changes

rparolin removed this from the cuda.bindings backlog milestone Apr 2, 2026

abhilash1910 added 3 commits April 6, 2026 09:29

renaming

4f3b442

exceptions

686c62a

refresh

3c5043c

[pre-commit.ci] auto code formatting

b1747af

rwgk reviewed Apr 6, 2026

View reviewed changes

Comment thread cuda_bindings/cuda/bindings/utils/__init__.py

Comment thread cuda_bindings/cuda/bindings/utils/_nvvm_utils.py Outdated

Comment thread cuda_bindings/cuda/bindings/utils/_nvvm_utils.py Outdated

refresh

d19960e

[pre-commit.ci] auto code formatting

4a7c3e9

rwgk reviewed Apr 7, 2026

View reviewed changes

Comment thread cuda_bindings/cuda/bindings/utils/_nvvm_utils.py Outdated

refresh

c13c860

rwgk added this to the cuda.bindings next milestone Apr 7, 2026

rwgk added awaiting-response Further information is requested and removed awaiting-response Further information is requested labels Apr 7, 2026

rwgk reviewed Apr 7, 2026

View reviewed changes

Comment thread cuda_bindings/cuda/bindings/utils/_nvvm_utils.py Outdated

This comment has been minimized.

Sign in to view

refresh

8239ee6

github-actions Bot added the Needs-Restricted-Paths-Review PR touches cuda_bindings or cuda_python; only NVIDIA employees may modify these paths; see LICENSEs label Apr 16, 2026

[pre-commit.ci] auto code formatting

5c5024c

rwgk removed the Needs-Restricted-Paths-Review PR touches cuda_bindings or cuda_python; only NVIDIA employees may modify these paths; see LICENSEs label Apr 16, 2026

rwgk reviewed Apr 16, 2026

View reviewed changes

Comment thread cuda_bindings/cuda/bindings/utils/_nvvm_utils.py Outdated

Comment thread cuda_bindings/tests/test_utils.py

Comment thread cuda_core/tests/test_program.py

Comment thread cuda_bindings/cuda/bindings/utils/_nvvm_utils.py Outdated

refresh

9003ff1

github-actions Bot added the Needs-Restricted-Paths-Review PR touches cuda_bindings or cuda_python; only NVIDIA employees may modify these paths; see LICENSEs label Apr 17, 2026

pre-commit-ci Bot and others added 2 commits April 17, 2026 06:55

[pre-commit.ci] auto code formatting

39633c2

refresh

9ad12b1

rwgk approved these changes Apr 17, 2026

View reviewed changes

rwgk enabled auto-merge (squash) April 17, 2026 13:45

Merge branch 'main' into nvvm_fix

d821db1

rwgk removed the Needs-Restricted-Paths-Review PR touches cuda_bindings or cuda_python; only NVIDIA employees may modify these paths; see LICENSEs label Apr 17, 2026

rwgk merged commit a18022c into NVIDIA:main Apr 17, 2026
92 checks passed

This comment has been minimized.

Sign in to view

		@@ -0,0 +1,93 @@
		# SPDX-FileCopyrightText: Copyright (c) 2026-2027 NVIDIA CORPORATION & AFFILIATES. All rights reserved.

Conversation

abhilash1910 commented Mar 31, 2026

Description

Uh oh!

copy-pr-bot Bot commented Mar 31, 2026

Uh oh!

abhilash1910 commented Mar 31, 2026

Uh oh!

Uh oh!

rwgk Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

abhilash1910 commented Apr 6, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

abhilash1910 commented Apr 7, 2026

Uh oh!

Uh oh!

rwgk commented Apr 7, 2026

Uh oh!

Uh oh!

rwgk commented Apr 7, 2026

Uh oh!

This comment has been minimized.

rwgk commented Apr 7, 2026

Uh oh!

rwgk commented Apr 7, 2026

Uh oh!

abhilash1910 commented Apr 16, 2026

Uh oh!

rwgk left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

abhilash1910 commented Apr 17, 2026

Uh oh!

abhilash1910 commented Apr 17, 2026

Uh oh!

rwgk commented Apr 17, 2026

Uh oh!

rwgk commented Apr 17, 2026

Uh oh!

copy-pr-bot Bot commented Apr 17, 2026

Uh oh!

rwgk commented Apr 17, 2026

Uh oh!

Uh oh!

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

github-actions Bot commented Apr 20, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants