public inbox for gcc-prs@sourceware.org
help / color / mirror / Atom feed
From: gback@cs.utah.edu
To: java-gnats@sourceware.cygnus.com
Cc: gback@cs.utah.edu
Subject: java/1117: don't exclude private methods from method table when optimizing
Date: Wed, 20 Dec 2000 12:09:00 -0000	[thread overview]
Message-ID: <19991128194351.4018.qmail@sourceware.cygnus.com> (raw)

>Number:         1117
>Category:       java
>Synopsis:       don't exclude private methods from method table when optimizing
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    apbianco
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Dec 20 12:06:10 PST 2000
>Closed-Date:    Tue Dec 14 14:28:54 PST 1999
>Last-Modified:  Tue Dec 14 14:30:00 PST 1999
>Originator:     gback@cs.utah.edu
>Release:        CVS
>Organization:
>Environment:
Linux
>Description:
Private methods are excluded from the method table
when inlining or optimization is turned on.
This breaks reflection.  On the other hand, inlined
methods may not exist in the executable at all, so
it would be "ok" to not report them.
>How-To-Repeat:

>Fix:
Per Bothner suggests to keep methods in the method
table when the -fkeep-inline-functions flag is given.
Refer to
http://sourceware.cygnus.com/ml/java-discuss/1999-q4/msg00372.html

>Release-Note:

>Audit-Trail:

Formerly PR gcj/110

State-Changed-From-To: open->feedback
State-Changed-By: apbianco
State-Changed-When: Fri Dec 10 21:17:19 1999
State-Changed-Why:
    The patch looks OK to me. Does it work for you? I'd like
    to close this PR (I'll check the patch in and write a
    Changelog entry on behalf of Per Bothner, unless Per wants
    to do it himself.)

From: apbianco@cygnus.com
To: apbianco@cygnus.com, gback@cs.utah.edu, java-gnats@sourceware.cygnus.com
Cc:  
Subject: Re: gcj/110
Date: 11 Dec 1999 05:17:19 -0000

 Synopsis: don't exclude private methods from method table when optimizing
 
 State-Changed-From-To: open->feedback
 State-Changed-By: apbianco
 State-Changed-When: Fri Dec 10 21:17:19 1999
 State-Changed-Why:
     The patch looks OK to me. Does it work for you? I'd like
     to close this PR (I'll check the patch in and write a
     Changelog entry on behalf of Per Bothner, unless Per wants
     to do it himself.)
 
 http://sourceware.cygnus.com/cgi-bin/gnatsweb.pl?cmd=view&pr=110&database=java

From: Godmar Back <gback@cs.utah.edu>
To: apbianco@cygnus.com
Cc: gback@cs.utah.edu, java-gnats@sourceware.cygnus.com
Subject: Re: gcj/110
Date: Fri, 10 Dec 1999 22:55:09 -0700 (MST)

  
  I've been using the patch successfully.
 
 	- Godmar
 
 > 
 > Synopsis: don't exclude private methods from method table when optimizing
 > 
 > State-Changed-From-To: open->feedback
 > State-Changed-By: apbianco
 > State-Changed-When: Fri Dec 10 21:17:19 1999
 > State-Changed-Why:
 >     The patch looks OK to me. Does it work for you? I'd like
 >     to close this PR (I'll check the patch in and write a
 >     Changelog entry on behalf of Per Bothner, unless Per wants
 >     to do it himself.)
 > 
 > http://sourceware.cygnus.com/cgi-bin/gnatsweb.pl?cmd=view&pr=110&database=java
 > 
 
State-Changed-From-To: feedback->closed
State-Changed-By: apbianco
State-Changed-When: Tue Dec 14 14:28:54 1999
State-Changed-Why:
    I checked in Per's suggested patch:
    
      http://gcc.gnu.org/ml/gcc-patches/1999-12/msg00527.html

From: apbianco@cygnus.com
To: apbianco@cygnus.com, gback@cs.utah.edu, java-gnats@sourceware.cygnus.com
Cc:  
Subject: Re: gcj/110
Date: 14 Dec 1999 22:28:55 -0000

 Synopsis: don't exclude private methods from method table when optimizing
 
 State-Changed-From-To: feedback->closed
 State-Changed-By: apbianco
 State-Changed-When: Tue Dec 14 14:28:54 1999
 State-Changed-Why:
     I checked in Per's suggested patch:
     
       http://gcc.gnu.org/ml/gcc-patches/1999-12/msg00527.html
 
 http://sourceware.cygnus.com/cgi-bin/gnatsweb.pl?cmd=view&pr=110&database=java
>Unformatted:



                 reply	other threads:[~2000-12-20 12:09 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=19991128194351.4018.qmail@sourceware.cygnus.com \
    --to=gback@cs.utah.edu \
    --cc=java-gnats@sourceware.cygnus.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).