[PATCH] D41606: [COFF] support /ignore:4217

[Bob Haarman via Phabricator via llvm-commits]

inglorion created this revision.
inglorion added reviewers: rnk, ruiu.

lld-link accepts link.exe's /ignore option, but used to ignore
it. This can lead to semantic differences when warnings are treated as
fatal errors. One such case is when we resolve an __imp_ symbol to a
local definition. We emit a warning in that case, which /wx turns into
a fatal. This change makes lld-link accept /ignore:4217 to suppress
that warning, so that code that links with link.exe /wx /ignore:4217
links with lld-link, too.

Fixes PR35762.


https://reviews.llvm.org/D41606

Files:
  lld/COFF/Config.h
  lld/COFF/Driver.cpp
  lld/COFF/Options.td
  lld/COFF/SymbolTable.cpp
  lld/test/COFF/ignore4217.yaml

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41606.128256.patch
Type: text/x-patch
Size: 5853 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171228/5519dfcc/attachment.bin>