This revision was automatically updated to reflect the committed changes.
Closed by commit rGe6020b2a4224: [clang][NFC] refactor GlobalMethodPool to
encapsulate its map (authored by rmaz, committed by drodriguez).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.
rmaz updated this revision to Diff 372999.
rmaz added a comment.
- keep `std::pair`, but move to
`GlobalMethodPool::Lists`
- add `const`
- `val` -> `&&Val`
- move comment
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109898/new/
https://reviews.ll
dexonsmith accepted this revision.
dexonsmith added a comment.
This revision is now accepted and ready to land.
Thanks for splitting this out -- left some comments inline, LGTM after that.
Comment at: clang/include/clang/Sema/Sema.h:1422-1427
/// Method Pool - allows efficie
rmaz created this revision.
rmaz added reviewers: dexonsmith, manmanren, vsapsai.
rmaz requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This refactor changes the GlobalMethodPool to a class that contains
the DenseMap of methods. This is to a
