[llvm-commits] [PATCH] RFC: fix some problems in GeneralHash

Duncan Sands baldrick at free.fr
Wed Feb 22 07:53:17 PST 2012


Hi Jay,

On 22/02/12 16:50, Jay Foad wrote:
> Here's an updated patch. I've rewritten the contentious comment, and
> re-inlined finish() and mix().

> --- include/llvm/ADT/Hashing.h	(revision 151155)
> +++ include/llvm/ADT/Hashing.h	(working copy)
> @@ -14,6 +14,7 @@
>  #ifndef LLVM_ADT_HASHING_H
>  #define LLVM_ADT_HASHING_H
>
> +#include <cstring>
>  #include "llvm/ADT/ArrayRef.h"
>  #include "llvm/ADT/StringRef.h"

usually system headers are listed last (style).

Ciao, Duncan.



More information about the llvm-commits mailing list