Am Dienstag, dem 03.06.2025 um 11:22 -0700 schrieb Andrew Pinski:
> On Tue, Jun 3, 2025 at 11:18 AM Martin Uecker <uec...@tugraz.at> wrote:
> >
> > Am Dienstag, dem 03.06.2025 um 10:56 -0700 schrieb Andrew Pinski:
> > > On Tue, Jun 3, 2025 at 10:45 AM Martin Uecker <uec...@tugraz.at> wrote:
> > > >
> > > >
> > > > This version only contains the fix for -Wc++-compat.
> > >
> > > Note this was recorded as PR 120078. I didn't notice the typo when I
> > > filed the bug report; otherwise I would have fixed it.
> > > Note I think the typo fix is obvious and should be backported to the
> > > open branches too as it was working with GCC 4.7.0 and was only broken
> > > with r0-116778-gf2bc201f53e2b8 which introduced the typo.
> >
> > Is there a reason component was set to middle-end for the bug?
> > I usually only have to look at C FE bugs.
>
> Because at the time I filed it, I thought it was an issue with
> processing the .opt file rather than an issue inside the .opt file.
> I still didn't see the typo until you posted the patch.
ok, committed with the PR and the semicolon.
Martin
>
> Thanks,
> Andrew
>
> >
> > Martin
> >
> > >
> > > Thanks,
> > > Andrew Pinski
> > >
> > > >
> > > > Bootstrapped and regression tested for x86_64.
> > > >
> > > > Martin
> > > >
> > > >
> > > > c: Enable -Wjump-misses-init for -Wc++-compat
> > > >
> > > > Fix a typo that prevented the warning from being activated with
> > > > -Wc++compat.
> > > >
> > > > gcc/c-family/ChangeLog:
> > > > * c.opts (Wjump-misses-init): Fix typo.
> > > >
> > > > gcc/testsuite/ChangeLog:
> > > > * gcc.dg/Wjump-misses-init-3.c: New test.
> > > >
> > > > diff --git a/gcc/c-family/c.opt b/gcc/c-family/c.opt
> > > > index 75b6531860e..50ba856fedb 100644
> > > > --- a/gcc/c-family/c.opt
> > > > +++ b/gcc/c-family/c.opt
> > > > @@ -938,7 +938,7 @@ C ObjC C++ ObjC++ CPP(cpp_warn_invalid_utf8)
> > > > CppReason(CPP_W_INVALID_UTF8) Var(w
> > > > Warn about invalid UTF-8 characters.
> > > >
> > > > Wjump-misses-init
> > > > -C ObjC Var(warn_jump_misses_init) Warning LangEnabledby(C
> > > > ObjC,Wc++-compat)
> > > > +C ObjC Var(warn_jump_misses_init) Warning LangEnabledBy(C
> > > > ObjC,Wc++-compat)
> > > > Warn when a jump misses a variable initialization.
> > > >
> > > > Enum
> > > > diff --git a/gcc/testsuite/gcc.dg/Wjump-misses-init-3.c
> > > > b/gcc/testsuite/gcc.dg/Wjump-misses-init-3.c
> > > > new file mode 100644
> > > > index 00000000000..36ebd371cb6
> > > > --- /dev/null
> > > > +++ b/gcc/testsuite/gcc.dg/Wjump-misses-init-3.c
> > > > @@ -0,0 +1,10 @@
> > > > +/* { dg-do compile } */
> > > > +/* { dg-options "-Wc++-compat" } */
> > > > +
> > > > +void f()
> > > > +{
> > > > + goto skip; /* { dg-warning "jump skips variable
> > > > initialization" } */
> > > > + int i = 1;
> > > > +skip:
> > > > +}
> > > > +
> > > >
> >
