[moco-tf] Do not reset input for BiasAdd Canonicalizer (#6470)
author박세희/On-Device Lab(SR)/Principal Engineer/삼성전자 <saehie.park@samsung.com>
Mon, 12 Aug 2019 04:28:10 +0000 (13:28 +0900)
committerGitHub Enterprise <noreply-CODE@samsung.com>
Mon, 12 Aug 2019 04:28:10 +0000 (13:28 +0900)
This will remove reset input for old replaced BiasAdd node in Canonicalization and update the diagram

Signed-off-by: SaeHie Park <saehie.park@samsung.com>
compiler/moco-tf/src/Canonicalization/BiasAddCanonicalizer.cpp

index 936db01..a42891a 100644 (file)
@@ -46,7 +46,8 @@ bool canonicalize_biasadd(loco::Graph *graph, moco::tf::TFBiasAdd *node)
    *                 B -/
    *
    *       After
-   *                    - TFBiasAdd -
+   *                 A -- TFBiasAdd -
+   *                 B -/
    *                 A --------------- TensorBiasAdd - C
    *                 B - BiasEncode -/
    *
@@ -54,7 +55,8 @@ bool canonicalize_biasadd(loco::Graph *graph, moco::tf::TFBiasAdd *node)
    *                 A : value of TFBiasAdd
    *                 B : bias of TFBiasAdd
    *                 C : a node that uses TFBiasAdd as an input
-   *                 TFBiasAdd is disconnected from other nodes
+   *                 TFBiasAdd is disconnected from node C
+   *                 A and B are drawn twice to simplify the diagram
    */
 
   INFO(l) << "TFNodeCanonicalize TFBiasAdd begin";
@@ -93,8 +95,6 @@ bool canonicalize_biasadd(loco::Graph *graph, moco::tf::TFBiasAdd *node)
 
   // replace old with new : about C in above note
   replace(node).with(bias_add);
-  node->value(nullptr);
-  node->bias(nullptr);
 
   INFO(l) << "TFNodeCanonicalize TFBiasAdd done";