Re: [PATCH, ARM] Fix arm_subsi3_insn alternatives

2015-04-13 Thread Yvan Roux
On 13 April 2015 at 12:55, Kyrill Tkachov wrote: > > On 13/04/15 11:47, Yvan Roux wrote: >> >> Ping (now that stage1 is open) > > Hi Yvan, > > This patch is ok, but can you wait until GCC 5.1 is released before > committing please? > That way trunk and the GCC 5 branch won't diverge too much if th

Re: [PATCH, ARM] Fix arm_subsi3_insn alternatives

2015-04-13 Thread Kyrill Tkachov
On 13/04/15 11:47, Yvan Roux wrote: Ping (now that stage1 is open) Hi Yvan, This patch is ok, but can you wait until GCC 5.1 is released before committing please? That way trunk and the GCC 5 branch won't diverge too much if there are any last minute (hopefully none in this area!) fixes that

Re: [PATCH, ARM] Fix arm_subsi3_insn alternatives

2015-04-13 Thread Yvan Roux
Ping (now that stage1 is open) On 27 March 2015 at 19:14, Kyrill Tkachov wrote: > > On 24/03/15 19:53, Yvan Roux wrote: >> >> Hi, >> >> after the issue with duplicated alternatives exhibited by PR64208, I >> checked the arm.md file and found that *arm_subsi3_insn has a >> duplication where alt 4

Re: [PATCH, ARM] Fix arm_subsi3_insn alternatives

2015-03-27 Thread Kyrill Tkachov
On 24/03/15 19:53, Yvan Roux wrote: Hi, after the issue with duplicated alternatives exhibited by PR64208, I checked the arm.md file and found that *arm_subsi3_insn has a duplication where alt 4 is (r,rI,r) and alt 6 is (r,r,r), this results in emitting an rsb instruction instead of a sub one,