================ @@ -697,27 +698,61 @@ static bool isTokenAtLoc(const SourceManager &SM, const LangOptions &LangOpts, return !Invalid && Text == TokenText; } -std::optional<SourceLocation> -getExpansionLocOfMacro(StringRef MacroName, SourceLocation Loc, - const ASTContext &Context) { +namespace { +struct SourceLocationHash { + std::size_t operator()(const SourceLocation &Loc) const { ---------------- AaronBallman wrote:
```suggestion std::size_t operator()(SourceLocation Loc) const { ``` A `SourceLocation` is a 32-bit value, so no need to pass by const reference. The same change can be made elsewhere in the patch. https://github.com/llvm/llvm-project/pull/117143 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits