[PATCH] D36045: Extend syntax of response files
Serge Pavlov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 21 10:24:47 PST 2017
sepavloff added a comment.
In https://reviews.llvm.org/D36045#962255, @hfinkel wrote:
> I like the extension, but making our parsing of response files differ from GCC's in this regard might not be an improvements (e.g., the handling of the escaped new lines, and even the comments, could be a breaking change). Can you add a new parameter (e.g., bool ExtendedSyntax = false), and then predicate the new behavior on that? Then set that parameter to true when parsing config files from Clang?
This is not a problem, I will revert the other dependency https://reviews.llvm.org/D24926 to its previous version, where config file was parsed by dedicated function.
https://reviews.llvm.org/D36045
More information about the llvm-commits
mailing list