lanl/singularity-eos

Consider replacing ternary expressions with multiplicative conditionals

Ouverte

#319 ouverte le 27 oct. 2023

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 personne assignée)C++ (21 forks)auto 404
Performancegood first issue

Métriques du dépôt

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

Description

Per @Yurlungur we may want to replace

x = cond ? a : b

with

x = (cond) * a + (1 - cond) * b

to reduce branching and improve GPU performance.

This might be premature optimization, but it also might be a good way to specify default handling of NaNs as well.

Guide contributeur

Direction de recherche
Recherchez les expressions ternaires dans les chemins de code liés au GPU dans le depot, evaluez si le remplacement par des conditionnelles multiplicatives ameliore les performances et testez sur du materiel GPU.
Stack technique
c
Domaine
backend
Type d'issue
Performance
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
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65