This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug target/54963] [4.8 Regression] Wrong code generated for libgfortran/generated/eoshift3_8.c on SH
- From: "olegendo at gcc dot gnu.org" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: Sun, 28 Oct 2012 23:01:24 +0000
- Subject: [Bug target/54963] [4.8 Regression] Wrong code generated for libgfortran/generated/eoshift3_8.c on SH
- Auto-submitted: auto-generated
- References: <bug-54963-4@http.gcc.gnu.org/bugzilla/>
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54963
--- Comment #3 from Oleg Endo <olegendo at gcc dot gnu.org> 2012-10-28 23:01:24 UTC ---
Created attachment 28551
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=28551
Proposed patch
This patch fixes the problem, by using 'emit_move_insn' instead of manually
doing the DImode reg copy.
I've seized the moment and refactored the abs patterns -- the T_REG clobber
handling was a bit confusing and using mode iterators saves a few lines.
Kaz, could you please have a look at this one?
Only briefly tested with make-gcc and compiling CSiBE. There are no code size
changes in the CSiBE set, except for one:
jikespg-1.3
src/prntstat 3576 -> 3568 -8 / -0.223714 %
I guess it's the T_REG clobber thing that has a positive impact on register
allocation in this case.