llvm/llvm-project

[NVPTX] Implement `TLI.isTruncateFree(EVT, EVT)` and `TLI.isZExtFree()`

Fermée

#114 339 ouverte le 31 oct. 2024

Voir sur GitHubGuide contributeur
 (11 commentaires) (0 réaction) (2 personnes assignées)C++ (10 782 forks)batch import
backend:NVPTXgood first issue

Métriques du dépôt

Stars
 (26 378 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

NVPTX currently does not override TLI.isTruncateFree(EVT, EVT) and TLI.isZExtFree(), and it probably should.

zext/truncate between i32/i64 can probably be considered free, as the values already live in separate 32-bit registers in hardware.

Originally posted by @Artem-B in https://github.com/llvm/llvm-project/pull/98666#discussion_r1681805956

Guide contributeur

Direction de recherche
Examinez l'implémentation par défaut de isTruncateFree et isZExtFree pour la cible NVPTX; étudiez comment d'autres backends (par exemple X86) gèrent les conversions i32/i64; implémentez les surcharges dans NVPTXTargetLowering.cpp.
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70