llvm/llvm-project

MetaRenamer command line options should be replaced with pass parameters

Aperta

#133.875 aperta il 1 apr 2025

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (1 assegnatario)C++ (10.782 fork)batch import
good first issuellvm:transforms

Metriche repository

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

Descrizione

The metarenamer pass currently defines 5 cl::opts.

In general with the new pass manager, we should prefer to use named pass parameters and avoid using global options. For utility passes like this, which are only useful when manually running opt, there should be low friction to replacing them.

https://github.com/llvm/llvm-project/pull/133874 is an example of a simpler migration for another utility pass for reference.

Guida contributor

Direzione di ricerca
Studia il codice sorgente del pass MetaRenamer, identifica i 5 cl::opts e segui il pattern della PR #133874 per sostituirli con parametri del pass. Aggiorna tutti gli utilizzi nel codebase.
Tech stack
ccpp
Dominio
compilersbackend
Tipo issue
Refactoring
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++LLVM pass developmentGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65