golang/go

cmd/compile: OpInlMark can get removed by SSA dead code elimination, resulting in invalid inlining trees

Open

#54.625 aperta il 23 ago 2022

Vedi su GitHub
 (19 commenti) (0 reazioni) (0 assegnatari)Go (19.008 fork)batch import
NeedsFixcompiler/runtimehelp wanted

Metriche repository

Star
 (133.883 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

See discussion at https://github.com/golang/go/issues/46234.

The immediate issue is that https://go.dev/play/p/OikFLBmKguY?v=gotip causes the loop at https://github.com/golang/go/blob/7f632f76db65f28038b8797fbfe5e15a9f63e387/src/runtime/traceback.go#L362 to never terminate, because the inlining "tree" has cycles.

The bigger issue is that we're generating trees like this for kube-apiserver, and this is presumably the root cause of #54593.

Guida contributor

Direzione di ricerca
Traccia il passaggio di eliminazione del codice morto SSA per capire come viene rimosso OpInlMark ed esamina la generazione dell'albero di inlining nel compilatore per identificare la causa del ciclo.
Tech stack
go
Dominio
compilers
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
5
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Oltre 1 settimana
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
Gocompiler internalsSSA
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
20