Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-07-24 Thread Uros Bizjak
On Mon, Jul 23, 2012 at 10:17 PM, Teresa Johnson tejohn...@google.com wrote: Resending in plain text mode so it goes through. Teresa On Mon, Jul 23, 2012 at 12:03 PM, Teresa Johnson tejohn...@google.com wrote: Any possibility of getting these patches (186979 and 186993), along with r184891

Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-07-24 Thread Teresa Johnson
On Tue, Jul 24, 2012 at 3:18 AM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Jul 23, 2012 at 10:17 PM, Teresa Johnson tejohn...@google.com wrote: Resending in plain text mode so it goes through. Teresa On Mon, Jul 23, 2012 at 12:03 PM, Teresa Johnson tejohn...@google.com wrote: Any

Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-07-14 Thread H.J. Lu
On Wed, Apr 25, 2012 at 12:14 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! We have a splitter for reg1 = reg2 0x, but only if regnums are different. But movl %edi, %edi is a cheaper variant of andq $0x, %rdi even with the same register and doesn't clobber flags, so this

Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-04-30 Thread Uros Bizjak
On Thu, Apr 26, 2012 at 5:34 PM, Uros Bizjak ubiz...@gmail.com wrote: On Thu, Apr 26, 2012 at 5:28 PM, Jakub Jelinek ja...@redhat.com wrote: We have a splitter for reg1 = reg2 0x, but only if regnums are different.  But movl %edi, %edi is a cheaper variant of andq $0x,

Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-04-30 Thread Jakub Jelinek
On Mon, Apr 30, 2012 at 02:54:05PM +0200, Uros Bizjak wrote: My recent changes to zero_extend expanders should handle this automatically, and will undo generation of zero_extend pattern. Please see zero_extendmodesi2_and expander, and how it handles TARGET_ZERO_EXTEND_WITH_AND targets.

Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-04-30 Thread Uros Bizjak
On Mon, Apr 30, 2012 at 3:10 PM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Apr 30, 2012 at 02:54:05PM +0200, Uros Bizjak wrote: My recent changes to zero_extend expanders should handle this automatically, and will undo generation of zero_extend pattern. Please see zero_extendmodesi2_and

Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-04-30 Thread Uros Bizjak
On Mon, Apr 30, 2012 at 3:34 PM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Apr 30, 2012 at 3:10 PM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Apr 30, 2012 at 02:54:05PM +0200, Uros Bizjak wrote: My recent changes to zero_extend expanders should handle this automatically, and will undo

Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-04-26 Thread Uros Bizjak
On Wed, Apr 25, 2012 at 9:14 PM, Jakub Jelinek ja...@redhat.com wrote: We have a splitter for reg1 = reg2 0x, but only if regnums are different.  But movl %edi, %edi is a cheaper variant of andq $0x, %rdi even with the same register and doesn't clobber flags, so this patch

Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-04-26 Thread Jakub Jelinek
On Thu, Apr 26, 2012 at 05:25:31PM +0200, Uros Bizjak wrote: On Wed, Apr 25, 2012 at 9:14 PM, Jakub Jelinek ja...@redhat.com wrote: We have a splitter for reg1 = reg2 0x, but only if regnums are different.  But movl %edi, %edi is a cheaper variant of andq $0x, %rdi even

Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-04-26 Thread Uros Bizjak
On Thu, Apr 26, 2012 at 5:28 PM, Jakub Jelinek ja...@redhat.com wrote: We have a splitter for reg1 = reg2 0x, but only if regnums are different.  But movl %edi, %edi is a cheaper variant of andq $0x, %rdi even with the same register and doesn't clobber flags, so this

[PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-04-25 Thread Jakub Jelinek
Hi! We have a splitter for reg1 = reg2 0x, but only if regnums are different. But movl %edi, %edi is a cheaper variant of andq $0x, %rdi even with the same register and doesn't clobber flags, so this patch attempts to expand it as a zero extension early. Bootstrapped/regtested

Re: [PATCH] Improve andq $0xffffffff, %reg handling (PR target/53110)

2012-04-25 Thread Richard Henderson
On 04/25/12 12:14, Jakub Jelinek wrote: 2012-04-25 Jakub Jelinek ja...@redhat.com PR target/53110 * config/i386/i386.md (andmode3): For andq $0x, reg instead expand it as zero extension. Ok. r~