This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch] make it possible for the target to rename ".tm_clone_table"
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~