Skip to content
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

Revert "Add SIMD to LowerCallMemcmp" #84595

Merged
merged 1 commit into from
Apr 11, 2023
Merged

Conversation

sbomer
Copy link
Member

@sbomer sbomer commented Apr 11, 2023

Reverts #84530

This is causing PR builds to fail with

error C2660: 'Compiler::gtNewSimdCmpOpAllNode': function does not take 7 arguments

See for example #84591, #84588, #84148.

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Apr 11, 2023
@ghost ghost assigned sbomer Apr 11, 2023
@ghost
Copy link

ghost commented Apr 11, 2023

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch, @kunalspathak
See info in area-owners.md if you want to be subscribed.

Issue Details

Reverts #84530

This is causing PR builds to fail with

error C2660: 'Compiler::gtNewSimdCmpOpAllNode': function does not take 7 arguments

See for example #84591, #84588, #84148.

Author: sbomer
Assignees: -
Labels:

area-CodeGen-coreclr

Milestone: -

@EgorBo
Copy link
Member

EgorBo commented Apr 11, 2023

It's just a bad timing of two PRs merged together, #84212 removed last argument in gtNewSimdCmpOpAllNode that I set to false in my PR, so the fix is just to remove , false); in gtNewSimdCmpOpAllNode and gtNewSimdBinOpNode - feel free to merge this revert or remove the arg to unblock the CI

@sbomer sbomer merged commit f8435f4 into main Apr 11, 2023
@sbomer sbomer deleted the revert-84530-SIMD-LowerCallMemcmp branch April 11, 2023 00:49
@sbomer
Copy link
Member Author

sbomer commented Apr 11, 2023

Thanks, I thought that might be the fix, but wasn't sure and the revert seemed like a safe fix.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants