[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