diff mbox

[libstdc++] : Fix PR/51673

Message ID CAEwic4borDXPhwyOVorBudwJd1xiM0uvbRquxxBcdNbbjE0ALA@mail.gmail.com
State New
Headers show

Commit Message

Kai Tietz Jan. 1, 2012, 10:40 p.m. UTC
Hello,

additionally to the suggested patch by Pawel Sikora, I added the
adjustments for mt-allocator to it too.

ChangeLog

2012-01-01  Kai Tietz  <ktietz@redhat.com>

        PR libstc++/51673
      * config/abi/pre/gnu-versioned-namespace.ver: Adjusted new/delete
      operators signature for LLP64 size_t, and adjusted signatures for
      mt-allocator using size_t.

Tested for x86_64-w64-mingw32, and regression tested also for
x86_64-unknown-linux-gnu.  Ok for apply to trunk and 4.6.x?

Kai

Comments

Paolo Carlini Jan. 1, 2012, 11:39 p.m. UTC | #1
Hi,
> Hello,
>
> additionally to the suggested patch by Pawel Sikora, I added the
> adjustments for mt-allocator to it too.
>
> ChangeLog
>
> 2012-01-01  Kai Tietz<ktietz@redhat.com>
>
>          PR libstc++/51673
>        * config/abi/pre/gnu-versioned-namespace.ver: Adjusted new/delete
>        operators signature for LLP64 size_t, and adjusted signatures for
>        mt-allocator using size_t.
>
> Tested for x86_64-w64-mingw32, and regression tested also for
> x86_64-unknown-linux-gnu.  Ok for apply to trunk and 4.6.x?
Benjamin, can you have a look to this? I suspect the other established 
allocators (like, eg, pool) should be also dealt with, but I don't know 
versioned namespace well enough to properly review the patch.

Thanks,
Paolo.
Kai Tietz Jan. 5, 2012, 11:18 a.m. UTC | #2
Ping!

2012/1/2 Paolo Carlini <paolo.carlini@oracle.com>:
> Hi,
>
>> Hello,
>>
>> additionally to the suggested patch by Pawel Sikora, I added the
>> adjustments for mt-allocator to it too.
>>
>> ChangeLog
>>
>> 2012-01-01  Kai Tietz<ktietz@redhat.com>
>>
>>         PR libstc++/51673
>>       * config/abi/pre/gnu-versioned-namespace.ver: Adjusted new/delete
>>       operators signature for LLP64 size_t, and adjusted signatures for
>>       mt-allocator using size_t.
>>
>> Tested for x86_64-w64-mingw32, and regression tested also for
>> x86_64-unknown-linux-gnu.  Ok for apply to trunk and 4.6.x?
>
> Benjamin, can you have a look to this? I suspect the other established
> allocators (like, eg, pool) should be also dealt with, but I don't know
> versioned namespace well enough to properly review the patch.
>
> Thanks,
> Paolo.
diff mbox

Patch

Index: gnu-versioned-namespace.ver
===================================================================
--- gnu-versioned-namespace.ver (revision 182720)
+++ gnu-versioned-namespace.ver (working copy)
@@ -42,9 +42,9 @@ 
     __once_proxy;

     # operator new(size_t)
-    _Znw[jm];
+    _Znw[jmy];
     # operator new(size_t, std::nothrow_t const&)
-    _Znw[jm]RKSt9nothrow_t;
+    _Znw[jmy]RKSt9nothrow_t;

     # operator delete(void*)
     _ZdlPv;
@@ -52,9 +52,9 @@ 
     _ZdlPvRKSt9nothrow_t;

     # operator new[](size_t)
-    _Zna[jm];
+    _Zna[jmy];
     # operator new[](size_t, std::nothrow_t const&)
-    _Zna[jm]RKSt9nothrow_t;
+    _Zna[jmy]RKSt9nothrow_t;

     # operator delete[](void*)
     _ZdaPv;
@@ -93,16 +93,16 @@ 

     # __gnu_cxx::__pool
     _ZN9__gnu_cxx3__76__poolILb[01]EE13_M_initializeEv;
-    _ZN9__gnu_cxx3__76__poolILb[01]EE16_M_reserve_blockE[jm][jm];
-    _ZN9__gnu_cxx3__76__poolILb[01]EE16_M_reclaim_blockEPc[jm];
+    _ZN9__gnu_cxx3__76__poolILb[01]EE16_M_reserve_blockE[jmy][jmy];
+    _ZN9__gnu_cxx3__76__poolILb[01]EE16_M_reclaim_blockEPc[jmy];
     _ZN9__gnu_cxx3__76__poolILb[01]EE10_M_destroyEv;
     _ZN9__gnu_cxx3__76__poolILb1EE16_M_get_thread_idEv;

-    _ZN9__gnu_cxx3__717__pool_alloc_base9_M_refillE[jm];
-    _ZN9__gnu_cxx3__717__pool_alloc_base16_M_get_free_listE[jm];
+    _ZN9__gnu_cxx3__717__pool_alloc_base9_M_refillE[jmy];
+    _ZN9__gnu_cxx3__717__pool_alloc_base16_M_get_free_listE[jmy];
     _ZN9__gnu_cxx3__717__pool_alloc_base12_M_get_mutexEv;

-    _ZN9__gnu_cxx3__79free_list6_M_getE[jm];
+    _ZN9__gnu_cxx3__79free_list6_M_getE[jmy];
     _ZN9__gnu_cxx3__79free_list8_M_clearEv;

     # __gnu_cxx::stdio_sync_filebuf