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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
