[PATCH] D100657: [Support] Don't include <algorithm> in Hashing.h

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 16 09:05:11 PDT 2021


thakis created this revision.
thakis added a reviewer: hans.
Herald added a subscriber: dexonsmith.
thakis requested review of this revision.
Herald added a project: LLVM.

The include is for std::swap(), but that's in <utility> in C++11,
and Hashing.h already includes that.


https://reviews.llvm.org/D100657

Files:
  llvm/include/llvm/ADT/Hashing.h


Index: llvm/include/llvm/ADT/Hashing.h
===================================================================
--- llvm/include/llvm/ADT/Hashing.h
+++ llvm/include/llvm/ADT/Hashing.h
@@ -48,7 +48,6 @@
 #include "llvm/Support/ErrorHandling.h"
 #include "llvm/Support/SwapByteOrder.h"
 #include "llvm/Support/type_traits.h"
-#include <algorithm>
 #include <cassert>
 #include <cstring>
 #include <string>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D100657.338143.patch
Type: text/x-patch
Size: 399 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210416/44072886/attachment.bin>


More information about the llvm-commits mailing list