[SH] PR 51244 - Improve conditional branches

Oleg Endo oleg.endo@t-online.de
Thu Mar 8 08:32:00 GMT 2012


On Tue, 2012-03-06 at 08:13 +0900, Kaz Kojima wrote:
> > Oleg Endo <oleg.endo@t-online.de> wrote:
> >> The attached patch is the same as the last one proposed in the PR.
> >> Tested against rev 184877 with
> >> 
> >> make -k check RUNTESTFLAGS="--target_board=sh-sim
> >> \{-m2/-ml,-m2/-mb,-m2a-single/-mb,
> >> -m4-single/-ml,-m4-single/-mb,
> >> -m4a-single/-ml,-m4a-single/-mb}"
> >> 
> >> and no new failures.
> >> OK?
> > 
> > OK.
> 
> I've tested your latest patch on sh4-unknown-linux-gnu with trunk
> revision 184872.  It looks that some new failures are poping up:
> 
> New tests that FAIL:
> 
> 22_locale/ctype/is/char/3.cc execution test
> 27_io/basic_filebuf/underflow/wchar_t/9178.cc execution test
> gfortran.dg/widechar_intrinsics_6.f90  -Os  execution test
> 
> Pehaps failures might be ones you've suggested in #10 in
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51244
> 

This is the patch for the patch, as attached in the PR.
Tested against rev 184966 as before and no changes in the test results
for me (i.e. no new failures).

ChangeLog:

	PR target/51244
	* config/sh/sh.md (movnegt): Expand into respective insns 
	immediately.  Use movrt_negc instead of negc pattern for
	non-SH2A.
	(*movnegt): Remove.
	(*movrt_negc, *negnegt, *movtt, *movt_qi): New insns and splits.

testsuite/ChangeLog:

	PR target/51244
	* gcc.target/sh/pr51244-1.c: Fix thinkos.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: sh_pr51244_2.patch
Type: text/x-patch
Size: 4416 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20120308/387f05ac/attachment.bin>


More information about the Gcc-patches mailing list