[all-commits] [llvm/llvm-project] e8049d: [NewPM][Inliner] Move the 'always inliner' case in...

Mircea Trofin via All-commits all-commits at lists.llvm.org
Fri Jan 15 18:35:38 PST 2021


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e8049dc3c8a46ccd75ce2a4f438d695d20feb660
      https://github.com/llvm/llvm-project/commit/e8049dc3c8a46ccd75ce2a4f438d695d20feb660
  Author: Mircea Trofin <mtrofin at google.com>
  Date:   2021-01-15 (Fri, 15 Jan 2021)

  Changed paths:
    M clang/test/CodeGen/thinlto-distributed-newpm.ll
    M clang/test/Frontend/optimization-remark-line-directive.c
    M clang/test/Frontend/optimization-remark-new-pm.c
    M clang/test/Frontend/optimization-remark-with-hotness-new-pm.c
    M clang/test/Frontend/optimization-remark.c
    M llvm/include/llvm/Analysis/InlineAdvisor.h
    M llvm/include/llvm/Analysis/MLInlineAdvisor.h
    M llvm/include/llvm/Analysis/ReplayInlineAdvisor.h
    M llvm/include/llvm/Passes/PassBuilder.h
    M llvm/include/llvm/Transforms/IPO/Inliner.h
    M llvm/lib/Analysis/DevelopmentModeInlineAdvisor.cpp
    M llvm/lib/Analysis/InlineAdvisor.cpp
    M llvm/lib/Analysis/MLInlineAdvisor.cpp
    M llvm/lib/Analysis/ReplayInlineAdvisor.cpp
    M llvm/lib/Passes/PassBuilder.cpp
    M llvm/lib/Passes/PassRegistry.def
    M llvm/lib/Transforms/IPO/Inliner.cpp
    M llvm/test/Other/new-pm-defaults.ll
    M llvm/test/Other/new-pm-lto-defaults.ll
    R llvm/test/Other/new-pm-module-inliner-wrapper.ll
    M llvm/test/Other/new-pm-pgo-preinline.ll
    M llvm/test/Other/new-pm-thinlto-defaults.ll
    M llvm/test/Other/new-pm-thinlto-postlink-pgo-defaults.ll
    M llvm/test/Other/new-pm-thinlto-postlink-samplepgo-defaults.ll
    M llvm/test/Other/new-pm-thinlto-prelink-pgo-defaults.ll
    M llvm/test/Other/new-pm-thinlto-prelink-samplepgo-defaults.ll
    M llvm/test/Transforms/Inline/inline_stats.ll
    M llvm/test/Transforms/Inline/optimization-remarks-with-hotness.ll
    M llvm/test/Transforms/Inline/optimization-remarks.ll
    M llvm/test/Transforms/Inline/pr46945.ll

  Log Message:
  -----------
  [NewPM][Inliner] Move the 'always inliner' case in the same CGSCC pass as 'regular' inliner

Expanding from D94808 - we ensure the same InlineAdvisor is used by both
InlinerPass instances. The notion of mandatory inlining is moved into
the core InlineAdvisor: advisors anyway have to handle that case, so
this change also factors out that a bit better.

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




More information about the All-commits mailing list