dexonsmith added a comment.

In D109632#3013523 <https://reviews.llvm.org/D109632#3013523>, @vsapsai wrote:

> 2. Serialize only methods owned by the current module (and change 
> `ReadMethodPoolVisitor` appropriately).

Would that require visiting all in-memory modules every time there's a global 
method pool lookup?

If so, that sounds expensive... and similar to the problem that had to be 
solved to make identifier lookup fast. Maybe the same approach can/should be 
used here?

Or if not, can you clarify for me?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109632/new/

https://reviews.llvm.org/D109632

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to