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

Đang mở Phù hợp với người mới
#388 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
88/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
c, rust
Lĩnh vực
backend

Hướng nghiên cứu

Bắt đầu tại src/core/backend.rs:7 và so sánh af_set_backend với các khai báo lân cận trong cùng extern block. Kiểm tra include/af/backend.h:24 để xác nhận kiểu enum C, sau đó cập nhật khai báo Rust cho khớp với ABI và chạy các bước kiểm tra hoặc test Rust của repository. Hoàn tất khi khai báo sử dụng độ rộng tham số chính xác và các bước kiểm tra đều đạt.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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.

Ngôn ngữ chính
Rust
Star
827
Fork
59
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của arrayfire/arrayfire-rust

Tất cả issue của arrayfire/arrayfire-rust

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.