[PATCH] D41926: [DWARFv5] CodeGen support for MD5 checksum
Adrian Prantl via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 10 17:16:12 PST 2018
aprantl added a comment.
This was not immediately obvious to me: Why can we now have empty DIFiles?
LGTM otherwise.
================
Comment at: llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp:277
+ memcpy(CKMem, Checksum.data(), Checksum.size());
+ MD5Checksum = reinterpret_cast<MD5::MD5Result *>(CKMem);
+ }
----------------
do we need error handling here or is it okay to silently drop malformed input?
https://reviews.llvm.org/D41926
More information about the llvm-commits
mailing list