ioeric added inline comments.

================
Comment at: lib/Tooling/Core/Lookup.cpp:139
+    const NamespaceDecl *NS = *I;
+    auto LookupRes = NS->lookup(DeclarationName(&AST.Idents.get(Head)));
+    if (!LookupRes.empty()) {
----------------
ilya-biryukov wrote:
> This will not take using namespaces into account, right? Do we care about 
> those or is this a known limitation of the tools?
Right, I think we don't handle using namespaces in this library due to 
limitation/importance etc. Added a FIXME.


Repository:
  rC Clang

https://reviews.llvm.org/D50189



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to