[PATCH] D75590: [Attributor] IPO across definition boundary of a function marked alwaysinline
Stefanos Baziotis via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 6 15:29:43 PST 2020
baziotis added inline comments.
================
Comment at: llvm/include/llvm/Transforms/IPO/Attributor.h:1105
+ if (FnScope && FnScope->hasFnAttribute(Attribute::AlwaysInline))
+ if (InfoCache.InlineableFunctions.find(FnScope) ==
+ InfoCache.InlineableFunctions.end())
----------------
Nit: You can use `!InfoCache.InlineableFunctions.count(FnScope)` (it is shorter)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75590/new/
https://reviews.llvm.org/D75590
More information about the llvm-commits
mailing list