Log to the right stream in DwarfTransformer::handleDie().

[clayborg]

Summary

Since we might end up using multiple threads when logging information in the DWARFTransformer, the handleDie() method must use the supplied stream named "OS" when logging warnings and errors. When we use multiple threads, we log to a thread specific stream buffer and then use a mutex to ensure our output doesn't overlap when we emit warnings and errors after a thread is done.

[wallace]

makes sense!

llvm/lib/DebugInfo/GSYM/DwarfTransformer.cpp

for (const DWARFAddressRange &Range : Ranges) {
       if (!Gsym.IsValidTextAddress(Range.LowPC)) {
         // We expect zero and -1 to be invalid addresses in DWARF depending
         // on the linker of the DWARF. This indicates a function was stripped
         // and the debug info wasn't able to be stripped from the DWARF. If
         // the LowPC isn't zero or -1, then we should emit an error.
         if (Range.LowPC != 0) {
           if (!Gsym.isQuiet()) {
             // Unexpected invalid address, emit a warning
-            Log << "warning: DIE has an address range whose start address is "
-                   "not in any executable sections ("
-                << *Gsym.GetValidTextRanges()
-                << ") and will not be processed:\n";
-            Die.dump(Log, 0, DIDumpOptions::getForSingleDIE());
+            OS << "warning: DIE has an address range whose start address is "
+            Die.dump(OS, 0, DIDumpOptions::getForSingleDIE());
           }
         }
         break;
       }
 
       FunctionInfo FI;
       FI.setStartAddress(Range.LowPC);
       FI.setEndAddress(Range.HighPC);