[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