[BUG] `af_set_backend` is declared with the wrong parameter width

未关闭 适合新手
#388 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
1/5
预计耗时
1 小时以内
新手友好度
88/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
冷清
技术栈
c, rust
领域
backend

调研方向

从 src/core/backend.rs:7 开始，将 af_set_backend 与同一 extern 块中的相邻声明进行比较。检查 include/af/backend.h:24 以确认 C enum 类型，然后更新 Rust 声明使其与 ABI 匹配，接着运行仓库的 Rust 检查或测试。当该声明使用正确的参数宽度且检查通过时，即表示完成。

由索引模型根据 Issue 内容生成。

描述

Bug

src/core/backend.rs:7:

extern "C" {
    fn af_set_backend(bknd: u8) -> c_int;

The C signature (include/af/backend.h:24) is:

AFAPI af_err af_set_backend(const af_backend bknd);

af_backend is a C enum, i.e. 4-byte int under both MSVC and GCC. Declaring the parameter as u8 is an ABI type mismatch.

In practice it works, because rustc emits zeroext for the u8 argument and the callee reads a clean value from the full register — I checked the IR (declare noundef i32 @af_set_backend(i8 noundef zeroext)). So this is as a latent correctness issue rather than an active bug, but it should still be c_uint to match the C ABI.

The three neighbouring declarations in the same extern block already use c_uint/c_int correctly.

Found by Claude Opus 5.

主要语言
Rust
星标
827
派生
59
30 天内没有已合并 PR

贡献指南

这个仓库没有索引到贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

arrayfire/arrayfire-rust 的其他 Issue

查看 arrayfire/arrayfire-rust 的全部 Issue

相似的 Issue

更多 Rust Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。