-
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
[mono] Test llvm bump #105867
[mono] Test llvm bump #105867
Conversation
Tagging subscribers to this area: @lambdageek, @steveisok |
/azp run runtime-llvm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-llvm |
Azure Pipelines successfully started running 1 pipeline(s). |
The EmptyStruct crash isn't related, I missed it in the previous triage and it's a known issue #106071. The dwarf error could be related. |
The dwarf error looks like some failure in gdb, so llvm x64 looks good. |
@BrzVlad if you think the fix is good, can you take this out of draft and move it forward? I'd like to get this in before RC1 snaps. |
Something seems to be wrong with llvm instruction emitting.
/azp run runtime-llvm |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run runtime-llvm |
Azure Pipelines successfully started running 1 pipeline(s). |
The llvm x64 lane is green now. This PR disables emitting of some llvm intrinsic as a current workaround, @tannergooding will take a look at this when there is some time available. For now this looks good, a review is needed so we can merge. cc @radekdoulik |
👍, can we make sure a tracking issue is logged for this? |
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, please add comment with issue number to the disabled shuffle intrinsic.
Issue for follow up: #106270 |
No description provided.