Skip to content

Add clang-tidy correctness CI - #308

Merged
scal444 merged 1 commit into
NVIDIA-BioNeMo:mainfrom
scal444:codex/clang-tidy-ci
Sep 16, 2026
Merged

scal444 merged 1 commit into
NVIDIA-BioNeMo:mainfrom
scal444:codex/clang-tidy-ci

Conversation

@scal444

@scal444 scal444 commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Updates to use clang-tidy 22 which has limited CUDA support.

We're starting with correctness-only items, then will expand with coverage and fixes .

Only true bug fixed in this diff is an unlikely overrun adding 1 to maxBatchSize BEFORE converting to size_t instead of after. The rest are defensive, or needed to get Clang CUDA compiling.

@greptile-apps

greptile-apps Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge, with no outstanding correctness, security, or repository-rule violations identified.

Summary

Adds correctness-focused clang-tidy 22 analysis for host C++ and CUDA code.

  • Introduces a pinned static-analysis CI workflow and CUDA compile-database conversion tooling.
  • Narrows clang-tidy configuration to analyzer and bug-prone correctness checks.
  • Resolves analyzer findings and compatibility issues across C++ and CUDA sources.
  • Corrects integer conversion ordering when allocating and calculating consolidated substructure buffers.
  • Removes an optional local pre-push hook while retaining CI enforcement.

Reviews (3) · Last reviewed commit: "Add clang-tidy correctness CI"

Comment thread .github/workflows/static-analysis.yml Outdated
@scal444
scal444 force-pushed the codex/clang-tidy-ci branch from 75ce898 to ed0c4a6 Compare September 14, 2026 20:39

@evasnow1992 evasnow1992 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you adding this additional gate. Changes look good to me.

@scal444
scal444 merged commit 9df86b2 into NVIDIA-BioNeMo:main Sep 16, 2026
15 checks passed
@scal444
scal444 deleted the codex/clang-tidy-ci branch September 18, 2026 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants