ioeric added inline comments.
================ Comment at: unittests/Tooling/LookupTest.cpp:215 + + // Potentially ambiguous symbols that are not visible at reference location + // are not considered ambiguous. ---------------- hokein wrote: > The test seems hard to understand what it actually does, could you explain > it? which symbol is ambiguous here? What's the behavior of the test before vs > after the fix? added comment. PTAL Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60257/new/ https://reviews.llvm.org/D60257 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
