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 6/6] combine: allow combining two insns into two in some cases (PR52714)


On 12/02/14 18:11, Segher Boessenkool wrote:
On Mon, Dec 01, 2014 at 10:39:58AM -0700, Jeff Law wrote:
Also OK with a testcase.  I'd recommend just using the one from 52714,
make it m68k specific.  Not sure if it's best to scan the assembly or
.combine dump -- your call.

Here is the testcase.  I cannot actually test it on an m68k build, should
really do something about that (I build lots of cross compilers but they
cannot run the testsuite).  Does it look okay, can you test it yourself?
[I did of course test the generated asm is indeed okay; just cannot test
the dejagnu stuff].


Segher


2014-12-02  Segher Boessenkool  <segher@kernel.crashing.org>

gcc/testsuite/
	PR rtl-optimization/52714
	* gcc.target/m68k/pr52714.c: New testcase.
THanks.  Based on a later message, you were able to do the testing.

At some point during the gcc-5 release process, I'll bootstrap m68k via Aranym.

Jeff


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