llvm/llvm-project

[DAG] computeKnownFPClass - add ISD::MAXNUM/MINNUM handling + test coverage

Open

#189 694 ouverte le 31 mars 2026

Voir sur GitHub
 (9 commentaires) (0 réactions) (1 assigné)C++ (10 782 forks)batch import
good first issuellvm:SelectionDAG

Métriques du dépôt

Stars
 (26 378 stars)
Métriques de merge PR
 (Merge moyen 1j 2h) (1 000 PRs mergées en 30 j)

Description

ISD::FMAXIMUM/ISD::FMINIMUM + ISD::FMAXIMUMNUM/ISD::FMINIMUMNUM handling can be added as well

This will need to be based off the implementation in computeKnownFPClass in ValueTracking.cpp - DenormalMode access might take some additional patches / test coverage

This should be tested with IR tests and not unit tests

Guide contributeur

Direction de recherche
Étudiez l'implémentation existante de computeKnownFPClass dans ValueTracking.cpp, comprenez comment ISD::FMAXIMUM et autres sont traités, puis ajoutez un traitement analogue pour ISD::MAXNUM/MINNUM, ISD::FMAXIMUMNUM/FMINIMUMNUM. Écrivez des tests IR pour couvrir les nouveaux cas.
Stack technique
Aucun
Domaine
backend
Type d'issue
Fonctionnalité
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.
3-5 jours
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.
Plutôt claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30