[PATCH] D97436: [lld-link] Fix addrsig symbols merging in ICF.

Zequan Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 26 16:43:19 PST 2021


zequanwu updated this revision to Diff 326844.
zequanwu added a comment.
Herald added subscribers: steven_wu, hiraditya.

- Add `/OPT:safeicf`, which will mark all sections as unique if `.addrsig` doesn't exist. By default, `/OPT:icf` will do aggressive icf, matching the behavior of MSVC linker's `/OPT:icf`.
- Update test cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97436

Files:
  lld/COFF/Config.h
  lld/COFF/Driver.cpp
  lld/COFF/ICF.cpp
  lld/test/COFF/icf-data.test
  lld/test/COFF/icf-local.test
  lld/test/COFF/icf-pdata.s
  lld/test/COFF/icf-safe.s
  lld/test/COFF/icf-vtables.s
  lld/test/COFF/lto-icf.ll
  lld/test/COFF/string-tail-merge.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97436.326844.patch
Type: text/x-patch
Size: 10834 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210227/5cf08ab9/attachment.bin>


More information about the llvm-commits mailing list