Displaying 2 results from an estimated 2 matches for "d23052".
2016 Aug 03
2
RFC: We should stop merging allocas in the inliner
...er Carruth <chandlerc at gmail.com> wrote:
> Seeing no big concerns raised with this direction...
>
> On Sun, Jul 31, 2016 at 9:47 PM Chandler Carruth <chandlerc at gmail.com>
> wrote:
>
>> 1) Add a flag to turn this off.
>>
>
> https://reviews.llvm.org/D23052
>
> Will proceed from there.
>
> 2) Run benchmarks with flag to make sure things are actually working.
>> 3) Change default for the flag, make sure chaos doesn't ensue.
>> 4) Delete the (quite substantial) complexity associated with this and the
>> flag.
>> 5...
2016 Aug 01
16
RFC: We should stop merging allocas in the inliner
Chris added alloca merging in the inliner a looooong time ago, 2009. The
reason he added it was because at the time we didn't do stack coloring and
without it we had serious stack size problems in LLVM.
Since then, a few rather important things have changed:
- We have reasonably powerful stack coloring support in the backend based
on lifetime markers
- Clang (the primary frontend I'm