Re: [PATCH] Handle andn and ~ in 32-bit stv pass (PR target/70322)

2016-12-02 Thread Uros Bizjak
On Fri, Dec 2, 2016 at 5:30 PM, Jakub Jelinek wrote: > On Fri, Dec 02, 2016 at 05:12:20PM +0100, Uros Bizjak wrote: >> >> This patch: >> >> 1) adds one_cmpldi2 pattern for stv purposes (which splits into two >> >>one_cmplsi2 after reload) >> >> 2) teaches the 32-bit stv pass

Re: [PATCH] Handle andn and ~ in 32-bit stv pass (PR target/70322)

2016-12-02 Thread Jakub Jelinek
On Fri, Dec 02, 2016 at 05:12:20PM +0100, Uros Bizjak wrote: > >> This patch: > >> 1) adds one_cmpldi2 pattern for stv purposes (which splits into two > >>one_cmplsi2 after reload) > >> 2) teaches the 32-bit stv pass to handle NOT (as xor all-ones) > >> 3) renames the old *andndi3_doubleword

Re: [PATCH] Handle andn and ~ in 32-bit stv pass (PR target/70322)

2016-12-02 Thread Uros Bizjak
On Fri, Dec 2, 2016 at 5:11 PM, Uros Bizjak wrote: > On Fri, Dec 2, 2016 at 3:21 PM, Jakub Jelinek wrote: >> Hi! >> >> While the https://gcc.gnu.org/ml/gcc-patches/2016-01/msg01087.html >> added *andndi3_doubleword, I don't understand how it can actually

Re: [PATCH] Handle andn and ~ in 32-bit stv pass (PR target/70322)

2016-12-02 Thread Uros Bizjak
On Fri, Dec 2, 2016 at 3:21 PM, Jakub Jelinek wrote: > Hi! > > While the https://gcc.gnu.org/ml/gcc-patches/2016-01/msg01087.html > added *andndi3_doubleword, I don't understand how it can actually work. > The problem is that this pattern is for combine, and needs combining > of

[PATCH] Handle andn and ~ in 32-bit stv pass (PR target/70322)

2016-12-02 Thread Jakub Jelinek
Hi! While the https://gcc.gnu.org/ml/gcc-patches/2016-01/msg01087.html added *andndi3_doubleword, I don't understand how it can actually work. The problem is that this pattern is for combine, and needs combining of DImode NOT setter with DImode AND user. But there is no DImode pattern for