llvm/llvm-project

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

Open

#84.940 aperta il 12 mar 2024

Vedi su GitHub
 (13 commenti) (0 reazioni) (1 assegnatario)C++ (10.782 fork)batch import
good first issuellvm:SelectionDAG

Metriche repository

Star
 (26.378 star)
Metriche merge PR
 (Merge medio 1g 2h) (1000 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Ispeziona i callback TLI.isBinOp e isCommutativeBinOp nel target lowering, quindi aggiungi le corrispondenti funzioni di pattern matching m BinOp/m c BinOp in SDPatternMatch.h. Segui il pattern degli operatori esistenti come m Add, m Sub, ecc.
Tech stack
cpp
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++LLVMGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40