llvm/llvm-project

Move JITLink from StringRefs to SymbolStringPtrs

Open

#55 533 ouverte le 17 mai 2022

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

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

JITLink is using the lllvm::StringRef type in many places where llvm::orc::SymbolStringPtr would result in better performance and memory footprint, e.g. https://github.com/llvm/llvm-project/blob/77480556c41fbca36b918323c69cb77f8e02b56c/llvm/include/llvm/ExecutionEngine/JITLink/JITLink.h#L648-L649

In such cases the string should be interned in a llvm::orc::SymbolStringPool and referenced via llvm::orc::SymbolStringPtr.

Guide contributeur

Direction de recherche
Identifiez tous les emplacements dans la base de code JITLink où StringRef est utilisé à des endroits appropriés pour SymbolStringPtr, en commençant par l'exemple dans `llvm/include/llvm/ExecutionEngine/JITLink/JITLink.h` (lignes 648-649). Internalisez les chaînes de caractères en utilisant `llvm::orc::SymbolStringPool` et mettez à jour les signatures de fonction en conséquence. Consultez les 11 commentaires sur l'issue pour obtenir un contexte supplémentaire ou des fichiers spécifiques mentionnés. Assurez vous de ne pas introduire de modifications fonctionnelles et concentrez vous sur les améliorations de performances et d'empreinte mémoire.
Stack technique
cpp
Domaine
backend
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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++LLVMJITLink
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30