Age | Commit message (Collapse) | Author |
|
after r351047
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@351307 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Removed the uses of the allOf() matcher inside node matchers that are implicit
allOf(). Replaced uses of allOf() with the explicit node matcher where it makes
matchers more readable. Replace anyOf(hasName(), hasName(), ...) with the more
efficient and readable hasAnyName().
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@347520 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Subscribers: nemanjai, ioeric, kbarton, cfe-commits
Differential Revision: https://reviews.llvm.org/D50355
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@339401 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Reviewers: javed.absar
Subscribers: nemanjai, kbarton, ilya-biryukov, ioeric, jkorous, arphaman, jfb, cfe-commits
Differential Revision: https://reviews.llvm.org/D50354
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@339400 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary: See the test case for a repro.
Reviewers: juliehockett, ioeric, hokein, aaron.ballman
Reviewed By: hokein
Subscribers: lebedev.ri, xazax.hun, cfe-commits
Differential Revision: https://reviews.llvm.org/D49862
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@338124 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Summary:
This patch removes a private matcher in fuchsia/TrailingReturnType check because
the matcher is now in ASTMatchers
Reviewers: aaron.ballman, alexfh, hokein
Reviewed By: aaron.ballman
Subscribers: xazax.hun, cfe-commits
Differential Revision: https://reviews.llvm.org/D49618
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@337707 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
This relands r332125 with a fixed test.
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@332141 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
This reverts commit r332125 for a failing test.
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@332131 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Adding a check to restrict system includes to a whitelist. Given a list
of includes that are explicitly allowed, the check issues a fixit to
remove any system include not on that list from the source file.
Differential Revision: https://reviews.llvm.org/D43778
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@332125 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
This reverts commit r331930, which was landed by accident.
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@331934 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Adding a check to restrict system includes to a whitelist. Given a list
of includes that are explicitly allowed, the check issues a fixit to
remove any system include not on that list from the source file.
Differential Revision: https://reviews.llvm.org/D43778
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@331930 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Fixes build for me:
[1/7] Linking CXX shared library lib/libclangTidyFuchsiaModule.so.7svn
FAILED: lib/libclangTidyFuchsiaModule.so.7svn
<...>
/usr/local/bin/ld.lld: error: undefined symbol: clang::tidy::google::build::UnnamedNamespaceInHeaderCheck::UnnamedNamespaceInHeaderCheck(llvm::StringRef, clang::tidy::ClangTidyContext*)
>>> referenced by FuchsiaTidyModule.cpp
>>> tools/clang/tools/extra/clang-tidy/fuchsia/CMakeFiles/clangTidyFuchsiaModule.dir/FuchsiaTidyModule.cpp.o:(std::_Function_handler<clang::tidy::ClangTidyCheck* (llvm::StringRef, clang::tidy::ClangTidyContext*), void clang::tidy::ClangTidyCheckFactories::registerCheck<clang::tidy::google::build::UnnamedNamespaceInHeaderCheck>(llvm::StringRef)::'lambda'(llvm::StringRef, clang::tidy::ClangTidyContext*)>::_M_invoke(std::_Any_data const&, llvm::StringRef&&, clang::tidy::ClangTidyContext*&&))
Refs. D45447, rCTE329720
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@329749 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Adding alias to google-build-namespaces to the Fuchsia module (checks
for anonymous namespaces in headers).
Differential Revision: https://reviews.llvm.org/D45447
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@329720 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
conflicts.
No functionality change intended.
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@325467 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Updating the fuchsia-multiple-inheritance to gracefully handle unknown
record types (e.g. templatized classes) by simply continuing, rather
than asserting and failing.
Fixes PR36052.
Differential Revision: https://reviews.llvm.org/D43223
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@325015 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Updating fuchsia-multiple-inheritance to not crash when a record
inherits a template.
Fixes PR36052.
Differential Revision: https://reviews.llvm.org/D42918
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@324432 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Adds a check to the Fuchsia module to warn when a class
inherits from multiple classes that are not pure virtual.
See https://fuchsia.googlesource.com/zircon/+/master/docs/cxx.md
for reference.
Differential Revision: https://reviews.llvm.org/D40580
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@323011 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Adds a check to the Fuchsia module to warn if a function has a trailing
return.
See https://fuchsia.googlesource.com/zircon/+/master/docs/cxx.md for
reference.
Differential Revision: https://reviews.llvm.org/D42116
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@322759 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Updating Fuchsia overloaded operator warning message for clarity.
Differential Revision: https://reviews.llvm.org/D42120
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@322692 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Adds a check to the Fuchsia module to warn if statically-stored objects
are created, unless constructed with `constexpr`.
See https://fuchsia.googlesource.com/zircon/+/master/docs/cxx.md for
reference.
Differential Revision: https://reviews.llvm.org/D41546
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@322310 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Updating fuchsia-overloaded-operator check to not issue warnings for
invalid locations.
Fixes PR35803.
Differential Revision: https://reviews.llvm.org/D41708
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@321762 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Adds a check to the Fuchsia module to warn if an operator is overloaded,
except move and copy operators.
See https://fuchsia.googlesource.com/zircon/+/master/docs/cxx.md for
reference.
Differential Revision: https://reviews.llvm.org/D41363
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@321363 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
Adds a check to the Fuchsia module to warn if classes are defined
with virtual inheritance.
See https://fuchsia.googlesource.com/zircon/+/master/docs/cxx.md for
reference.
Differential Revision: https://reviews.llvm.org/D40813
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@320841 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@319812 91177308-0d34-0410-b5e6-96231b3b80d8
|
|
with the Fuschia and Zircon coding guidelines (https://fuchsia.googlesource.com/zircon/+/master/docs/cxx.md). Adds the first of such checkers, which detects when default arguments are declared in a function declaration or when default arguments are used at call sites.
Patch by Julie Hockett
git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@319225 91177308-0d34-0410-b5e6-96231b3b80d8
|