Fwd: Re[3]: Boehm GC API update backport

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Fwd: Re[3]: Boehm GC API update backport

Ivan Maidanski
Hello,

Is there anything more preventing it to be merged to trunk?

The patches (rebased to recent trunk) are attached. (Actually only the first one is updated due to merge conflict in ChangeLog files.)

Thank you.

Regards,
Ivan

>Wed, 20 Jan 2016, 1:41 +03:00 from Ivan Maidanski < [hidden email] >:
>
>Hello Matthias,
>
>I've checked objc tests - same as without the patches.
>Actually objc uses only GC_malloc/realloc and GC typed API - these are not affected.
>
>Regards,
>Ivan
>
>> Sun, 17 Jan 2016, 1:55 +01:00 from Matthias Klose < [hidden email] >:
>>
>> On 16.01.2016 13:19, Ivan Maidanski wrote:
>> > Hello,
>> >
>> > Proposed 4 commits are code refactoring of boehm-gc (and libjava/boehm.cc, accordingly) to match API of recent BDWGC master (  https://github.com/ivmai/bdwgc ). This should simplify optional replacement of internal boehm-gcc with the external one.
>> > All gcj tests pass.
>> >
>> > Some of the benefits of using mainline boehm gc:
>> > *  has no compiler warning
>> > *  more targets supported (e.g. arm64)
>> > *  no trouble with back-porting gc patches Similar thing was proposed for the Mono community -  https://github.com/mono/mono/pull/2247 (the patches were accepted recently)
>> >
>> > References:
>> > [1] The proposed patches in GCC unofficial git mirror fork -  https://github.com/ivmai/gcc/tree/match-bdwgc-api
>>
>> for completeness, please check an ObjC enabled build configured with
>> --enable-objc-gc
>>
>> Matthias
>>

=?UTF-8?B?MDAwMS1DaGFuZ2UtR0Nfc2V0X2ZyZWVfc3BhY2VfZGl2aXNvci1wcm90b3R5?= =?UTF-8?B?cGUtaW4tYm9laG0tLnBhdGNo?= (5K) Download Attachment
=?UTF-8?B?MDAwMi1Nb3ZlLWV4dGVybi1DLWZyb20tYm9laG0uY2MtdG8tYm9laG0tZ2Mt?= =?UTF-8?B?aGVhZGVyLWZpbGVzLnBhdGNo?= (11K) Download Attachment
=?UTF-8?B?MDAwNC1DaGFuZ2UtR0NfLXVuLXJlZ2lzdGVyX215X3RocmVhZC1wcm90b3R5?= =?UTF-8?B?cGVzLWluLWJvZWhtLnBhdGNo?= (12K) Download Attachment
=?UTF-8?B?MDAwMy1DaGFuZ2UtR0NfZ2V0Xy10aHJlYWQtbWFpbl8tc3RhY2tfYmFzZS1w?= =?UTF-8?B?cm90b3R5cGVzLnBhdGNo?= (22K) Download Attachment