[PATCH] D158909: [JITLink] Honor shouldAddDefaultPasses in x86-64 ELF

Lang Hames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 27 13:58:10 PDT 2023


lhames added a subscriber: rafaelauler.
lhames added a comment.

Hi @rafaelauler,

Sounds like a good idea to me, but we shouldn't need to add a new argument for it. I've modified the `JITLinker` base class to expose this info with a method in commit 265d6bda95b <https://reviews.llvm.org/rG265d6bda95b1dc84fb37ac8a5f18d2b283a7aa3f> (with a follow-up in 35c9072c908 <https://reviews.llvm.org/rG35c9072c908fac5f10704f1b3912011c67be549c> to fix a bug in the initial commit).

Please let me know if that works for you?

Will you be at the Dev Meeting? I'd love to chat some more about how bolt's using JITLink.

- Lang.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158909



More information about the llvm-commits mailing list