[lld] r240901 - COFF: Remove a function that doesn't do much itself. NFC.

Rui Ueyama ruiu at google.com
Sat Jun 27 20:05:38 PDT 2015


Author: ruiu
Date: Sat Jun 27 22:05:38 2015
New Revision: 240901

URL: http://llvm.org/viewvc/llvm-project?rev=240901&view=rev
Log:
COFF: Remove a function that doesn't do much itself. NFC.

Modified:
    lld/trunk/COFF/Driver.cpp
    lld/trunk/COFF/Driver.h
    lld/trunk/COFF/DriverUtils.cpp

Modified: lld/trunk/COFF/Driver.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/COFF/Driver.cpp?rev=240901&r1=240900&r2=240901&view=diff
==============================================================================
--- lld/trunk/COFF/Driver.cpp (original)
+++ lld/trunk/COFF/Driver.cpp Sat Jun 27 22:05:38 2015
@@ -233,7 +233,7 @@ bool LinkerDriver::link(llvm::ArrayRef<c
     return llvm::libDriverMain(ArgsArr.slice(1)) == 0;
 
   // Parse command line options.
-  auto ArgsOrErr = Parser.parseLINK(ArgsArr);
+  auto ArgsOrErr = Parser.parseLINK(ArgsArr.slice(1));
   if (auto EC = ArgsOrErr.getError()) {
     llvm::errs() << EC.message() << "\n";
     return false;

Modified: lld/trunk/COFF/Driver.h
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/COFF/Driver.h?rev=240901&r1=240900&r2=240901&view=diff
==============================================================================
--- lld/trunk/COFF/Driver.h (original)
+++ lld/trunk/COFF/Driver.h Sat Jun 27 22:05:38 2015
@@ -53,7 +53,6 @@ public:
   }
 
 private:
-  ErrorOr<llvm::opt::InputArgList> parse(std::vector<const char *> Argv);
   std::vector<const char *> tokenize(StringRef S);
 
   ErrorOr<std::vector<const char *>>

Modified: lld/trunk/COFF/DriverUtils.cpp
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/COFF/DriverUtils.cpp?rev=240901&r1=240900&r2=240901&view=diff
==============================================================================
--- lld/trunk/COFF/DriverUtils.cpp (original)
+++ lld/trunk/COFF/DriverUtils.cpp Sat Jun 27 22:05:38 2015
@@ -542,15 +542,15 @@ public:
 
 // Parses a given list of options.
 ErrorOr<llvm::opt::InputArgList>
-ArgParser::parse(std::vector<const char *> Argv) {
+ArgParser::parse(ArrayRef<const char *> ArgsArr) {
   // First, replace respnose files (@<file>-style options).
-  auto ArgvOrErr = replaceResponseFiles(Argv);
+  auto ArgvOrErr = replaceResponseFiles(ArgsArr);
   if (auto EC = ArgvOrErr.getError()) {
     llvm::errs() << "error while reading response file: " << EC.message()
                  << "\n";
     return EC;
   }
-  Argv = std::move(ArgvOrErr.get());
+  std::vector<const char *> Argv = std::move(ArgvOrErr.get());
 
   // Make InputArgList from string vectors.
   COFFOptTable Table;
@@ -571,20 +571,13 @@ ArgParser::parse(std::vector<const char
 }
 
 ErrorOr<llvm::opt::InputArgList>
-ArgParser::parse(llvm::ArrayRef<const char *> Args) {
-  Args = Args.slice(1);
-  std::vector<const char *> V(Args.begin(), Args.end());
-  return parse(V);
-}
-
-ErrorOr<llvm::opt::InputArgList>
 ArgParser::parseLINK(ArrayRef<const char *> Args) {
   // Concatenate LINK env and given arguments and parse them.
   Optional<std::string> Env = Process::GetEnv("LINK");
   if (!Env)
     return parse(Args);
   std::vector<const char *> V = tokenize(*Env);
-  V.insert(V.end(), Args.begin() + 1, Args.end());
+  V.insert(V.end(), Args.begin(), Args.end());
   return parse(V);
 }
 





More information about the llvm-commits mailing list