Re: [Patch AArch64] Fixup to fcvt patterns added in r237200

2016-06-20 Thread Richard Earnshaw (lists)
On 10/06/16 13:29, James Greenhalgh wrote: > > Hi, > > My autotester picked up some issues with the vcvt{ds}_n_* intrinsics > added in r237200. > > The iterators in this pattern do not resolve, as they have not been > explicitly tied to the mode iterator (rather than the code iterator) > used

Re: [Patch AArch64] Fixup to fcvt patterns added in r237200

2016-06-20 Thread James Greenhalgh
On Fri, Jun 10, 2016 at 01:29:39PM +0100, James Greenhalgh wrote: > > Hi, > > My autotester picked up some issues with the vcvt{ds}_n_* intrinsics > added in r237200. > > The iterators in this pattern do not resolve, as they have not been > explicitly tied to the mode iterator (rather than the

Re: [Patch AArch64] Fixup to fcvt patterns added in r237200

2016-06-17 Thread Christophe Lyon
On 17 June 2016 at 16:44, James Greenhalgh wrote: > On Fri, Jun 17, 2016 at 04:25:31PM +0200, Christophe Lyon wrote: >> On 10 June 2016 at 14:29, James Greenhalgh wrote: >> > >> > Hi, >> > >> > My autotester picked up some issues with the

Re: [Patch AArch64] Fixup to fcvt patterns added in r237200

2016-06-17 Thread James Greenhalgh
On Fri, Jun 17, 2016 at 04:25:31PM +0200, Christophe Lyon wrote: > On 10 June 2016 at 14:29, James Greenhalgh wrote: > > > > Hi, > > > > My autotester picked up some issues with the vcvt{ds}_n_* intrinsics > > added in r237200. > > > Hi, > > What tests does your

Re: [Patch AArch64] Fixup to fcvt patterns added in r237200

2016-06-17 Thread Christophe Lyon
On 10 June 2016 at 14:29, James Greenhalgh wrote: > > Hi, > > My autotester picked up some issues with the vcvt{ds}_n_* intrinsics > added in r237200. > Hi, What tests does your autotester perform? I haven't noticed these problems when running the GCC testsuite on the

Re: [Patch AArch64] Fixup to fcvt patterns added in r237200

2016-06-13 Thread Kyrill Tkachov
On 10/06/16 13:29, James Greenhalgh wrote: Hi, My autotester picked up some issues with the vcvt{ds}_n_* intrinsics added in r237200. The iterators in this pattern do not resolve, as they have not been explicitly tied to the mode iterator (rather than the code iterator) used by the pattern.

[Patch AArch64] Fixup to fcvt patterns added in r237200

2016-06-10 Thread James Greenhalgh
Hi, My autotester picked up some issues with the vcvt{ds}_n_* intrinsics added in r237200. The iterators in this pattern do not resolve, as they have not been explicitly tied to the mode iterator (rather than the code iterator) used by the pattern. This fixup adds the attribute tags, allowing