[all-commits] [llvm/llvm-project] eefda6: [flang][driver] Add support for `-fget-symbols-sou...

Andrzej WarzyƄski via All-commits all-commits at lists.llvm.org
Thu Mar 18 07:14:08 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: eefda605fe1701937a8fe5379357e0990ad2fb4e
      https://github.com/llvm/llvm-project/commit/eefda605fe1701937a8fe5379357e0990ad2fb4e
  Author: Andrzej Warzynski <andrzej.warzynski at arm.com>
  Date:   2021-03-18 (Thu, 18 Mar 2021)

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

  Log Message:
  -----------
  [flang][driver] Add support for `-fget-symbols-sources`

Adds support for `-fget-symbols-sources` in the new Flang driver. All
relevant tests are updated to use the new driver when
`FLANG_BUILD_NEW_DRIVER` is set.

`RUN` lines in tests are updated so `-fsyntax-only`
comes before `-fget-symbols-sources`. That's because:
  * both `-fsyntax-only` and `-fget-symbols-sources` are
    action flags, and
  * the new driver, flang-new, will only consider the right-most
    action flag.
In other words, this change is needed so that the tests work with both
`f18` (requires both flags) and `flang-new` (only considers the last
action flag).

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




More information about the All-commits mailing list