From: Dmitry Kurtaev <dmitry.kurtaev+github@gmail.com>
Date: Tue, 26 Dec 2017 13:49:33 +0000 (+0300)
Subject: Limit Concat layer optimization
X-Git-Tag: accepted/tizen/6.0/unified/20201030.111113~236^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=70c605a03d02c41000d9338f428834426c49aa05;p=platform%2Fupstream%2Fopencv.git

Limit Concat layer optimization
---

diff --git a/modules/dnn/src/dnn.cpp b/modules/dnn/src/dnn.cpp
index b9360b1..32b09aa 100644
--- a/modules/dnn/src/dnn.cpp
+++ b/modules/dnn/src/dnn.cpp
@@ -1398,7 +1398,8 @@ struct Net::Impl
                         LayerPin pin = ld.inputBlobsId[i];
                         LayerData* inp_i_data = &layers[pin.lid];
                         while(inp_i_data->skipFlags[DNN_BACKEND_DEFAULT] &&
-                              inp_i_data->inputBlobsId.size() == 1)
+                              inp_i_data->inputBlobsId.size() == 1 &&
+                              inp_i_data->consumers.size() == 1)
                         {
                             pin = inp_i_data->inputBlobsId[0];
                             inp_i_data = &layers[pin.lid];
@@ -1428,15 +1429,11 @@ struct Net::Impl
                             Mat output_slice = output(chrange);
                             Mat& curr_output = inp_i_data->outputBlobs[pin.oid];
                             CV_Assert(output_slice.isContinuous() && output_slice.size == curr_output.size);
+                            Mat* oldPtr = &curr_output;
                             curr_output = output_slice;
-
-                            pin = ld.inputBlobsId[i];
-                            inp_i_data = &layers[pin.lid];
-                            for (int j = 0; j < inp_i_data->consumers.size(); ++j)
-                            {
-                                LayerPin consumer = inp_i_data->consumers[j];
-                                layers[consumer.lid].inputBlobs[consumer.oid] = &curr_output;
-                            }
+                            // Layers that refer old input Mat will refer to the
+                            // new data but the same Mat object.
+                            CV_Assert(curr_output.data == output_slice.data, oldPtr == &curr_output);
                         }
                         ld.skipFlags[DNN_BACKEND_DEFAULT] = true;
                         printf_(("\toptimized out Concat layer %s\n", concatLayer->name.c_str()));
diff --git a/modules/dnn/test/test_layers.cpp b/modules/dnn/test/test_layers.cpp
index 670ccc2..c55e7bc 100644
--- a/modules/dnn/test/test_layers.cpp
+++ b/modules/dnn/test/test_layers.cpp
@@ -314,6 +314,7 @@ TEST(Layer_Test_Fused_Concat, Accuracy)
     //
 
     testLayerUsingCaffeModels("layer_concat_optim", DNN_TARGET_CPU, true, false);
+    testLayerUsingCaffeModels("layer_concat_shared_input", DNN_TARGET_CPU, true, false);
 }
 
 TEST(Layer_Test_Eltwise, Accuracy)