[Bug objc/37460] New: [4.4 Regression]: objc.dg/super-class-2.m objc.dg/zero-link-1.m objc.dg/zero-link-2.m ICE

[Bug objc/37460] New: [4.4 Regression]: objc.dg/super-class-2.m objc.dg/zero-link-1.m objc.dg/zero-link-2.m ICE

[hp at gcc dot gnu dot org]

With revision 140146 these tests passed.
>From revision 140155 and on, these tests have failed as follows:

Running /tmp/hpautotest-gcc1/gcc/gcc/testsuite/objc.dg/dg.exp ...
FAIL: objc.dg/super-class-2.m (internal compiler error)
FAIL: objc.dg/super-class-2.m (test for excess errors)
FAIL: objc.dg/zero-link-1.m (internal compiler error)
FAIL: objc.dg/zero-link-1.m (test for excess errors)
ERROR: objc.dg/zero-link-1.m: error executing dg-final: couldn't open
"zero-link-1.s": no such file or directory
FAIL: objc.dg/zero-link-2.m (internal compiler error)
FAIL: objc.dg/zero-link-2.m (test for excess errors)
ERROR: objc.dg/zero-link-2.m: error executing dg-final: couldn't open
"zero-link-2.s": no such file or directory

With the message in the logfile being (cutnpasted):

Executing on host: /tmp/hpautotest-gcc1/cris-elf/gccobj/gcc/xgcc
-B/tmp/hpautotest-gcc1/cris-elf/gccobj/gcc/ /tmp/hpaut\
otest-gcc1/gcc/gcc/testsuite/objc.dg/super-class-2.m   -fnext-runtime
-I/tmp/hpautotest-gcc1/gcc/gcc/testsuite/../../li\
bobjc -L/tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/./libobjc/.libs -S  
-isystem /tmp/hpautotest-gcc1/cris-elf/gccob\
j/cris-elf/./newlib/targ-include -isystem
/tmp/hpautotest-gcc1/gcc/newlib/libc/include  -o super-class-2.s    (timeout \
= 300)
/tmp/hpautotest-gcc1/gcc/gcc/testsuite/objc.dg/super-class-2.m: In function
'+[NSMenuItem test:func0]':^M
/tmp/hpautotest-gcc1/gcc/gcc/testsuite/objc.dg/super-class-2.m:22: error:
invalid conversion in gimple call^M
int^M
^M
struct objc_object *^M
^M
D.1299 = objc_msgSendSuper (objc_super.0, _OBJC_SELECTOR_REFERENCES_0.1);^M
^M
/tmp/hpautotest-gcc1/gcc/gcc/testsuite/objc.dg/super-class-2.m:22: internal
compiler error: verify_gimple failed^M

Messages from the other two ICEs are similar.

Author of patches in suspect revision range as well as objc maintainer CC:ed.


-- 
           Summary: [4.4 Regression]: objc.dg/super-class-2.m objc.dg/zero-
                    link-1.m objc.dg/zero-link-2.m ICE
           Product: gcc
           Version: 4.4.0
            Status: UNCONFIRMED
          Keywords: ice-checking
          Severity: normal
          Priority: P3
         Component: objc
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hp at gcc dot gnu dot org
  GCC host triplet: x86_64-unknown-linux-gnu
GCC target triplet: cris-axis-elf


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

[Bug objc/37460] [4.4 Regression]: objc.dg/super-class-2.m objc.dg/zero-link-1.m objc.dg/zero-link-2.m ICE

[hp at gcc dot gnu dot org]

------- Comment #1 from hp at gcc dot gnu dot org  2008-09-10 16:19 -------
Oops, pinskia is libobjc maintainer, not objc maintainer...
CC:ed the listed maintainers.


-- 

hp at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mrs at gcc dot gnu dot org,
                   |                            |shebs at gcc dot gnu dot org


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

[Bug objc/37460] [4.4 Regression]: objc.dg/super-class-2.m objc.dg/zero-link-1.m objc.dg/zero-link-2.m ICE

[dominiq at lps dot ens dot fr]

------- Comment #2 from dominiq at lps dot ens dot fr  2008-09-11 11:45 -------
I see the same errors (a lot of them) on Darwin (see also
http://gcc.gnu.org/ml/gcc-testresults/2008-09/msg00938.html).


-- 


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

[Bug objc/37460] [4.4 Regression]: objc.dg/super-class-2.m objc.dg/zero-link-1.m objc.dg/zero-link-2.m ICE

[rguenth at gcc dot gnu dot org]

-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   GCC host triplet|x86_64-unknown-linux-gnu    |
 GCC target triplet|cris-axis-elf               |
   Last reconfirmed|0000-00-00 00:00:00         |2008-09-13 12:52:21
               date|                            |
   Target Milestone|---                         |4.4.0


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

[Bug objc/37460] [4.4 Regression]: objc.dg/super-class-2.m objc.dg/zero-link-1.m objc.dg/zero-link-2.m ICE

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2008-09-14 01:31 -------
This is the same fall out from the tuplification as I mention at
http://gcc.gnu.org/ml/gcc/2008-06/msg00460.html just now we are getting it for
all NeXT runtime testcases.  I am going to work on this.  It is just a matter
of changing how OBJ_TYPE_REF is used for the NeXT runtime.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |pinskia at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED


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

[Bug objc/37460] [4.4 Regression]: objc.dg/super-class-2.m objc.dg/zero-link-1.m objc.dg/zero-link-2.m ICE

[pinskia at gcc dot gnu dot org]

------- Comment #4 from pinskia at gcc dot gnu dot org  2008-09-14 01:48 -------
Created an attachment (id=16316)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16316&action=view)
Patch which I am testing to fix this and all tuplification related failures

This fixes the issue by just removing objc_get_callee_fndecl which is not
needed really as expand does the correct thing using OBJ_TYPE_REF_EXPR already.
 I have not figured out why this was added in the first place if
OBJ_TYPE_REF_EXPR had the correct answer already.


-- 


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

[Bug objc/37460] [4.4 Regression]: NeXT runtime ICEs

[pinskia at gcc dot gnu dot org]

------- Comment #5 from pinskia at gcc dot gnu dot org  2008-09-14 01:49 -------
And yes we still get -O0, a call:
        call    _objc_msgSend


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.4 Regression]:           |[4.4 Regression]: NeXT
                   |objc.dg/super-class-2.m     |runtime ICEs
                   |objc.dg/zero-link-1.m       |
                   |objc.dg/zero-link-2.m ICE   |


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

[Bug objc/37460] [4.4 Regression]: NeXT runtime ICEs

[dominiq at lps dot ens dot fr]

------- Comment #6 from dominiq at lps dot ens dot fr  2008-09-16 12:50 -------
With the patch in comment #4, the number of failures is back to the previous
situation (i686-apple-darwin9):

=== objc Summary for unix/-m64 ===

# of expected passes            1999
# of unexpected failures        570
# of expected failures          7
# of unresolved testcases       464
# of unsupported tests          2

=== objc Summary ===

# of expected passes            4895
# of unexpected failures        655
# of expected failures          14
# of unresolved testcases       538
# of unsupported tests          4
/opt/gcc/i686-darwin/gcc/xgcc  version 4.4.0 [revision 140386p8] 20080916
(experimental) (GCC) 

=== obj-c++ Summary for unix/-m64 ===

# of expected passes            416
# of unexpected failures        25
# of unexpected successes       6
# of expected failures          16
# of unresolved testcases       1
# of unsupported tests          3

=== obj-c++ Summary ===

# of expected passes            852
# of unexpected failures        48
# of unexpected successes       12
# of expected failures          21
# of unresolved testcases       2
# of unsupported tests          4
/opt/gcc/i686-darwin/gcc/testsuite/obj-c++/../../g++  version 4.4.0 [revision
140386p8] 20080916 (experimental) (GCC) 


-- 


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

[Bug objc/37460] [4.4 Regression]: NeXT runtime ICEs

[pinskia at gcc dot gnu dot org]

------- Comment #7 from pinskia at gcc dot gnu dot org  2008-09-17 21:39 -------
Fixed.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


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

[Bug objc/37460] [4.4 Regression]: NeXT runtime ICEs

[pinskia at gcc dot gnu dot org]

------- Comment #8 from pinskia at gcc dot gnu dot org  2008-09-17 21:40 -------
Subject: Bug 37460

Author: pinskia
Date: Wed Sep 17 21:39:13 2008
New Revision: 140430

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=140430
Log:
2008-09-17  Andrew Pinski  <andrew_pinski@playstation.sony.com>

        * tree.c (get_callee_fndecl): Don't call the language hook.
        * langhooks.h (lang_hooks): Remove lang_get_callee_fndecl.
        * langhooks-def.h (LANG_HOOKS_GET_CALLEE_FNDECL): Kill.
        (LANG_HOOKS_INITIALIZER): Remove LANG_HOOKS_GET_CALLEE_FNDECL.


2008-09-17  Andrew Pinski  <andrew_pinski@playstation.sony.com>

        PR objc/37460
        * objc-lang.c (LANG_HOOKS_GET_CALLEE_FNDECL): Don't define.
        * objc-act.h (objc_get_callee_fndecl): Remove prototype.
        * objc-act.c (objc_get_callee_fndecl): Kill.

2008-09-17  Andrew Pinski  <andrew_pinski@playstation.sony.com>

        * lang.c (LANG_HOOKS_GET_CALLEE_FNDECL): Don't define.
        (java_get_callee_fndecl): Kill.

2008-09-17  Andrew Pinski  <andrew_pinski@playstation.sony.com>

        PR objc/37460
        * objcp-lang.c (LANG_HOOKS_GET_CALLEE_FNDECL): Don't define.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/java/ChangeLog
    trunk/gcc/java/lang.c
    trunk/gcc/langhooks-def.h
    trunk/gcc/langhooks.h
    trunk/gcc/objc/ChangeLog
    trunk/gcc/objc/objc-act.c
    trunk/gcc/objc/objc-act.h
    trunk/gcc/objc/objc-lang.c
    trunk/gcc/objcp/ChangeLog
    trunk/gcc/objcp/objcp-lang.c
    trunk/gcc/tree.c


-- 


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