fix permute copy name (#80)

Summary:
Pull Request resolved: #80

We have a node_visitor for permute already, it was mistakenly named static_transpose after its xnnpack counterpart. permute_copy should not be skipped. It should only be skipped when applied to linear weights.

Instead our linear conversion pass will convert away any permute_copies which are not actually to be executed.

Reviewed By: kirklandsign

Differential Revision: D48488930

fbshipit-source-id: a347d61842a805694c8bddee9fa0efd73ab706d7

Author: mcr229

backends/xnnpack/operators/__init__.py

@@ -30,6 +30,7 @@
     op_minimum,
     op_multiply,
     op_negate,
+    op_permute,
     op_prelu,
     op_quantize_per_tensor,
     op_relu,
@@ -42,7 +43,6 @@
     op_squeeze,
     op_static_constant_pad,
     op_static_resize_bilinear_2d,
-    op_static_transpose,
     op_sub,
     op_to_copy,
 )

backends/xnnpack/operators/op_static_transpose.py renamed to backends/xnnpack/operators/op_permute.py

@@ -20,7 +20,7 @@
 
 
 @register_node_visitor
-class StaticTransposeVisitor(NodeVisitor):
+class PermuteVisitor(NodeVisitor):
     target = "aten.permute_copy.default"
 
     def __init__(self, *args) -> None:

backends/xnnpack/operators/op_skip_ops.py

@@ -113,12 +113,3 @@ class OpSymSizeInt(OpSkipOps):
     """
 
     target = "sym_size.int"
-
-
-@register_node_visitor
-class OpPermuteCopyDefault(OpSkipOps):
-    """
-    do nothing if node is permute_copy.default
-    """
-
-    target = "aten.permute_copy.default"