[PATCH] Remove dead zero store to calloc initialized memory

Philip Reames listmail at philipreames.com
Tue Jun 10 10:57:23 PDT 2014


ignore last.  I apparently have review comments I didn't get notified about.

Should have checked phabricator and not relied on the commits list notices.

Philip

On 06/10/2014 10:55 AM, Philip Reames wrote:
> ping.
>
> On 05/29/2014 11:53 AM, Philip Reames wrote:
>> (Again, this doesn't appear to have gone to the commits list.  Anyone 
>> have a clue what's I'm doing wrong here?)
>>
>> Remove dead zero store to calloc initialized memory
>>
>> http://reviews.llvm.org/D3942
>>
>> Optimize the following IR:
>>
>> %1  =  tail  call  noalias  i8*  @calloc(i64  1,  i64  4)
>> %2  =  bitcast  i8*  %1  to  i32*
>> ;  This  store  is  dead  and  should  be  removed
>> store  i32  0,  i32*  %2,  align  4
>>
>> Memory returned by calloc is guaranteed to be zero initialized. If 
>> the value being stored is the constant zero (and the store is not 
>> otherwise observable across threads), we can delete the store.
>>
>> Note: I do not have commit rights and will need someone to commit 
>> this change after review.
>>
>> Philip
>>
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140610/51e00895/attachment.html>


More information about the llvm-commits mailing list