[PATCH] D100762: [clang][cli] Extract AST dump format into extra option

Jan Svoboda via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 22 02:23:23 PDT 2021


jansvoboda11 planned changes to this revision.
jansvoboda11 added a comment.

If `-ast-dump=json` was a driver flag, it would be trivial to pass `-ast-dump -ast-dump-format json` to -cc1 instead. However, aliasing a single option to two options within the -cc1 argument parser isn't possible at the moment AFAIK. I can look into how much work adding that capability would be.

@arichardson Can you point me to the external consumers?

@dexonsmith I agree checking the offset only in a couple of tests would be better in terms of churn.


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