[PATCH] D129556: [lld-macho] Fix loading same libraries from both LC_LINKER_OPTION and command line

Kaining Zhong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 13 01:37:28 PDT 2022


PRESIDENT810 updated this revision to Diff 444185.
PRESIDENT810 added a comment.

I just realized that -force_load should not work only if the same library is already explicitly passed from command line, but not when it is previously passed from LC_LINKER_OPTION. I revised how I determine if -force_load should work by saving libraries shown in command line in a DenseSet.


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

https://reviews.llvm.org/D129556

Files:
  lld/MachO/Driver.cpp
  lld/MachO/InputFiles.h
  lld/test/MachO/lc-linker-option.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D129556.444185.patch
Type: text/x-patch
Size: 6647 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220713/c71da499/attachment-0001.bin>


More information about the llvm-commits mailing list