DmitryPolukhin added a comment. In D154602#4483600 <https://reviews.llvm.org/D154602#4483600>, @MaskRay wrote:
> Thanks for the patch! But I share the concern that the test is added to a > wrong layer > (https://maskray.me/blog/2021-08-08-toolchain-testing#the-test-checks-at-the-wrong-layer). > > We can add a `clang::createInvocation` test to > `clang/unittests/Driver/ToolChainTest.cpp`. Thank you for the suggestion. Done, please take another look. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154602/new/ https://reviews.llvm.org/D154602 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
