ggml-org/llama.cpp

Llava functions compiled as extern "C" throw exceptions

Chiusa

#7073 aperta il 4 mag 2024

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)C++ (21.781 fork)batch import
buggood first issue

Metriche repository

Star
 (124.194 stelle)
Metriche merge PR
 (Merge medio 6g 8h) (389 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Il problema si trova in clip.cpp alle righe 1277 e 2075, dove le funzioni dichiarate con extern "C" (clip model load e clip n mmproj embd) hanno specifiche di eccezioni non lancianti ma possono comunque lanciare eccezioni. Per risolvere, avvolgere i corpi delle funzioni in blocchi try catch che restituiscano indicatori di errore (ad esempio 1 o nullptr) invece di consentire la propagazione delle eccezioni. Esaminare il modello di gestione degli errori esistente nel file e assicurarsi che tutte le funzioni visibili esternamente seguano questa regola. Al momento non esistono PR collegati o assegnatari.
Tech stack
cppc
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++ exception handlingextern "C" linkage
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40