ggml-org/llama.cpp

Llava functions compiled as extern "C" throw exceptions

已關閉

#7,073 建立於 2024年5月4日

在 GitHub 查看貢獻者指南
 (1 則留言) (0 個反應) (0 位負責人)C++ (21,786 個分叉)batch import
buggood first issue

倉庫指標

星標
 (124,222 顆星)
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