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

Ouverte Adaptée aux débutants
#388 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
1/5
Temps estimé
Moins d'une heure
Accessibilité débutants
88/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Calme
Stack technique
c, rust
Domaine
backend

Piste de recherche

Commencez dans src/core/backend.rs:7 et comparez af_set_backend aux déclarations voisines du même bloc extern. Consultez include/af/backend.h:24 pour confirmer le type enum C, mettez ensuite à jour la déclaration Rust afin qu’elle corresponde à l’ABI, puis exécutez les vérifications ou tests Rust du dépôt. C’est terminé lorsque la déclaration utilise la largeur de paramètre correcte et que les vérifications passent.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
Rust
Étoiles
827
Forks
59
Aucune PR mergée en 30 j

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de arrayfire/arrayfire-rust

Toutes les issues de arrayfire/arrayfire-rust

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.