golang/go

cmd/compile: mips/mips64 floating point negation does not handle NaN correctly

Open

#58 466 ouverte le 10 févr. 2023

Voir sur GitHub
 (5 commentaires) (0 réactions) (1 assigné)Go (19 008 forks)batch import
NeedsDecisionarch-mipscompiler/runtimehelp wanted

Métriques du dépôt

Stars
 (133 883 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

As discussed on https://github.com/golang/go/issues/56491#issuecomment-1426492508, on MIPS the gc compiler uses neg.d (which it calls NEGD) to negate a floating-point value. However, that instruction does not actually negate a NaN. That suggests that we should not use neg.d for floating point negation, but should instead xor with the high bit.

CC @randall77 @golang/mips @golang/runtime

Guide contributeur

Direction de recherche
Tracez la génération de code de négation en virgule flottante MIPS dans cmd/compile/internal/ssagen et remplacez NEGD par un XOR du bit de poids fort.
Stack technique
go
Domaine
tooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GoMIPS assembly
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35