https://github.com/igorkudrin closed
https://github.com/llvm/llvm-project/pull/90370
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/90370
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
This simplification seems desired. https://reviews.llvm.org/D00#3302749 had
a `LLVM_INSTALL_TOOLCHAIN_ONLY` condition, which seems unneeded.
https://github.com/llvm/llvm-project/pull/90370
___
cfe-commits mailing list
igorkudrin wrote:
> I think out of tree builds of clang-tidy (back in the svn days, when people
> did partial checkouts of individual projects) probably needed this to be able
> to correctly correctly find the right headers. Maybe standalone builds aren't
> permitted anymore? I think I
vtjnash wrote:
I think out of tree builds of clang-tidy (back in the svn days, when people did
partial checkouts of individual projects) probably needed this to be able to
correctly correctly find the right headers. Maybe standalone builds aren't
permitted anymore? I think I mentioned this a
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Igor Kudrin (igorkudrin)
Changes
The only reason for the removed condition was that there was a dependency for
`CTTestTidyModule` on the `clang-tidy-headers` target, which was only created
under the same `NOT
https://github.com/igorkudrin created
https://github.com/llvm/llvm-project/pull/90370
The only reason for the removed condition was that there was a dependency for
`CTTestTidyModule` on the `clang-tidy-headers` target, which was only created
under the same `NOT LLVM_INSTALL_TOOLCHAIN_ONLY`
