[clang-tools-extra] [clang-tidy] [NFC] Potential dereference of nullptr. (PR #143145)

[Baranov Victor via cfe-commits]

vbvictor wrote:

I think we generally discussed such changes in https://github.com/llvm/llvm-project/pull/138283,
CC @nicovank.

Maybe we need to come to consensus in clang-tidy whether we should write `assert()` after `Result.Nodes.getNodeAs<>`. (There are many-many places that has `assert` and that doesn't have `assert`)

Personally, I don't have a strong opinion on such changes, so from LGTM without explicit approval, please wait for a second opinion. 


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