Delete some unused/redundant code.

PiperOrigin-RevId: 578003516
jax-ml · Oct 31, 2023 · 3d74b01 · 3d74b01
1 parent 171155f
commit 3d74b01
Showing 1 changed file with 4 additions and 21 deletions.
diff --git a/ml_dtypes/include/float8.h b/ml_dtypes/include/float8.h
@@ -882,18 +882,10 @@ struct IdentityConversion {
   static EIGEN_DEVICE_FUNC inline Scalar run(Scalar from) { return from; }
 };
 
-template <typename Scalar>
-struct ConvertImpl<Scalar, Scalar, /*kSaturate=*/false, /*kTruncate=*/false,
-                   /*EnableIf=*/void> : public IdentityConversion<Scalar> {};
-template <typename Scalar>
-struct ConvertImpl<Scalar, Scalar, /*kSaturate=*/false, /*kTruncate=*/true,
-                   /*EnableIf=*/void> : public IdentityConversion<Scalar> {};
-template <typename Scalar>
-struct ConvertImpl<Scalar, Scalar, /*kSaturate=*/true, /*kTruncate=*/false,
-                   /*EnableIf=*/void> : public IdentityConversion<Scalar> {};
-template <typename Scalar>
-struct ConvertImpl<Scalar, Scalar, /*kSaturate=*/true, /*kTruncate=*/true,
-                   /*EnableIf=*/void> : public IdentityConversion<Scalar> {};
+template <typename Scalar, bool kSaturate, bool kTruncate>
+struct ConvertImpl<Scalar, Scalar, /*kSaturate=*/kSaturate,
+                   /*kTruncate=*/kTruncate>
+    : public IdentityConversion<Scalar> {};
 
 template <typename Float>
 struct TraitsBase {
@@ -1221,15 +1213,6 @@ struct ConvertImpl<Eigen::half, float8_e5m2, kSaturate, kTruncate> {
   }
 };
 
-template <>
-struct ConvertImpl<float8_e5m2, Eigen::half, /*kSaturate=*/false,
-                   /*kTruncate=*/false> {
-  static EIGEN_DEVICE_FUNC inline Eigen::half run(float8_e5m2 from) {
-    return Eigen::numext::bit_cast<Eigen::half>(
-        static_cast<uint16_t>(static_cast<uint16_t>(from.rep()) << 8));
-  }
-};
-
 // Direct casts of e5m2 to Eigen::half simply shifts bits over.
 template <bool kSaturate, bool kTruncate>
 struct ConvertImpl<float8_e5m2, Eigen::half, kSaturate, kTruncate> {