[all-commits] [llvm/llvm-project] ceec43: [AlwaysInliner] Enable call site inlining to make ...

Dávid Bolvanský via All-commits all-commits at lists.llvm.org
Tue Jan 25 13:56:41 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ceec4383681c42bfd3d06a6913ce7554bea160b0
      https://github.com/llvm/llvm-project/commit/ceec4383681c42bfd3d06a6913ce7554bea160b0
  Author: Dávid Bolvanský <david.bolvansky at gmail.com>
  Date:   2022-01-25 (Tue, 25 Jan 2022)

  Changed paths:
    M clang/test/CodeGen/flatten.c
    M clang/test/CodeGenCXX/flatten.cpp
    M llvm/lib/Transforms/IPO/AlwaysInliner.cpp
    M llvm/test/Transforms/Coroutines/coro-retcon-once-private.ll
    M llvm/test/Transforms/Inline/always-inline.ll

  Log Message:
  -----------
  [AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)

Problem: Migration to new PM broke flatten attribute.

This is one use case why LLVM should support inlining call-site with alwaysinline.  The flatten attribute is nowdays broken, so we should either land patch like this one or remove everything related to  flatten attribute from Clang.

Second use case is something like "per call site inlining intrinsics" to control inlining even more; mentioned in
https://lists.llvm.org/pipermail/cfe-dev/2018-September/059232.html

Fixes https://github.com/llvm/llvm-project/issues/53360

Reviewed By: aeubanks

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




More information about the All-commits mailing list