[PATCH] InstCombine: don't drop 'inalloca' in PromoteCastOfAllocation (PR19569)

Hans Wennborg hans at chromium.org
Mon Apr 28 10:49:24 PDT 2014


On Mon, Apr 28, 2014 at 10:34 AM, Chandler Carruth <chandlerc at google.com> wrote:
> LGTM.

r207426

> Is there a way to switch to clone or something to isolate this logic? Not
> necessary for commit, just a nice-to-have.

I'll take a look.

> On Mon, Apr 28, 2014 at 10:28 AM, Hans Wennborg <hans at chromium.org> wrote:
>>
>> (Not using phabricator because I just get exceptions when trying to log
>> in.)
>>
>> This fixes the instcombine vs. inalloca issue in PR19569 and checks in
>> David's test.
>>
>> Thanks,
>> Hans
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>
>



More information about the llvm-commits mailing list