[PATCH] D92657: Fix interaction between clang and some inline builtins from glibc under _FORTIFY_SOURCE
serge via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 7 09:17:26 PST 2020
serge-sans-paille marked 2 inline comments as done.
serge-sans-paille added inline comments.
================
Comment at: llvm/lib/Analysis/InlineCost.cpp:2452-2455
Function *Callee = Call->getCalledFunction();
+ // Disallow recursive calls.
if (&F == Callee)
return InlineResult::failure("recursive call");
----------------
nickdesaulniers wrote:
> shouldn't we do something here with `RecursiveIsViable`?
Updated to match the change at clang level.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92657/new/
https://reviews.llvm.org/D92657
More information about the llvm-commits
mailing list