[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (PR53360)
Dávid Bolvanský via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Jan 22 12:47:04 PST 2022
xbolva00 added inline comments.
================
Comment at: llvm/lib/Transforms/IPO/AlwaysInliner.cpp:95
- // Remember to try and delete this function afterward. This both avoids
- // re-walking the rest of the module and avoids dealing with any iterator
- // invalidation issues while deleting functions.
- InlinedFunctions.push_back(&F);
+ if (F.hasFnAttribute(Attribute::AlwaysInline))
+ // Remember to try and delete this function afterward. This both avoids
----------------
avoid DCE of internal functions without always_inline; check test pr2945 in always-inline.ll
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117965/new/
https://reviews.llvm.org/D117965
More information about the cfe-commits
mailing list