On Tue, Jun 11, 2019 at 01:30:41AM +0000, Feng Xue OS wrote: > Some minor changes. > > Feng > > --- > diff --git a/gcc/ChangeLog b/gcc/ChangeLog > index 526ed45be89..469b54ddf93 100644 > --- a/gcc/ChangeLog > +++ b/gcc/ChangeLog > @@ -1,3 +1,14 @@ > +2019-06-04 Feng Xue <f...@os.amperecomputing.com> > + > + PR ipa/90401 > + * ipa-prop.c (add_to_agg_contents_list): New function. > + (clobber_by_agg_contents_list_p): Likewise. > + (extract_mem_content): Likewise. > + (get_place_in_agg_contents_list): Delete. > + (determine_known_aggregate_parts): Renamed from > + determine_locally_known_aggregate_parts. New parameter > + aa_walk_budget_p. > + > 2019-06-04 Segher Boessenkool <seg...@kernel.crashing.org> > > * config/rs6000/constraints.md (define_register_constraint "wp"): > diff --git a/gcc/ipa-prop.c b/gcc/ipa-prop.c > index d86c2f3db55..2fce69547dd 100644 > --- a/gcc/ipa-prop.c > +++ b/gcc/ipa-prop.c > @@ -1458,7 +1458,7 @@ get_ssa_def_if_simple_copy (tree rhs) > return rhs; > } > > -/* Simple linked list, describing known contents of an aggregate beforere > +/* Simple linked list, describing known contents of an aggregate before > call. */ > > struct ipa_known_agg_contents_list > @@ -1471,41 +1471,48 @@ struct ipa_known_agg_contents_list > struct ipa_known_agg_contents_list *next; > }; > > -/* Find the proper place in linked list of ipa_known_agg_contents_list > - structures where to put a new one with the given LHS_OFFSET and LHS_SIZE, > - unless there is a partial overlap, in which case return NULL, or such > - element is already there, in which case set *ALREADY_THERE to true. */ > +/* Add a known content item into a linked list of ipa_known_agg_contents_list > + structure, in which all elements are sorted ascendingly by offset. */
For future reference, there should be two spaces at the end of the sentence before */. You can use gcc/contrib/check_GNU_style.sh foo.patch to catch stuff like this before posting a final patch. Thanks, Marek
