[clang-tools-extra] [clang-tidy] Prefer the faster LLVM ADT sets and maps over `std::` ones (PR #174357)

[Carlos Galvez via cfe-commits]

================
@@ -10,14 +10,11 @@
 #define LLVM_CLANG_TOOLS_EXTRA_CLANG_TIDY_MISC_NEWDELETEOVERLOADSCHECK_H
 
 #include "../ClangTidyCheck.h"
-#include "llvm/ADT/SmallVector.h"
-#include <map>
 
 namespace clang::tidy::misc {
 
 class NewDeleteOverloadsCheck : public ClangTidyCheck {
-  std::map<const clang::CXXRecordDecl *,
-           llvm::SmallVector<const clang::FunctionDecl *, 4>>
+  llvm::DenseMap<const CXXRecordDecl *, SmallVector<const FunctionDecl *, 4>>
----------------
carlosgalvezp wrote:

Nit: inconsistent usage of `llvm` namespace, e.g. `llvm::DenseMap` vs `SmallVector`.

https://github.com/llvm/llvm-project/pull/174357