Skip to content

Disable AVX/AVX2 on interpreter-only x64 builds to fix Vector256 NRE #129574

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
kotlarmilos merged 10 commits into from
Jun 30, 2026
Merged

Disable AVX/AVX2 on interpreter-only x64 builds to fix Vector256 NRE #129574

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
dd4dac3
Disable AVX/AVX2 on interpreter-only x64 builds to fix Vector256 NRE
kotlarmilos Jun 18, 2026
80c93a7
Merge branch 'main' into fix-128901-interpreter-avx2
kotlarmilos Jun 19, 2026
eb36d66
Merge branch 'main' into fix-128901-interpreter-avx2
kotlarmilos Jun 19, 2026
d190c1c
Avoid setting AVX for interpreter-only instead of set-then-clear
kotlarmilos Jun 29, 2026
b3ca3da
Restore TARGET_X86/TARGET_AMD64 guard on interpreter-only AVX check
kotlarmilos Jun 29, 2026
f8d7bc6
Drop parentheses from interpreter-only AVX comment
kotlarmilos Jun 29, 2026
6829655
Clamp preferred vector width to 128 for interpreter-only
kotlarmilos Jun 29, 2026
c1c0c1e
Match guard style on interpreter-only preferred vector clamp
kotlarmilos Jun 30, 2026
019a8c7
Simplify interpreter-only ISA comments
kotlarmilos Jun 30, 2026
cc1310c
Drop redundant target guards on interpreter-only ISA blocks
kotlarmilos Jun 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 18 additions & 1 deletion src/coreclr/vm/codeman.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1539,7 +1539,15 @@ void EEJitManager::SetCpuInfo()

// x86-64-v3

if (((cpuFeatures & XArchIntrinsicConstants_Avx) != 0) && CLRConfig::GetConfigValue(CLRConfig::EXTERNAL_EnableAVX))
bool enableAVX = ((cpuFeatures & XArchIntrinsicConstants_Avx) != 0) && CLRConfig::GetConfigValue(CLRConfig::EXTERNAL_EnableAVX);

#if defined(FEATURE_INTERPRETER)
// The interpreter only supports 128-bit vectors, so don't enable AVX. The ISA
// dependency normalization below then removes the dependent AVX2/AVX512.
enableAVX = enableAVX && !interpreterOnly;
#endif
Comment thread
kotlarmilos marked this conversation as resolved.

if (enableAVX)
{
CPUCompileFlags.Set(InstructionSet_AVX);
}
Expand Down Expand Up @@ -1883,6 +1891,15 @@ void EEJitManager::SetCpuInfo()
preferredVectorBitWidth = 256;
}

#if defined(FEATURE_INTERPRETER)
if (interpreterOnly && (preferredVectorBitWidth > 128))
{
// Limit the preferred vector width so the Vector256/Vector512 marker ISAs
// set below are not reported after AVX was left disabled above.
preferredVectorBitWidth = 128;
}
#endif

if (preferredVectorBitWidth >= 512)
{
CPUCompileFlags.Set(InstructionSet_Vector512);
Expand Down