[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