Use ErrorOS for log messages as well as error

[sbc100]

Summary

These log messages are also diagnostics so it seems like
they should to the same place as errors and debug
messages.

Without this change when I enable --verbose those messages
go to stdout, but when I enable "-mllvm -debug" those messages
go to stderr (because dbgs() goes to stderr by default).

So I end up having to do this a lot:
lld <args> > output_message 2>&1

[sbc100]

• cleanup

[ruiu]

I agree that log should go to stderr, but I think message should still go to stdout. We have that function to print out a regular message, such as printing out version information or some hint message. We have this function mainly because we want to acquire the global lock before printing out something in the multithreaded environment.

[sbc100]

• feedback

[ruiu]

LGTM

[sbc100]

• update tests

Common/ErrorHandler.cpp

void ErrorHandler::print(StringRef S, raw_ostream::Colors C) {
   } else {
     *ErrorOS << S;
   }
 }
 
 void ErrorHandler::log(const Twine &Msg) {
   if (Verbose) {
     std::lock_guard<std::mutex> Lock(Mu);
-    outs() << LogName << ": " << Msg << "\n";
-    outs().flush();
+    *ErrorOS << LogName << ": " << Msg << "\n";
   }
 }
 
 void ErrorHandler::message(const Twine &Msg) {
   std::lock_guard<std::mutex> Lock(Mu);
   outs() << Msg << "\n";
   outs().flush();
 }