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: IRA conflict graph, multiple alternatives and commutative operands

From: Bernd Schmidt <bernds at codesourcery dot com>
To: "Vladimir N. Makarov" <vmakarov at redhat dot com>
Cc: Jie Zhang <jie at codesourcery dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Fri, 19 Mar 2010 19:20:10 +0100
Subject: Re: IRA conflict graph, multiple alternatives and commutative operands
References: <4B951A49.6070200@codesourcery.com> <4BA2A106.1050606@codesourcery.com> <4BA37383.7010005@redhat.com>

On 03/19/2010 01:52 PM, Vladimir N. Makarov wrote:
> After some reasoning I got a conclusion that this patch has more
> advantages than disadvantages.  The patch should not reproduce PR22072
> and will result in worse code only when clobbered classes are different
> from matched classes and pseudos get one of the clobbered classes (I
> think the probability of this is very small).

> So I guess the patch is ok.  It needs a blank line after alt1
> declaration.  It would be nice to add some comment too.

I've also added the testcase from the PR, bootstrapped & regression
tested on i686-linux, ran arm-elf regression tests and committed the
patch below.  Thanks!


Bernd

Attachment: ira-42258.diff
Description: Text document

References:
- Re: IRA conflict graph, multiple alternatives and commutative operands
  - From: Bernd Schmidt
- Re: IRA conflict graph, multiple alternatives and commutative operands
  - From: Vladimir N. Makarov

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