llvm/llvm-project

Move JITLink from StringRefs to SymbolStringPtrs

Open

#55.533 geöffnet am 17. Mai 2022

Auf GitHub ansehen
 (11 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (10.782 Forks)batch import
good first issuejitlinkorcjit

Repository-Metriken

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

Beschreibung

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.

Contributor Guide

Research-Richtung
Identifizieren Sie alle Stellen in der JITLink Codebasis, an denen StringRef an Stellen verwendet wird, die für SymbolStringPtr geeignet sind, beginnend mit dem Beispiel in `llvm/include/llvm/ExecutionEngine/JITLink/JITLink.h` (Zeilen 648-649). Internalisieren Sie Zeichenketten mit `llvm::orc::SymbolStringPool` und passen Sie die Funktionssignaturen entsprechend an. Überprüfen Sie die 11 Kommentare zum Issue für zusätzlichen Kontext oder erwähnte spezifische Dateien. Stellen Sie sicher, dass keine funktionalen Änderungen eingeführt werden, und konzentrieren Sie sich auf Leistungs und Speicherverbrauchsverbesserungen.
Tech Stack
cpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
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++LLVMJITLink
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30