This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PR43597, ARM, TESTCASE]
- From: Tom de Vries <vries at codesourcery dot com>
- To: Richard Earnshaw <rearnsha at arm dot com>
- Cc: Ramana Radhakrishnan <ramana dot radhakrishnan at linaro dot org>, Janis Johnson <janisjo at codesourcery dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Sun, 31 Jul 2011 13:18:02 +0300
- Subject: Re: [PR43597, ARM, TESTCASE]
- References: <4E24146F.8000508@codesourcery.com> <4E2424DF.3070808@arm.com> <4E2D95C3.8030705@codesourcery.com>
On 07/25/2011 07:11 PM, Tom de Vries wrote:
> Hi,
>
> thanks for the review.
>
> On 07/18/2011 03:19 PM, Richard Earnshaw wrote:
>> On 18/07/11 12:09, Tom de Vries wrote:
>>> Hi,
>>>
>>> PR43597 was fixed by http://gcc.gnu.org/viewcvs?view=revision&revision=172032.
>>>
>>> This patch adds a testcase.
>>>
>>> OK for trunk?
>>>
>>> Thanks,
>>> - Tom
>>>
>>> 2011-07-18 Tom de Vries <tom@codesourcery.com>
>>>
>>> PR target/43597
>>> * gcc.target/arm/pr43597.c: New test.
>>>
>>>
>>
>> No, don't pass -mthumb through dg-options unless you're using something
>> like require-effective-target.
>>
>
> OK, I see.
>
>> In this case the post-compile tests are all gated on thumb2. So why not
>> make the whole test just require arm_thumb2_ok?
>>
>> R.
>>
>
> Done. OK for trunk?
>
Ping. http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02173.html OK for trunk?
Thanks,
-Tom
> 2011-07-25 Tom de Vries <tom@codesourcery.com>
>
> PR target/43597
> * gcc.target/arm/pr43597.c: New test.