[PATCH] D75087: [ORC] Add LookupKind::TransitiveStatic for linker-resolved transitive dependencies

[Lang Hames via Phabricator via llvm-commits]

lhames added a comment.

So the aim here is to enable generation of lazy stubs without having to materialize any IR, right?

I think the right way to do this: They’re supposed to have just enough information to decide whether the materialization unit being added should be part of the JIT’d program or not. They’re not intended for making smart decisions about how to build the materialization units.

I think a better approach would be to define a ModuleSummaryMaterializationUnit (or something similar) and have that be the thing that you add to your JIT layer. The layer would build stubs based on the interface defined in the module summary and defer compilation of the IR.

Let me know if I’m missing problems with that approach though: I’m afraid I haven’t had time to think about this deeply.

btw — I’m on #llvm <https://reviews.llvm.org/tag/llvm/> on discord a lot these days. This might be a handy one to chat about in real time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75087