Re: [PATCH][ARM] Cleanup multiply patterns

2019-09-19 Thread Kyrill Tkachov
On 9/18/19 7:19 PM, Wilco Dijkstra wrote: Hi Kyrill,  + (mult:SI (match_operand:SI 3 "s_register_operand" "r")  +  (match_operand:SI 2 "s_register_operand" "r"] Looks like we'll want to mark operand 2 here with '%' as well? That doesn't make any difference sinc

Re: [PATCH][ARM] Cleanup multiply patterns

2019-09-18 Thread Wilco Dijkstra
Hi Kyrill, >>  + (mult:SI (match_operand:SI 3 "s_register_operand" "r") >>  +  (match_operand:SI 2 "s_register_operand" "r"] > > Looks like we'll want to mark operand 2 here with '%' as well? That doesn't make any difference since both operands are identical. It only h

Re: [PATCH][ARM] Cleanup multiply patterns

2019-09-18 Thread Kyrill Tkachov
Hi Wilco, On 9/9/19 6:07 PM, Wilco Dijkstra wrote: ping Cleanup the 32-bit multiply patterns.  Merge the pre-Armv6 with the Armv6  patterns, remove useless alternatives and order the accumulator operands  to prefer MLA Ra, Rb, Rc, Ra whenever feasible.  Bootstrap OK on armhf, regress passes.

Re: [PATCH][ARM] Cleanup multiply patterns

2019-09-09 Thread Wilco Dijkstra
ping   Cleanup the 32-bit multiply patterns.  Merge the pre-Armv6 with the Armv6 patterns, remove useless alternatives and order the accumulator operands to prefer MLA Ra, Rb, Rc, Ra whenever feasible. Bootstrap OK on armhf, regress passes. ChangeLog: 2019-09-03  Wilco Dijkstra    

[PATCH][ARM] Cleanup multiply patterns

2019-09-03 Thread Wilco Dijkstra
Cleanup the 32-bit multiply patterns. Merge the pre-Armv6 with the Armv6 patterns, remove useless alternatives and order the accumulator operands to prefer MLA Ra, Rb, Rc, Ra whenever feasible. Bootstrap OK on armhf, regress passes. ChangeLog: 2019-09-03 Wilco Dijkstra * config/arm/a