[PATCH] D108388: [Support] Move `MD5` members in `InternalState`.

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

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

LGTM, with a minor nit.



================
Comment at: llvm/include/llvm/Support/MD5.h:89-103
+  // Any 32-bit or wider unsigned integer data type will do.
+  typedef uint32_t MD5_u32plus;
+
+  // Internal State
+  struct {
+    MD5_u32plus a = 0x67452301;
+    MD5_u32plus b = 0xefcdab89;
----------------
I suggest leaving the location in the file these in the file above. That'll remove the diff entirely on the `typedef`, and make much of the diff for the members whitespace-only. Unless there's a specific reason you were moving it?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108388/new/

https://reviews.llvm.org/D108388