llvm/llvm-project

[InstCombiner] foldIntrinsicIsFPClass - incorrect submasking of FPClassTest masks

Geschlossen

#189.949 geöffnet am 01.04.2026

Auf GitHub ansehenContributor Guide
 (15 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (10.782 Forks)batch import
floating-pointgood first issuellvm:instcombine

Repository-Metriken

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

Beschreibung

As mentioned here when creating the DAG equivalent: https://github.com/llvm/llvm-project/pull/189944#discussion_r3021637130

We shouldn't be looking for equality, we should be testing that they are a subset of the known classes:

https://github.com/llvm/llvm-project/blob/03b5539dd58584e6821e11123d5c8546147a6f41/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp#L1180-L1184

Contributor Guide

Research-Richtung
Überprüfen Sie den Code an der referenzierten Stelle und ändern Sie die Bedingung von einer Gleichheitsprüfung (==) zu einer Teilmengenprüfung unter Verwendung von bitweisen Operationen wie `(Mask & Known) == Mask`.
Tech Stack
cpp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50