Re: [PATCH, PR65915] Fix float conversion split.

2015-05-05 Thread Uros Bizjak
On Tue, May 5, 2015 at 3:03 PM, Ilya Tocar wrote: >> >> +++ b/gcc/testsuite/gcc.target/i386/pr65915.c >> >> @@ -0,0 +1,6 @@ >> >> +/* { dg-do run } */ >> >> +/* { dg-options "-O2 -mavx512f -fpic -mcmodel=medium" } */ >> >> +/* { dg-require-effective-target avx512f } */ >> >> +/* { dg-require-effec

Re: [PATCH, PR65915] Fix float conversion split.

2015-05-05 Thread Ilya Tocar
> >> +++ b/gcc/testsuite/gcc.target/i386/pr65915.c > >> @@ -0,0 +1,6 @@ > >> +/* { dg-do run } */ > >> +/* { dg-options "-O2 -mavx512f -fpic -mcmodel=medium" } */ > >> +/* { dg-require-effective-target avx512f } */ > >> +/* { dg-require-effective-target lp64 } */ > >> + > >> +#include "avx512f-vrnd

Re: [PATCH, PR65915] Fix float conversion split.

2015-05-04 Thread Uros Bizjak
On Thu, Apr 30, 2015 at 5:18 PM, H.J. Lu wrote: > On Thu, Apr 30, 2015 at 8:15 AM, Ilya Tocar wrote: >>> Hi, >>> >>> Looks like I missed some splits, which caused PR65915. >>> Patch below fixes it. >>> Ok for trunk? >>> >>> 2015-04-28 Ilya Tocar >>> >>> * config/i386/i386.md (define_spli

Re: [PATCH, PR65915] Fix float conversion split.

2015-04-30 Thread H.J. Lu
On Thu, Apr 30, 2015 at 8:15 AM, Ilya Tocar wrote: >> Hi, >> >> Looks like I missed some splits, which caused PR65915. >> Patch below fixes it. >> Ok for trunk? >> >> 2015-04-28 Ilya Tocar >> >> * config/i386/i386.md (define_split): Check for xmm16+, >> when splitting scalar float c

Re: [PATCH, PR65915] Fix float conversion split.

2015-04-30 Thread Ilya Tocar
> Hi, > > Looks like I missed some splits, which caused PR65915. > Patch below fixes it. > Ok for trunk? > > 2015-04-28 Ilya Tocar > > * config/i386/i386.md (define_split): Check for xmm16+, > when splitting scalar float conversion. > > > --- > gcc/config/i386/i386.md | 8 +

Re: [PATCH, PR65915] Fix float conversion split.

2015-04-28 Thread H.J. Lu
On Tue, Apr 28, 2015 at 9:20 AM, Ilya Tocar wrote: >> > I've renamed EXT_SSE_REG_P into EXT_REX_SSE_REG_P for consistency. >> > Ok for stage1? >> Patch is OK for stage1. >> >> -- >> Thanks, K >> >> >> > On 19 Mar 12:24, Ilya Tocar wrote: >> > > Hi, >> > > >> > > There were some discussion about "x

[PATCH, PR65915] Fix float conversion split.

2015-04-28 Thread Ilya Tocar
> > I've renamed EXT_SSE_REG_P into EXT_REX_SSE_REG_P for consistency. > > Ok for stage1? > Patch is OK for stage1. > > -- > Thanks, K > > > > On 19 Mar 12:24, Ilya Tocar wrote: > > > Hi, > > > > > > There were some discussion about "x" constraints being too conservative > > > for some patterns