[all-commits] [llvm/llvm-project] d52f50: [NewPM] Use parameterized syntax for a couple of m...

[Björn Pettersson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d52f506192336e75296a951d8ee02fff3c28571d
      https://github.com/llvm/llvm-project/commit/d52f506192336e75296a951d8ee02fff3c28571d
  Author: Bjorn Pettersson <bjorn.a.pettersson at ericsson.com>
  Date:   2021-08-20 (Fri, 20 Aug 2021)

  Changed paths:
    M llvm/lib/Passes/PassBuilder.cpp
    M llvm/lib/Passes/PassRegistry.def
    M llvm/test/Analysis/MemorySSA/invalidate.ll
    M llvm/test/Transforms/EarlyCSE/AArch64/intrinsics.ll
    M llvm/test/Transforms/EarlyCSE/memoryssa.ll
    M llvm/test/Transforms/EntryExitInstrumenter/debug-info.ll
    M llvm/test/Transforms/EntryExitInstrumenter/mcount.ll
    M llvm/test/Transforms/LowerMatrixIntrinsics/multiply-minimal.ll
    M llvm/tools/opt/NewPMDriver.cpp

  Log Message:
  -----------
  [NewPM] Use parameterized syntax for a couple of more passes

A couple of passes that are parameterized in new-PM used different
pass names (in cmd line interface) while using the same pass class
name. This patch updates the PassRegistry to model pass parameters
more properly using PASS_WITH_PARAMS.

Reason for the change is to ensure that we have a 1-1 mapping
between class name and pass name (when disregarding the params).
With a 1-1 mapping it is more obvious which pass name to use in
options such as -debug-only, -print-after etc.

The opt -passes syntax is changed for the following passes:
  early-cse-memssa => early-cse<memssa>
  post-inline-ee-instrument => ee-instrument<post-inline>
  loop-extract-single => loop-extract<single>
  lower-matrix-intrinsics-minimal => lower-matrix-intrinsics<minimal>

This patch is not updating pass names in docs/Passes.rst. Not quite
sure what the status is for that document (e.g. when it comes to
listing pass paramters). It is only loop-extract-single that is
mentioned in Passes.rst today, out of the passes mentioned above.

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