[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