[PATCH] D44286: [lld-link] Add support for /ignore:4037.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 14 14:23:18 PDT 2018


ruiu added a comment.

> In the future I think this should be fixed once for all by tagging the messages with the corresponding Microsoft LNK number.
> 
> In that case, warn("/order...") could become lnkWarn(4217, "/order...") and automatically handle /ignore.

Adding a helper function that takes an error number might make sense, but I think that the use of the function is fairly limited, because lld doesn't try to do the exact same thing as MSVC link.exe. We sometimes emit a warning on which MSVC would emit a warning, but that's not too many.


https://reviews.llvm.org/D44286





More information about the llvm-commits mailing list