llvm/llvm-project

[libc] Clean up copt naming

Open

#133 505 ouverte le 28 mars 2025

Voir sur GitHub
 (6 commentaires) (0 réactions) (1 assigné)C++ (10 782 forks)batch import
good first issuelibc

Métriques du dépôt

Stars
 (26 378 stars)
Métriques de merge PR
 (Merge moyen 1j 2h) (1 000 PRs mergées en 30 j)

Description

Our code style document says all our compile flags set by the build system should start LIBC_COPT_ but not all of them do. We should update them to fit our naming scheme.

Steps to fix:

  1. go through the flags in config.json
  2. For each flag, check what copt it defines
  3. If that copt doesn't start with LIBC_COPT_, rename it so it does.
  4. Make sure the flag is renamed everywhere it's used in code, cmake, and bazel.

Guide contributeur

Direction de recherche
Le problème fournit une séquence claire d'étapes pour renommer les indicateurs de compilation dans config.json. La correction implique de rechercher toutes les occurrences de chaque indicateur renommé dans les fichiers cmake et bazel. Un responsable est déjà assigné, donc les contributeurs intéressés doivent se coordonner via les commentaires pour éviter un travail en double.
Stack technique
c
Domaine
build systemtooling
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.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Gitcmakebazel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80