[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