[PATCH] D50906: Update MemorySSA in LoopInstSimplify.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 20 16:11:30 PDT 2018


chandlerc added a comment.

In https://reviews.llvm.org/D50906#1206772, @asbirlea wrote:

> My bad, previous update won't work, updater needs to be allocated. Please comment on the use of make_unique (= operator in Optional does an std::move).


Can we make MemorySSAUpdater movable? This seems likely to be a pretty common pattern...


Repository:
  rL LLVM

https://reviews.llvm.org/D50906





More information about the llvm-commits mailing list