[PATCH] Fix PR17239 by changing the semantics of the RemainingArgsClass Option kind (clang side)
Reid Kleckner
rnk at google.com
Fri Aug 22 11:45:32 PDT 2014
This is tied to D4899, so I'll land it too. Thanks!
================
Comment at: tools/driver/driver.cpp:406-409
@@ -391,1 +405,6 @@
+ // Remove markers, if we added them
+ if (MarkEOLs) {
+ auto newEnd = std::remove(argv.begin(), argv.end(), nullptr);
+ argv.resize(newEnd - argv.begin());
+ }
return ExecuteCC1Tool(argv, argv[1] + 4);
----------------
This seems impossible. If we invoked a cc1 tool, we shouldn't have added EOL markers, right?
http://reviews.llvm.org/D4900
More information about the cfe-commits
mailing list