[Lldb-commits] [PATCH] D56229: [PECOFF] Implementation of ObjectFilePECOFF:: GetUUID()
Hui Huang via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Apr 17 07:50:06 PDT 2019
Hui added inline comments.
Comment at: source/Plugins/ObjectFile/PECOFF/ObjectFilePECOFF.cpp:891-892
+ if (!CreateBinary())
+ return UUID();
+ auto COFFObj =
> I don't think this is necessary as `CreateInstance` will refuse to return the ObjectFile instance if the creation of the coff binary object failed. (You could theoretically assert that the binary is really there if you want extra security).
There is no cached binary for memory instance (by CreateMemoryInstance). Is there any chance that any JIT-ed codes will call module or UUID related API?
CHANGES SINCE LAST ACTION
More information about the lldb-commits