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

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 15 10:23:39 PDT 2020


reames added a comment.

Just a thought, feel free to consider or ignore.

It occurs to me that the lowering of static alloca is under compiler control.  Maybe we shouldn't look at this as being an inlining problem, but instead look at it as being a lowering problem?  If we have a static alloca which is huge and only used in a rare codepath, maybe we should be converting that to a dynamic alloca if legal for the target?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81765





More information about the llvm-commits mailing list