Kill off more cerr/cout uses and prune includes a bit.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@79852 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Benjamin Kramer 2009-08-23 11:37:21 +00:00
parent 895ae9b30d
commit cfa6ec92e6
13 changed files with 41 additions and 66 deletions

View File

@ -233,7 +233,7 @@ namespace {
bool Error = false; bool Error = false;
for (unsigned i = 0, e = tri_->getNumRegs(); i != e; ++i) { for (unsigned i = 0, e = tri_->getNumRegs(); i != e; ++i) {
if (regUse_[i] != 0) { if (regUse_[i] != 0) {
cerr << tri_->getName(i) << " is still in use!\n"; errs() << tri_->getName(i) << " is still in use!\n";
Error = true; Error = true;
} }
} }

View File

@ -9,13 +9,19 @@
#define DEBUG_TYPE "virtregrewriter" #define DEBUG_TYPE "virtregrewriter"
#include "VirtRegRewriter.h" #include "VirtRegRewriter.h"
#include "llvm/Function.h"
#include "llvm/CodeGen/MachineFrameInfo.h"
#include "llvm/CodeGen/MachineInstrBuilder.h"
#include "llvm/CodeGen/MachineRegisterInfo.h"
#include "llvm/Support/Compiler.h" #include "llvm/Support/Compiler.h"
#include "llvm/Support/CommandLine.h"
#include "llvm/Support/Debug.h"
#include "llvm/Support/ErrorHandling.h" #include "llvm/Support/ErrorHandling.h"
#include "llvm/Support/raw_ostream.h" #include "llvm/Support/raw_ostream.h"
#include "llvm/Target/TargetInstrInfo.h"
#include "llvm/Target/TargetLowering.h" #include "llvm/Target/TargetLowering.h"
#include "llvm/ADT/DepthFirstIterator.h" #include "llvm/ADT/DepthFirstIterator.h"
#include "llvm/ADT/Statistic.h" #include "llvm/ADT/Statistic.h"
#include "llvm/ADT/STLExtras.h"
#include <algorithm> #include <algorithm>
using namespace llvm; using namespace llvm;

View File

@ -10,27 +10,9 @@
#ifndef LLVM_CODEGEN_VIRTREGREWRITER_H #ifndef LLVM_CODEGEN_VIRTREGREWRITER_H
#define LLVM_CODEGEN_VIRTREGREWRITER_H #define LLVM_CODEGEN_VIRTREGREWRITER_H
#include "llvm/Target/TargetRegisterInfo.h"
#include "llvm/ADT/BitVector.h"
#include "llvm/ADT/IndexedMap.h"
#include "llvm/ADT/SmallPtrSet.h"
#include "llvm/ADT/SmallVector.h"
#include "llvm/Support/Streams.h"
#include "llvm/Function.h"
#include "llvm/CodeGen/LiveIntervalAnalysis.h" #include "llvm/CodeGen/LiveIntervalAnalysis.h"
#include "llvm/CodeGen/MachineFrameInfo.h"
#include "llvm/CodeGen/MachineFunction.h" #include "llvm/CodeGen/MachineFunction.h"
#include "llvm/CodeGen/MachineInstrBuilder.h"
#include "llvm/CodeGen/MachineRegisterInfo.h"
#include "llvm/Target/TargetMachine.h"
#include "llvm/Target/TargetInstrInfo.h"
#include "llvm/Support/CommandLine.h"
#include "llvm/Support/Debug.h"
#include "llvm/ADT/BitVector.h"
#include "llvm/ADT/DenseMap.h"
#include "llvm/ADT/SmallSet.h"
#include "VirtRegMap.h" #include "VirtRegMap.h"
#include <map>
namespace llvm { namespace llvm {

View File

@ -12,7 +12,6 @@
//===----------------------------------------------------------------------===// //===----------------------------------------------------------------------===//
#include "llvm/Support/StringPool.h" #include "llvm/Support/StringPool.h"
#include "llvm/Support/Streams.h"
#include "llvm/ADT/StringRef.h" #include "llvm/ADT/StringRef.h"
using namespace llvm; using namespace llvm;

View File

@ -16,7 +16,7 @@
#include "llvm/Pass.h" #include "llvm/Pass.h"
#include "llvm/Function.h" #include "llvm/Function.h"
#include "llvm/ADT/StringExtras.h" #include "llvm/ADT/StringExtras.h"
#include "llvm/Support/Streams.h" #include "llvm/Support/raw_ostream.h"
#include "llvm/ADT/Statistic.h" #include "llvm/ADT/Statistic.h"
using namespace llvm; using namespace llvm;
@ -32,7 +32,7 @@ namespace {
HelloCounter++; HelloCounter++;
std::string fname = F.getName(); std::string fname = F.getName();
EscapeString(fname); EscapeString(fname);
cerr << "Hello: " << fname << "\n"; errs() << "Hello: " << fname << "\n";
return false; return false;
} }
}; };
@ -51,7 +51,7 @@ namespace {
HelloCounter++; HelloCounter++;
std::string fname = F.getName(); std::string fname = F.getName();
EscapeString(fname); EscapeString(fname);
cerr << "Hello: " << fname << "\n"; errs() << "Hello: " << fname << "\n";
return false; return false;
} }

View File

@ -19,13 +19,11 @@
// //
//===----------------------------------------------------------------------===// //===----------------------------------------------------------------------===//
#include "llvm/Constants.h"
#include "llvm/DerivedTypes.h" #include "llvm/DerivedTypes.h"
#include "llvm/LLVMContext.h"
#include "llvm/Module.h" #include "llvm/Module.h"
#include "llvm/Pass.h" #include "llvm/Pass.h"
#include "llvm/Support/Compiler.h" #include "llvm/Support/Compiler.h"
#include "llvm/Support/Streams.h" #include "llvm/Support/raw_ostream.h"
#include "llvm/Transforms/Instrumentation.h" #include "llvm/Transforms/Instrumentation.h"
#include "RSProfiling.h" #include "RSProfiling.h"
#include "ProfilingUtils.h" #include "ProfilingUtils.h"
@ -53,8 +51,8 @@ ModulePass *llvm::createFunctionProfilerPass() {
bool FunctionProfiler::runOnModule(Module &M) { bool FunctionProfiler::runOnModule(Module &M) {
Function *Main = M.getFunction("main"); Function *Main = M.getFunction("main");
if (Main == 0) { if (Main == 0) {
cerr << "WARNING: cannot insert function profiling into a module" errs() << "WARNING: cannot insert function profiling into a module"
<< " with no main function!\n"; << " with no main function!\n";
return false; // No main, no instrumentation! return false; // No main, no instrumentation!
} }
@ -100,8 +98,8 @@ ModulePass *llvm::createBlockProfilerPass() { return new BlockProfiler(); }
bool BlockProfiler::runOnModule(Module &M) { bool BlockProfiler::runOnModule(Module &M) {
Function *Main = M.getFunction("main"); Function *Main = M.getFunction("main");
if (Main == 0) { if (Main == 0) {
cerr << "WARNING: cannot insert block profiling into a module" errs() << "WARNING: cannot insert block profiling into a module"
<< " with no main function!\n"; << " with no main function!\n";
return false; // No main, no instrumentation! return false; // No main, no instrumentation!
} }

View File

@ -18,13 +18,10 @@
//===----------------------------------------------------------------------===// //===----------------------------------------------------------------------===//
#include "ProfilingUtils.h" #include "ProfilingUtils.h"
#include "llvm/Constants.h"
#include "llvm/DerivedTypes.h"
#include "llvm/LLVMContext.h"
#include "llvm/Module.h" #include "llvm/Module.h"
#include "llvm/Pass.h" #include "llvm/Pass.h"
#include "llvm/Support/Compiler.h" #include "llvm/Support/Compiler.h"
#include "llvm/Support/Streams.h" #include "llvm/Support/raw_ostream.h"
#include "llvm/Transforms/Utils/BasicBlockUtils.h" #include "llvm/Transforms/Utils/BasicBlockUtils.h"
#include "llvm/Transforms/Instrumentation.h" #include "llvm/Transforms/Instrumentation.h"
#include <set> #include <set>
@ -48,8 +45,8 @@ ModulePass *llvm::createEdgeProfilerPass() { return new EdgeProfiler(); }
bool EdgeProfiler::runOnModule(Module &M) { bool EdgeProfiler::runOnModule(Module &M) {
Function *Main = M.getFunction("main"); Function *Main = M.getFunction("main");
if (Main == 0) { if (Main == 0) {
cerr << "WARNING: cannot insert edge profiling into a module" errs() << "WARNING: cannot insert edge profiling into a module"
<< " with no main function!\n"; << " with no main function!\n";
return false; // No main, no instrumentation! return false; // No main, no instrumentation!
} }

View File

@ -14,19 +14,15 @@
#define DEBUG_TYPE "condprop" #define DEBUG_TYPE "condprop"
#include "llvm/Transforms/Scalar.h" #include "llvm/Transforms/Scalar.h"
#include "llvm/Constants.h"
#include "llvm/Function.h"
#include "llvm/Instructions.h" #include "llvm/Instructions.h"
#include "llvm/IntrinsicInst.h" #include "llvm/IntrinsicInst.h"
#include "llvm/Pass.h" #include "llvm/Pass.h"
#include "llvm/Type.h" #include "llvm/Type.h"
#include "llvm/Transforms/Utils/BasicBlockUtils.h" #include "llvm/Transforms/Utils/BasicBlockUtils.h"
#include "llvm/Transforms/Utils/Local.h" #include "llvm/Transforms/Utils/Local.h"
#include "llvm/ADT/STLExtras.h"
#include "llvm/ADT/Statistic.h" #include "llvm/ADT/Statistic.h"
#include "llvm/ADT/SmallVector.h" #include "llvm/ADT/SmallVector.h"
#include "llvm/Support/Compiler.h" #include "llvm/Support/Compiler.h"
#include "llvm/Support/Streams.h"
using namespace llvm; using namespace llvm;
STATISTIC(NumBrThread, "Number of CFG edges threaded through branches"); STATISTIC(NumBrThread, "Number of CFG edges threaded through branches");

View File

@ -17,8 +17,8 @@
#include "llvm/ADT/FoldingSet.h" #include "llvm/ADT/FoldingSet.h"
#include "llvm/System/Atomic.h" #include "llvm/System/Atomic.h"
#include "llvm/System/Mutex.h" #include "llvm/System/Mutex.h"
#include "llvm/Support/Streams.h"
#include "llvm/Support/ManagedStatic.h" #include "llvm/Support/ManagedStatic.h"
#include "llvm/Support/raw_ostream.h"
using namespace llvm; using namespace llvm;
//===----------------------------------------------------------------------===// //===----------------------------------------------------------------------===//
@ -316,11 +316,11 @@ AttrListPtr AttrListPtr::removeAttr(unsigned Idx, Attributes Attrs) const {
} }
void AttrListPtr::dump() const { void AttrListPtr::dump() const {
cerr << "PAL[ "; errs() << "PAL[ ";
for (unsigned i = 0; i < getNumSlots(); ++i) { for (unsigned i = 0; i < getNumSlots(); ++i) {
const AttributeWithIndex &PAWI = getSlot(i); const AttributeWithIndex &PAWI = getSlot(i);
cerr << "{" << PAWI.Index << "," << PAWI.Attrs << "} "; errs() << "{" << PAWI.Index << "," << PAWI.Attrs << "} ";
} }
cerr << "]\n"; errs() << "]\n";
} }

View File

@ -16,7 +16,6 @@
#include "llvm/ADT/SmallPtrSet.h" #include "llvm/ADT/SmallPtrSet.h"
#include "llvm/Support/Compiler.h" #include "llvm/Support/Compiler.h"
#include "llvm/Support/ManagedStatic.h" #include "llvm/Support/ManagedStatic.h"
#include "llvm/Support/Streams.h"
#include "llvm/System/Mutex.h" #include "llvm/System/Mutex.h"
#include "llvm/System/Threading.h" #include "llvm/System/Threading.h"
#include "llvm/Value.h" #include "llvm/Value.h"
@ -64,8 +63,8 @@ void LeakDetector::checkForGarbageImpl(LLVMContext &Context,
// use non-short-circuit version so that both checks are performed // use non-short-circuit version so that both checks are performed
if (Objects->hasGarbage(Message) | if (Objects->hasGarbage(Message) |
pImpl->LLVMObjects.hasGarbage(Message)) pImpl->LLVMObjects.hasGarbage(Message))
cerr << "\nThis is probably because you removed an object, but didn't " errs() << "\nThis is probably because you removed an object, but didn't "
<< "delete it. Please check your code for memory leaks.\n"; << "delete it. Please check your code for memory leaks.\n";
// Clear out results so we don't get duplicate warnings on // Clear out results so we don't get duplicate warnings on
// next call... // next call...

View File

@ -18,7 +18,6 @@
#include "llvm/Module.h" #include "llvm/Module.h"
#include "llvm/ModuleProvider.h" #include "llvm/ModuleProvider.h"
#include "llvm/Support/ErrorHandling.h" #include "llvm/Support/ErrorHandling.h"
#include "llvm/Support/Streams.h"
#include "llvm/Support/ManagedStatic.h" #include "llvm/Support/ManagedStatic.h"
#include "llvm/Support/raw_ostream.h" #include "llvm/Support/raw_ostream.h"
#include "llvm/System/Mutex.h" #include "llvm/System/Mutex.h"

View File

@ -16,7 +16,7 @@
#include "llvm/ADT/StringExtras.h" #include "llvm/ADT/StringExtras.h"
#include "llvm/ADT/StringRef.h" #include "llvm/ADT/StringRef.h"
#include "llvm/Support/ManagedStatic.h" #include "llvm/Support/ManagedStatic.h"
#include "llvm/Support/Streams.h" #include "llvm/Support/raw_ostream.h"
#include "llvm/System/RWMutex.h" #include "llvm/System/RWMutex.h"
#include "llvm/System/Threading.h" #include "llvm/System/Threading.h"
#include <algorithm> #include <algorithm>
@ -79,7 +79,7 @@ Type* TypeSymbolTable::remove(iterator Entry) {
#if DEBUG_SYMBOL_TABLE #if DEBUG_SYMBOL_TABLE
dump(); dump();
cerr << " Removing Value: " << Result->getName() << "\n"; errs() << " Removing Value: " << Result->getName() << "\n";
#endif #endif
tmap.erase(Entry); tmap.erase(Entry);
@ -90,9 +90,9 @@ Type* TypeSymbolTable::remove(iterator Entry) {
// list... // list...
if (Result->isAbstract()) { if (Result->isAbstract()) {
#if DEBUG_ABSTYPE #if DEBUG_ABSTYPE
cerr << "Removing abstract type from symtab" errs() << "Removing abstract type from symtab"
<< Result->getDescription() << Result->getDescription()
<< "\n"; << "\n";
#endif #endif
cast<DerivedType>(Result)->removeAbstractTypeUser(this); cast<DerivedType>(Result)->removeAbstractTypeUser(this);
} }
@ -112,7 +112,7 @@ void TypeSymbolTable::insert(const StringRef &Name, const Type* T) {
#if DEBUG_SYMBOL_TABLE #if DEBUG_SYMBOL_TABLE
dump(); dump();
cerr << " Inserted type: " << Name << ": " << T->getDescription() << "\n"; errs() << " Inserted type: " << Name << ": " << T->getDescription() << "\n";
#endif #endif
} else { } else {
// If there is a name conflict... // If there is a name conflict...
@ -124,8 +124,8 @@ void TypeSymbolTable::insert(const StringRef &Name, const Type* T) {
#if DEBUG_SYMBOL_TABLE #if DEBUG_SYMBOL_TABLE
dump(); dump();
cerr << " Inserting type: " << UniqueName << ": " errs() << " Inserting type: " << UniqueName << ": "
<< T->getDescription() << "\n"; << T->getDescription() << "\n";
#endif #endif
// Insert the tmap entry // Insert the tmap entry
@ -138,7 +138,7 @@ void TypeSymbolTable::insert(const StringRef &Name, const Type* T) {
if (T->isAbstract()) { if (T->isAbstract()) {
cast<DerivedType>(T)->addAbstractTypeUser(this); cast<DerivedType>(T)->addAbstractTypeUser(this);
#if DEBUG_ABSTYPE #if DEBUG_ABSTYPE
cerr << "Added abstract type to ST: " << T->getDescription() << "\n"; errs() << "Added abstract type to ST: " << T->getDescription() << "\n";
#endif #endif
} }
} }
@ -156,14 +156,14 @@ void TypeSymbolTable::refineAbstractType(const DerivedType *OldType,
for (iterator I = begin(), E = end(); I != E; ++I) { for (iterator I = begin(), E = end(); I != E; ++I) {
if (I->second == (Type*)OldType) { // FIXME when Types aren't const. if (I->second == (Type*)OldType) { // FIXME when Types aren't const.
#if DEBUG_ABSTYPE #if DEBUG_ABSTYPE
cerr << "Removing type " << OldType->getDescription() << "\n"; errs() << "Removing type " << OldType->getDescription() << "\n";
#endif #endif
OldType->removeAbstractTypeUser(this); OldType->removeAbstractTypeUser(this);
I->second = (Type*)NewType; // TODO FIXME when types aren't const I->second = (Type*)NewType; // TODO FIXME when types aren't const
if (NewType->isAbstract()) { if (NewType->isAbstract()) {
#if DEBUG_ABSTYPE #if DEBUG_ABSTYPE
cerr << "Added type " << NewType->getDescription() << "\n"; errs() << "Added type " << NewType->getDescription() << "\n";
#endif #endif
cast<DerivedType>(NewType)->addAbstractTypeUser(this); cast<DerivedType>(NewType)->addAbstractTypeUser(this);
} }
@ -184,13 +184,13 @@ void TypeSymbolTable::typeBecameConcrete(const DerivedType *AbsTy) {
} }
static void DumpTypes(const std::pair<const std::string, const Type*>& T ) { static void DumpTypes(const std::pair<const std::string, const Type*>& T ) {
cerr << " '" << T.first << "' = "; errs() << " '" << T.first << "' = ";
T.second->dump(); T.second->dump();
cerr << "\n"; errs() << "\n";
} }
void TypeSymbolTable::dump() const { void TypeSymbolTable::dump() const {
cerr << "TypeSymbolPlane: "; errs() << "TypeSymbolPlane: ";
sys::SmartScopedReader<true> Reader(*TypeSymbolTableLock); sys::SmartScopedReader<true> Reader(*TypeSymbolTableLock);
for_each(tmap.begin(), tmap.end(), DumpTypes); for_each(tmap.begin(), tmap.end(), DumpTypes);
} }

View File

@ -56,7 +56,6 @@
#include "llvm/Support/CallSite.h" #include "llvm/Support/CallSite.h"
#include "llvm/Support/CFG.h" #include "llvm/Support/CFG.h"
#include "llvm/Support/InstVisitor.h" #include "llvm/Support/InstVisitor.h"
#include "llvm/Support/Streams.h"
#include "llvm/ADT/SmallPtrSet.h" #include "llvm/ADT/SmallPtrSet.h"
#include "llvm/ADT/SmallVector.h" #include "llvm/ADT/SmallVector.h"
#include "llvm/ADT/StringExtras.h" #include "llvm/ADT/StringExtras.h"
@ -85,9 +84,9 @@ namespace { // Anonymous namespace for class
for (Function::iterator I = F.begin(), E = F.end(); I != E; ++I) { for (Function::iterator I = F.begin(), E = F.end(); I != E; ++I) {
if (I->empty() || !I->back().isTerminator()) { if (I->empty() || !I->back().isTerminator()) {
cerr << "Basic Block does not have terminator!\n"; errs() << "Basic Block does not have terminator!\n";
WriteAsOperand(errs(), I, true); WriteAsOperand(errs(), I, true);
cerr << "\n"; errs() << "\n";
Broken = true; Broken = true;
} }
} }