[all-commits] [llvm/llvm-project] 548549: [flang][f18] Make -fdebug-dump-{symbols|parse-tree...

[Andrzej Warzyński via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 548549e88ac4a1908206b888be24592e1dbcc38f
      https://github.com/llvm/llvm-project/commit/548549e88ac4a1908206b888be24592e1dbcc38f
  Author: Andrzej Warzynski <andrzej.warzynski at arm.com>
  Date:   2021-02-16 (Tue, 16 Feb 2021)

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

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

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>
```

Differential Revision: https://reviews.llvm.org/D96528