[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.

I also wanted to mention that there's one more barrier to getting stack unwinding working on AArch64 for JIT'd functions:

We need to force the emission of the EH Frame section (currently done via `SupportsCompactUnwindWithoutEHFrame` in the `MCObjectFileInfo` for the target), since libunwind doesn't yet support dynamically registering compact unwind information at run-time. For now I've been doing this with a simple patch to `MCObjectFileInfo.cpp`, commenting out the logic that enables the flag on AArch64. However, for typical JIT users, it'd probably be preferable to configure this correctly at run-time, without the compile-time patching.

Is there precedent for updating these kinds of flags/settings during the JIT setup? If so I'm happy to include that in this change, as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103052