-
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
Assertion failed 'compIsaSupportedDebugOnly(InstructionSet_AVX2)' in Libraries Test Run checked coreclr OSX x64 Debug #64175
Comments
Tagging subscribers to this area: @dotnet/area-system-runtime-intrinsics Issue Details
|
This assert seems massively off. It claims the assert is in gentree, but there is no equivalent assert anywhere near there: https://github.com/dotnet/runtime/blob/main/src/coreclr/jit/gentree.cpp#L20737 There is a known issue here, but the assert that catches it is being added here in codegen: #64140 and the assert is being added because nothing else was known to raise the issue |
Ah, I didn't realize the PR had been run 2 days ago. This particular assert was already fixed in #63972 |
Ah thanks, I didn't see that in my search. |
runtime/src/coreclr/jit/gentree.cpp Line 20655 in bb5ca4c
This assert is still failing on PRs based off of current main branch, e.g.: https://helix.dot.net/api/2019-06-17/jobs/4ca0619d-cc63-48a1-b0a7-100192c57fd7/workitems/System.Runtime.Intrinsics.Tests/console |
This is a different, but related, assert being handled by #64140. 64140 also notably adds validation to help validate we aren't introducing failures like this in the future given that we only have a very small pool of OSX machines that have AVX but not AVX2. |
#64113
The text was updated successfully, but these errors were encountered: