llvm/llvm-project

Move JITLink from StringRefs to SymbolStringPtrs

Open

#55,533 创建于 2022年5月17日

在 GitHub 查看
 (11 评论) (0 反应) (1 负责人)C++ (10,782 fork)batch import
good first issuejitlinkorcjit

仓库指标

Star
 (26,378 star)
PR 合并指标
 (平均合并 1天 2小时) (30 天内合并 1,000 个 PR)

描述

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.

贡献者指南

研究方向
识别 JITLink 代码库中所有使用 StringRef 但适合使用 SymbolStringPtr 的位置，从 `llvm/include/llvm/ExecutionEngine/JITLink/JITLink.h`（第 648-649 行）的示例开始。使用 `llvm::orc::SymbolStringPool` 对字符串进行内部化，并相应更新函数签名。查看该议题的 11 条评论以获取额外上下文或提及的特定文件。确保不引入功能变更，专注于性能和内存占用的改进。
技术栈
cpp
领域
backend
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-2 天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
C++LLVMJITLink
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
30