llvm/llvm-project

[InstCombiner] foldIntrinsicIsFPClass - incorrect submasking of FPClassTest masks

Open

#189 949 ouverte le 1 avr. 2026

Voir sur GitHub
 (15 commentaires) (0 réactions) (1 assigné)C++ (10 782 forks)batch import
floating-pointgood first issuellvm:instcombine

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

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

Guide contributeur

Direction de recherche
Examinez le code à l'emplacement référencé et modifiez la condition d'une vérification d'égalité (==) à une vérification de sous ensemble à l'aide d'opérations binaires telles que `(Mask & Known) == Mask`.
Stack technique
cpp
Domaine
backend
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50