KhronosGroup/SPIRV-LLVM-Translator

[Coverity] Copy instead of move

Open

#3.180 geöffnet am 16. Mai 2025

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)LLVM (270 Forks)auto 404
buggood first issue

Repository-Metriken

Stars
 (616 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfen Sie jeden Coverity CID Link, untersuchen Sie den Code an der angegebenen Zeile und entscheiden Sie, ob eine Kopie durch std::move ersetzt werden kann. Wenden Sie die Änderungen für jeden gültigen Fall an.
Tech Stack
cpp
Domain
compilers
Issue Type
Fehler
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.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++move semantics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60