[PATCH] Fix PR17239 by changing the semantics of the RemainingArgsClass Option kind (clang side)

Rafael Auler rafaelauler at gmail.com
Tue Aug 19 12:22:56 PDT 2014


This is the clang side of my updated patch in D4899. I updated the Driver to cope with the new LLVM API proposed in that patch: 

- Call ExpandResponseFiles with MarkEOLs set to true to allow correct expansion of the /LINK argument, fixing PR17239. Only do this for the clang driver, not for clang -cc1 tools.
- Ignore nullptrs in the argv vector because they are now used to indicate end of lines.

I also updated the Driver::ParseArgStrings parameter name that had a confusing argument name that was the same of a known class used in the same scope (NFC).

http://reviews.llvm.org/D4900

Files:
  lib/Driver/Driver.cpp
  test/Driver/cl-link-at-file.c
  tools/driver/driver.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4900.12677.patch
Type: text/x-patch
Size: 4948 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140819/e8cbf357/attachment.bin>


More information about the cfe-commits mailing list