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

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


  Branch: refs/heads/arcpatch-D117965
  Home:   https://github.com/llvm/llvm-project
  Commit: 7a61357ea2c3b737ce338452aee058033119fa28
      https://github.com/llvm/llvm-project/commit/7a61357ea2c3b737ce338452aee058033119fa28
  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