[all-commits] [llvm/llvm-project] d06e94: [flang][driver] Add PrescanAction frontend action ...

Andrzej WarzyƄski via All-commits all-commits at lists.llvm.org
Thu Feb 4 06:24:18 PST 2021


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

  Changed paths:
    M flang/include/flang/Frontend/FrontendAction.h
    M flang/include/flang/Frontend/FrontendActions.h
    M flang/lib/Frontend/CompilerInstance.cpp
    M flang/lib/Frontend/FrontendAction.cpp
    M flang/lib/Frontend/FrontendActions.cpp

  Log Message:
  -----------
  [flang][driver] Add PrescanAction frontend action (nfc)

This new action encapsulates all actions that require the prescanner to
be run before proceeding with other processing. By adding this new
action, we are better equipped to control which actions _do_ run the
prescanner and which _do not_.

The following actions that require the prescanner are refactored to
inherit from `PrescanAction`:
  * `PrintPreprocessedAction`
  * `ParseSyntaxOnlyAction` .

New virtual method is introduced to facilitate all this:
  * `BeginSourceFileAction`
Like in Clang, this method is run inside `BeginSourceFile`. In other
words, it is invoked before `ExecuteAction` for the corresponding
frontend action is run. This method allows us to:
  * carry out any processing that is always required by the action (e.g.
    run the prescanner)
  * fine tune the settings/options on a file-by-file basis (e.g. to
    decide between fixed-form and free-form based on file extension)

This patch implements non-functional-changes.

Reviewed By: FarisRehman

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




More information about the All-commits mailing list