Skip to content

fix: Remove deprecated clang-tidy config option#1276

Open
Sqvid wants to merge 1 commit intoARM-software:mainfrom
Sqvid:rm-bad-tidy-config
Open

fix: Remove deprecated clang-tidy config option#1276
Sqvid wants to merge 1 commit intoARM-software:mainfrom
Sqvid:rm-bad-tidy-config

Conversation

@Sqvid
Copy link
Copy Markdown
Contributor

@Sqvid Sqvid commented Apr 9, 2026

The option AnalyzeTemporaryDtors has been deprecated since clang-tidy version 16 and was removed in version 18. As a result, this config option will throw errors with newer versions of these tools while not providing any practical benefit.

Copy link
Copy Markdown
Contributor

@gunes-arm gunes-arm left a comment

Choose a reason for hiding this comment

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

Can you make "R" in "remove dep..." uppercase in the commit and PR titles?

@Sqvid Sqvid changed the title fix: remove deprecated clang-tidy config option fix: Remove deprecated clang-tidy config option Apr 9, 2026
The option `AnalyzeTemporaryDtors` has been deprecated since
`clang-tidy` version 16 and was removed in version 18. As a result, this
config option will throw errors with newer versions of these tools while
not providing any practical benefit.

Change-Id: I096407f9dc41e4564a4fc89f5a9f39a834b5e20a
Signed-off-by: Siddhartha Menon <siddhartha.menon@arm.com>
@Sqvid Sqvid force-pushed the rm-bad-tidy-config branch from afa3728 to 96bf319 Compare April 9, 2026 11:54
@Sqvid Sqvid requested a review from gunes-arm April 9, 2026 11:55
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