Initialize base class by move constructor instead of copy constructor

[aaron.ballman]

Summary

The move constructor for FullDependence is initializing the base class via a copy constructor instead of a move constructor. This patch adds a default move constructor to the base class, and makes use of it from the derived class instead of half-moving/half-copying the object.

I'm not certain if this was accidentally forgotten as part of r243791, or was purposeful.

~Aaron

[aaron.ballman]

This was commit in r244812.

include/llvm/Analysis/DependenceAnalysis.h

   class Dependence {
   protected:
     Dependence(const Dependence &) = default;
+    Dependence(Dependence &&) = default;
 
   public:
     Dependence(Instruction *Source,

                    unsigned Levels);
 
     FullDependence(FullDependence &&RHS)
-        : Dependence(RHS), Levels(RHS.Levels),
+        : Dependence(std::move(RHS)), Levels(RHS.Levels),
           LoopIndependent(RHS.LoopIndependent), Consistent(RHS.Consistent),
           DV(std::move(RHS.DV)) {}