public inbox for java-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Boehm GC API update backport
@ 2016-01-16 12:19 Ivan Maidanski
  2016-01-17  0:55 ` Matthias Klose
  0 siblings, 1 reply; 3+ messages in thread
From: Ivan Maidanski @ 2016-01-16 12:19 UTC (permalink / raw)
  To: GCJ-patches

[-- Attachment #1: Type: text/plain, Size: 712 bytes --]

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

--
Best regards,
Ivan

[-- Attachment #2: 0001-Change-GC_set_free_space_divisor-prototype-in-boehm-.patch --]
[-- Type: application/x-patch, Size: 3726 bytes --]

[-- Attachment #3: 0002-Move-extern-C-from-boehm.cc-to-boehm-gc-header-files.patch --]
[-- Type: application/x-patch, Size: 8536 bytes --]

[-- Attachment #4: 0004-Change-GC_-un-register_my_thread-prototypes-in-boehm.patch --]
[-- Type: application/x-patch, Size: 9439 bytes --]

[-- Attachment #5: 0003-Change-GC_get_-thread-main_-stack_base-prototypes.patch --]
[-- Type: application/x-patch, Size: 16411 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Boehm GC API update backport
  2016-01-16 12:19 Boehm GC API update backport Ivan Maidanski
@ 2016-01-17  0:55 ` Matthias Klose
  2016-01-19 22:41   ` Re[2]: " Ivan Maidanski
  0 siblings, 1 reply; 3+ messages in thread
From: Matthias Klose @ 2016-01-17  0:55 UTC (permalink / raw)
  To: Ivan Maidanski, GCJ-patches

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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re[2]: Boehm GC API update backport
  2016-01-17  0:55 ` Matthias Klose
@ 2016-01-19 22:41   ` Ivan Maidanski
  0 siblings, 0 replies; 3+ messages in thread
From: Ivan Maidanski @ 2016-01-19 22:41 UTC (permalink / raw)
  To: Matthias Klose; +Cc: GCJ-patches

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 <doko@ubuntu.com>:
> 
> 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
> 
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-01-19 22:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-16 12:19 Boehm GC API update backport Ivan Maidanski
2016-01-17  0:55 ` Matthias Klose
2016-01-19 22:41   ` Re[2]: " Ivan Maidanski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).