llvm/llvm-project

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

Open

#189.694 geöffnet am 31. März 2026

Auf GitHub ansehen
 (9 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (10.782 Forks)batch import
good first issuellvm:SelectionDAG

Repository-Metriken

Stars
 (26.378 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1T 2h) (1.000 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Studieren Sie die vorhandene Implementierung von computeKnownFPClass in ValueTracking.cpp und verstehen Sie, wie ISD::FMAXIMUM und ähnliche behandelt werden. Fügen Sie dann eine analoge Behandlung für ISD::MAXNUM/MINNUM, ISD::FMAXIMUMNUM/FMINIMUMNUM hinzu. Schreiben Sie IR Tests, um die neuen Fälle abzudecken.
Tech Stack
Keine
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
3-5 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30