Skip to content

Conversation

@nealef
Copy link
Contributor

@nealef nealef commented Jul 27, 2021

Reworking of previous (withdrawn) PR to address some floating point test failures:

  • Add rmove specification in cpu-s390x.md
  • OP_RMOVE implementation in mini-s390x.c
  • Correct float instruction lengths
  • Disable debug code
  • Correct RCONV_TO_I4/RCONV_TO_U4

    - Add rmove specification in cpu-s390x.md
    - OP_RMOVE implementation in mini-s390x.c
    - Correct float instruction lengths
    - Disable debug code
    - Correct RCONV_TO_I4/RCONV_TO_U4
@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Jul 27, 2021
@ghost
Copy link

ghost commented Jul 27, 2021

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@uweigand
Copy link
Contributor

I can confirm that this PR fixes all regression I've been seeing after #56111 was merged. Thanks!

@akoeplinger akoeplinger changed the title Additional float32 fixes s390x: Additional float32 fixes Jul 30, 2021
@akoeplinger akoeplinger merged commit 727bc6c into dotnet:main Jul 30, 2021
@nealef nealef deleted the s390x-fixes branch August 2, 2021 05:44
@ghost ghost locked as resolved and limited conversation to collaborators Sep 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Codegen-JIT-mono community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants