[Bug java/9577] Linker error for legal Java/CNI code if method parameters are reversed

[Bug java/9577] Linker error for legal Java/CNI code if method parameters are reversed

[pinskia@physics.uc.edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9577


pinskia@physics.uc.edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
   Last reconfirmed|0000-00-00 00:00:00         |2003-05-26 00:27:07
               date|                            |


------- Additional Comments From pinskia@physics.uc.edu  2003-05-26 00:27 -------
still happens on the mainline (20030525):

tin:~/src/gnu/gcctest/9577>gcj --main=Foo Foo.java snafu.cc
/tmp/ccUDft0d.o(.data+0xc): undefined reference to `Foo::bar(JArray<java::lang::String*>
*, java::io::File*)'
/tmp/ccUDft0d.o(.data+0x64): undefined reference to `Foo::bar(JArray<java::lang::String*
>*, java::io::File*)'


Foo.o:
                   U _ZN3Foo3barEP6JArrayIPN4java4lang6StringEEPN4java2io4FileE
snafu.o:
00000000 T _ZN3Foo3barEP6JArrayIPN4java4lang6StringEEPNS1_2io4FileE

Looks like someone is mangling the function wrong.




------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug java/9577] Linker error for legal Java/CNI code if method parameters are reversed

[pinskia at physics dot uc dot edu]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9577


pinskia at physics dot uc dot edu changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |wrong-code
   Last reconfirmed|2003-05-26 00:27:07         |2003-08-03 18:25:49
               date|                            |


------- Additional Comments From pinskia at physics dot uc dot edu  2003-08-03 18:25 -------
In fact the two symbols demangle to the same function.

[Bug java/9577] Linker error for legal Java/CNI code if method parameters are reversed

[rmathew at hotmail dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9577



------- Additional Comments From rmathew at hotmail dot com  2003-08-03 18:32 -------
Subject: Re:  Linker error for legal Java/CNI code if method parameters ar

>------- Additional Comments From pinskia at physics dot uc dot edu  
>2003-08-03 18:25 -------
>In fact the two symbols demangle to the same function.

Curious - I was looking at the possible causes
of the same bug today!

Yes, they do demangle to the same
function - what is incorrect is that the Java
mangler does not compress the common
"java" between "java.lang.String" and
"java.io.File", while the C++ mangler
does.

Strangely, this seems to happen only
if the array argument precedes the
"normal" argument, but not vice versa.

I looked through the code a few times
but could not find an obvious cause for
the behaviour.

I'll keep trying.

Ranjit.

_________________________________________________________________
Going on a holiday? Want to study abroad? 
http://server1.msn.co.in/msnleads/citibankpersonalloan/citibankploanjuly03.asp?type=txt 
Need a personal loan?

[Bug java/9577] Linker error for legal Java/CNI code if method parameters are reversed

[rmathew at hotmail dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9577



------- Additional Comments From rmathew at hotmail dot com  2003-08-11 10:07 -------
I have just submitted a patch for this bug:

http://gcc.gnu.org/ml/java-patches/2003-q3/msg00317.html

The problem was that find_compression_record_match( )
in mangle.c was incorrectly assuming that all
IDENTIFIER_NODEs in the compression table are
partial package names - a "6JArray" (the mangled
name for an array template) breaks this
assumption.

[Bug java/9577] Linker error for legal Java/CNI code if method parameters are reversed

[cvs-commit at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9577



------- Additional Comments From cvs-commit at gcc dot gnu dot org  2003-09-17 15:25 -------
Subject: Bug 9577

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	tromey@gcc.gnu.org	2003-09-17 15:25:08

Modified files:
	libjava/testsuite: ChangeLog 
Added files:
	libjava/testsuite/libjava.cni: PR9577.java PR9577.out 
	                               natPR9577.cc 

Log message:
	2003-09-17  Ranjit Mathew  <rmathew@hotmail.com>
	
	PR java/9577
	* libjava.cni/PR9577.java: New file.
	* libjava.cni/natPR9577.cc: New file.
	* libjava.cni/PR9577.out: New file.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/testsuite/ChangeLog.diff?cvsroot=gcc&r1=1.290&r2=1.291
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/testsuite/libjava.cni/PR9577.java.diff?cvsroot=gcc&r1=NONE&r2=1.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/testsuite/libjava.cni/PR9577.out.diff?cvsroot=gcc&r1=NONE&r2=1.1
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/testsuite/libjava.cni/natPR9577.cc.diff?cvsroot=gcc&r1=NONE&r2=1.1

[Bug java/9577] Linker error for legal Java/CNI code if method parameters are reversed

[cvs-commit at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9577



------- Additional Comments From cvs-commit at gcc dot gnu dot org  2003-09-17 15:27 -------
Subject: Bug 9577

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	tromey@gcc.gnu.org	2003-09-17 15:26:59

Modified files:
	gcc/java       : ChangeLog mangle.c 

Log message:
	2003-09-17  Ranjit Mathew  <rmathew@hotmail.com>
	
	Fixes PR java/9577
	* mangle.c (find_compression_record_match): Skip
	over a "6JArray" (the array template mangled string)
	IDENTIFIER_NODE.
	(mangle_array_type): Correct minor typo.
	(atms): Move definition to the beginning.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/java/ChangeLog.diff?cvsroot=gcc&r1=1.1248&r2=1.1249
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/java/mangle.c.diff?cvsroot=gcc&r1=1.29&r2=1.30

[Bug java/9577] Linker error for legal Java/CNI code if method parameters are reversed

[tromey at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9577


tromey at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |3.4


------- Additional Comments From tromey at gcc dot gnu dot org  2003-09-17 16:20 -------
I've checked in the test case and fix.