This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][ARM][1/4] PR target/65932: Add testcase
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
- Cc: Nick Clifton <nickc at redhat dot com>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 3 Feb 2016 10:51:30 +0100
- Subject: Re: [PATCH][ARM][1/4] PR target/65932: Add testcase
- Authentication-results: sourceware.org; auth=none
- References: <871t8ukx86 dot fsf at redhat dot com> <56B1CC77 dot 4030201 at foss dot arm dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Wed, Feb 03, 2016 at 09:46:31AM +0000, Kyrill Tkachov wrote:
> Hi Nick,
>
> On 03/02/16 08:35, Nick Clifton wrote:
> >Hi Kyrill,
> >
> > I would like to approve this patch, but cannot, since it is not ARM
> > specific. I think that if you ping the list you may be able to get a
> > response, and it would be nice to see this whole patch series checked
> > in before the gcc 6 branch occurs.
> >
> >Cheers
> > Nick
> >
> >PS. If necessary you could always move the test to gcc.target/arm...
> >
>
> Thanks again for looking at these.
> I'd like to keep this test in the generic directory as we're
> trying to avoid cluttering the gcc.target directories with
> tests that are not arm-specific.
>
> CC'ing Jakub then. Jakub, is it ok to have the test from
> https://gcc.gnu.org/ml/gcc-patches/2016-01/msg01717.html
> in gcc.c-torture/execute/ ?
The test is not portable, so not in this form.
Does the test reproduce what you want if you change
0xfffffff1 to -15? If yes, the test is ok with that change.
Jakub