[all-commits] [llvm/llvm-project] f6b514: [AlwaysInliner] Emit inline remark only when succe...

[Ellis Hoag via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f6b5142ac2e4c49b9eff4c22d699f464a6812feb
      https://github.com/llvm/llvm-project/commit/f6b5142ac2e4c49b9eff4c22d699f464a6812feb
  Author: Ellis Hoag <ellis.sparky.hoag at gmail.com>
  Date:   2022-03-17 (Thu, 17 Mar 2022)

  Changed paths:
    M llvm/lib/Transforms/IPO/AlwaysInliner.cpp
    M llvm/test/Transforms/Inline/always-inline-remark.ll

  Log Message:
  -----------
  [AlwaysInliner] Emit inline remark only when successful

Failures in `InlineFunction()` are caught after D121722, but `emitInlinedIntoBasedOnCost()` should only be called when inlining is successful. This also removes an unnecessary call to `shouldInline()` which always returned `InlineCost::getAlways()`.

Reviewed By: kyulee, nikic

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