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
macOS lazily saves the AVX512 context #36330
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. Do note that since it seems that loop vectorizer might be involved there'll still be other issues that you'll need to hack around until it is moved to the LLVM level.
We should also back-port the llvm patch. |
This seems like a terrible idea on their part, but I guess it's what we have to do. |
IIUC it sounds like they are reimplementing I assume it also means that running a darwin vm that doesn't support avx512 on hardware that does is basically unsupported then... (which is probably something that apple couldn't care less about...) |
I doubt this is the last we've heard along these lines, but let's try this for now, since it matches LLVM. |
Is it possible to back port this change? |
Fix #36329