[llvm] r278571 - [Inliner] Don't treat inalloca allocas as static

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 16 11:15:04 PDT 2016


On Fri, Aug 12, 2016 at 3:23 PM, Reid Kleckner via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: rnk
> Date: Fri Aug 12 17:23:04 2016
> New Revision: 278571
>
> URL: http://llvm.org/viewvc/llvm-project?rev=278571&view=rev
> Log:
> [Inliner] Don't treat inalloca allocas as static
>
> They aren't static, and moving them to the entry block across something
> else will only result in tears.
>
> Root cause of http://crbug.com/636558.
>
> Added:
>     llvm/trunk/test/Transforms/Inline/inalloca-not-static.ll
> Modified:
>     llvm/trunk/lib/Transforms/Utils/InlineFunction.cpp

This seemed like a nasty miscompile, so I merged it to 3.9 in r278831.


More information about the llvm-commits mailing list