llvm/llvm-project

[AArch64] Add SimplifyDemandedVectorEltsForTargetNode support for AArch64ISD::DUPLANE nodes

Open

#87.497 aperta il 3 apr 2024

Vedi su GitHub
 (13 commenti) (0 reazioni) (1 assegnatario)C++ (10.782 fork)batch import
backend:AArch64good first issue

Metriche repository

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

Descrizione

Noticed on #86284 - aarch64 is missing SimplifyDemandedVectorElts handling to/from AArch64ISD::DUPLANE nodes

  • Extend performDUPCombine to call SimplifyDemandedVectorElts from AArch64ISD::DUPLANE nodes as we only demand that one lane element.
  • Add AArch64TargetLowering::SimplifyDemandedVectorEltsForTargetNode to handle AArch64ISD::DUPLANE nodes (and AArch64ISD::DUP?) - if we only demand the dup lane index we might be able to remove the node entirely.
  • The hadd-combine.ll tests might be enough, but we could require some additional test coverage.

Guida contributor

Direzione di ricerca
Esamina la funzione performDUPCombine esistente in AArch64ISelLowering.cpp e comprendi come SimplifyDemandedVectorElts viene utilizzato in altri target. Quindi implementa le modifiche passo dopo passo: prima estendi performDUPCombine per chiamare SimplifyDemandedVectorElts, poi aggiungi il callback specifico del target e infine verifica con i test hadd combine.ll. Concentrati su come i nodi DUPLANE richiedono solo un elemento di lane e come rimuoverli se solo quella lane è richiesta.
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.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40