Make EValue template constructor MSVC friendly

Differential Revision: D65393407

Author: tarun292

runtime/core/evalue.h

@@ -242,17 +242,20 @@ struct EValue {
   // Template constructor that allows construction from types that can be
   // dereferenced to produce a type that EValue can be implicitly constructed
   // from.
-  template <typename T>
-  /*implicit*/ EValue(
-      T&& value,
-      typename std::enable_if<std::is_convertible<
-          decltype(*std::forward<T>(value)),
-          EValue>::value>::type* = 0) {
+template <typename T,
+    typename = typename std::enable_if<
+        std::is_convertible<
+            decltype(*std::forward<T>(std::declval<T>())), // declval to simulate forwarding
+            EValue
+        >::value
+    >::type>
+/*implicit*/ EValue(T&& value) {
     ET_CHECK_MSG(value != nullptr, "Pointer is null.");
     // Note that this ctor does not initialize this->tag directly; it is set by
     // moving in the new value.
     moveFrom(*std::forward<T>(value));
-  }
+}
+
 
   // Delete constructor for raw pointers to ensure they cannot be used.
   template <typename T>

runtime/core/exec_aten/exec_aten.h

@@ -9,6 +9,7 @@
 #pragma once
 
 #include <executorch/runtime/core/tensor_shape_dynamism.h> // @manual
+#include <executorch/runtime/platform/compiler.h>
 #ifdef USE_ATEN_LIB
 #include <ATen/Tensor.h> // @manual
 #include <c10/core/Device.h>