[PATCH] D51015: [MemorySSA] Make the Updater movable.

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added inline comments.


================
Comment at: include/llvm/Analysis/MemorySSAUpdater.h:73
   MemorySSAUpdater(MemorySSA *MSSA) : MSSA(MSSA) {}
+  MemorySSAUpdater(MemorySSAUpdater &&MSSAU) : MSSA(MSSAU.MSSA) {}
+  MemorySSAUpdater& operator=(MemorySSAUpdater&& MSSAU) {
----------------
george.burgess.iv wrote:
> Can we `= default;`? I'd imagine the associated Small* data structures are really cheap to move/etc. if they're empty.
> 
> If not, that's sad, but OK, this works. :)
We shouldn't need to write it at all if that worked....

I think the issue is the SmallSet of AssertingVH's?


Repository:
  rL LLVM

https://reviews.llvm.org/D51015