From 43905744da45d248ecb5cc20d61423d90c4b7582 Mon Sep 17 00:00:00 2001 From: Tanner Gooding Date: Tue, 13 Jun 2023 10:43:42 -0700 Subject: [PATCH] Don't return `0` from `getMaxVectorByteLength` when intrinsics are disabled (#87420) --- src/coreclr/jit/compiler.h | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/src/coreclr/jit/compiler.h b/src/coreclr/jit/compiler.h index 65379c6..049b630 100644 --- a/src/coreclr/jit/compiler.h +++ b/src/coreclr/jit/compiler.h @@ -8719,8 +8719,11 @@ private: } else { + // TODO: We should be returning 0 here, but there are a number of + // places that don't quite get handled correctly in that scenario + assert((JitConfig.EnableHWIntrinsic() == 0) || (JitConfig.EnableSSE() == 0)); - return 0; + return XMM_REGSIZE_BYTES; } #elif defined(TARGET_ARM64) if (compOpportunisticallyDependsOn(InstructionSet_AdvSimd)) @@ -8729,8 +8732,11 @@ private: } else { + // TODO: We should be returning 0 here, but there are a number of + // places that don't quite get handled correctly in that scenario + assert((JitConfig.EnableHWIntrinsic() == 0) || (JitConfig.EnableArm64AdvSimd() == 0)); - return 0; + return FP_REGSIZE_BYTES; } #else assert(!"getMaxVectorByteLength() unimplemented on target arch"); -- 2.7.4