COMPMID-3953: Move assignment operator failures on android
authorMichalis Spyrou <michalis.spyrou@arm.com>
Wed, 4 Nov 2020 18:55:34 +0000 (18:55 +0000)
committerMichalis Spyrou <michalis.spyrou@arm.com>
Wed, 4 Nov 2020 20:16:12 +0000 (20:16 +0000)
Signed-off-by: Michalis Spyrou <michalis.spyrou@arm.com>
Change-Id: Id37d90e29749c50eb58084ae9a1ef78e84dbdcd7
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/4326
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
arm_compute/runtime/NEON/functions/NEDeconvolutionLayer.h
arm_compute/runtime/NEON/functions/NEFFTConvolutionLayer.h
arm_compute/runtime/NEON/functions/NEFullyConnectedLayer.h
arm_compute/runtime/NEON/functions/NEGEMMConvolutionLayer.h
arm_compute/runtime/NEON/functions/NEHOGMultiDetection.h
arm_compute/runtime/NEON/functions/NELSTMLayerQuantized.h
arm_compute/runtime/NEON/functions/NELocallyConnectedLayer.h
arm_compute/runtime/NEON/functions/NEQLSTMLayer.h
arm_compute/runtime/NEON/functions/NERNNLayer.h

index 378fce70b3889efc5d8b2b86c0f2f12457a2b05f..97b1a47f64dbc466e9f5e4c4d70a8480dcb7d31e 100644 (file)
@@ -82,10 +82,10 @@ public:
     NEDeconvolutionLayer(const NEDeconvolutionLayer &) = delete;
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEDeconvolutionLayer &operator=(const NEDeconvolutionLayer &) = delete;
-    /** Allow instances of this class to be moved */
-    NEDeconvolutionLayer(NEDeconvolutionLayer &&) = default;
-    /** Allow instances of this class to be moved */
-    NEDeconvolutionLayer &operator=(NEDeconvolutionLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NEDeconvolutionLayer(NEDeconvolutionLayer &&) = delete;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NEDeconvolutionLayer &operator=(NEDeconvolutionLayer &&) = delete;
     /** Default destructor */
     virtual ~NEDeconvolutionLayer() = default;
 
index b3e98fc2d61e48291126a07b749ffc29f1e08639..37750e243bc69b7dc0dbef712957cc6351e72cac 100644 (file)
@@ -63,12 +63,12 @@ public:
     NEFFTConvolutionLayer(std::shared_ptr<IMemoryManager> memory_manager = nullptr);
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEFFTConvolutionLayer(const NEFFTConvolutionLayer &) = delete;
-    /** Default move constructor */
-    NEFFTConvolutionLayer(NEFFTConvolutionLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains non movable objects) */
+    NEFFTConvolutionLayer(NEFFTConvolutionLayer &&) = delete;
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEFFTConvolutionLayer &operator=(const NEFFTConvolutionLayer &) = delete;
-    /** Default move assignment operator */
-    NEFFTConvolutionLayer &operator=(NEFFTConvolutionLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains non movable objects) */
+    NEFFTConvolutionLayer &operator=(NEFFTConvolutionLayer &&) = delete;
     /** Default destructor */
     ~NEFFTConvolutionLayer();
     /** Set the input and output tensors.
index 3ab3d8126230b50c2bc4521905742a08a33e3de1..0a7748a94bf8f0c53089256ea904b8ae16af4993 100644 (file)
@@ -127,12 +127,12 @@ public:
     NEFullyConnectedLayer(std::shared_ptr<IMemoryManager> memory_manager = nullptr, IWeightsManager *weights_manager = nullptr);
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEFullyConnectedLayer(const NEFullyConnectedLayer &) = delete;
-    /** Default move constructor */
-    NEFullyConnectedLayer(NEFullyConnectedLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NEFullyConnectedLayer(NEFullyConnectedLayer &&) = delete;
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEFullyConnectedLayer &operator=(const NEFullyConnectedLayer &) = delete;
-    /** Default move assignment operator */
-    NEFullyConnectedLayer &operator=(NEFullyConnectedLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NEFullyConnectedLayer &operator=(NEFullyConnectedLayer &&) = delete;
     /** Default destructor */
     ~NEFullyConnectedLayer();
     /** Set the input and output tensors.
index 6bcf56fb0bfe8679c3e0bc20f0dbbc615ebfa7ac..59d83ed68de2aca71f185b8a19779c8225591698 100644 (file)
@@ -54,12 +54,12 @@ public:
     NEConvolutionLayerReshapeWeights();
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEConvolutionLayerReshapeWeights(const NEConvolutionLayerReshapeWeights &) = delete;
-    /** Default move constructor */
-    NEConvolutionLayerReshapeWeights(NEConvolutionLayerReshapeWeights &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains non movable objects) */
+    NEConvolutionLayerReshapeWeights(NEConvolutionLayerReshapeWeights &&) = delete;
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEConvolutionLayerReshapeWeights &operator=(const NEConvolutionLayerReshapeWeights &) = delete;
-    /** Default move assignment operator */
-    NEConvolutionLayerReshapeWeights &operator=(NEConvolutionLayerReshapeWeights &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains non movable objects) */
+    NEConvolutionLayerReshapeWeights &operator=(NEConvolutionLayerReshapeWeights &&) = delete;
     /** Default destructor */
     ~NEConvolutionLayerReshapeWeights();
     /** Set the input and output tensors.
@@ -167,12 +167,12 @@ public:
     NEGEMMConvolutionLayer(const std::shared_ptr<IMemoryManager> &memory_manager = nullptr, IWeightsManager *weights_manager = nullptr);
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEGEMMConvolutionLayer(const NEGEMMConvolutionLayer &) = delete;
-    /** Default move constructor */
-    NEGEMMConvolutionLayer(NEGEMMConvolutionLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains non movable objects) */
+    NEGEMMConvolutionLayer(NEGEMMConvolutionLayer &&) = delete;
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEGEMMConvolutionLayer &operator=(const NEGEMMConvolutionLayer &) = delete;
-    /** Default move assignment operator */
-    NEGEMMConvolutionLayer &operator=(NEGEMMConvolutionLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains non movable objects) */
+    NEGEMMConvolutionLayer &operator=(NEGEMMConvolutionLayer &&) = delete;
     /** Default destructor */
     ~NEGEMMConvolutionLayer();
     /** Set the input and output tensors.
index 0fb3edd490045c63c57c9185763bb80a2872ffb9..f273e55e77edda8c011f3230d23e2fd39d1fe982 100644 (file)
@@ -62,12 +62,12 @@ public:
     NEHOGMultiDetection(std::shared_ptr<IMemoryManager> memory_manager = nullptr);
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEHOGMultiDetection(const NEHOGMultiDetection &) = delete;
-    /** Default move constructor */
-    NEHOGMultiDetection(NEHOGMultiDetection &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NEHOGMultiDetection(NEHOGMultiDetection &&) = delete;
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEHOGMultiDetection &operator=(const NEHOGMultiDetection &) = delete;
-    /** Default move assignment operator */
-    NEHOGMultiDetection &operator=(NEHOGMultiDetection &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NEHOGMultiDetection &operator=(NEHOGMultiDetection &&) = delete;
     /** Default destructor */
     ~NEHOGMultiDetection();
     /** Initialise the function's source, destination, detection window strides, border mode, threshold and non-maxima suppression
index 39fafef773c27de6eec20315c8e400efc9bd5fc8..a354a4df7b39a25818095bc094026c57c9d1aa84 100644 (file)
@@ -67,12 +67,12 @@ public:
     NELSTMLayerQuantized(std::shared_ptr<IMemoryManager> memory_manager = nullptr);
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NELSTMLayerQuantized(const NELSTMLayerQuantized &) = delete;
-    /** Default move constructor */
-    NELSTMLayerQuantized(NELSTMLayerQuantized &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NELSTMLayerQuantized(NELSTMLayerQuantized &&) = delete;
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NELSTMLayerQuantized &operator=(const NELSTMLayerQuantized &) = delete;
-    /** Default move assignment operator */
-    NELSTMLayerQuantized &operator=(NELSTMLayerQuantized &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NELSTMLayerQuantized &operator=(NELSTMLayerQuantized &&) = delete;
     /** Default destructor */
     ~NELSTMLayerQuantized();
     /** Initialize function's tensors.
index e9f3e934748fc859d68e0249527528f188be4d2a..86e630013025b7f3e233f41f574a5c936944cb8d 100644 (file)
@@ -55,12 +55,12 @@ public:
     NELocallyConnectedLayer(std::shared_ptr<IMemoryManager> memory_manager = nullptr);
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NELocallyConnectedLayer(const NELocallyConnectedLayer &) = delete;
-    /** Default move constructor */
-    NELocallyConnectedLayer(NELocallyConnectedLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NELocallyConnectedLayer(NELocallyConnectedLayer &&) = delete;
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NELocallyConnectedLayer &operator=(const NELocallyConnectedLayer &) = delete;
-    /** Default move assignment operator */
-    NELocallyConnectedLayer &operator=(NELocallyConnectedLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NELocallyConnectedLayer &operator=(NELocallyConnectedLayer &&) = delete;
     /** Default destructor */
     ~NELocallyConnectedLayer();
     /** Set the input and output tensors.
index 17ad5a354b9fbee1de71d123fb3f89f46bb1d9cc..fcabc1d0c4c09d0de53d5b6df478cf59cda270d7 100644 (file)
@@ -67,12 +67,12 @@ public:
     NEQLSTMLayer(std::shared_ptr<IMemoryManager> memory_manager = nullptr);
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEQLSTMLayer(const NEQLSTMLayer &) = delete;
-    /** Default move constructor */
-    NEQLSTMLayer(NEQLSTMLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NEQLSTMLayer(NEQLSTMLayer &&) = delete;
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NEQLSTMLayer &operator=(const NEQLSTMLayer &) = delete;
-    /** Default move assignment operator */
-    NEQLSTMLayer &operator=(NEQLSTMLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NEQLSTMLayer &operator=(NEQLSTMLayer &&) = delete;
     /** Default destructor */
     ~NEQLSTMLayer();
     /** Initialize function's tensors.
index 74fdc59af6da6fa0924cc273602f87741ef4bdf0..c42b303a89b3574cee19189d682a4cffddc2cb7c 100644 (file)
@@ -44,12 +44,12 @@ public:
     NERNNLayer(std::shared_ptr<IMemoryManager> memory_manager = nullptr);
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NERNNLayer(const NERNNLayer &) = delete;
-    /** Default move constructor */
-    NERNNLayer(NERNNLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NERNNLayer(NERNNLayer &&) = delete;
     /** Prevent instances of this class from being copied (As this class contains pointers) */
     NERNNLayer &operator=(const NERNNLayer &) = delete;
-    /** Default move assignment operator */
-    NERNNLayer &operator=(NERNNLayer &&) = default;
+    /** Prevent instances of this class from being moved (As this class contains pointers) */
+    NERNNLayer &operator=(NERNNLayer &&) = delete;
     /** Default destructor */
     ~NERNNLayer();
     /** Initialize the function