[PATCH] D105465: [RuntimeDyld] Added support for relocation of indirect functions

Fri Jul 15 00:54:23 PDT 2022

MoritzS added a comment.

In D105465#3653761 <https://reviews.llvm.org/D105465#3653761>, @lhames wrote:

> Does the `Indirect` flag really need to go onto `JITSymbolFlags`? My mental model of ifuncs (very limited -- I haven't used them myself) is "self-updating PLT" -- its ifunc-ness is opaque to callers right?

Yes, you are right, ifuncs are completely opaque to callers, so we probably shouldn't add that flag. I will change that.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105465