Add support for the following f18 options:
- -fdebug-module-writer
We also add a new Setter for the -J/module-dir option.
Paths
| Differential D96875
[flang][driver] Add -fdebug-module-writer option ClosedPublic Authored by arnamoy10 on Feb 17 2021, 9:28 AM.
Details Summary Add support for the following f18 options:
We also add a new Setter for the -J/module-dir option.
Diff Detail Event TimelineComment Actions Added some setters to work with. Modified the test case to include flang-new -fc1 as well.
Comment Actions Thank you for submitting this @arnamoy10 ! Comments inline.
Comment Actions Minor changes as per comments, also updated commit message to include description of changes related to Setters. Comment Actions LGTM, thank you! (I believe that you've addressed all of Tim's comments too, but please wait a bit before merging in case I missed something) This revision is now accepted and ready to land.Mar 5 2021, 9:58 AM
Revision Contents
Diff 327230 clang/include/clang/Driver/Options.td
clang/lib/Driver/ToolChains/Flang.cpp
flang/include/flang/Frontend/CompilerInvocation.h
flang/lib/Frontend/CompilerInvocation.cpp
flang/lib/Frontend/FrontendActions.cpp
flang/test/Flang-Driver/driver-help.f90
flang/test/Semantics/mod-file-rewriter.f90
|
IMO this should be a frontend-only option. Compiler end-users are unlikely to need this in their regular workflows, right? I recommend moving it here.