[luci] Notice comment for bias arg of circle.TransposeConv (#4022)
authorCheongyo Bahk <cg.bahk@gmail.com>
Sun, 30 Aug 2020 21:31:31 +0000 (06:31 +0900)
committerGitHub <noreply@github.com>
Sun, 30 Aug 2020 21:31:31 +0000 (06:31 +0900)
This commit adds notice comment for the `bias()` argument of
`circle.TransposeConv`.

ONE-DCO-1.0-Signed-off-by: Cheongyo Bahk <cg.bahk@gmail.com>

compiler/luci/lang/include/luci/IR/Nodes/CircleTransposeConv.h

index 1972a7b..e355102 100644 (file)
@@ -47,6 +47,18 @@ public:
   loco::Node *outBackprop(void) const { return at(2)->node(); }
   void outBackprop(Node *node) { at(2)->node(node); }
 
+  /**
+   * @note  "bias" is optional. When this node has no conceptual bias, "bias()"
+   *        expected to be `luci::CircleOutputExclude` type.
+   *
+   * <Comment on tflite TRANSPOSE_CONV>
+   *
+   * (Circle node has no dependency on tflite, but just for information on converting)
+   * Before TF v2.3.0, tflite TRANSPOSE_CONV didn't support fused bias as argument.
+   * From TF v2.3.0, tflite TRANSPOSE_CONV supports bias as optional 4th argument.
+   *
+   * Ref: https://github.com/tensorflow/tensorflow/commit/43b8f6e710
+   */
   loco::Node *bias(void) const override { return at(3)->node(); }
   void bias(loco::Node *node) override { at(3)->node(node); }