6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-07-26 05:25:47 +00:00
Code Issues Projects Releases Wiki Activity

Return true on success in cl::ExpandResponseFiles

Browse Source 
This fixes a logic bug pointed out by Juraj Ivancic.

No behavior change because none of the in-tree clients of
cl::ExpandResponseFiles check the return value.  In this case, the
@prefixed arguments are left in the command line.  Downstream command
line processing has the opportunity to emit errors about it, so this
isn't that bad.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@196306 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Reid Kleckner
2013-12-03 19:13:18 +00:00
parent 2575bc901b
commit ca3da883b8
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/Support/CommandLine.cpp
View File

@@ -634,7 +634,7 @@ static bool ExpandResponseFile(const char *FName, StringSaver &Saver,
bool cl::ExpandResponseFiles(StringSaver &Saver, TokenizerCallback Tokenizer, bool cl::ExpandResponseFiles(StringSaver &Saver, TokenizerCallback Tokenizer,
SmallVectorImpl<const char *> &Argv) { SmallVectorImpl<const char *> &Argv) {
unsigned RspFiles = 0; unsigned RspFiles = 0;
bool AllExpanded = false; bool AllExpanded = true;
// Don't cache Argv.size() because it can change. // Don't cache Argv.size() because it can change.
for (unsigned I = 0; I != Argv.size(); ) { for (unsigned I = 0; I != Argv.size(); ) {