[PATCH] D55050: [LoopSimplifyCFG] Update MemorySSA in terminator folding. PR39783

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 29 22:31:29 PST 2018


asbirlea added a comment.

Based on the corresponding Phi updates, the MemoryPhi updates look correct.

Nit: any chance we could merge pr39783_1.ll  and pr39783_2.ll into a single file and simplify the tests? e.g. can we keep only @1 in the first file and @func76 in the second; clean-up attributes?

Thanks for the patch!


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

https://reviews.llvm.org/D55050





More information about the llvm-commits mailing list