[m-rev.] diff: upgrade to boehm gc 7.2

Julien Fischer juliensf at csse.unimelb.edu.au
Wed Jul 18 19:05:04 AEST 2012

On Wed, 18 Jul 2012, Peter Ross wrote:

> On 18 July 2012 09:55, Julien Fischer <juliensf at csse.unimelb.edu.au> wrote:
>> On Mon, 16 Jul 2012, Peter Ross wrote:
>>> Please find attached the interesting bits (ie those bits where I had to
>>> resolve
>>> a non-obvious conflict) of the diff to upgrade to version 7.2 of the boehm
>>> gc
>>> collector.
>>> The full diff can be found at uranus:~petdr/DIFF.GC72
>>> This change successfully bootchecked in the asm_fast.gc and hlc.gc grades.
>>> I also installed the resulting compiler in the asm_fast.gc and hlc.gc
>>> grades
>>> and then built some test programs in those grades, on my local machine.
>>> So far this has only been tested extensively on linux.
>>> Julien, can you test on Mac OS X and Windows?
>>> I will do the test on Solaris 10.
>> One thing I have noticed is that it looks like
>> scripts/prepare_install_dir.in is trying to copy over things from the
>> boehm_gc directory that no longer exist, e.g.
>> boehm_gc/configure_atomic_ops.sh.
> Good spot

Actually, make install on one of the machines I was testing it on was
having whinge, but I'll claim the credit for it ;-)

> The following files were removed during the update
> boehm_gc/bdw-gc.pc
> boehm_gc/configure_atomic_ops.sh
> boehm_gc/hpux_test_and_clear.s
> It looks like only the configure_atomic_ops.sh is mentioned explicitly.

I suggest adding a note to compiler/notes/upgrade_boehm_gc.html that
the boehm GC files listed in scripts/prepare_install.in should be checked
against the updated version of the collector.

mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au

More information about the reviews mailing list