[all-commits] [llvm/llvm-project] 133172: [NewPM] Use the default AA pipeline by default

[aeubanks via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 13317286f8298eb3bafa9ddebd1c03bef4918948
      https://github.com/llvm/llvm-project/commit/13317286f8298eb3bafa9ddebd1c03bef4918948
  Author: Arthur Eubanks <aeubanks at google.com>
  Date:   2021-11-04 (Thu, 04 Nov 2021)

  Changed paths:
    M clang/lib/CodeGen/BackendUtil.cpp
    M llvm/lib/LTO/LTOBackend.cpp
    M llvm/lib/LTO/ThinLTOCodeGenerator.cpp
    M llvm/lib/Passes/PassBuilder.cpp
    M llvm/tools/llvm-opt-fuzzer/llvm-opt-fuzzer.cpp

  Log Message:
  -----------
  [NewPM] Use the default AA pipeline by default

We almost always want to use the default AA pipeline. It's very easy for
users of PassBuilder to forget to customize the AAManager to use the
default AA pipeline (for example, the NewPM C API forgets to do this).

If somebody wants a custom AA pipeline, similar to what is being done
now with the default AA pipeline registration, they can

  FAM.registerPass([&] { return std::move(MyAA); });

before calling

  PB.registerFunctionAnalyses(FAM);

For example, LTOBackend.cpp and NewPMDriver.cpp do this.

Reviewed By: asbirlea

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