[PATCH] D101391: [DSE] Eliminate zero memset after calloc

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 27 13:32:21 PDT 2021


xbolva00 added a comment.

In D101391#2720758 <https://reviews.llvm.org/D101391#2720758>, @nikic wrote:

> I think it would be good to add a negative test where there is a store to the calloc in the meantime.

zero_memset_and_store_after_calloc?
store is killed by memset and then we have a calloc with memset.

In D101391#2720758 <https://reviews.llvm.org/D101391#2720758>, @nikic wrote:

> I think it would be good to add a negative test where there is a store to the calloc in the meantime.

Yes, good idea. Prepared partial_zero_memset_and_store_after_calloc and zero_memset_and_store_with_dyn_index_after_calloc.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101391



More information about the llvm-commits mailing list