-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Ensure we account for intrinsics which have gtType=TYP_SIMD16 but gtSIMDSize=32 #35904
Conversation
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.
LGTM - thanks!
Hello @tannergooding! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
|
@bartonjs it seems like this is that cryptic error from the OS that happens once in million runs?
It seems like this is the only instance in 200 runs. |
Yep. I've tried getting OS support from it, they say it's unique to the abuse that test machines have and real callers never see problems like that. Makes it hard to do anything other than let the OS error leak out. |
@jaredpar another example where we have an unavoidable test failure that we need to be resilient around. |
jitstress failure on is also a separate issue: #35798 (comment) |
This resolves #35877