Re: [Patch ARM] Fix PR19599 tail

2013-08-01 Thread Ramana Radhakrishnan
On 07/29/13 11:05, Mikael Pettersson wrote: Ramana Radhakrishnan writes: > Hi, > > This fixes up the issues with PR target/19599 and the issues we've had > around it. ... > 2013-07-25 Ramana Radhakrishnan > > PR target/19599 > PR target/57731 > P

Re: [Patch ARM] Fix PR19599 tail

2013-07-29 Thread Mikael Pettersson
Ramana Radhakrishnan writes: > Hi, > > This fixes up the issues with PR target/19599 and the issues we've had > around it. ... > 2013-07-25 Ramana Radhakrishnan > > PR target/19599 > PR target/57731 > PR target/57748 Are you sure about the 57748 refe

Re: [Patch ARM] Fix PR19599 tail

2013-07-25 Thread Ramana Radhakrishnan
On Thu, Jul 25, 2013 at 12:40 PM, Bernd Edlinger wrote: > Hi Ramana, > > why did you leave the space before the bx? > this ends up in the .s file making it look ugly.. Ooops - thanks for noticing - wasn't deliberate - fixed as obvious with this. Ramana Index: gcc/config/arm/arm.md =

Re: [Patch ARM] Fix PR19599 tail

2013-07-25 Thread Bernd Edlinger
Hi Ramana, why did you leave the space before the bx? this ends up in the .s file making it look ugly.. + return \" bx%?\\t%0\\t%@ indirect register sibling call\"; Regards Bernd.

[Patch ARM] Fix PR19599 tail

2013-07-25 Thread Ramana Radhakrishnan
Hi, This fixes up the issues with PR target/19599 and the issues we've had around it. This fixes up some of the current issues we have around this. I'll follow up with a separate patch around longjmps. This puts in predicates for bx where needed - missed out CCFSM (sigh :( ), replaces uses