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 324335 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/Semantics/mod-file-rewriter.f90
|
[nit] The convention here seems to be: no . at the end
[nit] Perhaps a bit shorter: Enable debug messages while writhing module files?