From 72ebcefb3889b7a17f44a346184354d88744923e Mon Sep 17 00:00:00 2001
From: Jack <32371937+jackzhxng@users.noreply.github.com>
Date: Wed, 23 Apr 2025 14:26:08 -0700
Subject: [PATCH] =?UTF-8?q?Revert=20"Save=20some=20size=20in=20dtype=5Futi?=
 =?UTF-8?q?l=20when=20dtype=20selective=20build=20is=20not=20in=20use?=
 =?UTF-8?q?=E2=80=A6"?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This reverts commit 4fe7367adfaa592c492930025d06a37ba994daff.
---
 kernels/portable/cpu/util/dtype_util.h | 39 ++------------------------
 1 file changed, 2 insertions(+), 37 deletions(-)

diff --git a/kernels/portable/cpu/util/dtype_util.h b/kernels/portable/cpu/util/dtype_util.h
index 167d30611be..2286ca50bee 100644
--- a/kernels/portable/cpu/util/dtype_util.h
+++ b/kernels/portable/cpu/util/dtype_util.h
@@ -228,7 +228,7 @@ enum class SupportedTensorDtypes {
 namespace internal {
 
 template <typename CTYPE_COMPUTE, const char* op_name>
-load_to_compute_fn<CTYPE_COMPUTE> get_load_to_compute_fn_impl(
+load_to_compute_fn<CTYPE_COMPUTE> get_load_to_compute_fn(
     const Tensor& t,
     SupportedTensorDtypes dtypes) {
   switch (dtypes) {
@@ -252,7 +252,7 @@ load_to_compute_fn<CTYPE_COMPUTE> get_load_to_compute_fn_impl(
 }
 
 template <typename CTYPE_COMPUTE, const char* op_name>
-store_compute_to_tensor_fn<CTYPE_COMPUTE> get_store_compute_to_tensor_fn_impl(
+store_compute_to_tensor_fn<CTYPE_COMPUTE> get_store_compute_to_tensor_fn(
     const Tensor& t,
     SupportedTensorDtypes dtypes) {
   switch (dtypes) {
@@ -285,41 +285,6 @@ store_compute_to_tensor_fn<CTYPE_COMPUTE> get_store_compute_to_tensor_fn_impl(
   return nullptr;
 }
 
-#ifndef EXECUTORCH_SELECTIVE_BUILD_DTYPE
-constexpr const char kGenericElementwiseOpName[] = "generic_elementwise_op";
-#endif // EXECUTORCH_SELECTIVE_BUILD_DTYPE
-
-template <typename CTYPE_COMPUTE, const char* op_name>
-load_to_compute_fn<CTYPE_COMPUTE> get_load_to_compute_fn(
-    const Tensor& t,
-    SupportedTensorDtypes dtypes) {
-  // NOTE: Selective build relies on the operator name being passed
-  // here. When it's *not* active, using the same operator name
-  // everywhere saves on size because we don't require a new template
-  // instantiation for every operator.
-  return get_load_to_compute_fn_impl<
-      CTYPE_COMPUTE,
-#ifdef EXECUTORCH_SELECTIVE_BUILD_DTYPE
-      op_name
-#else // EXECUTORCH_SELECTIVE_BUILD_DTYPE
-      kGenericElementwiseOpName
-#endif // EXECUTORCH_SELECTIVE_BUILD_DTYPE
-      >(t, dtypes);
-}
-
-template <typename CTYPE_COMPUTE, const char* op_name>
-store_compute_to_tensor_fn<CTYPE_COMPUTE> get_store_compute_to_tensor_fn(
-    const Tensor& t,
-    SupportedTensorDtypes dtypes) {
-  return get_store_compute_to_tensor_fn_impl<
-      CTYPE_COMPUTE,
-#ifdef EXECUTORCH_SELECTIVE_BUILD_DTYPE
-      op_name
-#else // EXECUTORCH_SELECTIVE_BUILD_DTYPE
-      kGenericElementwiseOpName
-#endif // EXECUTORCH_SELECTIVE_BUILD_DTYPE
-      >(t, dtypes);
-}
 bool check_tensor_dtype(
     const Tensor t,
     SupportedTensorDtypes dtypes,