llvm/llvm-project

llvm/lib/CodeGen/LiveRangeUtils.h:40: Iterators applied to wrong data structure ?

Open

#122 937 ouverte le 14 janv. 2025

Voir sur GitHub
 (8 commentaires) (0 réactions) (2 assignés)C++ (10 782 forks)batch import
code-qualitygood first issuellvm:codegen

Métriques du dépôt

Stars
 (26 378 stars)
Métriques de merge PR
 (Merge moyen 1j 2h) (1 000 PRs mergées en 30 j)

Description

Static analyser cppcheck says:

llvm/lib/CodeGen/LiveRangeUtils.h:40:6: error: Same iterator is used with different containers 'LR' and 'LR.segments'. [iterators1]

Source code is

typename LiveRangeT::iterator J = LR.begin(), E = LR.end(); // ... LR.segments.erase(J, E);

So J and E are iterators for LR, but get used on LR.segments.

I am surprised this compiles.

Guide contributeur

Direction de recherche
Le bug se trouve dans `llvm/lib/CodeGen/LiveRangeUtils.h` à la ligne 40, où des itérateurs de `LR` sont utilisés avec `LR.segments`. Pour corriger, remplacez `LR.segments.erase(J, E);` par `LR.segments.erase(LR.segments.begin(), LR.segments.end());` ou par des itérateurs appropriés. Vérifiez le type `LiveRangeT` pour vous assurer que les itérateurs sont compatibles.
Stack technique
cpp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55