[PATCH] D157802: [JITLink][EHFrameSupport] Accept multiple relocations

[Jonas Hahnfeld via Phabricator via llvm-commits]

Hahnfeld added a comment.

I'm not proposing a general solution, just looking at the one case we need in `eh_frame` to get the PC offset from the start of the `.text` section (as far as I understand). There `ADD32` and `SUB32` are at exactly the same offset (that's the condition that currently makes `EHFrameEdgeFixer` fail) with the `SUB32` pointing to the start of the block with a zero addend. For everything else, I agree with your previous statement that storing two arbitrary symbols is a problem, and I don't think we want to go into splitting blocks to fuse edges.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157802