This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, boehm-gc, AArch64] Add AArch64 support
- From: Christophe Lyon <christophe dot lyon at linaro dot org>
- To: Yvan Roux <yvan dot roux at linaro dot org>
- Cc: Marcus Shawcroft <marcus dot shawcroft at gmail dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Patch Tracking <patches at linaro dot org>
- Date: Thu, 2 May 2013 21:51:28 +0200
- Subject: Re: [PATCH, boehm-gc, AArch64] Add AArch64 support
- References: <CAD57uCeB4rgg=XmHEyHYZwU9ZhnSsaS1vBTf+qgdgWgMWYzXoA at mail dot gmail dot com> <CAD57uCfa2K7bux0siwxK_+ktDzwxbN9VQJTTU=SMOOXkWCm2iQ at mail dot gmail dot com> <CAD57uCfN=5QG0TmOa3h_J1AvxzqvvH2mOHQmUFM7HCwwURPHSw at mail dot gmail dot com> <CAFqB+PwDWa2swxvOLF52VzDE=_Xbj0ajmGyU8agy9qnqvkR_fA at mail dot gmail dot com> <CAD57uCd4pVs=OkeQUc9XXnrSGK6v6FFzfO-A4sftNxs6CXKwug at mail dot gmail dot com>
I have just added the ChangeLog entry (in boehm-gc/), which I forgot
to add when I committed this patch for you.
(as r198562)
Christophe.
On 11 April 2013 13:20, Yvan Roux <yvan.roux@linaro.org> wrote:
> Hi Marcus,
>
> thanks for pointing this, but unfortunately I don't have the svn write
> access for the moment, thus
> if somebody can commit it for me, i would be very thankful.
>
> Cheers,
> Yvan
>
> On 11 April 2013 13:02, Marcus Shawcroft <marcus.shawcroft@gmail.com> wrote:
>> Hi Yvan, My understanding of the 'Free for all' section of
>> http://gcc.gnu.org/svnwrite.html#policies is that since this is a
>> backport from an upstream project you do not need to seek further
>> approval to commit this change.
>>
>> Cheers
>> /Marcus
>>
>> On 2 April 2013 11:50, Yvan Roux <yvan.roux@linaro.org> wrote:
>>> Ping (second try)
>>>
>>> Sorry if you received it twice, it seems that my gmail account
>>> switched in text/html mode :(
>>>
>>> Many thanks,
>>> Yvan
>>>
>>> On 2 April 2013 11:21, Yvan Roux <yvan.roux@linaro.org> wrote:
>>>>
>>>> Ping
>>>>
>>>>
>>>> On 17 March 2013 21:34, Yvan Roux <yvan.roux@linaro.org> wrote:
>>>>>
>>>>> Hi,
>>>>>
>>>>> this is a backport from gc mainline of the basic AArch64 support (it
>>>>> covers the Linux and bare machine mode). I tested it on the Foundation
>>>>> model with enabling the objc frontend, and passing the testsuite
>>>>> manually (maybe I miss-configured it, but it seems that the boehm-gc
>>>>> testsuite is not cross-environment friendly, as the gctest script
>>>>> looks for the host gcc build tree), and everything is fine, except the
>>>>> thread_leak_test which has a different output than the x86 one:
>>>>>
>>>>> x86 thread_leak_test output
>>>>>
>>>>> Leaked composite object at 0x2aaaaab05fe0
>>>>>
>>>>> (/work/sources/gcc-fsf/bgc/boehm-gc/testsuite/boehm-gc.c/thread_leak_test.c:21,
>>>>> sz=4, NORMAL)
>>>>>
>>>>> Leaked composite object at start: 0x2aaaaab03fa0, appr. length: 40
>>>>> Leaked composite object at 0x2aaaaab05ec0
>>>>>
>>>>> (/work/sources/gcc-fsf/bgc/boehm-gc/testsuite/boehm-gc.c/thread_leak_test.c:21,
>>>>> sz=4, NORMAL)
>>>>>
>>>>> Leaked composite object at 0x2aaaaab05f20
>>>>>
>>>>> (/work/sources/gcc-fsf/bgc/boehm-gc/testsuite/boehm-gc.c/thread_leak_test.c:21,
>>>>> sz=4, NORMAL)
>>>>>
>>>>> Leaked composite object at start: 0x2aaaaab03f50, appr. length: 40
>>>>> Leaked composite object at start: 0x2aaaaab03f78, appr. length: 40
>>>>> Leaked composite object at 0x2aaaaab05ef0
>>>>>
>>>>> (/work/sources/gcc-fsf/bgc/boehm-gc/testsuite/boehm-gc.c/thread_leak_test.c:21,
>>>>> sz=4, NORMAL)
>>>>>
>>>>> Leaked composite object at start: 0x2aaaaab03fa0, appr. length: 40
>>>>> Leaked composite object at 0x2aaaaab05e00
>>>>>
>>>>> (/work/sources/gcc-fsf/bgc/boehm-gc/testsuite/boehm-gc.c/thread_leak_test.c:21,
>>>>> sz=4, NORMAL)
>>>>>
>>>>> Leaked composite object at start: 0x2aaaaab03f78, appr. length: 40
>>>>>
>>>>> AArch64 thread_leak_test output:
>>>>>
>>>>> Leaked composite object at 0x7f91e14ef0
>>>>>
>>>>> (/work/sources/gcc-fsf/bgc/boehm-gc/testsuite/boehm-gc.c/thread_leak_test.c:21,
>>>>> sz=4, NORMAL)
>>>>>
>>>>> Leaked composite object at 0x7f91e14fe0
>>>>>
>>>>> (/work/sources/gcc-fsf/bgc/boehm-gc/testsuite/boehm-gc.c/thread_leak_test.c:21,
>>>>> sz=4, NORMAL)
>>>>>
>>>>> Leaked composite object at 0x7f91e14e00
>>>>>
>>>>> (/work/sources/gcc-fsf/bgc/boehm-gc/testsuite/boehm-gc.c/thread_leak_test.c:21,
>>>>> sz=4, NORMAL)
>>>>>
>>>>> Leaked composite object at 0x7f91e14e30
>>>>>
>>>>> (/work/sources/gcc-fsf/bgc/boehm-gc/testsuite/boehm-gc.c/thread_leak_test.c:21,
>>>>> sz=4, NORMAL)
>>>>>
>>>>> Leaked composite object at 0x7f91e14fe0
>>>>>
>>>>> (/work/sources/gcc-fsf/bgc/boehm-gc/testsuite/boehm-gc.c/thread_leak_test.c:21,
>>>>> sz=4, NORMAL)
>>>>>
>>>>>
>>>>> Regards,
>>>>> Yvan
>>>>>
>>>>> 2013-03-16 Yvan Roux <yvan.roux@linaro.org>
>>>>>
>>>>> * include/private/gcconfig.h (AARCH64): New macro (defined
>>>>> only if
>>>>> __aarch64__).
>>>>> * include/private/gcconfig.h (mach_type_known): Update comment
>>>>> adding
>>>>> ARM AArch64 target.
>>>>> * include/private/gcconfig.h (NOSYS, mach_type_known,
>>>>> CPP_WORDSZ,
>>>>> MACH_TYPE, ALIGNMENT, HBLKSIZE, OS_TYPE, LINUX_STACKBOTTOM,
>>>>> USE_GENERIC_PUSH_REGS, DYNAMIC_LOADING, DATASTART, DATAEND,
>>>>> STACKBOTTOM): Define for AArch64.
>>>>
>>>>