Not seeing any benefit to using DTYPE selection in add_mul model

Author: BujSet

examples/selective_build/test_selective_build.sh

@@ -138,7 +138,7 @@ aten,aten::clone.out" \
     rm "./mv2.pte"
 }
 
-test_cmake_select_ops_add_in_yaml() {
+test_cmake_select_ops_add_in_list() {
     echo "Exporting add_mul"
     ${PYTHON_EXECUTABLE} -m examples.portable.scripts.export --model_name="add_mul"
     local example_dir=examples/selective_build

kernels/portable/cpu/selective_build.h

@@ -21,10 +21,22 @@ inline constexpr bool should_include_kernel_dtype(
   executorch::aten::ScalarType scalar_type
 ) {
   return ((executorch::aten::string_view(operator_name).compare("add.out") == 0)
-        && (true))
- || ((executorch::aten::string_view(operator_name).compare("mm.out") == 0)
-        && (true));
+        && (scalar_type == executorch::aten::ScalarType::Float || scalar_type == executorch::aten::ScalarType::Int))
+ || ((executorch::aten::string_view(operator_name).compare("mul.out") == 0)
+        && (scalar_type == executorch::aten::ScalarType::Float));
+// || ((executorch::aten::string_view(operator_name).compare("sub.out") == 0)
+//        && (true));
 }
+
+//inline constexpr bool should_include_kernel_dtype(
+//  const char *operator_name,
+//  executorch::aten::ScalarType scalar_type
+//) {
+//  return ((executorch::aten::string_view(operator_name).compare("add.out") == 0)
+//        && (true))
+// || ((executorch::aten::string_view(operator_name).compare("mm.out") == 0)
+//        && (true));
+//}
 /*
 inline constexpr bool should_include_kernel_dtype(
   const char* ,//operator_name,