https://github.com/carlosgalvezp approved this pull request.
I think this is the correct thing to do. It's a programming error if these pointers are null, and so assertions are the correct tool for this. Other checks do `if (const auto* Found = ...)`, but that's not correct, since it silences programming errors. https://github.com/llvm/llvm-project/pull/143145 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits