llvm/llvm-project

MetaRenamer command line options should be replaced with pass parameters

Ouverte

#133 875 ouverte le 1 avr. 2025

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (1 personne assignée)C++ (10 782 forks)batch import
good first issuellvm:transforms

Métriques du dépôt

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

Description

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.

Guide contributeur

Direction de recherche
Étudiez le code source du pass MetaRenamer, identifiez les 5 cl::opts et suivez le modèle de la PR #133874 pour les remplacer par des paramètres de pass. Mettez à jour toutes les utilisations dans la base de code.
Stack technique
ccpp
Domaine
compilersbackend
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.
2
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.
Plutôt claire
Prérequis
C++LLVM pass developmentGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65