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

Lang Hames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 13 15:26:23 PDT 2023


lhames added a comment.

> No, very unlikely that I'll be there, not planned at the moment.

Ok. We'll have to stick to forums / reviews for now then.

Regarding the edge design: I think the aim for the medium term should be to define whether there can be multiple edges at one location or not, and if so what the expected behavior is. The ultimate goal is to allow plugins to effectively modify fixups, so we'll need either (1) some sort of pattern matching to deal with multiple edges, or (2) a more elaborate edge type and more edge kinds.

For now we can kick the can down the road a bit by taking the `AddFDEToCIEEdges` approach. Did you get a chance to look at that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157802



More information about the llvm-commits mailing list