[PATCH] D16325: Add support for computing SHA1 in LLVM

[Rui Ueyama via llvm-commits]

ruiu added a subscriber: ruiu.

================
Comment at: include/llvm/Support/SHA1.h:34
@@ +33,3 @@
+  /// Digest more data.
+  void update(const char *data, size_t len);
+
----------------
Why don't you pass ArrayRef<uint8_t>?

================
Comment at: include/llvm/Support/SHA1.h:41-45
@@ +40,7 @@
+
+  /// Return a reference to the current SHA1 for the digested data since the
+  /// last call to init(). This is suitable for getting the SHA1 at any time
+  /// without invalidating the internal state so that more calls can be made
+  /// into update.
+  StringRef result();
+
----------------
It is not clear in this comment whether this function returns a hex string or raw bytes.


http://reviews.llvm.org/D16325