-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Fix the operands of NI_AVX512F_TernaryLogic during lowering #107813
Conversation
This reverts commit 5b01643.
@tannergooding @BruceForstall PTAL. We want to merge before 4pm today. |
[Fact] | ||
public static void TestEntryPoint() | ||
{ | ||
try |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should rather be checking if (Avx512F.IsSupported)
, rather than doing try/catch, no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, I misread the error of a different issue about "NullReferenceException". Will fix it.
This reverts commit 3319875.
/azp run Fuzzlyn |
/backport to release/9.0 |
Started backporting to release/9.0: https://github.com/dotnet/runtime/actions/runs/10890038223 |
We were not correctly translating the logical operation into control word leading to assert getting hit during containment.
Fixes: #107587
Fixes: #107324