This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Patch to categorize builtins.def
"Kaveh R. Ghazi" <ghazi@caip.rutgers.edu> writes:
> > From: "Zack Weinberg" <zack@codesourcery.com>
> >
> > "Kaveh R. Ghazi" <ghazi@caip.rutgers.edu> writes:
> >
> > > In working on builtins.def, it's much easier for me to work on
> > > conceptual classes of builtins. So this patch breaks the builtins
> > > list into groups.
> >
> > Um. Didn't you argue for not having any groupings just this morning?
> > zw
>
> Seems like an eternity...
>
> I'm not against groupings per se. I just noted that we didn't do it
> at the time and that was in support of sorting atan2 alphabetically in
> between atan and atanf. I.e. not having scattered one-off exceptions
> to an automated sort. Perhaps I'm contradicting myself, can I change
> my mind? :-)
>
> Regarding categories, what I'm finding is that when I want to
> e.g. work on a testcase for math builtins, it helps to be able to
> quickly scan the list of just math builtins in one section rather than
> read every single line in the file trying to pick out certain ones.
I thought it might be something like this, but I'm very tired and
might have been confused, so I wanted to make sure. The patch is
OK.
zw