[PATCH] D62536: Only passes that preserve MemorySSA must mark it as preserved.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 8 01:51:57 PDT 2019


chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

LGTM

Any chance you can add a test that shows one of the problems where verifying MemorySSA fails because we get a "preserved" copy after a loop pass that doesn't actually preserve it?

Fine to submit the patch whenever, and just try ta add a test case when its convenient.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62536





More information about the llvm-commits mailing list