[PATCH] D111863: [libunwind] Add an interface for dynamic .eh_frame registration

Lang Hames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 18 16:51:22 PDT 2021


lhames added a comment.

In D111863#3071353 <https://reviews.llvm.org/D111863#3071353>, @joerg wrote:

> I would strongly prefer if ORCv2 doesn't depend on this. It essentially forces libunwind to parse the whole section just to find the delimiters of the FDEs. That's a lot of unnecessary work as JIT use normally allows registering functions individually.

I don't follow this. Does libunwind provide some way to register FDEs without parsing the FDE content? If so we can definitely use that, but we should still process the whole section: ORC links objects (not functions), and we should register every FDE for an object when it's linked in.

It's also worth noting that ORC and MCJIT have always called `__register_frame` on every frame, which seems like it should be at least as much work as this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111863



More information about the llvm-commits mailing list