On Fri, Sep 30, 2016 at 06:49:17PM +0200, Jan-Benedict Glaw wrote:
> On Thu, 2016-09-29 18:01:46 +0200, Marek Polacek <pola...@redhat.com> wrote:
> > My upcoming fix revealed more places that were missing a fall through
> > marker.
> >
> > Bootstrapped/regtested on x86_64-linux, ppc64-linux, and aarch64-linux-gnu
> > with
> > my fix, applying to trunk.
> >
> > 2016-09-29 Marek Polacek <pola...@redhat.com>
> >
> > * rtti.c (involves_incomplete_p): Add fall through comment.
> >
> > * dwarf2out.c (loc_descriptor): Add fall through comment.
> > (add_const_value_attribute): Likewise.
>
> Maybe gcc/config/alpha/predicates.md:184, too?
>
> /* ... fall through ... */
>
> This showed up on my build robot recently.
Thanks, fixed:
2016-09-30 Marek Polacek <pola...@redhat.com>
* config/alpha/predicates.md: Adjust fall through comment.
diff --git gcc/config/alpha/predicates.md gcc/config/alpha/predicates.md
index 24fa3c2..ca14fad 100644
--- gcc/config/alpha/predicates.md
+++ gcc/config/alpha/predicates.md
@@ -181,7 +181,7 @@
case SUBREG:
if (register_operand (op, mode))
return 1;
- /* ... fall through ... */
+ /* fall through */
case MEM:
return ((TARGET_BWX || (mode != HImode && mode != QImode))
&& general_operand (op, mode));
Marek
