diff options
author | Justin Bogner <mail@justinbogner.com> | 2015-07-02 22:52:08 +0000 |
---|---|---|
committer | Justin Bogner <mail@justinbogner.com> | 2015-07-02 22:52:08 +0000 |
commit | 0cd9248d9e1c4a35ea7633a11de3fb394b48f452 (patch) | |
tree | 1230089592673217be3821c094d26745ae06d150 /clang/lib/Driver/Job.cpp | |
parent | 94817612e2b45c3e447399359aa117f386ab58c4 (diff) | |
download | bcm5719-llvm-0cd9248d9e1c4a35ea7633a11de3fb394b48f452.tar.gz bcm5719-llvm-0cd9248d9e1c4a35ea7633a11de3fb394b48f452.zip |
Driver: Remove the Job class. NFC
We had a strange relationship here where we made a list of Jobs
inherit from a single Job, but there weren't actually any places where
this arbitrary nesting was used or needed.
Simplify all of this by removing Job entirely and updating all of the
users to either work with a JobList or a single Command.
llvm-svn: 241310
Diffstat (limited to 'clang/lib/Driver/Job.cpp')
-rw-r--r-- | clang/lib/Driver/Job.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/clang/lib/Driver/Job.cpp b/clang/lib/Driver/Job.cpp index 4b5b839e524..ac18e1eb56a 100644 --- a/clang/lib/Driver/Job.cpp +++ b/clang/lib/Driver/Job.cpp @@ -25,12 +25,10 @@ using llvm::raw_ostream; using llvm::StringRef; using llvm::ArrayRef; -Job::~Job() {} - Command::Command(const Action &Source, const Tool &Creator, const char *Executable, const ArgStringList &Arguments) - : Job(CommandClass), Source(Source), Creator(Creator), - Executable(Executable), Arguments(Arguments), ResponseFile(nullptr) {} + : Source(Source), Creator(Creator), Executable(Executable), + Arguments(Arguments), ResponseFile(nullptr) {} static int skipArgs(const char *Flag, bool HaveCrashVFS) { // These flags are all of the form -Flag <Arg> and are treated as two @@ -293,8 +291,6 @@ int FallbackCommand::Execute(const StringRef **Redirects, std::string *ErrMsg, return SecondaryStatus; } -JobList::JobList() : Job(JobListClass) {} - void JobList::Print(raw_ostream &OS, const char *Terminator, bool Quote, CrashReportInfo *CrashInfo) const { for (const auto &Job : *this) |