llvm/llvm-project

GlobalISel should have G_POISON

Open

#127.486 aperta il 17 feb 2025

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

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Cerca tutti i riferimenti a G IMPLICIT DEF nel codice sorgente di LLVM e aggiungi la gestione parallela per G POISON. Inizia dalle definizioni delle istruzioni GlobalISel, poi aggiorna MachineVerifier, legalizer, combiner e tutti gli altri pass che gestiscono G IMPLICIT DEF. Infine, migra l'allargamento dei vettori per utilizzare G POISON.
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.
Mezza giornata
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
C++LLVMGlobalISel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60