[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.
@@ -727,7 +727,7 @@
std::vector<const char *> Argv = replaceResponseFiles(ArgsArr);
// Make InputArgList from string vectors.
- COFFOptTable Table;
+ static COFFOptTable Table;
opt::InputArgList Args = Table.ParseArgs(Argv, MissingIndex, MissingCount);
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 476 bytes
Desc: not available
More information about the cfe-commits