[Coverity] Copy instead of move · KhronosGroup/SPIRV-LLVM-Translator#3180 | Good First Issue

(0 评论) (0 反应) (0 负责人)LLVM (270 fork)auto 404

buggood first issue

仓库指标

Star: (616 star)
PR 合并指标: (PR 指标待抓取)

描述

See issues at https://scan.coverity.com/projects/intel-llvm?tab=overview

CID 489856:

https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blame/590068b12402262dd9bd0aa1386f2e89babfa8a1/lib/SPIRV/SPIRVWriter.cpp#L2430

CID 489869:

https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blame/590068b12402262dd9bd0aa1386f2e89babfa8a1/lib/SPIRV/SPIRVWriter.cpp#L6425

CID 490409:

This one looks a bit weird to me, not sure if its a valid hit:

https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blame/590068b12402262dd9bd0aa1386f2e89babfa8a1/lib/SPIRV/SPIRVWriter.cpp#L5148

CID 490034:

https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blob/590068b12402262dd9bd0aa1386f2e89babfa8a1/lib/SPIRV/libSPIRV/SPIRVEntry.cpp#L637

CID 490504:

https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blob/590068b12402262dd9bd0aa1386f2e89babfa8a1/lib/SPIRV/libSPIRV/SPIRVInstruction.h#L1197

CID 490545:

https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blob/590068b12402262dd9bd0aa1386f2e89babfa8a1/lib/SPIRV/libSPIRV/SPIRVInstruction.h#L1804

CID 489914:

https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blob/590068b12402262dd9bd0aa1386f2e89babfa8a1/lib/SPIRV/libSPIRV/SPIRVModule.cpp#L718

CID 490348:

https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blob/590068b12402262dd9bd0aa1386f2e89babfa8a1/lib/SPIRV/libSPIRV/SPIRVUtil.h#L164

CID 439509:

https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blob/590068b12402262dd9bd0aa1386f2e89babfa8a1/lib/SPIRV/libSPIRV/SPIRVUtil.h#L167

CID 489584:

https://github.com/KhronosGroup/SPIRV-LLVM-Translator/blob/590068b12402262dd9bd0aa1386f2e89babfa8a1/lib/SPIRV/libSPIRV/SPIRVValue.h#L310

贡献者指南

研究方向: 检查每个Coverity CID链接，查看指定行的代码，并判断是否可以将拷贝替换为std::move。对每个有效案例应用更改。
技术栈: cpp
领域: compilers
议题类型: 缺陷
难度: 3
预计时间: 半天
活动状态: 新近可参与
清晰度: 清晰
前置要求: C++move semantics
新手友好度: 60