[PATCH] D130120: [2/3] [LLD] [COFF] Add support for a new, mingw specific embedded directive -exclude-symbols:

[Mateusz Mikuła via Phabricator via llvm-commits]

mati865 accepted this revision.
mati865 added a comment.
This revision is now accepted and ready to land.

Even though the changes are small I not sure if approval from an outsider (me) should suffice here.



================
Comment at: lld/test/COFF/exclude-symbols-embedded.s:24-25
+.section .drectve,"yn"
+.ascii " -exclude-symbols:sym2"
+.ascii " -exclude-symbols:sym3"
+
----------------
Maybe one of these lines should list multiple symbols to cover that case?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130120