This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH]: Fix alias grouping of name tags.
Eric Botcazou <ebotcazou@libertysurf.fr> writes:
> > I've already said why i disagree that it is non-obvious, and not to
> > belabor the pointer, but:
> > I think it's actually obvious to since it is just used for sorting a
> > list of pointers and removing duplicates.
>
> The problem is of course that the resulting order "escapes" from the piece of
> code if there are duplicates in the list so, asumming this case, if you enter
> compact_name_tags with the exact same lists on different hosts, you can exit
> with lists sorted differently. That this doesn't have any impact whatsoever
> on code generation is IMHO non-obvious.
I think you are both talking past each other. Eric, I agree that a
comment would be appropriate. However, I think Danny is correct that
you should write that comment. The code is obvious to Danny, so he
won't write the right comment. Eric, you should have enough
information to write the comment at this point.
Ian