llvm/llvm-project

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

Open

#87 497 ouverte le 3 avr. 2024

Voir sur GitHub
 (13 commentaires) (0 réactions) (1 assigné)C++ (10 782 forks)batch import
backend:AArch64good first issue

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

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.

Guide contributeur

Direction de recherche
Examinez la fonction existante performDUPCombine dans AArch64ISelLowering.cpp et comprenez comment SimplifyDemandedVectorElts est utilisé dans d'autres cibles. Ensuite, implémentez les modifications étape par étape : d'abord étendez performDUPCombine pour appeler SimplifyDemandedVectorElts, puis ajoutez le callback spécifique à la cible, et enfin vérifiez avec les tests hadd combine.ll. Concentrez vous sur la façon dont les nœuds DUPLANE ne demandent qu'un seul élément lane et comment les supprimer si seule cette lane est demandée.
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.
Claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40