lanl/singularity-eos

Should we remove forwarding references from PTE solvers?

Aperta

#438 aperta il 29 nov 2024

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)C++ (21 fork)auto 404
bugclean-updiscussionhelp wantedinterfacequestion

Metriche repository

Star
 (36 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Leggi il problema #437 per capire il segmentation fault causato dal passaggio di puntatori per riferimento. Quindi esamina il codice del risolutore PTE, concentrandoti su come vengono utilizzati i riferimenti di inoltro. Pianifica di modificare solo i risolutori PTE per passare per valore, tranne PTE Solver Base. Testa le modifiche su HIP per assicurarti che non ci siano segmentation fault.
Tech stack
cpp
Dominio
hpc
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40