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

[Andrzej Warzynski via Phabricator via llvm-commits]

awarzynski added a comment.

In D96528#2566877 <https://reviews.llvm.org/D96528#2566877>, @klausler wrote:

> What if a compiler developer wants multiple dumps?

This is tricky. `flang-new -cc1` models `clang -cc1` in this respect and runs one action at a time.  Diverging from this design would be challenging because then the driver would have to:

- schedule the actions (and keep track of them)
- parse options for every action separately
- resolve potential conflicts with options (e.g. one option is fine with one of the requested actions, but causes a driver/compilation error for the other)

Alternatively, we could make `-fdebug-dump-parse-tree` and `-fdebug-dump-symbols` non-action flags in `flang-new`. This wouldn't fit nicely into the model implemented in `libclangDriver` where compiler flags/options are split into:

- action flags
- set-up/config/feature flags.

I've reverted this change in https://reviews.llvm.org/D96870. Moving forward, I would like to keep the current approach in `flang-new -fc1`. This may mean some inconsistencies with `f18` in terms of user-experience, but the alternative would be costly to implement.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96528