[PATCH] D103052: Do not patch FDE symbols in RuntimeDyld, on targets that use non-absolute symbol relocations in `.eh_frame`

[Cody Tapscott via Phabricator via llvm-commits]

topolarity added a comment.

In D103052#2846623 <https://reviews.llvm.org/D103052#2846623>, @lhames wrote:

> did you happen to check whether your test cases worked under JITLink?

I checked this out briefly, but I hit a different issue for JITLink.

For some reason the `.eh_frame` section is empty when the EHFrameRegistrar is notified of the emitted object (`EHFrameSupport.cpp:796-800`), so no frames end up getting registered. When I did some debugging, it looked like the `compact_unwind` section is empty at that point too, so something may be preventing debug frames from being generated but I'm not sure what.

I'll try to dig further and see if I can find the root cause, but it sounds like that would be a separate fix, along with the open mentioned above about how to support forcing FDE Frame generation in the JIT, at least until libunwind supports dynamic registration of compact_unwind frames.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103052