lanl/singularity-eos

Should we remove forwarding references from PTE solvers?

Ouverte

#438 ouverte le 29 nov. 2024

Voir sur GitHub
 (4 commentaires) (0 réaction) (0 personne assignée)C++ (21 forks)auto 404
bugclean-updiscussionhelp wantedinterfacequestion

Métriques du dépôt

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

Description

The segfault on HIP was caused by an access-out-of-scope error caused by passing pointers by reference into the PTE solvers. See #437 . Perhaps we should consider removing the forwarding references and pass only by value into the PTE solvers (with the exception of PTE Solver Base, which requires pass-by-reference as it is used as the first part of the derived class constructors).

Ping @jhp-lanl @jdolence @dholladay00 @mauneyc-LANL and @rbberger for feedback

Guide contributeur

Direction de recherche
Lisez le problème n°437 pour comprendre le segfault causé par le passage de pointeurs par référence. Examinez ensuite le code du solveur PTE, en vous concentrant sur la façon dont les références de transfert sont utilisées. Prévoyez de modifier uniquement les solveurs PTE pour les passer par valeur, à l'exception de PTE Solver Base. Testez les modifications sur HIP pour vous assurer qu'aucun segfault ne se produit.
Stack technique
cpp
Domaine
hpc
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
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
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40