This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH]: Put i386 insn prefixes on the same line as the insn

From: "Uros Bizjak" <ubizjak at gmail dot com>
To: "Andreas Schwab" <schwab at suse dot de>
Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
Date: Thu, 16 Aug 2007 14:56:42 +0200
Subject: Re: [PATCH]: Put i386 insn prefixes on the same line as the insn
References: <5787cf470708160527j2c919c05r96ed926ee5e7abc9@mail.gmail.com> <je3ayjbqsj.fsf@sykes.suse.de>

On 8/16/07, Andreas Schwab <schwab@suse.de> wrote:

> >    [(return)
> >     (unspec [(const_int 0)] UNSPEC_REP)]
> >    "reload_completed"
> > -  "rep {;} ret"
> > +  "rep{\;| }ret"
>
> Didn't you mean to use "rep ret" here?

No, because ret is not a string isns and putting "rep ret" on the same
line triggers an assembler error (in fact the check that the 'as'
manual is talking about) - at least with version 2.17 of gnu
assembler.

Uros.

References:
- [PATCH]: Put i386 insn prefixes on the same line as the insn
  - From: Uros Bizjak
- Re: [PATCH]: Put i386 insn prefixes on the same line as the insn
  - From: Andreas Schwab

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]