[PATCH] D110925: [clangd] Follow-up on rGdea48079b90d

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 5 00:58:47 PDT 2021


sammccall added inline comments.


================
Comment at: llvm/include/llvm/Support/FileSystem/UniqueID.h:17
 
+#include "llvm/ADT/DenseMap.h"
 #include <cstdint>
----------------
kbobyrev wrote:
> sammccall wrote:
> > you only need DenseMapInfo here
> I think I do need utility because I'm using `std::pair`
I meant this file can include DenseMapInfo.h instead of DenseMap.h. Fixed in 4e91035387faf9e18134b1d46ce0fa803a6e9122


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110925



More information about the cfe-commits mailing list