Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-09-13 Thread Kyrill Tkachov
Hi Wilco, On 04/09/17 20:56, Wilco Dijkstra wrote: Kyrill Tkachov wrote: After Bernd's change almost all DI mode instructions are split before register allocation. So instructions using DI mode no longer exist and thus these extend variants can never be matched and are thus redundant.

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-09-11 Thread Wilco Dijkstra
Any further comments?   Kyrill Tkachov wrote: > > After Bernd's change almost all DI mode instructions are split before > > register > > allocation. So instructions using DI mode no longer exist and thus these > > extend variants can never be matched and are thus redundant. > > Bernd's patch

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-09-04 Thread Wilco Dijkstra
Kyrill Tkachov wrote: > > After Bernd's change almost all DI mode instructions are split before > > register > > allocation. So instructions using DI mode no longer exist and thus these > > extend variants can never be matched and are thus redundant. > > Bernd's patch splits them when we don't

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-09-04 Thread Kyrill Tkachov
On 04/09/17 18:07, Wilco Dijkstra wrote: Kyrill Tkachov wrote: -(define_insn_and_split "*iordi_notsesidi_di" - [(set (match_operand:DI 0 "s_register_operand" "=,") - (ior:DI (not:DI (sign_extend:DI -(match_operand:SI 2 "s_register_operand" "r,r"))) -

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-09-04 Thread Wilco Dijkstra
Kyrill Tkachov wrote: > -(define_insn_and_split "*iordi_notsesidi_di" > -  [(set (match_operand:DI 0 "s_register_operand" "=,") > -   (ior:DI (not:DI (sign_extend:DI > -    (match_operand:SI 2 "s_register_operand" "r,r"))) > -   (match_operand:DI 1

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-09-04 Thread Kyrill Tkachov
Hi Wilco, Sorry for the delay... On 27/06/17 16:38, Wilco Dijkstra wrote: ping From: Wilco Dijkstra Sent: 17 January 2017 18:00 To: GCC Patches Cc: nd; Kyrylo Tkachov; Richard Earnshaw Subject: [PATCH][ARM] Remove Thumb-2 iordi_not patterns After Bernd's DImode patch [1] almost all DImode

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-06-27 Thread Wilco Dijkstra
  ping From: Wilco Dijkstra Sent: 17 January 2017 18:00 To: GCC Patches Cc: nd; Kyrylo Tkachov; Richard Earnshaw Subject: [PATCH][ARM] Remove Thumb-2 iordi_not patterns     After Bernd's DImode patch [1] almost all DImode operations are expanded early (except for -mfpu=neon). This means

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-06-13 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 17 January 2017 18:00 To: GCC Patches Cc: nd; Kyrylo Tkachov; Richard Earnshaw Subject: [PATCH][ARM] Remove Thumb-2 iordi_not patterns     After Bernd's DImode patch [1] almost all DImode operations are expanded early (except for -mfpu=neon). This means

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-04-20 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 17 January 2017 18:00 To: GCC Patches Cc: nd; Kyrylo Tkachov; Richard Earnshaw Subject: [PATCH][ARM] Remove Thumb-2 iordi_not patterns   After Bernd's DImode patch [1] almost all DImode operations are expanded early (except for -mfpu=neon). This means

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-02-23 Thread Wilco Dijkstra
  ping From: Wilco Dijkstra Sent: 17 January 2017 18:00 To: GCC Patches Cc: nd; Kyrylo Tkachov; Richard Earnshaw Subject: [PATCH][ARM] Remove Thumb-2 iordi_not patterns     After Bernd's DImode patch [1] almost all DImode operations are expanded early (except for -mfpu=neon). This means

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-02-02 Thread Wilco Dijkstra
ping From: Wilco Dijkstra Sent: 17 January 2017 18:00 To: GCC Patches Cc: nd; Kyrylo Tkachov; Richard Earnshaw Subject: [PATCH][ARM] Remove Thumb-2 iordi_not patterns   After Bernd's DImode patch [1] almost all DImode operations are expanded early (except for -mfpu=neon). This means the Thumb

[PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-01-17 Thread Wilco Dijkstra
After Bernd's DImode patch [1] almost all DImode operations are expanded early (except for -mfpu=neon). This means the Thumb-2 iordi_notdi_di patterns are no longer used - the split ORR and NOT instructions are merged into ORN by Combine. With -mfpu=neon the iordi_notdi_di patterns are used on