[PATCH] D154198: [JITLink][PowerPC] Add TOC and relocations for ppc64

Kai Luo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 10 01:15:00 PDT 2023


lkail marked an inline comment as done.
lkail added inline comments.


================
Comment at: llvm/include/llvm/ExecutionEngine/JITLink/ppc64.h:254
+    if (K == CallBranchDeltaRestoreTOC) {
+      uint32_t NopInst = support::endian::read32<Endianness>(FixupPtr + 4);
+      assert(NopInst == 0x60000000 &&
----------------
chapuni wrote:
> NopInst is unused in -Asserts
Should be fixed by https://github.com/llvm/llvm-project/commit/79786c4d23f1fd7af438e4fd4e33ec109626bee4.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154198



More information about the llvm-commits mailing list