[m-rev.] for review: .target_debug

Julien Fischer jfischer at opturion.com
Sun Aug 10 02:40:55 AEST 2025


On Sat, 9 Aug 2025 at 22:39, Zoltan Somogyi <zoltan.somogyi at runbox.com> wrote:
>
> This implements what we recently agreed on. For review by anyone.
> One question: given that .c_debug was documented as being mainly
> for Mercury implementors, does this warrant a NEWS.md entry?

We could get away without one; I don't think we ever announced the
ll_debug / c_debug grades.


> Rename and generalize .c_debug to .target_debug.

...

> diff --git a/scripts/mgnuc_file_opts.sh-subr b/scripts/mgnuc_file_opts.sh-subr
> index f217af317..34c2d42c5 100644
> --- a/scripts/mgnuc_file_opts.sh-subr
> +++ b/scripts/mgnuc_file_opts.sh-subr
> @@ -10,9 +10,11 @@
>  # mgnuc_file_opts.sh-subr:
>  #
>  # An `sh' subroutine for processing the mgnuc options that can be set from
> -# .mgnuc_opts files. It is included twice in mgnuc. Its implementation is
> -# bound tightly to the code of mgnuc.in; all the variables it manipulates
> -# come from there.
> +# .mgnuc_opts files. It is included twice in mgnuc; once to process options
> +# from the command line, and once to process options fro .mgnuc_opts files.

s/fro/from/

The rest looks fine.

Julien.


More information about the reviews mailing list