[Bash-autocompletion] Follow up patch for D36782

[yamaguchi]

Summary

D36782 broke lld buildbots, because I've changed OptionInfo's type
from ArrayRef to std::vector, so it became a copy, not a reference.
In this patch, I changed Table to static, so that it won't be freed
after the return. However, I'm not sure if this is Ok for lld, because
it requires this function to be called atomic.

[ruiu]

Technically, this patch fixes your issue, but I don't think this is a good way of fixing it because parse() function now has an internal state which is not obvious to the user of the function. Can you review https://reviews.llvm.org/D37217? This is an alternative fix that doesn't use a static local variables.

[yamaguchi]

D37217 has already landed, so we don't need this anymore.

lld/COFF/DriverUtils.cpp

 };
 
 // Parses a given list of options.
 opt::InputArgList ArgParser::parse(ArrayRef<const char *> ArgsArr) {
   // First, replace respnose files (@<file>-style options).
   std::vector<const char *> Argv = replaceResponseFiles(ArgsArr);
 
   // Make InputArgList from string vectors.
-  COFFOptTable Table;
+  static COFFOptTable Table;
   unsigned MissingIndex;
   unsigned MissingCount;
   opt::InputArgList Args = Table.ParseArgs(Argv, MissingIndex, MissingCount);
 
   // Print the real command line if response files are expanded.
   if (Args.hasArg(OPT_verbose) && ArgsArr.size() != Argv.size()) {
     std::string Msg = "Command line:";
     for (const char *S : Argv)