Tidy fix

src/include/migraphx/op/binary.hpp

@@ -67,12 +67,10 @@ struct binary : op_name<Derived>
             .same_dims();
         auto s0 = inputs.at(0);
         auto s1 = inputs.at(1);
-        if(s0.dynamic() and s1.dynamic() and s0 == s1)
-        {
-            return s0;
-        }
-        else if(s0.dynamic() or s1.dynamic())
+        if(s0.dynamic() or s1.dynamic())
         {
+            if(s0 == s1)
+                return s0;
             MIGRAPHX_THROW("BINARY: " + point_function() + ": fixed-dyn shape for inputs");
         }
         else if(s0 == s1 and s0.packed())

src/shape.cpp

@@ -507,7 +507,7 @@ bool shape::dynamic_dimension::has_optimal() const { return opt != 0; }
 bool operator==(const shape::dynamic_dimension& x, const shape::dynamic_dimension& y)
 {
     // don't check opt if both are fixed
-    bool check_opt = (x.is_fixed() and y.is_fixed()) ? false : true;
+    bool check_opt = not(x.is_fixed() and y.is_fixed());
     return (x.min == y.min and x.max == y.max and (check_opt ? x.opt == y.opt : true));
 }