[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