This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: Implement -Wimplicit-fallthrough (take 2): fix missing breaks

From: Jeff Law <law at redhat dot com>
To: Marek Polacek <polacek at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Wed, 27 Jul 2016 14:17:35 -0600
Subject: Re: Implement -Wimplicit-fallthrough (take 2): fix missing breaks
Authentication-results: sourceware.org; auth=none
References: <20160727165212.GW7007@redhat.com>

On 07/27/2016 10:52 AM, Marek Polacek wrote:

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.

Note that I think libgo's runtime is shared with golang and that Ianimports the runtime from that project.


jeff

Follow-Ups:
- Re: Implement -Wimplicit-fallthrough (take 2): fix missing breaks
  - From: Marek Polacek

References:
- Implement -Wimplicit-fallthrough (take 2): fix missing breaks
  - From: Marek Polacek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]