Skip to content

Commit 9695bae

Browse files
authored
[Proton] Rename knobs.proton.cupti_path to knobs.proton.cupti_dir to make clear that it should be the parent directory to .so file (#7155)
Other knobs with PATH seem to allow referencing the library directly. This name update clarifies that it must be the parent directory.
1 parent 418c127 commit 9695bae

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

python/triton/knobs.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -450,7 +450,7 @@ class amd_knobs(base_knobs):
450450

451451

452452
class proton_knobs(base_knobs):
453-
cupti_path: env_opt_str = env_opt_str("TRITON_CUPTI_LIB_PATH")
453+
cupti_dir: env_opt_str = env_opt_str("TRITON_CUPTI_LIB_PATH")
454454

455455

456456
build = build_knobs()

third_party/proton/proton/profile.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ def _get_backend_default_path(backend: str) -> str:
2626
lib_path = ""
2727
if backend == "cupti":
2828
# First try to get the path from the environment variable that overrides the default path
29-
lib_path = knobs.proton.cupti_path
29+
lib_path = knobs.proton.cupti_dir
3030
if lib_path is None:
3131
# Get the default path for the cupti backend,
3232
# which is the most compatible with the current CUPTI header file triton is compiled with

0 commit comments

Comments
 (0)