[PATCH] D56722: [lld] [ELF] Support for warn-once option

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added a comment.

> Rui, should we make -warn-once ON by default?

No, I don't think so. Either on or off could work as a default, but ON is currently the default, and I don't think we have a reason to change the default.



================
Comment at: ELF/Options.td:382
   HelpText<"Report unresolved symbols as warnings">;
+def warn_once: F<"warn-once">,
+  HelpText<"Warn only once for each undefined symbol">;
----------------
You should follow the local convention of a file. Please add a blank line before each `def` line.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D56722