[patch] Allocate identifier node for user-specified assembler
Mark Mitchell
mark@codesourcery.com
Tue Mar 1 22:27:00 GMT 2005
John David Anglin wrote:
>>It would be better if you could instead avoid relying on this kind of
>>use of identifiers. These uses are a significant performance impact
>>on GCC and the ability to do them will go away in the future.
>
>
> Are you suggesting that it would be better to do any necessary
> tracking of referenced symbols in the backend? In the PA case, we only
> need to do this for externals and the number usually isn't that large.
I'm not sure exacty what Geoff's suggesting, but it doesn't seem
practical to avoid creating the IDENTIFIER_NODE right now. (As Geoff
says, it would be nice to do so eventually, but the machinery doesn't
work that way yet.)
Fortunately, as you say, this code path effects only functions for which
the user has explicitly specified the assembler name; hopefully, there
are relatively few in most programs.
I think your patch is correct, and should be applied. Please go ahead,
but give Geoff twenty-four hours to object, in case he thinks there's a
better way to do this in the 4.0 timeframe.
Thanks,
--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com
(916) 791-8304
More information about the Gcc-patches
mailing list