> -----Original Message----- > From: Jakub Jelinek <[email protected]> > Sent: 01 July 2026 11:11 > To: [email protected]; Richard Earnshaw <[email protected]>; > Tamar Christina <[email protected]> > Cc: [email protected] > Subject: [PATCH] aarch64: Remove some spurious semicolons > > Hi! > > When building with GCC 8, I'm seeing > ../../gcc/config/aarch64/aarch64-elf-metadata.cc:71:2: warning: extra ‘;’ [- > Wpedantic] > ../../gcc/config/aarch64/aarch64-sve-builtins-shapes.cc:5456:17: warning: > extra ‘;’ [-Wpedantic] > ../../gcc/config/aarch64/aarch64-sve-builtins-shapes.cc:5493:22: warning: > extra ‘;’ [-Wpedantic] > ../../gcc/config/aarch64/tuning_models/neoversev2.h:320:2: warning: extra > ‘;’ [-Wpedantic] > warnings. The following patch fixes that. > > Ok for trunk?
Ok. Thanks, Tamar > > 2026-07-01 Jakub Jelinek <[email protected]> > > * config/aarch64/tuning_models/neoversev2.h > (neoversev2_dispatch_constraint_callback): Remove spurious > semicolon. > * config/aarch64/aarch64-elf-metadata.cc > (gnu_property_features_to_string): Likewise. > * config/aarch64/aarch64-sve-builtins-shapes.cc (SHAPE (write_zt), > SHAPE (write_lane_zt)): Likewise. > > --- gcc/config/aarch64/tuning_models/neoversev2.h.jj 2026-03-27 > 10:17:13.772336646 +0100 > +++ gcc/config/aarch64/tuning_models/neoversev2.h 2026-07-01 > 11:46:26.610063825 +0200 > @@ -317,7 +317,7 @@ neoversev2_dispatch_constraint_callback > constraints.safe_push ({TOTAL_SLOTS, total_slots}); > > return constraints; > -}; > +} > > /* Neoverse V2 dispatch constraints configuration. */ > static const struct dispatch_constraint_info > neoversev2_dispatch_constraint_info = { > --- gcc/config/aarch64/aarch64-elf-metadata.cc.jj 2026-03-27 > 10:17:13.756336907 +0100 > +++ gcc/config/aarch64/aarch64-elf-metadata.cc 2026-07-01 > 11:43:39.981150836 +0200 > @@ -68,7 +68,7 @@ gnu_property_features_to_string (unsigne > separator = ", "; > } > return s_features; > -}; > +} > > } // namespace anonymous > > --- gcc/config/aarch64/aarch64-sve-builtins-shapes.cc.jj 2026-06-02 > 08:15:04.502142863 +0200 > +++ gcc/config/aarch64/aarch64-sve-builtins-shapes.cc 2026-07-01 > 11:45:24.805837917 +0200 > @@ -5434,7 +5434,7 @@ struct write_zt_def : public overloaded_ > return c.require_immediate_range (0, 0, 0); > } > }; > -SHAPE (write_zt); > +SHAPE (write_zt) > > /* MOVT (vector to table) > Variants are also available for: > @@ -5471,5 +5471,5 @@ struct write_lane_zt_def : public overlo > && c.require_immediate_range (2, 0, 3); > } > }; > -SHAPE (write_lane_zt); > +SHAPE (write_lane_zt) > } > > Jakub
