Ahh, sorry, I thought you meant the other one :( Apologies for confusion On Tue, Oct 5, 2021 at 9:58 AM Sam McCall via Phabricator < revi...@reviews.llvm.org> wrote:
> 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
