llvm/llvm-project

Move JITLink from StringRefs to SymbolStringPtrs

Open

#55.533 aperta il 17 mag 2022

Vedi su GitHub
 (11 commenti) (0 reazioni) (1 assegnatario)C++ (10.782 fork)batch import
good first issuejitlinkorcjit

Metriche repository

Star
 (26.378 star)
Metriche merge PR
 (Merge medio 1g 2h) (1000 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Identifica tutte le posizioni nella codebase di JITLink in cui viene utilizzato StringRef in luoghi appropriati per SymbolStringPtr, partendo dall'esempio in `llvm/include/llvm/ExecutionEngine/JITLink/JITLink.h` (righe 648-649). Internalizza le stringhe utilizzando `llvm::orc::SymbolStringPool` e aggiorna di conseguenza le firme delle funzioni. Rivedi gli 11 commenti sull'issue per ulteriore contesto o file specifici menzionati. Assicurati di non introdurre modifiche funzionali e concentrati sui miglioramenti delle prestazioni e dell'impronta di memoria.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++LLVMJITLink
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30