ocaml/dune

Add support for assembly language for foreign_stubs

Ouverte

#3 342 ouverte le 8 avr. 2020

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)OCaml (478 forks)github user discovery
good first issue

Métriques du dépôt

Stars
 (1 882 étoiles)
Métriques de merge PR
 (Merge moyen 1j 20h) (345 PRs mergées en 30 j)

Description

The doc for foreign_stubs says:

language specifies the source language, where c means C and cxx means C++. In future, more languages may be supported.

It would be nice to add support for .S files too ; generally they can just be passed to the underlying C compiler just fine ; I'm not fully sure of what should be the assembly flags. I think keeping the flags as C by default should generally work.

Guide contributeur

Direction de recherche
Examinez comment dune gère les foreign stubs pour les langages C et C++, et étendez cette logique pour accepter les fichiers assembleur (.S). Il suffit probablement de les transmettre tels quels au compilateur C, mais vérifiez la gestion des indicateurs.
Stack technique
ccpp
Domaine
build system
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
OCamlC
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75