klimek added a reviewer: sammccall. klimek added a comment. +Sam for additional sanity checking.
================ Comment at: clang/lib/ASTMatchers/ASTMatchFinder.cpp:902 // Maps (matcher, node) -> the match result for memoization. - typedef std::map<MatchKey, MemoizedMatchResult> MemoizationMap; + typedef std::map<MatchKey, MemoizedMatchResult, std::less<>> MemoizationMap; MemoizationMap ResultCache; ---------------- Ok, if this actually matters, we should also not use a std::map here, but a llvm::DenseMap (or do we rely on iteration invalidation semantics here?). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80202/new/ https://reviews.llvm.org/D80202 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
