r263732 - Remove defaulted move ops, the type is zero-cost copyable anyway, so there's no need for specific move ops

[don hinton via cfe-commits]

Dave:

Won't this prevent UnresolvedSet from being movable?

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?

thanks...
don

On Thu, Mar 17, 2016 at 2:28 PM, David Blaikie via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: dblaikie
> Date: Thu Mar 17 13:28:16 2016
> New Revision: 263732
>
> URL: http://llvm.org/viewvc/llvm-project?rev=263732&view=rev
> Log:
> Remove defaulted move ops, the type is zero-cost copyable anyway, so
> there's no need for specific move ops
>
> (addresses MSVC build error, since MSVC 2013 can't generate default move
> ops)
>
> Modified:
>     cfe/trunk/include/clang/AST/UnresolvedSet.h
>
> Modified: cfe/trunk/include/clang/AST/UnresolvedSet.h
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/UnresolvedSet.h?rev=263732&r1=263731&r2=263732&view=diff
>
> ==============================================================================
> --- cfe/trunk/include/clang/AST/UnresolvedSet.h (original)
> +++ cfe/trunk/include/clang/AST/UnresolvedSet.h Thu Mar 17 13:28:16 2016
> @@ -61,9 +61,7 @@ private:
>    template <unsigned N> friend class UnresolvedSet;
>    UnresolvedSetImpl() = default;
>    UnresolvedSetImpl(const UnresolvedSetImpl &) = default;
> -  UnresolvedSetImpl(UnresolvedSetImpl &&) = default;
>    UnresolvedSetImpl &operator=(const UnresolvedSetImpl &) = default;
> -  UnresolvedSetImpl &operator=(UnresolvedSetImpl &&) = default;
>
>  public:
>    // We don't currently support assignment through this iterator, so we
> might
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160317/ecfaa768/attachment-0001.html>