<div dir="ltr">Dave:<div><br></div><div>Won't this prevent UnresolvedSet from being movable?</div><div><br></div><div>I was under the impression that all subobjects (base class in this case) had to be movable in order for the compiler to generate implicit move ops.  Is that not the case?</div><div><br></div><div>thanks...</div><div>don</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 17, 2016 at 2:28 PM, David Blaikie via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: dblaikie<br>
Date: Thu Mar 17 13:28:16 2016<br>
New Revision: 263732<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=263732&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=263732&view=rev</a><br>
Log:<br>
Remove defaulted move ops, the type is zero-cost copyable anyway, so there's no need for specific move ops<br>
<br>
(addresses MSVC build error, since MSVC 2013 can't generate default move<br>
ops)<br>
<br>
Modified:<br>
    cfe/trunk/include/clang/AST/UnresolvedSet.h<br>
<br>
Modified: cfe/trunk/include/clang/AST/UnresolvedSet.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/UnresolvedSet.h?rev=263732&r1=263731&r2=263732&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/UnresolvedSet.h?rev=263732&r1=263731&r2=263732&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/include/clang/AST/UnresolvedSet.h (original)<br>
+++ cfe/trunk/include/clang/AST/UnresolvedSet.h Thu Mar 17 13:28:16 2016<br>
@@ -61,9 +61,7 @@ private:<br>
   template <unsigned N> friend class UnresolvedSet;<br>
   UnresolvedSetImpl() = default;<br>
   UnresolvedSetImpl(const UnresolvedSetImpl &) = default;<br>
-  UnresolvedSetImpl(UnresolvedSetImpl &&) = default;<br>
   UnresolvedSetImpl &operator=(const UnresolvedSetImpl &) = default;<br>
-  UnresolvedSetImpl &operator=(UnresolvedSetImpl &&) = default;<br>
<br>
 public:<br>
   // We don't currently support assignment through this iterator, so we might<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div>