Re: patch for PR65729

2015-04-18 Thread Richard Earnshaw
On 15/04/15 13:51, Yvan Roux wrote: > Hi, > > On 14 April 2015 at 17:36, Vladimir Makarov wrote: >> On 04/14/2015 04:11 AM, Jakub Jelinek wrote: >>> >>> On Tue, Apr 14, 2015 at 10:08:24AM +0200, Yvan Roux wrote: --- a/gcc/lra-constraints.c +++ b/gcc/lra-constraints.c @@ -1656,

Re: patch for PR65729

2015-04-15 Thread Yvan Roux
Hi, On 14 April 2015 at 17:36, Vladimir Makarov wrote: > On 04/14/2015 04:11 AM, Jakub Jelinek wrote: >> >> On Tue, Apr 14, 2015 at 10:08:24AM +0200, Yvan Roux wrote: >>> >>> --- a/gcc/lra-constraints.c >>> +++ b/gcc/lra-constraints.c >>> @@ -1656,8 +1656,7 @@ prohibited_class_reg_set_mode_p (enu

Re: patch for PR65729

2015-04-14 Thread Vladimir Makarov
On 04/14/2015 04:11 AM, Jakub Jelinek wrote: On Tue, Apr 14, 2015 at 10:08:24AM +0200, Yvan Roux wrote: --- a/gcc/lra-constraints.c +++ b/gcc/lra-constraints.c @@ -1656,8 +1656,7 @@ prohibited_class_reg_set_mode_p (enum reg_class rclass, { HARD_REG_SET temp; - // ??? Is this assert r

Re: patch for PR65729

2015-04-14 Thread Jakub Jelinek
On Tue, Apr 14, 2015 at 10:08:24AM +0200, Yvan Roux wrote: > --- a/gcc/lra-constraints.c > +++ b/gcc/lra-constraints.c > @@ -1656,8 +1656,7 @@ prohibited_class_reg_set_mode_p (enum reg_class rclass, > { >HARD_REG_SET temp; > > - // ??? Is this assert right > - // lra_assert (hard_reg_set

Re: patch for PR65729

2015-04-14 Thread Yvan Roux
Hi, here is the patch that restore the assertion and swap its arguments as discussed in the PR. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65729 Bootstrapped and regtested on x86_64, cross built and regtested on i686, aarch64, arm and armeb. Is it ok for trunk (maybe after 5.1 is released) ?