[PATCH] D116660: [LoopFlatten] Update MemorySSA state

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 5 06:04:50 PST 2022


SjoerdMeijer added a comment.

Ah, forgot to mention tests. Without the update `MSSAU->removeEdge()`, I was expecting `AR.MSSA->verifyMemorySSA()` to run in an assert (with the expensive checks enabled in my build), but somewhat surprisingly to me it didn't.
So not sure yet if we need tests for this, but am looking into this.


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

https://reviews.llvm.org/D116660



More information about the llvm-commits mailing list