finegrainify namespacification

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@10839 91177308-0d34-0410-b5e6-96231b3b80d8

tools/bugpoint/BugDriver.cpp

@@ -22,7 +22,6 @@
 #include "Support/CommandLine.h"
 #include "Support/FileUtilities.h"
 #include <memory>
-
 using namespace llvm;
 
 // Anonymous namespace to define command line options for debugging.
@@ -38,12 +37,10 @@ namespace {
                                 "(for miscompilation detection)"));
 }
 
-namespace llvm {
-
 /// getPassesString - Turn a list of passes into a string which indicates the
 /// command line options that must be passed to add the passes.
 ///
-std::string getPassesString(const std::vector<const PassInfo*> &Passes) {
+std::string llvm::getPassesString(const std::vector<const PassInfo*> &Passes) {
   std::string Result;
   for (unsigned i = 0, e = Passes.size(); i != e; ++i) {
     if (i) Result += " ";
@@ -56,7 +53,7 @@ std::string getPassesString(const std::vector<const PassInfo*> &Passes) {
 // DeleteFunctionBody - "Remove" the function by deleting all of its basic
 // blocks, making it external.
 //
-void DeleteFunctionBody(Function *F) {
+void llvm::DeleteFunctionBody(Function *F) {
   // delete the body of the function...
   F->deleteBody();
   assert(F->isExternal() && "This didn't make the function external!");
@@ -184,4 +181,3 @@ void BugDriver::PrintFunctionList(const std::vector<Function*> &Funcs) {
   std::cout << std::flush;
 }
 
-} // End llvm namespace

tools/bugpoint/CodeGeneratorBug.cpp

@@ -32,9 +32,9 @@
 #include "Support/FileUtilities.h"
 #include <algorithm>
 #include <set>
+using namespace llvm;
 
 namespace llvm {
-
   extern cl::list<std::string> InputArgv;
 
   class ReduceMisCodegenFunctions : public ListReducer<Function*> {
@@ -54,7 +54,7 @@ public:
     bool TestFuncs(const std::vector<Function*> &CodegenTest,
                    bool KeepFiles = false);
   };
-
+}
 
 bool ReduceMisCodegenFunctions::TestFuncs(const std::vector<Function*> &Funcs,
                                           bool KeepFiles) {
@@ -328,7 +328,7 @@ namespace {
   };
 }
 
-void DisambiguateGlobalSymbols(Module *M) {
+static void DisambiguateGlobalSymbols(Module *M) {
   // First, try not to cause collisions by minimizing chances of renaming an
   // already-external symbol, so take in external globals and functions as-is.
   Disambiguator D;
@@ -406,5 +406,3 @@ bool BugDriver::debugCodeGenerator() {
 
   return false;
 }
-
-} // End llvm namespace

tools/bugpoint/CrashDebugger.cpp

@@ -28,21 +28,22 @@
 #include "Support/FileUtilities.h"
 #include <fstream>
 #include <set>
+using namespace llvm;
 
 namespace llvm {
-
   class DebugCrashes : public ListReducer<const PassInfo*> {
     BugDriver &BD;
   public:
     DebugCrashes(BugDriver &bd) : BD(bd) {}
     
-  // doTest - Return true iff running the "removed" passes succeeds, and running
-  // the "Kept" passes fail when run on the output of the "removed" passes.  If
-  // we return true, we update the current module of bugpoint.
+    // doTest - Return true iff running the "removed" passes succeeds, and
+    // running the "Kept" passes fail when run on the output of the "removed"
+    // passes.  If we return true, we update the current module of bugpoint.
     //
     virtual TestResult doTest(std::vector<const PassInfo*> &Removed,
                               std::vector<const PassInfo*> &Kept);
   };
+}
 
 DebugCrashes::TestResult
 DebugCrashes::doTest(std::vector<const PassInfo*> &Prefix,
@@ -82,6 +83,7 @@ DebugCrashes::doTest(std::vector<const PassInfo*> &Prefix,
   return NoFailure;
 }
 
+namespace llvm {
   class ReduceCrashingFunctions : public ListReducer<Function*> {
     BugDriver &BD;
   public:
@@ -98,6 +100,7 @@ public:
     
     bool TestFuncs(std::vector<Function*> &Prefix);
   };
+}
 
 bool ReduceCrashingFunctions::TestFuncs(std::vector<Function*> &Funcs) {
   // Clone the program to try hacking it apart...
@@ -143,10 +146,11 @@ bool ReduceCrashingFunctions::TestFuncs(std::vector<Function*> &Funcs) {
 }
 
 
-/// ReduceCrashingBlocks reducer - This works by setting the terminators of all
-/// terminators except the specified basic blocks to a 'ret' instruction, then
-/// running the simplify-cfg pass.  This has the effect of chopping up the CFG
-/// really fast which can reduce large functions quickly.
+namespace llvm {
+  /// ReduceCrashingBlocks reducer - This works by setting the terminators of
+  /// all terminators except the specified basic blocks to a 'ret' instruction,
+  /// then running the simplify-cfg pass.  This has the effect of chopping up
+  /// the CFG really fast which can reduce large functions quickly.
   ///
   class ReduceCrashingBlocks : public ListReducer<BasicBlock*> {
     BugDriver &BD;
@@ -164,6 +168,7 @@ public:
     
     bool TestBlocks(std::vector<BasicBlock*> &Prefix);
   };
+}
 
 bool ReduceCrashingBlocks::TestBlocks(std::vector<BasicBlock*> &BBs) {
   // Clone the program to try hacking it apart...
@@ -403,4 +408,3 @@ bool BugDriver::debugCrash() {
   return false;
 }
 
-} // End llvm namespace

tools/bugpoint/ExecutionDriver.cpp

@@ -29,7 +29,6 @@ BUGPOINT NOTES:
 #include "llvm/Support/ToolRunner.h"
 #include <fstream>
 #include <iostream>
-
 using namespace llvm;
 
 namespace {
@@ -61,12 +60,12 @@ namespace {
 }
 
 namespace llvm {
-
   // Anything specified after the --args option are taken as arguments to the
   // program being debugged.
   cl::list<std::string>
   InputArgv("args", cl::Positional, cl::desc("<program arguments>..."),
             cl::ZeroOrMore);
+}
 
 //===----------------------------------------------------------------------===//
 // BugDriver method implementation
@@ -237,4 +236,3 @@ bool BugDriver::isExecutingJIT() {
   return InterpreterSel == RunJIT;
 }
 
-} // End llvm namespace

tools/bugpoint/Miscompilation.cpp

@@ -18,6 +18,7 @@
 #include "llvm/Transforms/Utils/Cloning.h"
 #include "llvm/Transforms/Utils/Linker.h"
 #include "Support/FileUtilities.h"
+using namespace llvm;
 
 namespace llvm {
 
@@ -29,6 +30,7 @@ public:
     virtual TestResult doTest(std::vector<const PassInfo*> &Prefix,
                               std::vector<const PassInfo*> &Suffix);
   };
+}
 
 ReduceMiscompilingPasses::TestResult
 ReduceMiscompilingPasses::doTest(std::vector<const PassInfo*> &Prefix,
@@ -122,6 +124,7 @@ ReduceMiscompilingPasses::doTest(std::vector<const PassInfo*> &Prefix,
   return NoFailure;
 }
 
+namespace llvm {
   class ReduceMiscompilingFunctions : public ListReducer<Function*> {
     BugDriver &BD;
   public:
@@ -138,6 +141,7 @@ public:
     
     bool TestFuncs(const std::vector<Function*> &Prefix, bool EmitBytecode);
   };
+}
 
 bool ReduceMiscompilingFunctions::TestFuncs(const std::vector<Function*> &Funcs,
                                             bool EmitBytecode) {
@@ -315,4 +319,3 @@ bool BugDriver::debugMiscompilation() {
   return false;
 }
 
-} // End llvm namespace

tools/bugpoint/OptimizerDriver.cpp

@@ -25,8 +25,7 @@
 #include <unistd.h>
 #include <sys/types.h>
 #include <sys/wait.h>
-
-namespace llvm {
+using namespace llvm;
 
 /// writeProgramToFile - This writes the current "Program" to the named bytecode
 /// file.  If an error occurs, true is returned.
@@ -162,4 +161,3 @@ bool BugDriver::runPasses(const std::vector<const PassInfo*> &Passes,
   return !ExitedOK;
 }
 
-} // End llvm namespace