Tidy up debug printing and dumping a bit, moving from errs() to dbgs() (#17398)

Tidy up debug printing and dumping a bit, moving from errs() to dbgs() in a few places.

Author: lattner

lib/SILOptimizer/Mandatory/TFConstExpr.cpp

@@ -228,7 +228,8 @@ namespace {
     }
 
     void setValue(SILValue value, SymbolicValue symVal) {
-      assert(!value->getType().isAddress() &&"addresses are tracked separately");
+      assert(!value->getType().isAddress() &&
+             "addresses are tracked separately");
       calculatedValues.insert({ value, symVal });
     }
 
@@ -398,7 +399,7 @@ SymbolicValue ConstExprFunctionState::computeConstantValue(SILValue value) {
     if (fn)
       return SymbolicValue::getFunction(fn, conf);
 
-    DEBUG(llvm::errs() << "ConstExpr Unresolved witness: " << *value << "\n");
+    DEBUG(llvm::dbgs() << "ConstExpr Unresolved witness: " << *value << "\n");
     return SymbolicValue::getUnknown(value, UnknownReason::Default);
   }
 
@@ -416,7 +417,7 @@ SymbolicValue ConstExprFunctionState::computeConstantValue(SILValue value) {
     return calculatedValues[apply];
   }
 
-  DEBUG(llvm::errs() << "ConstExpr Unknown simple: " << *value << "\n");
+  DEBUG(llvm::dbgs() << "ConstExpr Unknown simple: " << *value << "\n");
 
   // Otherwise, we don't know how to handle this.
   return SymbolicValue::getUnknown(value, UnknownReason::Default);
@@ -753,7 +754,7 @@ ConstExprFunctionState::computeConstantValueBuiltin(BuiltinInst *inst) {
   }
 
 
-  DEBUG(llvm::errs() << "ConstExpr Unknown Builtin: " << *inst << "\n");
+  DEBUG(llvm::dbgs() << "ConstExpr Unknown Builtin: " << *inst << "\n");
 
   // Otherwise, we don't know how to handle this builtin.
   return unknownResult();
@@ -780,7 +781,7 @@ ConstExprFunctionState::computeOpaqueCallResult(ApplyInst *apply,
                                      UnknownReason::Trap);
   }
 
-  DEBUG(llvm::errs() << "ConstExpr Opaque Callee: " << *callee << "\n");
+  DEBUG(llvm::dbgs() << "ConstExpr Opaque Callee: " << *callee << "\n");
   return SymbolicValue::getUnknown((SILInstruction*)apply,
                                    UnknownReason::Default);
 }
@@ -936,8 +937,8 @@ SymbolicValue ConstExprFunctionState::getConstantValue(SILValue value) {
 
   // If this is the top-level lazy interpreter, output a debug trace.
   if (!fn) {
-    DEBUG(llvm::errs() << "ConstExpr top level: "; value->dump());
-    DEBUG(llvm::errs() << "  RESULT: ";  result.dump());
+    DEBUG(llvm::dbgs() << "ConstExpr top level: "; value->dump());
+    DEBUG(llvm::dbgs() << "  RESULT: ";  result.dump());
   }
 
   return calculatedValues[value] = result;
@@ -1096,7 +1097,7 @@ ConstExprFunctionState::computeSingleStoreAddressValue(SILValue addr) {
     }
 
 
-    DEBUG(llvm::errs() << "Unknown SingleStore ConstExpr user: "
+    DEBUG(llvm::dbgs() << "Unknown SingleStore ConstExpr user: "
                        << *user << "\n");
 
     // If this is some other user that we don't know about, then we should
@@ -1198,7 +1199,7 @@ AddressValue ConstExprFunctionState::computeAddressValue(SILValue addr) {
   if (auto *bai = dyn_cast<BeginAccessInst>(addr))
     return getAddressValue(bai->getOperand());
 
-  DEBUG(llvm::errs() << "ConstExpr Unknown simple addr: " << *addr << "\n");
+  DEBUG(llvm::dbgs() << "ConstExpr Unknown simple addr: " << *addr << "\n");
 
   // Otherwise, we don't know how to handle this.
   return getUnknownAddress(addr, UnknownReason::Default);
@@ -1299,20 +1300,20 @@ ConstExprFunctionState::evaluateFlowSensitive(SILInstruction *inst) {
       auto result = getConstantValue(oneResultVal);
       if (!result.isConstant())
         return result;
-      DEBUG(llvm::errs() << "  RESULT: ";  result.dump());
+      DEBUG(llvm::dbgs() << "  RESULT: ";  result.dump());
     } else {
       auto result = getAddressValue(oneResultVal);
       // If the address could not be resolved, puts the 'unknown' code into
       // the referenced memory object.
       auto memVal = memoryObjects[result.getObjectID()].first;
       if (memVal.isUnknown())
         return memVal;
-      DEBUG(llvm::errs() << "  RESULT: ";  result.dump());
+      DEBUG(llvm::dbgs() << "  RESULT: ";  result.dump());
     }
     return None;
   }
 
-  DEBUG(llvm::errs() << "ConstExpr Unknown FS: " << *inst << "\n");
+  DEBUG(llvm::dbgs() << "ConstExpr Unknown FS: " << *inst << "\n");
   // If this is an unknown instruction with no results then bail out.
   return SymbolicValue::getUnknown(inst, UnknownReason::Default);
 }
@@ -1340,10 +1341,10 @@ evaluateAndCacheCall(SILFunction &fn, SubstitutionMap substitutionMap,
   unsigned nextBBArg = 0;
   const auto &argList = fn.front().getArguments();
 
-  DEBUG(llvm::errs().changeColor(raw_ostream::SAVEDCOLOR, /*bold*/true)
+  DEBUG(llvm::dbgs().changeColor(raw_ostream::SAVEDCOLOR, /*bold*/true)
         << "\nConstExpr call fn: "
         << Demangle::demangleSymbolAsString(fn.getName());
-        llvm::errs().resetColor()
+        llvm::dbgs().resetColor()
         << "\n");
 
   for (unsigned i = 0, e = conventions.getNumIndirectSILResults(); i != e; ++i)
@@ -1371,7 +1372,7 @@ evaluateAndCacheCall(SILFunction &fn, SubstitutionMap substitutionMap,
 
   while (1) {
     SILInstruction *inst = &*nextInst++;
-    DEBUG(llvm::errs() << "ConstExpr interpret: "; inst->dump());
+    DEBUG(llvm::dbgs() << "ConstExpr interpret: "; inst->dump());
 
     // Make sure we haven't exceeded our interpreter iteration cap.
     if (++numInstEvaluated > ConstExprLimit)
@@ -1413,7 +1414,7 @@ evaluateAndCacheCall(SILFunction &fn, SubstitutionMap substitutionMap,
 
       // TODO: Handle caching of results.
 
-      DEBUG(llvm::errs() << "\n");
+      DEBUG(llvm::dbgs() << "\n");
       return None;
     }
 
@@ -1454,7 +1455,7 @@ evaluateAndCacheCall(SILFunction &fn, SubstitutionMap substitutionMap,
       continue;
     }
 
-    DEBUG(llvm::errs() << "ConstExpr: Unknown Terminator: " << *inst << "\n");
+    DEBUG(llvm::dbgs() << "ConstExpr: Unknown Terminator: " << *inst << "\n");
 
     // TODO: Enum switches when we support enums?
     return SymbolicValue::getUnknown(inst, UnknownReason::Default);

lib/SILOptimizer/Mandatory/TFDevicePartition.cpp

@@ -463,7 +463,7 @@ class DevicePartitionerImpl
     if (inst->getName().str() == "tf_tensor_to_i1")
       return;
 
-    inst->print(llvm::errs());
+    inst->dump();
     llvm_unreachable(
         "DevicePartitionerImpl cannot mark this instruction yet\n");
   }

lib/SILOptimizer/Mandatory/TFDifferentiation.cpp

@@ -383,8 +383,6 @@ class ADContext {
 ADContext::ADContext(SILModule &module, SILPassManager &passManager)
   : module(module), passManager(passManager), typeConverter(module)
 {
-  auto &ctx = getASTContext();
-      
   // Add local witness tables to the local map.
   for (auto &wt : module.getWitnessTables())
     witnessTables.insert({wt.getConformance(), &wt});

lib/SILOptimizer/Mandatory/TFLowerGraph.cpp

@@ -1492,7 +1492,7 @@ void TFGraphLowering::visitTFOpInst(BuiltinInst *inst) {
   }
 
   if (!hasDevice) {
-    inst->print(llvm::errs());
+    inst->dump();
     llvm_unreachable("The above tensor op has no device set");
   }
 

lib/SILOptimizer/Mandatory/TFPartition.cpp

@@ -3223,7 +3223,7 @@ void TFFunctionPartition::balanceRetainReleaseCount(SILValue oldResult,
       continue;
     }
 
-    oldResult->print(llvm::errs());
+    oldResult->dump();
     llvm_unreachable("Unhandled instruction type");
   }
 

lib/SILOptimizer/Mandatory/TFUtilities.cpp

@@ -486,7 +486,7 @@ static bool tryToRemoveArrayValue(SILValue value) {
         }
 
         // Otherwise we don't know what this is, conservatively bail out.
-        DEBUG(llvm::errs() << "Could not remove array because of: " << *user);
+        DEBUG(llvm::dbgs() << "Could not remove array because of: " << *user);
         return false;
       }
     }