reactorlabs/rir

ScopeResolution: unnecessary execution of code

Open

#1274 aperta il 1 apr 2024

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (19 fork)github user discovery
buggood first issuepir

Metriche repository

Star
 (61 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

The code here will run on every iteration of this pass. Even if it does not change the 'changed' flag , the method fromContext should not be invoked more than is required.

https://github.com/reactorlabs/rir/blob/0c2896fbc82f505e61ed8db416e113905d8f2237/rir/src/compiler/opt/scope_resolution.cpp#L317

Guida contributor

Direzione di ricerca
Rivedi il file scope resolution.cpp intorno alla riga 317. Comprendi il ciclo di iterazione e la chiamata a fromContext. Identifica se fromContext può essere spostato fuori dal ciclo o se una condizione può prevenirne l'esecuzione non necessaria. Considera come viene utilizzato il flag 'changed' e assicurati che la correzione non alteri la logica.
Tech stack
cpp
Dominio
compilers
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75