[m-rev.] for review: Disable uninitialized variable warnings in high-level C grades.

Peter Wang novalazy at gmail.com
Fri May 31 17:20:07 AEST 2019


On Fri, 31 May 2019 17:12:47 +1000 (AEST), Julien Fischer <jfischer at opturion.com> wrote:
> 
> Hi Peter,
> 
> On Fri, 31 May 2019, Peter Wang wrote:
> 
> > scripts/mgnuc.in:
> >    Disable uninitialized variable warnings in high-level C grades.
> >    This is necessary but went (mostly) unnoticed because we happen to
> >    disable uninitialized variable warnings for gcc on x86/x86_64 in
> >    support of low-level C grades, and we suppress any and all warnings
> >    if using clang.
> 
> IIRC, disabling all warnings was necessary with some very early versions
> of clang -- we should probably re-enable them at some
> point since we no longer support those versions of clang.

Yes, I sent an incomplete patch last year but then got distracted by
trying to "fix" fields in RTTI structures from signed to unsigned types,
which then got caught up how to deal with Java RTTI fields since Java
doesn't have unsigned types... that was a mistake.

Peter


More information about the reviews mailing list