Torvald Riegel <trie...@redhat.com> writes: > On Mon, 2011-11-21 at 15:06 +0100, Rainer Orth wrote: >> * _ITM_getThreadnum is the only symbol in libitm.map that isn't present >> in the library. It's documented as missing and should perhaps be >> removed from the map? > > Yes, this is not supported anymore.
Here's the patch that removes it. I think this should go in before 4.7.0 is released: if the function is added later, it would incorrectly end up in the LIBITM_1.0, defeating the purpose of symbol versioning. Bootstrapped without regressions on i386-pc-solaris2.11 and verified that the symbol version info is unchanged. Ok for mainline? Rainer 2012-01-14 Rainer Orth <r...@cebitec.uni-bielefeld.de> * libitm.h (_ITM_getThreadnum): Remove. * libitm.map (_ITM_getThreadnum): Remove from LIBITM_1.0.
# HG changeset patch # Parent e9235a85428f1c332d0b4346142aabc0d3c3990c Remove _ITM_getThreadnum diff --git a/libitm/libitm.h b/libitm/libitm.h --- a/libitm/libitm.h +++ b/libitm/libitm.h @@ -1,4 +1,4 @@ -/* Copyright (C) 2008, 2009, 2011 Free Software Foundation, Inc. +/* Copyright (C) 2008, 2009, 2011, 2012 Free Software Foundation, Inc. Contributed by Richard Henderson <r...@redhat.com>. This file is part of the GNU Transactional Memory Library (libitm). @@ -149,8 +149,6 @@ extern void _ITM_addUserCommitAction(_IT extern void _ITM_addUserUndoAction(_ITM_userUndoFunction, void *) ITM_REGPARM; -extern int _ITM_getThreadnum(void) ITM_REGPARM; - extern void _ITM_dropReferences (void *, size_t) ITM_REGPARM ITM_PURE; extern void *_ITM_malloc (size_t) diff --git a/libitm/libitm.map b/libitm/libitm.map --- a/libitm/libitm.map +++ b/libitm/libitm.map @@ -8,7 +8,6 @@ LIBITM_1.0 { _ITM_commitTransaction; _ITM_commitTransactionEH; _ITM_error; - _ITM_getThreadnum; _ITM_getTransactionId; _ITM_inTransaction; _ITM_libraryVersion;
-- ----------------------------------------------------------------------------- Rainer Orth, Center for Biotechnology, Bielefeld University