Core runtime changes for Windows MSVC support (#6624)

Summary:
Miscellaneous core runtime fixes to make sure we can build with MSVC.


Differential Revision: D65328601

Author: tarun292

kernels/prim_ops/register_prim_ops.cpp

@@ -148,7 +148,8 @@ static Kernel prim_ops[] = {
           EValue& out = *stack[2];
           if (a.isInt() && b.isInt()) {
             const int64_t quot = a.toInt() / b.toInt();
-            if (std::signbit(a.toInt()) == std::signbit(b.toInt())) {
+            if (std::signbit(static_cast<double>(a.toInt())) ==
+                std::signbit(static_cast<double>(b.toInt()))) {
               out = EValue(quot);
               return;
             }

kernels/quantized/cpu/op_quantize.cpp

@@ -281,7 +281,7 @@ Tensor& quantize_per_channel_out(
   check_quantize_per_tensor_args(input, quant_min, quant_max, dtype, out);
 
   // a list contains all dimensions except axis
-  int64_t dims[input.dim() - 1];
+  int64_t dims[kTensorDimensionLimit];
   for (int64_t i = 0; i < input.dim() - 1; i++) {
     if (i < axis) {
       dims[i] = i;

runtime/core/portable_type/tensor_impl.cpp

@@ -110,8 +110,11 @@ Error TensorImpl::internal_resize_contiguous(ArrayRef<SizesType> new_sizes) {
           numel_bound_);
 
       if (strides_ && dim_order_) {
-        ET_CHECK_OK_OR_RETURN_ERROR(
-            dim_order_to_stride(new_sizes.data(), dim_order_, dim_, strides_));
+        auto error =
+            dim_order_to_stride(new_sizes.data(), dim_order_, dim_, strides_);
+        if (error != Error::Ok) {
+          return error;
+        }
       }
       numel_ = new_numel;
       std::copy(new_sizes.begin(), new_sizes.end(), sizes_);

runtime/executor/memory_manager.h

@@ -71,7 +71,7 @@ class MemoryManager final {
       // bug that triggers a syntax error when using [[maybe_unused]] on the
       // first parameter of a constructor:
       // https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81429
-      __attribute__((unused)) MemoryAllocator* constant_allocator,
+      __ET_UNUSED MemoryAllocator* constant_allocator,
       HierarchicalAllocator* non_constant_allocator,
       MemoryAllocator* runtime_allocator,
       MemoryAllocator* temporary_allocator)

runtime/executor/program.cpp

@@ -27,7 +27,11 @@
 #define ET_ENABLE_PROGRAM_VERIFICATION 1
 #endif
 
+#ifdef _MSC_VER
+#pragma warning(disable : 4456 4457)
+#else
 #pragma clang diagnostic ignored "-Wshadow"
+#endif
 
 namespace executorch {
 namespace runtime {
@@ -88,7 +92,11 @@ Result<executorch_flatbuffer::ExecutionPlan*> get_execution_plan(
     } else if (eh.error() == Error::NotFound) {
       // No header; the program consumes the whole file, and there are no
       // segments.
-      program_size = ET_UNWRAP(loader->size());
+      auto result = loader->size();
+      if (!result.ok()) {
+        return result.error();
+      }
+      program_size = result.get();
     } else {
       ET_LOG(Error, "Extended header may be corrupt");
       return eh.error();