This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Implement -Wimplicit-fallthrough (take 2): fix missing breaks
- From: Marek Polacek <polacek at redhat dot com>
- To: Uros Bizjak <ubizjak at gmail dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 29 Jul 2016 14:35:09 +0200
- Subject: Re: Implement -Wimplicit-fallthrough (take 2): fix missing breaks
- Authentication-results: sourceware.org; auth=none
- References: <CAFULd4YGCnVkPtVW-hd-7O=YiTB6nTfQ8Q97_4N9m1uTRAL1_g@mail.gmail.com>
On Fri, Jul 29, 2016 at 02:14:01PM +0200, Uros Bizjak wrote:
> Hello!
>
> > This is what the new warning pointed out. I think all these are bugs.
> >
> > This patch has been tested on powerpc64le-unknown-linux-gnu, aarch64-linux-gnu,
> > and x86_64-redhat-linux.
> >
> > 2016-07-27 Marek Polacek <polacek@redhat.com>
> >
> > PR c/7652
> > gcc/
> > * config/i386/i386.c (ix86_expand_args_builtin): Add break.
> > (ix86_expand_round_builtin): Likewise.
> > * config/rs6000/rs6000.c (altivec_expand_ld_builtin): Add break.
> > (altivec_expand_st_builtin): Likewise.
> > * gengtype.c (dbgprint_count_type_at): Add break.
> > gcc/java/
> > * jcf-dump.c (print_constant): Add break.
> > libgo/
> > * runtime/heapdump.c (dumpefacetypes): Add break.
>
> x86 part is OK, also for backports to other release branches.
Thanks, will do (except 4.9, most likely).
Marek