[PATCH] D99645: [flang][driver] Add debug options not requiring semantic checks

[Andrzej Warzynski via Phabricator via cfe-commits]

awarzynski added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:4378
   HelpText<"Run the InputOuputTest action. Use for development and testing only.">;
+def fdebug_unparse_no_sema : Flag<["-"], "fdebug-unparse-no-sema">, Group<Action_Group>,
+  HelpText<"Unparse and stop (skips the semantic checks)">;
----------------
kiranchandramohan wrote:
> Does this flag actually mean, parse and construct the parse-tree and unparse and stop?
> 
> While the current name is OK now, when the rest of the pipeline (codegen, linking etc) is there will it be an issue?
> Does this flag actually mean, parse and construct the parse-tree and unparse and stop?

Yes

> While the current name is OK now, when the rest of the pipeline (codegen, linking etc) is there will it be an issue?
IIUC, there shouldn't be any. Currently, the `-fdebug-unparse` options have dedicated frontend actions 
* `-fdebug-unparse` runs [[ https://github.com/llvm/llvm-project/blob/main/flang/lib/Frontend/FrontendActions.cpp#L194-L208 | DebugUnparseAction ]]
* `-fdebug-unparse-no-sema` will run `DebugUnparseNoSemaAction` (introduced here)
I don't see why we'd change these actions once codegen or linking are available. That's something that will be implemented separately.

Does this help?




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99645