ocaml/dune

Add support for assembly language for foreign_stubs

Offen

#3.342 geöffnet am 08.04.2020

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)OCaml (478 Forks)github user discovery
good first issue

Repository-Metriken

Stars
 (1.882 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 20h) (345 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche, wie dune foreign stubs für C und C++ behandelt, und erweitere diese Logik, um Assembler Dateien (.S) zu akzeptieren. Wahrscheinlich genügt es, sie unverändert an den C Compiler zu übergeben, aber überprüfe die Flag Verarbeitung.
Tech Stack
ccpp
Domain
build system
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
OCamlC
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75