[PATCH] combine: Don't create (set (reg:CC) (compare (reg:CC) (const0)))

Jeff Law law@redhat.com
Mon May 11 21:16:00 GMT 2015


On 05/11/2015 03:11 PM, Segher Boessenkool wrote:
> On Mon, May 11, 2015 at 11:23:47AM -0600, Jeff Law wrote:
>>> 	* combine.c (simplify_set): When generating a CC set, if the
>>> 	source already is in the correct mode, do not wrap it in a
>>> 	compare.  Simplify the rest of that code.
>> Seems reasonable.  Might not hurt to do a little testing on a cc0 target
>> though.
>
> I tested on mn10300, cris, and m68k.  On m68k it triggers while
> building libgcc (for 040).  I verified the transform is correct.
Seems reasonable to me.

>
> I don't have a setup to actually bootstrap or regression check
> any cc0 target.  A recurring theme :-(
Understood.  As I mentioned in IRC and in another thread, it may be 
useful to set up an aranym instance for m68k bootstrap testing in the 
compile farm.

For other targets, building foo-elf in a single tree build with 
binutils, newlib and the simulator, then running the gcc testsuite with 
the simulator is often a good alternative.  Probably wise to run through 
this once or twice just to get familiar with it, then pull it out when 
really needed ;)

jeff



More information about the Gcc-patches mailing list