[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