jingham added a comment.

This seems good to me, but Greg is more expert than I so I'd wait on his okay.



================
Comment at: source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp:1901-1907
+        dwarf->GetTypeList()->Insert(type_sp);
+        // Cache the type if it isn't context-specific.
+        auto &cache = dwarf->GetDIEToType();
+        if (exe_ctx.GetFrameSP())
+          cache.erase(die.GetDIE());
+        else
+          cache[die.GetDIE()] = type_sp.get();
----------------
You have to do this twice, maybe there should be a method to add the results to 
the DIE to Type map?


https://reviews.llvm.org/D53530



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

Reply via email to