llvm/llvm-project

[DAG] SDPatternMatch - add m_BinOp/m_c_BinOp variants driven by TLI.isBinOp/isCommutativeBinOp

Ouverte

#84 940 ouverte le 12 mars 2024

Voir sur GitHubGuide contributeur
 (13 commentaires) (0 réaction) (1 personne assignée)C++ (10 782 forks)batch import
good first issuellvm:SelectionDAG

Métriques du dépôt

Stars
 (26 378 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

As noticed on #84759 - we should be able to allow folds to generally match any binary ops that are recognized by the TLI.isBinOp/isCommutativeBinOp callbacks, allowing us to match target binops in generic dag combines.

CC @mshockwave

Guide contributeur

Direction de recherche
Inspectez les callbacks TLI.isBinOp et isCommutativeBinOp dans le target lowering, puis ajoutez les fonctions de correspondance de motifs m BinOp/m c BinOp correspondantes dans SDPatternMatch.h. Suivez le modèle des opérateurs existants comme m Add, m Sub, etc.
Stack technique
cpp
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++LLVMGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40