ggml-org/llama.cpp

Llava functions compiled as extern "C" throw exceptions

Fermée

#7 073 ouverte le 4 mai 2024

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)C++ (21 874 forks)batch import
buggood first issue

Métriques du dépôt

Stars
 (124 541 étoiles)
Métriques de merge PR
 (Merge moyen 6j 8h) (389 PRs mergées en 30 j)

Description

Basically this: llama.cpp\examples\llava\clip.cpp(1277,13): warning : 'clip_model_load' has a non-throwing exception specification but can still throw [-Wexceptions] llama.cpp\examples\llava\clip.cpp(2075,5): warning : 'clip_n_mmproj_embd' has a non-throwing exception specification but can still throw [-Wexceptions]

As these are library exported functions and wrapped in extern "C", they should not allow exceptions to cross the boundary. C language has no idea what to do with them.

Compiled with clang-cl in windows.

Guide contributeur

Direction de recherche
Le problème se trouve dans clip.cpp aux lignes 1277 et 2075, où les fonctions déclarées avec extern "C" (clip model load et clip n mmproj embd) ont des spécifications d'exception non levantes mais peuvent toujours lever des exceptions. Pour corriger, encapsulez les corps des fonctions dans des blocs try catch qui renvoient des indicateurs d'erreur (par exemple 1 ou nullptr) au lieu de permettre la propagation des exceptions. Examinez le modèle de gestion des erreurs existant dans le fichier et assurez vous que toutes les fonctions visibles de l'extérieur suivent cette règle. Il n'existe pas encore de PR liés ou de personnes assignées.
Stack technique
cppc
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.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C++ exception handlingextern "C" linkage
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40