labath marked an inline comment as done.
labath added inline comments.

================
Comment at: source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.h:294
+
+  lldb_private::UUID m_uuid;
 };
----------------
asmith wrote:
> labath wrote:
> > Move UUID handling into a separate patch.
> The UUID change is being discussed here https://reviews.llvm.org/D56229
Ah, I completely forgot about that patch. Thanks for reminding me. This is 
actually very good timing because I was just diving into the topic of UUID 
matching in lldb on windows (in order to match up minidumps with respective 
exe/pdb files). I'll elaborate more there...


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56233/new/

https://reviews.llvm.org/D56233



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

Reply via email to