[PATCH] D52857: Deprecate 'set output foo' API of clang-query
Stephen Kelly via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 10 09:02:44 PDT 2018
steveire added a comment.
> I don't see this functionality being so critical that we need to deprecate the existing spelling when there are backwards compatible options available, which is why I'm opposed to this patch going in with the proposed syntax.
I don't think we're going to go anywhere except around in circles :). I don't see your comma separated syntax as viable. Maybe there is a third way that both of us don't see.
I tried to catch you on IRC without success. Thanks for the reviews! I appreciate your time. I'll see if I can find a different reviewer for this in a while.
Thanks,
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52857
More information about the cfe-commits
mailing list