ggml-org/llama.cpp

Llava functions compiled as extern "C" throw exceptions

已关闭

#7,073 创建于 2024年5月4日

在 GitHub 查看贡献者指南
 (1 条评论) (0 个反应) (0 位负责人)C++ (21,764 个派生)batch import
buggood first issue

仓库指标

星标
 (124,134 个星标)
PR 合并指标
 (平均合并 6天 8小时) (30 天内合并 389 个 PR)

描述

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.

贡献者指南

研究方向
问题出现在 clip.cpp 的第 1277 行和第 2075 行，其中使用 extern "C" 声明的函数（clip model load 和 clip n mmproj embd）具有非抛出异常规范，但仍可能抛出异常。修复方法是将函数体包装在 try catch 块中，返回错误指示符（例如 1 或 nullptr），而不是允许异常传播。检查文件中现有的错误处理模式，并确保所有外部可见的函数都遵循此规则。目前尚无关联的 PR 或分配者。
技术栈
cppc
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
新近可参与
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
C++ exception handlingextern "C" linkage
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
40