[PATCH] D105729: [AFDO] Merge function attributes after inlining

Kazu Hirata via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 9 14:28:09 PDT 2021


kazu added a comment.

In D105729#2868139 <https://reviews.llvm.org/D105729#2868139>, @wenlei wrote:

> I'm wondering whether it would be more future proof if `AttributeFuncs::mergeAttributesForInlining` is moved inside `InlineFunction`? Currently it's called right after `InlineFunction` for always inline, scc inline, and with this change sample loader inline too.

Great idea!  Let me post a follow-up patch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105729/new/

https://reviews.llvm.org/D105729



More information about the llvm-commits mailing list