[PATCH] D33056: [NewGVN] When a store becomes the leader, update the memory leader for the class

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

While looking at something completely unrelated I noticed:

  bool isDead() const {
    // If it's both dead from a value perspective, and dead from a memory
    // perspective, it's really dead.
    return empty() && memory_empty();
  }

shouldn't this also do `&& StoreCount == 0` ?


https://reviews.llvm.org/D33056