-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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.
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
Missing CI test coverage for Vector512 #88233
Comments
Tagging subscribers to this area: @dotnet/area-system-runtime-intrinsics Issue DetailsThe following test passed CI without any errors in #88166: [Fact]
[MethodImpl(MethodImplOptions.AggressiveOptimization)]
public static void SanityCheckAO()
{
if (Vector512.IsHardwareAccelerated && Avx512BW.IsSupported)
throw new Exception("Yay?");
} Dumping the CpuId, it looks like all processors in CI that report runtime/src/coreclr/vm/codeman.cpp Lines 1882 to 1892 in ddf47f0
With #86655, those no longer report
|
We should add an outerloop leg to the AVX-512 run that sets We are getting some stress coverage still, just randomly based on |
The following test passed CI without any errors in #88166:
Dumping the CpuId, it looks like all processors in CI that report
Avx512BW.IsSupported == true
also reportisGenuineIntel=True familyID=6 extendedModelID=5 model=5
.runtime/src/coreclr/vm/codeman.cpp
Lines 1882 to 1892 in ddf47f0
With #86655, those no longer report
Vector512.IsHardwareAccelerated
, therefore we have no CI coverage of such code paths?The text was updated successfully, but these errors were encountered: