[PATCH] D128968: [JITLink][COFF] Initial COFF support.

Sunho Kim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 04:11:04 PDT 2022


sunho added inline comments.


================
Comment at: llvm/lib/ExecutionEngine/JITLink/COFFLinkGraphBuilder.cpp:339
+      LLVM_DEBUG({
+        if (Offset + Symbol->getSize() > LastDifferentOffset)
+          dbgs() << "  Overlapping symbol range generated for the following "
----------------
chapuni wrote:
> LastDifferentOffset is not used with -Asserts.
@chapuni Hi, I'll fix this shortly. Thanks for letting me know.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128968



More information about the llvm-commits mailing list