public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug inline-asm/49142] New: Invalid 8bit register operand
@ 2011-05-24 14:34 hjl.tools at gmail dot com
  2011-05-25 16:39 ` [Bug inline-asm/49142] " hjl at gcc dot gnu.org
  2011-06-24 15:39 ` hjl.tools at gmail dot com
  0 siblings, 2 replies; 3+ messages in thread
From: hjl.tools at gmail dot com @ 2011-05-24 14:34 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49142

           Summary: Invalid 8bit register operand
           Product: gcc
           Version: 4.6.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: inline-asm
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: hjl.tools@gmail.com
            Target: x86_64


i386.md has

(define_insn "*movqi_extzv_2_rex64"
  [(set (match_operand:QI 0 "register_operand" "=Q,?R")
        (subreg:QI
          (zero_extract:SI (match_operand 1 "ext_register_operand" "Q,Q")
                           (const_int 8)
                           (const_int 8)) 0))]
  "TARGET_64BIT"
{
  switch (get_attr_type (insn))
    {
    case TYPE_IMOVX:
      return "movz{bl|x}\t{%h1, %k0|%k0, %h1}";
    default:
      return "mov{b}\t{%h1, %0|%0, %h1}";
    }
}

However, since %ah can only be used with legacy 8bit register,
it puts restriction on the other operand.  We shouldn't generate
"movb %ah, %dil".


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug inline-asm/49142] Invalid 8bit register operand
  2011-05-24 14:34 [Bug inline-asm/49142] New: Invalid 8bit register operand hjl.tools at gmail dot com
@ 2011-05-25 16:39 ` hjl at gcc dot gnu.org
  2011-06-24 15:39 ` hjl.tools at gmail dot com
  1 sibling, 0 replies; 3+ messages in thread
From: hjl at gcc dot gnu.org @ 2011-05-25 16:39 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49142

--- Comment #1 from hjl at gcc dot gnu.org <hjl at gcc dot gnu.org> 2011-05-25 16:19:40 UTC ---
Author: hjl
Date: Wed May 25 16:19:36 2011
New Revision: 174211

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=174211
Log:
Replace q_regs_operand with QIreg_operand in QI ext patterns.

2011-05-25  H.J. Lu  <hongjiu.lu@intel.com>

    PR target/49142
    * config/i386/i386.md (*movqi_extv_1_rex64): Remove
    "register_operand" check and replace q_regs_operand with
    QIreg_operand in "type" calculation.
    (*movqi_extv_1): Likewise.
    (*movqi_extzv_2_rex64): Likewise.
    (*movqi_extzv_2): Likewise.

    * config/i386/predicates.md (QIreg_operand): New.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/i386/i386.md
    trunk/gcc/config/i386/predicates.md


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug inline-asm/49142] Invalid 8bit register operand
  2011-05-24 14:34 [Bug inline-asm/49142] New: Invalid 8bit register operand hjl.tools at gmail dot com
  2011-05-25 16:39 ` [Bug inline-asm/49142] " hjl at gcc dot gnu.org
@ 2011-06-24 15:39 ` hjl.tools at gmail dot com
  1 sibling, 0 replies; 3+ messages in thread
From: hjl.tools at gmail dot com @ 2011-06-24 15:39 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49142

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.7.0

--- Comment #2 from H.J. Lu <hjl.tools at gmail dot com> 2011-06-24 15:39:25 UTC ---
Fixed.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-06-24 15:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-24 14:34 [Bug inline-asm/49142] New: Invalid 8bit register operand hjl.tools at gmail dot com
2011-05-25 16:39 ` [Bug inline-asm/49142] " hjl at gcc dot gnu.org
2011-06-24 15:39 ` hjl.tools at gmail dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).