[PATCH] D29149: NewGVN: Add basic dead store elimination

[Daniel Berlin via Phabricator via llvm-commits]

dberlin added a comment.

In https://reviews.llvm.org/D29149#656853, @efriedma wrote:

> Not sure I would call this dead store elimination; you aren't proving the store is dead, just that it's redundant.


We actually prove both in some cases?
Happy to call it whatever.

Most value numbering implementations call it dead store elimination.

Note that our current DSE is the same in some cases. Some stores it proves dead, some redundant.

See, e.g, eliminateNoopStore


https://reviews.llvm.org/D29149