futogergely marked 7 inline comments as done. futogergely added a comment. In D91000#3506605 <https://reviews.llvm.org/D91000#3506605>, @whisperity wrote:
> Just one question if you could try this out for me: what happens if you run > `clang-tidy a.c b.c` (two TUs in the invocation) where **one of them** > (preferably the later one, i.e. **`b.c`**) does //NOT// have Annex K enabled? > I believe the cached `IsAnnexKAvailable` (like any other TU-specific state of > the check instance) should be invalidated/cleared in an overridden `void > onStartTranslationUnit()` function. > > Also, what happens if the check is run for C++ code? It is working as is, a new ClangTidyCheck is created for every translation unit. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91000/new/ https://reviews.llvm.org/D91000 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
