This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Document how to build and run clang-tidy checks
Needs ReviewPublic

Authored by philnik on Jan 17 2023, 3:02 PM.

Details

Reviewers
ldionne
Mordante
Group Reviewers
Restricted Project
Summary

Getting clang-tidy to work can be a little bit more involved than most test utilies. This documents what is required to get clang-tidy working with system packages to make it a bit easier.

Diff Detail

Event Timeline

philnik created this revision.Jan 17 2023, 3:02 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 17 2023, 3:02 PM
philnik requested review of this revision.Jan 17 2023, 3:02 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 17 2023, 3:02 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision as: Mordante.Jan 18 2023, 9:44 AM

Thanks for working on this. LGTM, but I think a native speaker may want to have a pass over the text too.

libcxx/docs/TestingLibcxx.rst
129–130

I think the other part doesn't add much info.

132

Note it seems this gives issues on Debian, the first run works, then trying to run CMake again will try write things in
/usr/lib/llvm-16/include/c++/v1/__config_site.tmp, which for good reasons is not allowed.

Have you seen this before? If not I can file a bug report.

philnik added inline comments.Jan 24 2023, 10:19 PM
libcxx/docs/TestingLibcxx.rst
132

I've had similar problems, but I thought it was a problem with my setup. Given that you also have this problem, I'd say, file a bug report.

philnik updated this revision to Diff 504138.Mar 10 2023, 7:18 AM
philnik marked 2 inline comments as done.

Address comments