minor additional changes for improving the eliminate contiguous.

src/eliminate_contiguous.cpp

@@ -9,7 +9,7 @@
 namespace migraphx {
 inline namespace MIGRAPHX_INLINE_NS {
 
-bool try_compute_shape(instruction_ref ins, const std::vector<shape>& inputs)
+static bool try_compute_shape(instruction_ref ins, const std::vector<shape>& inputs)
 {
     try
     {
@@ -21,8 +21,10 @@ bool try_compute_shape(instruction_ref ins, const std::vector<shape>& inputs)
         }
 
         auto outputs = ins->outputs();
-        // If the current instruction has no output, it means the last output shape
-        // is non-standard, then we cannot eliminate the contiguous
+        // If the current instruction has no output, it means it is the last
+        // instruction and generates a non-standard output. But for unary
+        // and binary operators, we can still remove it and reshape the output
+        // to be standard since these operator can handle non-standard inputs
         if(outputs.empty())
         {
             return true;
@@ -51,7 +53,7 @@ bool try_compute_shape(instruction_ref ins, const std::vector<shape>& inputs)
     return true;
 }
 
-bool try_compute_shape(instruction_ref ins, const std::vector<instruction_ref>& args)
+static bool try_compute_shape(instruction_ref ins, const std::vector<instruction_ref>& args)
 {
     auto inputs = to_shapes(args);
     return try_compute_shape(ins, inputs);

src/include/migraphx/op/binary.hpp

@@ -21,14 +21,7 @@ struct binary
     shape compute_shape(std::vector<shape> inputs) const
     {
         check_shapes{inputs}.has(2).same_type().same_dims();
-        if(inputs.at(0) == inputs.at(1) and inputs.at(0).packed() and inputs.at(1).packed())
-        {
-            return inputs.at(0);
-        }
-        else
-        {
-            return {inputs.at(0).type(), inputs.at(0).lens()};
-        }
+        return {inputs.at(0).type(), inputs.at(0).lens()};
     }
 };
 

src/include/migraphx/op/unary.hpp

@@ -21,14 +21,7 @@ struct unary
     shape compute_shape(std::vector<shape> inputs) const
     {
         check_shapes{inputs}.has(1);
-        if(inputs.front().packed())
-        {
-            return inputs.at(0);
-        }
-        else
-        {
-            return {inputs.at(0).type(), inputs.at(0).lens()};
-        }
+        return {inputs.at(0).type(), inputs.at(0).lens()};
     }
 };