[all-commits] [llvm/llvm-project] 478863: [clangd] Basic IncludeCleaner support for c/c++ st...
Sam McCall via All-commits
all-commits at lists.llvm.org
Mon Jan 3 09:20:12 PST 2022
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 478863ef58c7f7314e0669d332a90d6e233d44fb
https://github.com/llvm/llvm-project/commit/478863ef58c7f7314e0669d332a90d6e233d44fb
Author: Sam McCall <sam.mccall at gmail.com>
Date: 2022-01-03 (Mon, 03 Jan 2022)
Changed paths:
M clang-tools-extra/clangd/Headers.cpp
M clang-tools-extra/clangd/Headers.h
M clang-tools-extra/clangd/IncludeCleaner.cpp
M clang-tools-extra/clangd/IncludeCleaner.h
M clang-tools-extra/clangd/tool/ClangdMain.cpp
M clang-tools-extra/clangd/unittests/HeadersTests.cpp
M clang-tools-extra/clangd/unittests/IncludeCleanerTests.cpp
Log Message:
-----------
[clangd] Basic IncludeCleaner support for c/c++ standard library
There are some limitations here, so this is behind a flag for now (in addition
to the config setting for the overall feature).
- symbols without exactly one associated header aren't handled right
- no macro support
- referencing std::size_t usually doesn't leave any trace in the AST that the
alias in std was used, so we associate with stddef.h instead of cstddef.
(An AST issue not specific to stdlib, but much worse there)
Differential Revision: https://reviews.llvm.org/D114077
More information about the All-commits
mailing list