jimingham wrote:

> It's not clear to me whether `TypeSystem::DeclContextIsContainedInLookup` 
> provides such a guarantee. Implementation suggests that it's only interested 
> in direct `DeclContext`, maybe ignoring transparent decl contexts like inline 
> namespace it already handles. I hope reviewers can clarify this.

It might be good to ask this as a separate question on the lldb project 
Discourse (https://discourse.llvm.org/c/subprojects/lldb/8).  That's likely to 
get more notice than a comment deep in a patch review.

https://github.com/llvm/llvm-project/pull/68705
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to