[Gc] Re: Re[2]: [Patch 2/2] Aarch64 libatomic_ops and Gc basic port

Yvan Roux yvan.roux at linaro.org
Thu Jan 24 03:55:27 PST 2013


Hi Ivan and Marcus,

Tanks for the comments and commiting.

I've a question for you two, on the proper way to have this merged
into GCC. Should it be first merged into Ivan's gcc-boehmgc github
branch or directly into the FSF repo ? Regarding the FSF, Marcus this
has to go into the AArch64 branch I guess ?

Cheers,
Yvan

On 23 January 2013 20:51, Ivan Maidanski <ivmai at mail.ru> wrote:
> Hi Marcus and Yvan,
>
> I've committed the proposed GC patch (with the 2 corrections mentioned
> below) to master branch:
> https://github.com/ivmai/bdwgc/commit/7b5acfba9a1df00f0427d1d2e1a92570da3ab2d1
>
> Regards,
> Ivan
>
> Wed, 23 Jan 2013, 18:15 UTC Marcus Shawcroft <marcus.shawcroft at arm.com>:
>
> On 23/01/13 17:53, Yvan Roux wrote:
>
>> @@ -557,6 +568,7 @@
>> /* running Amdahl UTS4 */
>> /* S390 ==> 390-like machine */
>> /* running LINUX */
>> + /* AARCH64 ==> ARM 64-bit */
>
> The define AARCH64 represents the ARM AArch64 execution state therefore
> I think this comment should read:
>
> AARCH64 -> ARM AArch64
>
>
>> +# define DATASTART ((ptr_t)__data_start)
>> + /* __stack_base__ is set in newlib/libc/sys/arm/crt0.S */
>
> That comment doesn't make sense in this context.
>
> Cheers
> /Marcus
>
>
>
> _______________________________________________
> Gc mailing list
> Gc at linux.hpl.hp.com
> http://www.hpl.hp.com/hosted/linux/mail-archives/gc/
>
>


More information about the Gc mailing list