[PATCH] D37203: [Bash-autocompletion] Follow up patch for D36782

Yuka Takahashi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 28 02:04:51 PDT 2017


yamaguchi created this revision.

https://reviews.llvm.org/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.


https://reviews.llvm.org/D37203

Files:
  lld/COFF/DriverUtils.cpp


Index: lld/COFF/DriverUtils.cpp
===================================================================
--- lld/COFF/DriverUtils.cpp
+++ lld/COFF/DriverUtils.cpp
@@ -727,7 +727,7 @@
   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);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37203.112859.patch
Type: text/x-patch
Size: 476 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170828/bf5a0772/attachment.bin>


More information about the cfe-commits mailing list