Szelethus added inline comments.
================ Comment at: clang/include/clang/Analysis/MacroExpansionContext.h:105 + using MacroExpansionText = SmallString<40>; + using ExpansionMap = llvm::DenseMap<SourceLocation, MacroExpansionText>; + using ExpansionRangeMap = llvm::DenseMap<SourceLocation, SourceLocation>; ---------------- Hmm, I'm by no means an expert, but isn't a string-like structure a bit big for a `DenseMap`? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93222/new/ https://reviews.llvm.org/D93222 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
