[unused-includes][ADT] Don't include DJB.h as it's not used. NFC.

[vsapsai]

StringMap uses xxHash instead of DJB.

[llvmbot]

@llvm/pr-subscribers-llvm-adt

Author: Volodymyr Sapsai (vsapsai)

Changes

StringMap uses xxHash instead of DJB.

---

Full diff: https://github.com/llvm/llvm-project/pull/88211.diff

1 Files Affected:

• (modified) llvm/include/llvm/ADT/StringMap.h (-1)

diff --git a/llvm/include/llvm/ADT/StringMap.h b/llvm/include/llvm/ADT/StringMap.h
index 32e611467ac58e..453d91349e3585 100644
--- a/llvm/include/llvm/ADT/StringMap.h
+++ b/llvm/include/llvm/ADT/StringMap.h
@@ -17,7 +17,6 @@
 #include "llvm/ADT/StringMapEntry.h"
 #include "llvm/ADT/iterator.h"
 #include "llvm/Support/AllocatorBase.h"
-#include "llvm/Support/DJB.h"
 #include "llvm/Support/PointerLikeTypeTraits.h"
 #include <initializer_list>
 #include <iterator>

[vsapsai]

DJB.h header include was added in 9a4a4c3

[vsapsai]

Buildkite reports

Total Discovered Tests: 36933
  Skipped          :    30 (0.08%)
  Unsupported      :   243 (0.66%)
  Passed           : 36624 (99.16%)
  Expectedly Failed:    36 (0.10%)

but still decided the build has failed.

Don't know how to re-trigger the tests.

[dwblaikie]

Thanks for the catch! Must've been a holdover from the original version of the patch.

[vsapsai]

Thanks for the catch! Must've been a holdover from the original version of the patch.

Thanks for taking a look that's not an upcoming use of DJB hashing.