erichkeane added inline comments.
================ Comment at: clang/lib/AST/DeclBase.cpp:1222 + DeclContext *DC = this; + while (DC && DC->isTransparentContext()) + DC = DC->getParent(); ---------------- cor3ntin wrote: > Can getParent() be null? If it cam, then this function can return null which > you don't check when you call that function. Or it can't and your test is not > doing anything. I don't think it can ever be null, a DC of transparent-context type is always going to have at least a TranslationUnit as a parent. I might think an assertion here to replace the `DC &&` part might be worth it. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108403/new/ https://reviews.llvm.org/D108403 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits