[Lldb-commits] [PATCH] D12658: Search variables based on clang::DeclContext and clang::Decl tree

[Paul Herman via lldb-commits]

paulherman added a comment.

I thought about the problem a bit more and I believe that having the call to DeclLinkDeclToObject inside ParseVariableDIE makes sense since this gets called only when the variable will actually be used in a search together with its context. Also, since ParseVariableDIE is lazy, this makes the linking to be lazy.

I don't really see how one could do this inside GetClangDeclForDIE or inside Variable::GetDecl as there is no access to the VariableSP in there.


http://reviews.llvm.org/D12658