Carlos =?utf-8?q?Gálvez?= <carlos.gal...@zenseact.com>, Carlos =?utf-8?q?Gálvez?= <carlos.gal...@zenseact.com> Message-ID: In-Reply-To: <llvm.org/llvm/llvm-project/pull/128...@github.com>
carlosgalvezp wrote: Thank you for the discussion, I'm okey with changing the code based on the feedback, namely: - move the logic to ASTMatchers, off by default - clang tidy actively enables it - document technical debt with FIXME/GH ticket > Properly update the documentation and changelog I did that in the release notes. It's there anything else you believe is missing? I will update the AST documentation accordingly as well. https://github.com/llvm/llvm-project/pull/128150 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
