llvm/llvm-project

GlobalISel should have G_POISON

Open

#127 486 ouverte le 17 févr. 2025

Voir sur GitHub
 (6 commentaires) (0 réactions) (1 assigné)C++ (10 782 forks)batch import
good first issuellvm:globalisel

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

Currently GlobalISel has G_IMPLICIT_DEF, which acts like undef. This is used to lower both poison and undef from the IR. To mirror the semantics of the IR, we need a separate representation of poison. We should introduce G_POISON to enable better optimizations, particularly when legalization introduces padding vector elements.

This change would mostly be finding all the places G_IMPLICIT_DEF is handled, and equivalently handling G_POISON. We should then migrate vector widening and other transforms to introduce poison instead of undef.

Guide contributeur

Direction de recherche
Recherchez toutes les references a G IMPLICIT DEF dans le code source de LLVM et ajoutez un traitement parallele pour G POISON. Commencez par les definitions d'instructions GlobalISel, puis mettez a jour MachineVerifier, legalizer, combiner et tous les autres passes qui traitent G IMPLICIT DEF. Enfin, migrez l'elargissement des vecteurs pour utiliser G POISON.
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.
Une demi journée
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
C++LLVMGlobalISel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60