[Lldb-commits] [PATCH] D55142: Minidump debugging using the native PDB reader
Leonard Mosescu via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Dec 7 14:52:44 PST 2018
lemo added inline comments.
Comment at: source/Plugins/SymbolFile/NativePDB/SymbolFileNativePDB.cpp:171-174
+ // TODO: we need to compare the age, in addition to the GUID
if (expected_info->getGuid() != guid)
> Mainly out of curiosity, what's the complication here? The llvm interface does not provide the means to retrieve the age?
Yes, the interface doesn't expose the age, although that's not not really the problem (the age is a detail - part of the generic "Debug ID")
We just need to make the handling of PE/COFF & VC UUIDs more consistent: right now most places expect a stripped version (just the GUID). This is wrong since a correct PDB match should take the age into account, but it's outside the scope of this change.
CHANGES SINCE LAST ACTION
More information about the lldb-commits