[PATCH] D130121: [3/3] [COFF] Emit embedded -exclude-symbols: directives for hidden visibility for MinGW

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 12 01:29:16 PDT 2022


mstorsjo added a comment.

In D130121#3717108 <https://reviews.llvm.org/D130121#3717108>, @MaskRay wrote:

> Reasonable to back port this patch, too.

Oh, do you think so? In one sense, I guess if we're going to do this, it doesn't really matter that much if we do it now or in 16.x. (Although by the time 16.x is out, binutils 2.40 with support for the linker directive might be out too, so it'd work better in Clang+ld.bfd cases.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130121



More information about the llvm-commits mailing list