llvm/llvm-project

MetaRenamer command line options should be replaced with pass parameters

Open

#133.875 geöffnet am 1. Apr. 2025

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (10.782 Forks)batch import
good first issuellvm:transforms

Repository-Metriken

Stars
 (26.378 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 1T 2h) (1.000 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche den Quellcode des MetaRenamer Passes, identifiziere die 5 cl::opts und folge dem Muster von PR #133874, um sie durch Pass Parameter zu ersetzen. Aktualisiere alle Verwendungen in der Codebasis.
Tech Stack
ccpp
Domain
compilersbackend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C++LLVM pass developmentGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65