Re: PATCH: PR target/49142: Invalid 8bit register operand

2011-05-25 Thread H.J. Lu
On Wed, May 25, 2011 at 12:11 PM, Uros Bizjak wrote: > On Wed, May 25, 2011 at 6:20 PM, H.J. Lu wrote: > >> We are working on a new optimization, which turns off TARGET_MOVX. >> GCC generates: >> >> movb %ah, %dil >> >> But %ah can only be used with %[abcd][hl].  This patc

Re: PATCH: PR target/49142: Invalid 8bit register operand

2011-05-25 Thread Uros Bizjak
On Wed, May 25, 2011 at 6:20 PM, H.J. Lu wrote: > We are working on a new optimization, which turns off TARGET_MOVX. > GCC generates: > > movb %ah, %dil > > But %ah can only be used with %[abcd][hl].  This patch adds QIreg_operand > and uses it in *movqi_extv_1_rex64/*

Re: PATCH: PR target/49142: Invalid 8bit register operand

2011-05-25 Thread H.J. Lu
On Wed, May 25, 2011 at 8:30 AM, Uros Bizjak wrote: > On Wed, May 25, 2011 at 4:42 PM, H.J. Lu wrote: >> On Wed, May 25, 2011 at 7:00 AM, Uros Bizjak wrote: >>> On Tue, May 24, 2011 at 5:54 PM, H.J. Lu wrote: Hi, We are working on a new optimization, which turns off TARGET_MOVX.

Re: PATCH: PR target/49142: Invalid 8bit register operand

2011-05-25 Thread Uros Bizjak
On Wed, May 25, 2011 at 4:42 PM, H.J. Lu wrote: > On Wed, May 25, 2011 at 7:00 AM, Uros Bizjak wrote: >> On Tue, May 24, 2011 at 5:54 PM, H.J. Lu wrote: >>> Hi, >>> >>> We are working on a new optimization, which turns off TARGET_MOVX. >>> GCC generates: >>> >>> movb %ah, %dil >>> >>> But %ah ca

Re: PATCH: PR target/49142: Invalid 8bit register operand

2011-05-25 Thread H.J. Lu
On Wed, May 25, 2011 at 7:00 AM, Uros Bizjak wrote: > On Tue, May 24, 2011 at 5:54 PM, H.J. Lu wrote: >> Hi, >> >> We are working on a new optimization, which turns off TARGET_MOVX. >> GCC generates: >> >> movb %ah, %dil >> >> But %ah can only be used with %[abcd][hl].  This patch adds QIreg_oper

Re: PATCH: PR target/49142: Invalid 8bit register operand

2011-05-25 Thread Uros Bizjak
On Tue, May 24, 2011 at 5:54 PM, H.J. Lu wrote: > Hi, > > We are working on a new optimization, which turns off TARGET_MOVX. > GCC generates: > > movb %ah, %dil > > But %ah can only be used with %[abcd][hl].  This patch adds QIreg_operand > and uses it in *movqi_extv_1_rex64/*movqi_extzv_2_rex64.

PATCH: PR target/49142: Invalid 8bit register operand

2011-05-24 Thread H.J. Lu
Hi, We are working on a new optimization, which turns off TARGET_MOVX. GCC generates: movb %ah, %dil But %ah can only be used with %[abcd][hl]. This patch adds QIreg_operand and uses it in *movqi_extv_1_rex64/*movqi_extzv_2_rex64. OK for trunk if there is no regression? Thanks. H.J. --- 201