[Lldb-commits] [PATCH] D15312: Fix scope-based lookup when more than one function is found.

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Thu Dec 10 09:29:32 PST 2015

clayborg added a subscriber: clayborg.
clayborg added a comment.

I like the clang specific patch here better. Mainly because if you add a function like this to TypeSystem.h:

  virtual uint32_t
  DeclContextCountDeclLevels (void *opaque_decl_ctx,
                              void *opaque_find_decl_ctx,
                              ConstString *find_name = nullptr,
                              CompilerType *find_type = nullptr) = 0;

It means that you will have a function in the CompilerDeclContext named "CountDeclLevels(...)". The "DeclContext" prefix is the convention in TypeSystem.h that says "I will add a corresponding function to CompilerDeclContext that will call through to this function.

I still like the clang specific version better because I still think it doesn't make a sensible API for CompilerDeclContext.




More information about the lldb-commits mailing list