target/10412: Renesas SH - Incorrect code generation

target/10412: Renesas SH - Incorrect code generation

[shrinivasa]

>Number:         10412
>Category:       target
>Synopsis:       Renesas SH - Incorrect code generation
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          wrong-code
>Submitter-Id:   net
>Arrival-Date:   Tue Apr 15 11:56:00 UTC 2003
>Closed-Date:
>Last-Modified:
>Originator:     shrinivasa@kpitcummins.com
>Release:        GCC Snapshor 14/04/2003 and GCC CVS
>Organization:
>Environment:
[shrinivasa@Linuxsrv5 bug]$ ~/sh-elf-cvs/bin/sh-elf-gcc -v
Reading specs from /home/shrinivasa/sh-elf-cvs/lib/gcc-lib/sh-elf/3.4/specs
Configured with: /home/shrinivasa/src/gcc/configure --enable-languages=c,c++ --t
arget=sh-elf --with-newlib --with-headers=/home/shrinivasa/src/newlib-1.11.0/new
lib/libc/include --prefix=/home/shrinivasa/sh-elf-cvs
Thread model: single
gcc version 3.4 20030415 (experimental)

and 


Reading specs from /home/shrinivasa/sh-elf-15042003/lib/gcc-lib/sh-elf/3.3/specs

Configured with: /home/shrinivasa/src/gcc-20030414/configure --enable-languages=
c,c++ --target=sh-elf --with-newlib --prefix=/home/shrinivasa/sh-elf-15042003 :
(reconfigured) /home/shrinivasa/src/gcc-20030414/configure --enable-languages=c,
c++ --target=sh-elf --with-newlib --prefix=/home/shrinivasa/sh-elf-15042003 : (r
econfigured) /home/shrinivasa/src/gcc-20030414/configure --enable-languages=c,c+
+ --target=sh-elf --with-newlib --with-headers=/home/shrinivasa/src/newlib-1.11.
0/newlib/libc/include --prefix=/home/shrinivasa/sh-elf-15042003
Thread model: single
gcc version 3.3 20030414 (prerelease)
>Description:
When following code is compiled  with
sh-elf-gcc  -S  -mhitachi -m2 -O2 -bug1-cvs.s bug1.c
generates an incorrect code.

int func0 (int x);

int func1 (unsigned long addr)
{
  int err;

  err = func0 (addr);
  if (err) return (err);

  err = func0 (addr * 7);
  return (err);
}

The generated code is 

	.file	"bug1.c"
	.text
	.text
	.align 1
	.align 4
	.global	_func1
	.type	_func1, @function
_func1:
	mov.l	r8,@-r15
	mov.l	r14,@-r15
	mov	r4,r8
	sts.l	macl,@-r15
	mov.l	.L6,r1
	sts.l	pr,@-r15
	jsr	@r1
	mov	r15,r14
	tst	r0,r0
	bt	.L5
	mov	r14,r15
	lds.l	@r15+,pr
	lds.l	@r15+,macl
	mov.l	@r15+,r14
	rts	
	mov.l	@r15+,r8
	.align 4
.L5:
	mov	#7,r1 ; <<<<< HERE  address of func0 is lost 
	mul.l	r1,r8
	sts	macl,r4
	mov	r14,r15
	lds.l	@r15+,pr
	lds.l	@r15+,macl
	mov.l	@r15+,r14
	jmp	@r4	; <<<<< R4 contains multiplication, incorrect jump
	mov.l	@r15+,r8
.L7:
	.align 2
.L6:
	.long	_func0
	.size	_func1, .-_func1
	.ident	"GCC: (GNU) 3.4 20030415 (experimental)"

The same problem is observed for current snapshot of GCC also. ( Dated 14th April 2003)
>How-To-Repeat:
Compile the attached bug1.c with following command.
sh-elf-gcc  -S  -mhitachi -m2 -O2 -.s bug1.c

Problem occurs for current CVS branch as well as latest GCC-3.3 snapshot.
>Fix:
None.
If -mhitachi switch is not used then the problem disappears.
>Release-Note:
>Audit-Trail:
>Unformatted:
----gnatsweb-attachment----
Content-Type: application/x-zip-compressed; name="bug.zip"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="bug.zip"

UEsDBBQAAAAIACeJjy75vYklEQEAADYCAAAKAAAAYnVnMS1jdnMuc6VRPW/CMBCdz78iShdQjRWD
A1GmSAwsqFvnKgkmMnI+dHYq2l9fO2kCtCOT7927u/fOB+ystISw6CvOypAAs/Jqb0+uVdUEfI6E
iyrdFrmGj3PflJ6xX538RTTI/GtV25AxkxKo20+mAROarZDHM+biLgEoKCYEjDWOq/NSP1az45Yi
n/gOJ/ZiEDJPDDN47IqcRWssYEQxIlBY1ztpOMmhS5/8FNcXv9IOH7HXnlTHzDASrYE/6eTuV5zG
uCm87Aajda+HJaetxp1QPO/kUncOin9m2HGXzo7WDm491G1Tjcdxn8GM+r6diq3mE6qTbCyEh/0+
DRaHt/dlsGEiWEfRJhI8Dhby2klUtSvK9TIkP1BLAwQUAAAACAAaiY8u3/+CSWQAAACpAAAABgAA
AGJ1ZzEuY3WMsQqAMAxE90D+4cbWSScH8WPEtlKQCNGCIP67WnUR3C6X9y7KgpCkL2HiGVfbMDHF
p61gksxxEO8wTjKgc04t08YEXJBXzQKuhPadyliToQBzvizUL0nlPv4UFKhv7UPvTAdQSwMEFAAA
AAgALomPLrq6YoIOAQAANAIAAAsAAABidWcxLXNuYXAuc61RPW+DMBCdz7/CIkuiOhYOpkVMSBmy
RN06V0AcRGQ+dDZR219fGwpJ2rXT3bt3H+/ugJ9rrSAohkrwMiDArfqwN5PrumqpWDzpvEp3Ra7h
/Ty0pWfsZ69+EKOZt7buWjJFUgJNd+UaMGHZFkW8YCHvAoCSYULAWOO4Ji/1YzY/xgzFzPc4sxeD
kHli7CF8kpNojQUMGYYECutq5cy7Ib5Kn3wXVxc/sR4fsZ89T50iY0u0Bn6Fk7uruBnTprB6GYU2
gx6XnLeadsJ/UHJpegflHzH8+JwuinYOxh7qrq2m57hjcFN/3V7Ft8sL65NqLQSH/T6l68Pr24ZG
PKK7MIxCKSRd96hQaZUbtQnIN1BLAQIUABQAAAAIACeJjy75vYklEQEAADYCAAAKAAAAAAAAAAEA
AAC2gQAAAABidWcxLWN2cy5zUEsBAhQAFAAAAAgAGomPLt//gklkAAAAqQAAAAYAAAAAAAAAAQAg
ALaBOQEAAGJ1ZzEuY1BLAQIUABQAAAAIAC6Jjy66umKCDgEAADQCAAALAAAAAAAAAAEAAAC2gcEB
AABidWcxLXNuYXAuc1BLBQYAAAAAAwADAKUAAAD4AgAAAAA=