[PATCH] D100762: [clang][cli] Extract AST dump format into extra option
Duncan P. N. Exon Smith via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 19 12:14:26 PDT 2021
dexonsmith added a comment.
In D100762#2699252 <https://reviews.llvm.org/D100762#2699252>, @arichardson wrote:
> I'm not sure it's a good idea to remove the `-ast-dump=json` option. While this is -cc1 option, there do seem to be external consumers based on a quick search for "-ast-dump=json". Keeping it would also reduce some of the test churn.
Maybe `-ast-dump=json` can be changed to an alias for `-ast-dump -ast-dump-format json`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100762/new/
https://reviews.llvm.org/D100762
More information about the cfe-commits
mailing list