[PATCH] D81765: Don't inline dynamic allocas that simplify to huge static allocas.

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 24 17:56:45 PDT 2020


aemerson updated this revision to Diff 273194.
aemerson edited the summary of this revision.
aemerson added a comment.
Herald added a subscriber: kristof.beyls.

Add some CHECK lines, add FIXME comment, update description/commit message.

I'm going to commit this now assuming that all feedback has been addressed, let me know if there's anything else needed.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81765/new/

https://reviews.llvm.org/D81765

Files:
  llvm/include/llvm/Analysis/InlineCost.h
  llvm/lib/Analysis/InlineCost.cpp
  llvm/test/Transforms/Inline/dynamic-alloca-simplified-large.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81765.273194.patch
Type: text/x-patch
Size: 7757 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200625/6cdf7db8/attachment.bin>


More information about the llvm-commits mailing list