[PATCH] D33324: [index] Avoid infinite recursion when looking up a dependent name

2017-05-18 Thread Alex Lorenz via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. arphaman marked an inline comment as done. Closed by commit rL303366: [index] Avoid one more crash caused by infinite recursion that happens when (authored by arphaman). Changed prior to commit: https://reviews.llvm.org/D

[PATCH] D33324: [index] Avoid infinite recursion when looking up a dependent name

2017-05-18 Thread Ben Langmuir via Phabricator via cfe-commits
benlangmuir accepted this revision. benlangmuir added a comment. This revision is now accepted and ready to land. LGTM. One stylistic comment, but I'll leave that up to you. Comment at: lib/AST/CXXInheritance.cpp:286 +BaseRecord = nullptr; + else if (Visi

[PATCH] D33324: [index] Avoid infinite recursion when looking up a dependent name

2017-05-18 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman created this revision. This is a regression introduced by r302632 . Repository: rL LLVM https://reviews.llvm.org/D33324 Files: include/clang/AST/CXXInheritance.h lib/AST/CXXInheritance.cpp test/Index/Core/index-dependent-source.cpp Index: test/Index/Core/index-dependent-sour