[PATCH] D127558: [JITLink][AArch64] Unify table managers of ELF and MachO.

Sunho Kim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 15 22:17:51 PDT 2022


sunho added inline comments.


================
Comment at: llvm/include/llvm/ExecutionEngine/JITLink/aarch64.h:79
+      KindToSet = aarch64::PageOffset12;
+      uint32_t RawInstr = *(const support::ulittle32_t *)FixupPtr;
+      assert(E.getAddend() == 0 &&
----------------
chapuni wrote:
> This is unused with -Asserts
Thanks for letting me know. I'll fix it shortly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127558



More information about the llvm-commits mailing list