| Use the range-based overload of llvm::sort where possible Reviewed By: MaskRay Differential Revision: https://reviews.llvm.org/D130403 | 3 years ago |
| Use drop_begin (NFC) | 3 years ago |
| [clang-doc] Add check for pointer validity clang-doc would SEGV when running over the Fuchsia code base. This patch adds a check to avoid dereferencing potentially null pointers in the Values vector. These pointers were either never valid or had been invalidated when the underlying pointer in std::unique_ptr was moved from, hence making it nullptr within the vector. Reviewed By: phosek Differential Revision: https://reviews.llvm.org/D130279 | 3 years ago |
| Use llvm::sort instead of std::sort where possible llvm::sort is beneficial even when we use the iterator-based overload, since it can optionally shuffle the elements (to detect non-determinism). However llvm::sort is not usable everywhere, for example, in compiler-rt. Reviewed By: nhaehnle Differential Revision: https://reviews.llvm.org/D130406 | 3 years ago |
| [clang][lex] NFCI: Use FileEntryRef in PPCallbacks::InclusionDirective() This patch changes type of the File parameter in PPCallbacks::InclusionDirective() from const FileEntry * to Optional<FileEntryRef>. With the API change in place, this patch then removes some uses of the deprecated FileEntry::getName() (e.g. in DependencyGraph.cpp and ModuleDependencyCollector.cpp). Reviewed By: dexonsmith, bnbarham Differential Revision: https://reviews.llvm.org/D123574 | 4 years ago |
| [clang-query] Add check to prevent setting srcloc when no introspection is available. Checks if introspection support is available set output kind parser. If it isn't present the auto complete will not suggest srcloc and an error query will be reported if a user tries to access it. Reviewed By: steveire Differential Revision: https://reviews.llvm.org/D101365 | 5 years ago |
| Use llvm::sort instead of std::sort where possible llvm::sort is beneficial even when we use the iterator-based overload, since it can optionally shuffle the elements (to detect non-determinism). However llvm::sort is not usable everywhere, for example, in compiler-rt. Reviewed By: nhaehnle Differential Revision: https://reviews.llvm.org/D130406 | 3 years ago |
| [clang-tidy] Fix a false positive in readability-simplify-boolean-expr Reviewed By: LegalizeAdulthood Differential Revision: https://reviews.llvm.org/D134590 (cherry picked from commit 8c783b8ec78ec857e446a89a35463baed8026f40) | 3 years ago |
| [clangd] Return earlier when snippet is empty Fixes github.com/clangd/clangd/issues/1216 If the Snippet string is empty, Snippet.front() would trigger a crash. Move the Snippet->empty() check up a few lines to avoid this. Should not break any existing behavior. Differential Revision: https://reviews.llvm.org/D134137 (cherry picked from commit 60528c690a4c334d2a3a2c22eb97af9e67d7a91d) | 3 years ago |
| [docs] improve documentation for misc-const-correctness Improve the documentation for 'misc-const-correctness' to: - include better examples - improve the english - fix links to other checks that were broken due to the directory-layout changes - mention the limitation that the check does not run on C code. Addresses #56749, #56958 Reviewed By: njames93 Differential Revision: https://reviews.llvm.org/D132244 (cherry picked from commit b5b750346346bfe95c7c6b1cceacc6cfccc8f4f4) | 3 years ago |
| [include-cleaner] Fix build error in unit test Reviewed By: nridge Differential Revision: https://reviews.llvm.org/D127217 | 4 years ago |
| Remove unneeded cl::ZeroOrMore for cl::opt/cl::list options | 4 years ago |
| [pp-trace] Print HashLoc in InclusionDirective callback The HashLoc in InclusionDirective callback is an unused parameter. Since pp-trace is also used as a test of Clang’s PPCallbacks interface, add it to the output of pp-trace could avoid some unintended change on it. This shuold resolves PR52673 Reviewed By: aaron.ballman Differential Revision: https://reviews.llvm.org/D125373 | 4 years ago |
| [pseudo] Allow opaque nodes to represent terminals This allows incomplete code such as namespace foo { to be modeled as a normal sequence with the missing } represented by an empty opaque node. Differential Revision: https://reviews.llvm.org/D130551 | 3 years ago |
| [clang-tidy] Fix a false positive in readability-simplify-boolean-expr Reviewed By: LegalizeAdulthood Differential Revision: https://reviews.llvm.org/D134590 (cherry picked from commit 8c783b8ec78ec857e446a89a35463baed8026f40) | 3 years ago |
| [OpenMP] "UnFix" layering problem with FrontendOpenMP This reverts commit 97aa593a8387586095b7eac12974ba2fdd08f4c3 as it causes problems (PR45453) https://reviews.llvm.org/D77574#1966321. This additionally adds an explicit reference to FrontendOpenMP to clang-tidy where ASTMatchers is used. This is hopefully just a temporary solution. The dependence on FrontendOpenMP from ASTMatchers should be handled by CMake implicitly, not us explicitly. Reviewed By: aheejin Differential Revision: https://reviews.llvm.org/D77666 | 6 years ago |
| [clang-tidy] Remove unnecessary code from ReadabilityModuleTest D56303 added testing code that was then made redundant by the changes in D125026. However this code wasn't completely removed in the latter patch. Reviewed By: aaron.ballman Differential Revision: https://reviews.llvm.org/D130026 | 3 years ago |
| .gitignore: ignore docs/_build That is where the documentation gets built to. I'm pretty much clueless about SVN; I would appreciate if someone more knowledgeable about SVN made the equivalent change. llvm-svn: 176417 | 13 years ago |
| Generalize "check-all" umbrella targets, use for check-clang-tools The mechanism behind "check-all" is recording params of add_lit_testsuite() calls in global variables LLVM_LIT_*, and then creating an extra suite with their union at the end. This avoids composing the check-* targets directly, which doesn't work well. We generalize this by allowing multiple families of variables LLVM_{name}_LIT_*: umbrella_lit_testsuite_begin(check-foo) ... test suites here will be added to LLVM_FOO_LIT_* variables ... umbrella_lit_testsuite_end(check-foo) (This also moves some implementation muck out of {llvm,clang}/CMakeLists.txt This patch also changes check-clang-tools to use be an umbrella test target, which means the clangd and clang-pseudo tests are included in it, along with the the other testsuites that already are (like check-clang-extra-clang-tidy). Differential Revision: https://reviews.llvm.org/D121838 | 4 years ago |
| [clangd] add CODE_OWNERS Reviewers: klimek Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, kadircet, usaxena95, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D73537 | 6 years ago |
| Fix typos throughout the license files that somehow I and my reviewers all missed! Thanks to Alex Bradbury for pointing this out, and the fact that I never added the intended legacy anchor to the developer policy. Add that anchor too. With hope, this will cause the links to all resolve successfully. llvm-svn: 351731 | 7 years ago |
| [NFC] update clang-tools-extra README.txt | 4 years ago |