[clang-tools-extra] [clang-tidy] Add modernize-use-std-tie check (PR #191218)

via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 10 00:54:19 PDT 2026


lucasvallejoo wrote:

Hi @unterumarmung,

Thank you so much for taking the time to write such a detailed and insightful review. You are completely right, my initial prototype relied too heavily on textual heuristics (token slicing) and is too fragile for the standard expected in clang-tidy.

I really appreciate the explanation on how this should be properly designed. I am going to step back, study Clang's AST matchers more deeply, and rewrite this to be 100% AST-driven so it can safely handle edge cases, macros, side effects, and complex syntax variations.

It will take me some time to learn the internals properly and expand both the positive and negative test coverage as you suggested, but I am on it. I'll update the PR once the new AST-based architecture is solid. Thanks again for pointing me in the right direction!

https://github.com/llvm/llvm-project/pull/191218


More information about the cfe-commits mailing list