[PATCH] D129721: [JITLink][COFF] Handle out-of-order COMDAT second symbol.

Sunho Kim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 13 19:42:44 PDT 2022


sunho created this revision.
sunho added reviewers: lhames, sgraenitz, v.g.vassilev.
Herald added subscribers: StephenFan, hiraditya.
Herald added a project: All.
sunho requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Handle out-of-order COMDAT second symbols. In llvm codegen, the second symbol of COMDAT sequence always follows the first symbol in the global symbol list. But, when the object file came from MSVC compiler, these can come in out of order.


https://reviews.llvm.org/D129721

Files:
  llvm/lib/ExecutionEngine/JITLink/COFFLinkGraphBuilder.cpp
  llvm/lib/ExecutionEngine/JITLink/COFFLinkGraphBuilder.h
  llvm/test/ExecutionEngine/JITLink/X86/COFF_comdat_intervene.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D129721.444496.patch
Type: text/x-patch
Size: 6896 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220714/9c05773f/attachment.bin>


More information about the llvm-commits mailing list