[PATCH] D98191: [flang][driver] Add support for `-fdebug-dump-symbols-sources`

Andrzej Warzynski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 8 09:06:00 PST 2021


awarzynski created this revision.
Herald added a reviewer: sscalpone.
Herald added subscribers: jansvoboda11, dang.
awarzynski requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

Note that the original spelling in `f18` is `-fget-symbols-sources`. The
new driver will only support the new spelling, which is also added to
`f18` as an alias for the existing one.

Tests are updated to use the new spelling and to use `%flang_fc1`
instead of `%f18`. This way both the new and the current driver are
tested.

As:

- both `-fsyntax-only` and `-fdebug-dump-symbols-sources` are action flags, and
- the new driver will only consider the right-most action flag,

`RUN` lines in tests are updated so that the tests work with both
`f18` (requires both flags) and `flang-new` (only considers the last
action flag).


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D98191

Files:
  clang/include/clang/Driver/Options.td
  flang/include/flang/Frontend/FrontendActions.h
  flang/include/flang/Frontend/FrontendOptions.h
  flang/lib/Frontend/CompilerInvocation.cpp
  flang/lib/Frontend/FrontendActions.cpp
  flang/lib/FrontendTool/ExecuteCompilerInvocation.cpp
  flang/test/Flang-Driver/driver-help.f90
  flang/test/Semantics/getsymbols01.f90
  flang/test/Semantics/getsymbols02.f90
  flang/test/Semantics/getsymbols03-a.f90
  flang/test/Semantics/getsymbols04.f90
  flang/test/Semantics/getsymbols05.f90
  flang/tools/f18/f18.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98191.329043.patch
Type: text/x-patch
Size: 8094 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210308/305f7050/attachment-0001.bin>


More information about the cfe-commits mailing list