[PATCH] D96528: [flang][f18] Make -fdebug-dump-{symbols|parse-tree} imply -fsyntax-only

[Faris Rehman via Phabricator via llvm-commits]

FarisRehman accepted this revision.
FarisRehman added a comment.
This revision is now accepted and ready to land.

This patch makes sense to me as the behaviour of the new driver is similar to clang in that only the last action-related option is run, e.g:

- Only `-emit-llvm` is run in: `clang -cc1 -ast-dump -emit-llvm file.c`
- Only `-ast-dump` is run in: `clang -cc1 -emit-llvm -ast-dump file.c`

This change will mean `-fsyntax-only` does not need to be specified when running `-fdebug-dump-parse-tree` or `-fdebug-dump-symbols` as it will always be run when either one of these action-related options are present.
LRTM!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96528/new/

https://reviews.llvm.org/D96528