Skip to content

Commit

Permalink
CI: Pin clang-tidy to 15.0.7. (#5661)
Browse files Browse the repository at this point in the history
Most recent supported version by libcudacxx.

Compilation introduced as a transitive dependency from rmm.

Authors:
  - Simon Adorf (https://github.com/csadorf)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - Dante Gama Dessavre (https://github.com/dantegd)
  - Jake Awe (https://github.com/AyodeAwe)

URL: #5661
  • Loading branch information
csadorf authored Nov 18, 2023
1 parent 8b07b00 commit c48eaa1
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
4 changes: 2 additions & 2 deletions conda/environments/clang_tidy_cuda-118_arch-x86_64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@ channels:
- nvidia
dependencies:
- c-compiler
- clang-tools==16.0.6
- clang==16.0.6
- clang-tools==15.0.7
- clang==15.0.7
- cmake>=3.26.4
- cuda-version=11.8
- cudatoolkit
Expand Down
2 changes: 1 addition & 1 deletion cpp/scripts/run-clang-tidy.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

import tomli

EXPECTED_VERSION = "16.0.6"
EXPECTED_VERSION = "15.0.7"
VERSION_REGEX = re.compile(r" LLVM version ([0-9.]+)")
GPU_ARCH_REGEX = re.compile(r"sm_(\d+)")
SPACES = re.compile(r"\s+")
Expand Down
5 changes: 3 additions & 2 deletions dependencies.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -95,8 +95,9 @@ dependencies:
common:
- output_types: [conda, requirements]
packages:
- clang==16.0.6
- clang-tools==16.0.6
# clang 15 required by libcudacxx.
- clang==15.0.7
- clang-tools==15.0.7
- ninja
- tomli
common_build:
Expand Down

0 comments on commit c48eaa1

Please sign in to comment.