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

[Andrzej Warzynski via Phabricator via llvm-commits]

awarzynski created this revision.
awarzynski requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

The following _action_ options are always used with `-fsyntax-only`
(also an _action_ option):

- -fdebug-dump-symbols
- -fdebug-dump-parse-tree

This patch makes the above options imply `-fsyntax-only`.

>From the perspective of `f18` this change saves typing and is otherwise
a non-functional change. But it will simplify things in the new driver,
`flang-new`, in which only the last action option is taken into account
and executed. In other words, the following would only run
`-fsyntax-only`:

  flang-new -fdebug-dump-symbols -fsyntax-only <input>

whereas this would only run `-fdebug-dump-symbols`:

  flang-new -fsyntax-only -fdebug-dump-symbols <input>


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D96528

Files:
  flang/test/Evaluate/test_folding.sh
  flang/test/Semantics/data05.f90
  flang/test/Semantics/data08.f90
  flang/test/Semantics/offsets01.f90
  flang/test/Semantics/offsets02.f90
  flang/test/Semantics/offsets03.f90
  flang/test/Semantics/oldparam01.f90
  flang/test/Semantics/oldparam02.f90
  flang/test/Semantics/resolve100.f90
  flang/test/Semantics/rewrite01.f90
  flang/test/Semantics/typeinfo01.f90
  flang/tools/f18/f18.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D96528.323073.patch
Type: text/x-patch
Size: 5445 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210211/b9bfa036/attachment.bin>