https://github.com/efriedma-quic commented: LGTM... but it would be nice to get a second opinion on whether `var->getDeclContext()->isDependentContext()` is the right way to check this. https://github.com/llvm/llvm-project/pull/161590