[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