[m-rev.] for review: rename the .ll_debug grade component for hlc grades to .c_debug

Julien Fischer jfischer at opturion.com
Wed Dec 28 13:29:04 AEDT 2022


On Wed, 28 Dec 2022, Zoltan Somogyi wrote:

> 2022-12-27 15:24 GMT+11:00 "Julien Fischer" <jfischer at opturion.com>:
>>>     Or should we have it also set the value
>>>     of the c_debug sh variable, thereby implying the value of this grade
>>>     component? The latter seems iffy, but I also can't think of any good
>>>     new name for mgnuc's own -c-debug option.
>>
>> I suggest something like:
>>
>>       --debug-info
>>       --include-debug-info
>>       --emit-debug-info
>
> I tried this out with --cdebug-info, but the result was less than satisfactory,
> due to the fact that the same option names now mean different things
> when given to mmc versus when given to mgnuc.
>
> I therefore propose the following scheme:
>
> - Both mmc and mgnuc should accept an option named --c-debug, which
>  should cause both to pass -g to the C compiler, *without* affecting the grade.
>
> - Both mmc and mgnuc should accept an option named --c-debug-grade,
>  which will add .c_debug to the grade (for hlc grades), and will therefore
>  also pass -g to the C compiler.
>
> The internal variables in the compiler and in shell scripts should also follow
> this naming scheme, which should
>
> - suggest what the difference between the two options is, and
> - should make it easy to fully document that difference in the user guide
>  and in help messages.
>
> Any objections, or suggestions for better naming schemes?

No, that scheme seems fine.

Julien.


More information about the reviews mailing list