Patch to allow Ada to work with tree-ssa
Mark Mitchell
mark@codesourcery.com
Wed Jun 23 23:10:00 GMT 2004
Richard Kenner wrote:
> You also managed to add another quadratic algorithm somewhere,
> see gcc.gnu.org/PR16163. Can you please fix this asap.
>
>Well, it's only quadratic due to the way the testcase is constructed. And
>the reason is due to something that I already know about and will be fixed by
>a change I plan to make soon anyway.
>
>The issue here is that the tree walker didn't used to look at the fields of
>types. So it missed variables in DECL_FIELD_OFFSET, for example. Now that
>it does, the cost of walking through a type is proportional to the number of
>fields in the type.
>
>
If that's the only reason you're doing this walk, then you should just
have a tag on types that indicates whether or not they have any fields
with run-time offsets. There's no reason to be penalizing every
standard C and C++ program for a feature used rarely in GNU C, never in
GNU C++, and never in ISO C.
Please fix this ASAP.
I consider this a major regression.
Is there another global write person who will support me in reverting
this patch, if 48 hours go by and this problem has not been solved?
--
Mark Mitchell
CodeSourcery, LLC
(916) 791-8304
mark@codesourcery.com
More information about the Gcc
mailing list