[PATCH] D78425: [ValueLattice] Add move constructor

[Nikita Popov via Phabricator via llvm-commits]

nikic marked an inline comment as done.
nikic added inline comments.


================
Comment at: include/llvm/Analysis/ValueLattice.h:145
+      break;
+    }
+    Other.Tag = unknown;
----------------
A better implementation of this would be:

    memcpy(this, &Other, sizeof(ValueLatticeElement));
    Other.Tag = unknown;

I'm not sure though whether this is legal C++ or not :)


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

https://reviews.llvm.org/D78425