This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]

Re: Patch: jcf-write and last_bc

To: apbianco at cygnus dot com
Subject: Re: Patch: jcf-write and last_bc
From: Per Bothner <per at bothner dot com>
Date: Mon, 08 Oct 2001 17:57:19 -0700
CC: Gcc Patch List <gcc-patches at gcc dot gnu dot org>, Java Patch List <java-patches at gcc dot gnu dot org>
References: <87bsjh1znm.fsf@creche.redhat.com> <3BC23C15.50404@bothner.com> <15298.16704.697050.187042@fencer.cygnus.com>

There's something screwy here.  The ChangeLog says

Insert `nop' if `jsr' immediately follows `monitorenter'.

But there is nothing about 'jsr' in the patch itself.  The
patch is for CASE CONVERT_EXPR etc.  What does that have to
do with monitorentry+jsr?  At the very least we need a comment
explaining what is going on.

I'm going to try to reproduce the bug without the patch
to see what is going on.

Follow-Ups:
- Re: Patch: jcf-write and last_bc
  - From: Alexandre Petit-Bianco

References:
- Patch: jcf-write and last_bc
  - From: Tom Tromey
- Re: Patch: jcf-write and last_bc
  - From: Per Bothner
- Re: Patch: jcf-write and last_bc
  - From: Alexandre Petit-Bianco

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]