clayborg accepted this revision. clayborg added a comment. This revision is now accepted and ready to land.
Looks good as long as my inlined comment doesn't point out an error, then this is good to go! Good stuff. ================ Comment at: source/Symbol/Variable.cpp:250-258 @@ -233,1 +249,11 @@ + +CompilerDecl +Variable::GetDecl () +{ + Type *type = GetType(); + CompilerDecl decl = type->GetSymbolFile()->GetDeclForUID(GetID()); + if (decl) + decl.GetTypeSystem()->DeclLinkToObject(decl.GetOpaqueDecl(), shared_from_this()); + return decl; +} ---------------- As long as calling DeclLinkToObject(...) more than once with the same args is ok, then this will be fine. http://reviews.llvm.org/D12658 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits