Re: [RS6000, PATCH] PR70052, ICE compiling _Decimal128 test case

2016-03-29 Thread David Edelsohn
On Tue, Mar 29, 2016 at 6:14 AM, Alan Modra wrote: > On Fri, Mar 25, 2016 at 07:36:34PM +1030, Alan Modra wrote: >> +2016-03-25 Alan Modra >> + >> + PR target/70052 >> + * config/rs6000/constraints.md (j): Simplify. >> + *

Re: [RS6000, PATCH] PR70052, ICE compiling _Decimal128 test case

2016-03-29 Thread Alan Modra
On Fri, Mar 25, 2016 at 07:36:34PM +1030, Alan Modra wrote: > +2016-03-25 Alan Modra > + > + PR target/70052 > + * config/rs6000/constraints.md (j): Simplify. > + * config/rs6000/predicates.md (easy_fp_constant): Exclude > + decimal float 0.D. > + *

Re: [RS6000, PATCH] PR70052, ICE compiling _Decimal128 test case

2016-03-25 Thread Alan Modra
On Thu, Mar 24, 2016 at 12:51:05PM -0400, David Edelsohn wrote: > The patch did not convert all "j" constraints, so the ChangeLog needs > to be a little clearer to explain which alternatives required the > change. Um, I did intend to change all "j" constraints in insns handling decimal float. I

Re: [RS6000, PATCH] PR70052, ICE compiling _Decimal128 test case

2016-03-24 Thread David Edelsohn
On Thu, Mar 24, 2016 at 7:01 AM, Alan Modra wrote: > This fixes the PR70052 ICE by modifying easy_fp_constant to correctly > return false for decimal floating point zero. 0.0D is not an all-zero > bit pattern, at least, not the canonical form. > > I've also taken on Mike's

[RS6000, PATCH] PR70052, ICE compiling _Decimal128 test case

2016-03-24 Thread Alan Modra
This fixes the PR70052 ICE by modifying easy_fp_constant to correctly return false for decimal floating point zero. 0.0D is not an all-zero bit pattern, at least, not the canonical form. I've also taken on Mike's suggestion of using a mode dependent constraint for insns that currently use "j".