llvm/llvm-project

AMDGPU should implement TargetLowering::hasAndNot

Open

#112.550 geöffnet am 16. Okt. 2024

Auf GitHub ansehen
 (17 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (10.782 Forks)batch import
backend:AMDGPUgood first issue

Repository-Metriken

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

Beschreibung

AMDGPU should implement this optimization hook: https://github.com/llvm/llvm-project/blob/b333edd0d6da744c099ad3ff3b5fbd2d4e4dd45a/llvm/include/llvm/CodeGen/TargetLowering.h#L797

This is available on the SALU for i32 and i64 (s_andn2_b32 and s_andn2_b64). The implementation therefore should be approximately !N->isDivergent() && (N->getValueType() == i32 || i64).

Contributor Guide

Research-Richtung
Implementieren Sie den hasAndNot Hook in der AMDGPU TargetLowering Unterklasse. Die Implementierung sollte prüfen, ob der Wert nicht divergent ist und den Typ i32 oder i64 hat, und dann true zurückgeben. Beziehen Sie sich auf den vorhandenen Hook in TargetLowering.h in Zeile 797. Überprüfen Sie den AMDGPU Backend Code in lib/Target/AMDGPU/, um die geeignete zu ändernde Unterklasse zu identifizieren. Klären Sie mit den Maintainern den Umgang mit divergenten Werten und etwaige zusätzliche Bedingungen.
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.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75