On Tue, 16 Apr 2024, Jakub Jelinek wrote: > Hi! > > While studying the TYPE_CANONICAL/TYPE_STRUCTURAL_EQUALITY_P stuff, > I've noticed some nits in comments, the following patch fixes them. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?
OK (IMO obvious). Richard. > 2024-04-16 Jakub Jelinek <ja...@redhat.com> > > * tree.cc (array_type_nelts): Ensure 2 spaces after . in comment > instead of just one. > (build_variant_type_copy): Likewise. > (tree_check_failed): Likewise. > (build_atomic_base): Likewise. > * ipa-free-lang-data.cc (fld_incomplete_type_of): Use an indefinite > article rather than a. > > --- gcc/tree.cc.jj 2024-04-15 14:25:21.721813546 +0200 > +++ gcc/tree.cc 2024-04-15 17:49:25.923935286 +0200 > @@ -3689,7 +3689,7 @@ int_byte_position (const_tree field) > } > > /* Return, as a tree node, the number of elements for TYPE (which is an > - ARRAY_TYPE) minus one. This counts only elements of the top array. */ > + ARRAY_TYPE) minus one. This counts only elements of the top array. */ > > tree > array_type_nelts (const_tree type) > @@ -5757,7 +5757,7 @@ build_variant_type_copy (tree type MEM_S > t = build_distinct_type_copy (type PASS_MEM_STAT); > > /* Since we're building a variant, assume that it is a non-semantic > - variant. This also propagates TYPE_STRUCTURAL_EQUALITY_P. */ > + variant. This also propagates TYPE_STRUCTURAL_EQUALITY_P. */ > TYPE_CANONICAL (t) = TYPE_CANONICAL (type); > /* Type variants have no alias set defined. */ > TYPE_ALIAS_SET (t) = -1; > @@ -8915,7 +8915,7 @@ get_file_function_name (const char *type > #if defined ENABLE_TREE_CHECKING && (GCC_VERSION >= 2007) > > /* Complain that the tree code of NODE does not match the expected 0 > - terminated list of trailing codes. The trailing code list can be > + terminated list of trailing codes. The trailing code list can be > empty, for a more vague error message. FILE, LINE, and FUNCTION > are of the caller. */ > > @@ -9332,7 +9332,7 @@ make_or_reuse_accum_type (unsigned size, > > /* Create an atomic variant node for TYPE. This routine is called > during initialization of data types to create the 5 basic atomic > - types. The generic build_variant_type function requires these to > + types. The generic build_variant_type function requires these to > already be set up in order to function properly, so cannot be > called from there. If ALIGN is non-zero, then ensure alignment is > overridden to this value. */ > --- gcc/ipa-free-lang-data.cc.jj 2024-04-15 14:25:21.668814259 +0200 > +++ gcc/ipa-free-lang-data.cc 2024-04-15 17:49:47.303647823 +0200 > @@ -234,7 +234,7 @@ fld_decl_context (tree ctx) > return ctx; > } > > -/* For T being aggregate type try to turn it into a incomplete variant. > +/* For T being aggregate type try to turn it into an incomplete variant. > Return T if no simplification is possible. */ > > static tree > > Jakub > > -- Richard Biener <rguent...@suse.de> SUSE Software Solutions Germany GmbH, Frankenstrasse 146, 90461 Nuernberg, Germany; GF: Ivo Totev, Andrew McDonald, Werner Knoblich; (HRB 36809, AG Nuernberg)