[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