Re: [PATCH, rs6000] Fix PR target/67808, LRA ICE on double to long double conversion

2015-10-06 Thread David Edelsohn
On Mon, Oct 5, 2015 at 6:36 PM, Michael Meissner wrote: > Ok, after spending the day on going down the rabbit hole of trying to optimize > just about every, here are my patches. > > Note, I simplified the constraints to eliminate some rare possibilities, like > optimizing converting from double to

Re: [PATCH, rs6000] Fix PR target/67808, LRA ICE on double to long double conversion

2015-10-05 Thread Michael Meissner
Ok, after spending the day on going down the rabbit hole of trying to optimize just about every, here are my patches. Note, I simplified the constraints to eliminate some rare possibilities, like optimizing converting from double to long double if the double happened to be in a GPR register and th

Re: [PATCH, rs6000] Fix PR target/67808, LRA ICE on double to long double conversion

2015-10-05 Thread Peter Bergner
On Mon, 2015-10-05 at 13:12 -0400, Michael Meissner wrote: > I have attached a better version of the patch. I'll note that I have not committed the earlier patch and will hold off while we sort out what is best here. > This gives the constraints: > > #1: op0 = m, op1 = d, op2 = d > #2: o

Re: [PATCH, rs6000] Fix PR target/67808, LRA ICE on double to long double conversion

2015-10-05 Thread Michael Meissner
On Fri, Oct 02, 2015 at 02:04:48PM -0500, Peter Bergner wrote: > PR67808 exposes a problem with the constraints in the *extenddftf2_internal > pattern, in that it allows TFmode operands to occupy Altivec registers > which they are not allowed to do. Reload was able to work around the > problem, bu

Re: [PATCH, rs6000] Fix PR target/67808, LRA ICE on double to long double conversion

2015-10-02 Thread David Edelsohn
On Fri, Oct 2, 2015 at 3:04 PM, Peter Bergner wrote: > PR67808 exposes a problem with the constraints in the *extenddftf2_internal > pattern, in that it allows TFmode operands to occupy Altivec registers > which they are not allowed to do. Reload was able to work around the > problem, but LRA is

[PATCH, rs6000] Fix PR target/67808, LRA ICE on double to long double conversion

2015-10-02 Thread Peter Bergner
PR67808 exposes a problem with the constraints in the *extenddftf2_internal pattern, in that it allows TFmode operands to occupy Altivec registers which they are not allowed to do. Reload was able to work around the problem, but LRA is more pedantic and it caused it to go into an infinite spill lo