[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
Fri Jun 12 18:15:30 PDT 2020


aemerson updated this revision to Diff 270555.
aemerson retitled this revision from "Don't hoist very large static allocas to the entry block during inlining" to "Don't inline dynamic allocas that simplify to huge static allocas.".
aemerson edited the summary of this revision.
aemerson added a comment.

Instead of avoiding hoisting, avoid inlining altogether.


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/lib/Transforms/Utils/InlineFunction.cpp
  llvm/test/Transforms/Inline/dynamic-alloca-simplified-large.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81765.270555.patch
Type: text/x-patch
Size: 4671 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200613/b84faae0/attachment.bin>


More information about the llvm-commits mailing list