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]

Re: [patch] h8300.md: Add new peepholes.


On Mon, Feb 19, 2001 at 03:45:34PM -0500, Kazu Hirata wrote:
> Hi Zack,
> 
> > > 2001-02-19  Kazu Hirata  <kazu@hxi.com>
> > > 
> > > 	* config/h8300/h8300.md: Add peepholes to optimize ior and
> > > 	xor.
> >
> > Shouldn't you be using define_peephole2?
> 
> Is it OK even if RTL generated by define_peephole2 is rather complex, like a
> parallel consisting of a zero extension and an ior?  (I am guessing it is
> because if you emit assembly language with define_peephole, then no other
> pass can modify it anyway.)

I believe it is okay, as long as the RTL is recognizable insns.  Worst
case, no later pass will be able to improve it.  If you use text
peepholes, then nothing gets a chance to improve it.  Also we'd like
to get rid of the text peephole machinery eventually.

zw


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