ggml-org/llama.cpp

Llava functions compiled as extern "C" throw exceptions

Fechada

#7.073 aberto em 4 de mai. de 2024

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)C++ (21.781 forks)batch import
buggood first issue

Métricas do repositório

Stars
 (124.194 estrelas)
Métricas de merge de PR
 (Mesclagem média 6d 8h) (389 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
O problema está em clip.cpp nas linhas 1277 e 2075, onde as funções declaradas com extern "C" (clip model load e clip n mmproj embd) possuem especificações de exceção não lançadoras, mas ainda podem lançar exceções. Para corrigir, envolva os corpos das funções em blocos try catch que retornem indicadores de erro (por exemplo, 1 ou nullptr) em vez de permitir que as exceções se propaguem. Examine o padrão de tratamento de erros existente no arquivo e garanta que todas as funções visíveis externamente sigam essa regra. Ainda não há PRs vinculados ou responsáveis atribuídos.
Pilha de tecnologia
cppc
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
C++ exception handlingextern "C" linkage
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40