Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Comments

Close side panel

Update to the Clang v20.1.7 version of the main#45

Merged
ktf merged 1 commit intomasterAliceO2Group/O2CodeChecker:masterfrom
pr45AliceO2Group/O2CodeChecker:pr45Copy head branch name to clipboard
Aug 29, 2025
Merged

Update to the Clang v20.1.7 version of the main#45
ktf merged 1 commit intomasterAliceO2Group/O2CodeChecker:masterfrom
pr45AliceO2Group/O2CodeChecker:pr45Copy head branch name to clipboard

Conversation

@ktf
Copy link
Member

@ktf ktf commented Aug 12, 2025 •
edited
Loading

No description provided.

@alibuild
Copy link
Collaborator

alibuild commented Aug 12, 2025

Error while checking build/o2codechecker/o2 for 85f3dd5 at 2025-08-12 09:38:

## sw/BUILD/o2codechecker-latest/log
CMake Error at /sw/slc9_x86-64/Clang/v18.1.8-21/lib/cmake/llvm/AddLLVM.cmake:596 (add_library):
CMake Error at /sw/slc9_x86-64/Clang/v18.1.8-21/lib/cmake/llvm/AddLLVM.cmake:720 (target_link_libraries):

Full log here.

@alibuild
Copy link
Collaborator

alibuild commented Aug 27, 2025

Error while checking build/o2codechecker/o2 for afd6c48 at 2025-08-27 09:56:

## sw/BUILD/o2codechecker-latest/log
/sw/SOURCES/o2codechecker/build_o2codechecker_o2-slc9_x86-64/0/tool/ClangTidyMain.cpp:386:18: error: 'struct clang::tidy::ClangTidyOptions' has no member named 'ExcludeHeaderFilterRegex'; did you mean 'HeaderFilterRegex'?
/sw/SOURCES/o2codechecker/build_o2codechecker_o2-slc9_x86-64/0/tool/ClangTidyMain.cpp:402:21: error: 'struct clang::tidy::ClangTidyOptions' has no member named 'ExcludeHeaderFilterRegex'; did you mean 'HeaderFilterRegex'?
/sw/SOURCES/o2codechecker/build_o2codechecker_o2-slc9_x86-64/0/tool/ClangTidyMain.cpp:492:33: error: 'class clang::tidy::GlobList' has no member named 'getItems'; did you mean 'llvm::SmallVector<clang::tidy::GlobList::GlobListItem, 0> clang::tidy::GlobList::Items'? (not accessible from this context)
/sw/SOURCES/o2codechecker/build_o2codechecker_o2-slc9_x86-64/0/tool/ClangTidyMain.cpp:496:53: error: 'Item' is not captured
/sw/SOURCES/o2codechecker/build_o2codechecker_o2-slc9_x86-64/0/tool/ClangTidyMain.cpp:672:5: error: 'ChecksAndOptions' was not declared in this scope; did you mean 'NamesAndOptions'?
/sw/SOURCES/o2codechecker/build_o2codechecker_o2-slc9_x86-64/0/tool/ClangTidyMain.cpp:677:36: error: 'Valid' was not declared in this scope
/sw/SOURCES/o2codechecker/build_o2codechecker_o2-slc9_x86-64/0/tool/ClangTidyMain.cpp:682:35: error: 'Valid' was not declared in this scope
make[2]: *** [tool/CMakeFiles/clangTidyMainO2.dir/build.make:79: tool/CMakeFiles/clangTidyMainO2.dir/ClangTidyMain.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:388: tool/CMakeFiles/clangTidyMainO2.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:146: all] Error 2

Full log here.

@ktf
Update to the Clang v20.1.7 version of the main
a4e578d 
Also make sure we build only the plugin, since the upstream
clang now supports loading clang-tidy plugins via the -load
option.
@ktf
Copy link
Member Author

ktf commented Aug 28, 2025

@sawenzel any objections to this? Basically the new versions of clang seem to have proper plugin management.

@ktf
Copy link
Member Author

ktf commented Aug 29, 2025

Discussed offline. Merging.

@ktf ktf merged commit 5ddf284 into master Aug 29, 2025
1 check passed
@ktf ktf deleted the pr45 branch August 29, 2025 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ktf @alibuild
Morty Proxy This is a proxified and sanitized view of the page, visit original site.