[Patch] make it possible for the target to rename ".tm_clone_table"

Richard Henderson rth@redhat.com
Sat Nov 19 20:24:00 GMT 2011


On 11/19/2011 02:04 AM, Iain Sandoe wrote:
> It should be possible for Rainer to define
> TARGET_ASM_TM_CLONE_TABLE_SECTION_NAME to NULL, and have varasm use
> the normal data section for tm_clone_table sections.    One might
> wish to re-adjust testsuite/gcc.dg/tm/20100615.c if that is done.

Well, not really.  OSF is going to need an extension to collect2 in
order to create a single table, or changes to other bits of the 
compiler to invoke _ITM_registerTMCloneTable for each object file's
table.

... actually, there seems to be some sort of merge error.  I don't
see the crtstuff.c changes that called _ITM_registerTMCloneTable in
the first place.

> +/* Set if the target wants to override the default ".tm_clone_table"
> +   section name.  */
> +DEFHOOKPOD
> +(tm_clone_table_section_name,
> + "Define this macro to specify the section name that should be used\
> +  for transactional memory clone tables.  If this is defined to NULL\
> +  the normal data section will be used for the tables instead.\
> +  The default section name is @code{\".tm_clone_table\"}.",
> + const char *, ".tm_clone_table")

I'd prefer we return a section object instead.  I.e. pull the
call to get_named_section into the hook as well.


r~



More information about the Gcc-patches mailing list