Re: java/4489: bytecode generation endless loop with certain switch() exprs

Re: java/4489: bytecode generation endless loop with certain switch() exprs

[tromey]

Synopsis: bytecode generation endless loop with certain switch() exprs

Responsible-Changed-From-To: unassigned->tromey
Responsible-Changed-By: tromey
Responsible-Changed-When: Mon Oct  8 13:38:34 2001
Responsible-Changed-Why:
    I fixed it.
State-Changed-From-To: open->closed
State-Changed-By: tromey
State-Changed-When: Mon Oct  8 13:38:34 2001
State-Changed-Why:
    I've checked in a fix.

http://gcc.gnu.org/cgi-bin/gnatsweb.pl?cmd=view&pr=4489&database=gcc

Re: java/4489: bytecode generation endless loop with certain switch() exprs

[Tom Tromey]

The following reply was made to PR java/4489; it has been noted by GNATS.

From: Tom Tromey <tromey@redhat.com>
To: apbianco@redhat.com
Cc: gcc-gnats@gcc.gnu.org
Subject: Re: java/4489: bytecode generation endless loop with certain switch() exprs
Date: 08 Oct 2001 13:03:54 -0600

 FYI, I have debugged this and I have a simple patch which I'll submit
 once `make check' has finished.
 
 Tom

java/4489: bytecode generation endless loop with certain switch() exprs

[apbianco]

>Number:         4489
>Category:       java
>Synopsis:       bytecode generation endless loop with certain switch() exprs
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    unassigned
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri Oct 05 19:56:01 PDT 2001
>Closed-Date:
>Last-Modified:
>Originator:     Alex Petit-Bianco
>Release:        unknown-1.0
>Organization:
>Environment:
all platforms
>Description:
Consider the following test case:

class Switch {
    void foo (int keyBits, int blockBits) {
      switch (keyBits >= blockBits ? keyBits : blockBits) {
	case 0:
	    break;
	case 1:
	    break;
	}
    }
}

It sends the bytecode generator into a endless loop.
Somethings appears to go wrong when the blocks are being
re-arranged in gcc/java/jcf-write.c:1814. 
>How-To-Repeat:
$ gcj -C Switch.java
>Fix:
Use a temporary variable to compute the expression the
switch is operating on.
>Release-Note:
>Audit-Trail:
>Unformatted:
----gnatsweb-attachment----
Content-Type: application/octet-stream; name="dummy"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="dummy"