llvm/llvm-project

GlobalISel should have G_POISON

Open

#127.486 geöffnet am 17. Feb. 2025

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (10.782 Forks)batch import
good first issuellvm:globalisel

Repository-Metriken

Stars
 (26.378 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1T 2h) (1.000 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Durchsuchen Sie den LLVM Quellcode nach allen Verweisen auf G IMPLICIT DEF und fügen Sie eine parallele Behandlung für G POISON hinzu. Beginnen Sie mit den GlobalISel Anweisungsdefinitionen, aktualisieren Sie dann MachineVerifier, Legalizer, Combiner und alle anderen Passes, die G IMPLICIT DEF behandeln. Migrieren Sie schließlich die Vektorerweiterung zur Verwendung von G POISON.
Tech Stack
cpp
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++LLVMGlobalISel
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60