[all-commits] [llvm/llvm-project] 6d6644: [LLD] [MinGW] Hook up --icf=safe to -opt:safeicf (...

Martin Storsjö via All-commits all-commits at lists.llvm.org
Wed Oct 25 04:38:45 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6d66440c50e3047b5ce152830d4ccae381c7c2bf
      https://github.com/llvm/llvm-project/commit/6d66440c50e3047b5ce152830d4ccae381c7c2bf
  Author: Martin Storsjö <martin at martin.st>
  Date:   2023-10-25 (Wed, 25 Oct 2023)

  Changed paths:
    M lld/MinGW/Driver.cpp
    M lld/test/MinGW/driver.test

  Log Message:
  -----------
  [LLD] [MinGW] Hook up --icf=safe to -opt:safeicf (#70037)

Back when the --icf= option was hooked up in the MinGW frontend in LLD,
in 2017, lld-link didn't support safe ICF, and mapping it to noicf was
suggested in review: https://reviews.llvm.org/D40019

In 2018, in ab038025a50c79a89edb5832b163e8c729ceb703,
lld-link did get support for handling address significance tables,
allowing the ICF to operate safely on more sections.

Later in 2021, lld-link did get support for a separate safe ICF mode in
5bdc5e7efda4100c4d11085c2da8f1fb932ccce4 /
https://reviews.llvm.org/D97436.

Hook this up for the MinGW frontend as well.




More information about the All-commits mailing list