[Driver] Drop redundant arguments in various helper functions

The OutputInfo, ToolChain, and Triple can all be retrieved from the
Compilation, so now that we're passing one of those around we don't
need to pass the others explicitly.

No functionality change.

Author: jrose-apple

include/swift/Driver/Driver.h

@@ -308,22 +308,19 @@ class Driver {
   ///
   /// \param C The Compilation which this Job will eventually be part of
   /// \param JA The Action for which a Job should be created
-  /// \param OI The OutputInfo for which a Job should be created
   /// \param OFM The OutputFileMap for which a Job should be created
-  /// \param TC The tool chain which should be used to create the Job
   /// \param AtTopLevel indicates whether or not this is a top-level Job
   /// \param JobCache maps existing Action/ToolChain pairs to Jobs
   ///
   /// \returns a Job for the given Action/ToolChain pair
   Job *buildJobsForAction(Compilation &C, const JobAction *JA,
-                          const OutputInfo &OI, const OutputFileMap *OFM,
-                          StringRef workingDirectory, const ToolChain &TC,
+                          const OutputFileMap *OFM,
+                          StringRef workingDirectory,
                           bool AtTopLevel, JobCacheMap &JobCache) const;
 
 private:
   void computeMainOutput(Compilation &C, const JobAction *JA,
-                         const OutputInfo &OI, const OutputFileMap *OFM,
-                         const ToolChain &TC, bool AtTopLevel,
+                         const OutputFileMap *OFM, bool AtTopLevel,
                          SmallVectorImpl<const Action *> &InputActions,
                          SmallVectorImpl<const Job *> &InputJobs,
                          const TypeToPathMap *OutputMap,
@@ -333,7 +330,7 @@ class Driver {
                          llvm::SmallString<128> &Buf,
                          CommandOutput *Output) const;
 
-  void chooseSwiftModuleOutputPath(Compilation &C, const OutputInfo &OI,
+  void chooseSwiftModuleOutputPath(Compilation &C,
                                    const OutputFileMap *OFM,
                                    const TypeToPathMap *OutputMap,
                                    StringRef workingDirectory,
@@ -343,39 +340,39 @@ class Driver {
                                       const TypeToPathMap *OutputMap,
                                       StringRef workingDirectory,
                                       CommandOutput *Output) const;
+
   void chooseRemappingOutputPath(Compilation &C, const TypeToPathMap *OutputMap,
                                  CommandOutput *Output) const;
 
   void chooseSerializedDiagnosticsPath(Compilation &C, const JobAction *JA,
-                                       const OutputInfo &OI,
                                        const TypeToPathMap *OutputMap,
                                        StringRef workingDirectory,
                                        CommandOutput *Output) const;
 
-  void chooseDependenciesOutputPaths(Compilation &C, const OutputInfo &OI,
+  void chooseDependenciesOutputPaths(Compilation &C,
                                      const TypeToPathMap *OutputMap,
                                      StringRef workingDirectory,
                                      llvm::SmallString<128> &Buf,
                                      CommandOutput *Output) const;
 
-  void chooseOptimizationRecordPath(Compilation &C, const OutputInfo &OI,
+  void chooseOptimizationRecordPath(Compilation &C,
                                     StringRef workingDirectory,
                                     llvm::SmallString<128> &Buf,
                                     CommandOutput *Output) const;
 
-  void chooseObjectiveCHeaderOutputPath(Compilation &C, const OutputInfo &OI,
+  void chooseObjectiveCHeaderOutputPath(Compilation &C,
                                         const TypeToPathMap *OutputMap,
                                         StringRef workingDirectory,
                                         CommandOutput *Output) const;
 
-  void chooseLoadedModuleTracePath(Compilation &C, const OutputInfo &OI,
+  void chooseLoadedModuleTracePath(Compilation &C,
                                    StringRef workingDirectory,
                                    llvm::SmallString<128> &Buf,
                                    CommandOutput *Output) const;
 
-  void chooseTBDPath(Compilation &C, const OutputInfo &OI,
-                     const TypeToPathMap *OutputMap, StringRef workingDirectory,
-                     llvm::SmallString<128> &Buf, CommandOutput *Output) const;
+  void chooseTBDPath(Compilation &C, const TypeToPathMap *OutputMap,
+                     StringRef workingDirectory, llvm::SmallString<128> &Buf,
+                     CommandOutput *Output) const;
 
 public:
   /// Handle any arguments which should be treated before building actions or