Skip to content

clang-tidy: remove AnalyzeTemporaryDtors key#1778

Merged
rapids-bot[bot] merged 1 commit intomainfrom
fix-AnalyzeTemporaryDtors
Feb 7, 2026
Merged

clang-tidy: remove AnalyzeTemporaryDtors key#1778
rapids-bot[bot] merged 1 commit intomainfrom
fix-AnalyzeTemporaryDtors

Conversation

@achirkin
Copy link
Copy Markdown
Contributor

@achirkin achirkin commented Feb 6, 2026

The AnalyzeTemporaryDtors was removed from clang-tidy since around ver 18.
This PR removes the option to make clang-tidy not crash on our codebase.

Reference: https://prereleases.llvm.org/16.0.0/rc3/tools/clang/tools/extra/docs/ReleaseNotes.html#improvements-to-clang-tidy

@achirkin achirkin self-assigned this Feb 6, 2026
@achirkin achirkin requested a review from a team as a code owner February 6, 2026 13:40
@achirkin achirkin added bug Something isn't working non-breaking Introduces a non-breaking change labels Feb 6, 2026
@achirkin
Copy link
Copy Markdown
Contributor Author

achirkin commented Feb 7, 2026

/merge

@rapids-bot rapids-bot Bot merged commit fa131a7 into main Feb 7, 2026
80 checks passed
benfred pushed a commit to benfred/cuvs that referenced this pull request Feb 10, 2026
The `AnalyzeTemporaryDtors` was removed from clang-tidy since around ver 18.
This PR removes the option to make clang-tidy not crash on our codebase.

Reference: https://prereleases.llvm.org/16.0.0/rc3/tools/clang/tools/extra/docs/ReleaseNotes.html#improvements-to-clang-tidy

Authors:
  - Artem M. Chirkin (https://github.com/achirkin)

Approvers:
  - Corey J. Nolet (https://github.com/cjnolet)

URL: rapidsai#1778
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants