[PATCH] D39259: [lld] unified COFF and ELF error handling on new Common/ErrorHandler

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added inline comments.


================
Comment at: lld/include/lld/Common/ErrorHandler.h:50-54
+  ErrorHandler(llvm::raw_ostream *Stream, StringRef LogName)
+      : ErrorLimitExceededMsg("too many errors emitted, stopping now (use "
+                              "-error-limit=0 to see all errors)"),
+        LogName(LogName), ErrorOS(Stream),
+        ColorDiagnostics(Stream->has_colors()) {}
----------------
inglorion wrote:
> ruiu wrote:
> > I believe you are not using this ctor.
> The default ctor delegates to it. I suppose I could just only have a default ctor.
Yeah, and initializing a member with an ELF-ish default value seems a bit odd if you move this file to Common directory.


https://reviews.llvm.org/D39259