[PATCH] D41719: [InlineCost] Prevent infinite recursion on function pointers

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 8 13:39:42 PST 2018


paquette updated this revision to Diff 128977.
paquette added a comment.

Got rid of the magic 100 for the maximum depth and put in a command line option (-inline-recursion-limit) instead. I tried a few different limits with the knob and found that this test crashes around a recursion depth of 3500 on my machine. A max depth of 2000 seemed like a good middle-ground.


https://reviews.llvm.org/D41719

Files:
  include/llvm/Analysis/InlineCost.h
  lib/Analysis/InlineCost.cpp
  test/Transforms/Inline/inline-functionptr.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41719.128977.patch
Type: text/x-patch
Size: 6745 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180108/b97e7b07/attachment.bin>


More information about the llvm-commits mailing list