r190907 - Revert r190902 and r190906

[Hans Wennborg]

Author: hans
Date: Tue Sep 17 19:21:51 2013
New Revision: 190907

URL: http://llvm.org/viewvc/llvm-project?rev=190907&view=rev
Log:
Revert r190902 and r190906

The first one broke the build, and the latter one made it worse.

Modified:
    cfe/trunk/include/clang/Driver/Job.h
    cfe/trunk/lib/Driver/Job.cpp

Modified: cfe/trunk/include/clang/Driver/Job.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Job.h?rev=190907&r1=190906&r2=190907&view=diff
==============================================================================
--- cfe/trunk/include/clang/Driver/Job.h (original)
+++ cfe/trunk/include/clang/Driver/Job.h Tue Sep 17 19:21:51 2013
@@ -24,8 +24,7 @@ class Action;
 class Command;
 class Tool;
 
-// Users of this class will use clang::driver::ArgStringList.
-typedef llvm::opt::ArgStringList ArgStringList;
+using llvm::opt::ArgStringList;
 
 class Job {
 public:
@@ -68,11 +67,11 @@ class Command : public Job {
 
   /// The list of program arguments (not including the implicit first
   /// argument, which will be the executable).
-  ArgStringList Arguments;
+  llvm::opt::ArgStringList Arguments;
 
 public:
   Command(const Action &_Source, const Tool &_Creator, const char *_Executable,
-          const ArgStringList &_Arguments);
+          const llvm::opt::ArgStringList &_Arguments);
 
   virtual void Print(llvm::raw_ostream &OS, const char *Terminator,
                      bool Quote, bool CrashReport = false) const;
@@ -86,7 +85,7 @@ public:
   /// getCreator - Return the Tool which caused the creation of this job.
   const Tool &getCreator() const { return Creator; }
 
-  const ArgStringList &getArguments() const { return Arguments; }
+  const llvm::opt::ArgStringList &getArguments() const { return Arguments; }
 
   static bool classof(const Job *J) {
     return J->getKind() == CommandClass;

Modified: cfe/trunk/lib/Driver/Job.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Job.cpp?rev=190907&r1=190906&r2=190907&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/Job.cpp (original)
+++ cfe/trunk/lib/Driver/Job.cpp Tue Sep 17 19:21:51 2013
@@ -21,7 +21,8 @@ using llvm::StringRef;
 Job::~Job() {}
 
 Command::Command(const Action &_Source, const Tool &_Creator,
-                 const char *_Executable, const ArgStringList &_Arguments)
+                 const char *_Executable,
+                 const llvm::opt::ArgStringList &_Arguments)
     : Job(CommandClass), Source(_Source), Creator(_Creator),
       Executable(_Executable), Arguments(_Arguments) {}
 
@@ -112,7 +113,7 @@ void Command::Print(raw_ostream &OS, con
   OS << Terminator;
 }
 
-int Command::Execute(const StringRef **Redirects, std::string *ErrMsg,
+int Command::Execute(const llvm::StringRef **Redirects, std::string *ErrMsg,
                      bool *ExecutionFailed) const {
   SmallVector<const char*, 128> Argv;
   Argv.push_back(Executable);