[PATCH] D31105: [Support] Add a function to MD5 a file's contents.

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 20 16:11:32 PDT 2017


mehdi_amini accepted this revision.
mehdi_amini added a comment.
This revision is now accepted and ready to land.

LGTM. See one comment inline.

(After reading more: this is not really the MD5 interface but the "result" object only.)



================
Comment at: llvm/unittests/Support/MD5Test.cpp:70
 }
 }
----------------
I feel we should have a test for with low() and high() here.


https://reviews.llvm.org/D31105





More information about the llvm-commits mailing list