[m-rev.] for post-commit review: handle all builtin types in library/term_to_xml.m

Zoltan Somogyi zoltan.somogyi at runbox.com
Wed Oct 16 18:06:39 AEDT 2019



On Wed, 16 Oct 2019 17:33:41 +1100 (AEDT), Julien Fischer <jfischer at opturion.com> wrote:
> +    ;       foreign(ftype)
> +    ;       primitives(
> +                int, uint, int8, int16, int32, int64,
> +                uint8, uint16, uint32, uint64).

Why is the uint field not with the uintN fields?

Apart from that, the diff is fine, though it would be nice if you could document
why the LLDS and MLDS backends need different expected output files.

Zoltan.


More information about the reviews mailing list