This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java 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


Tom Tromey wrote:

>While looking at some other problems with bytecode generation, I
>noticed that `state->last_bc' isn't always updated correctly.  OP1
>sets this, and OP1 is also used to emit data.  When this happens,
>last_bc will be set to some arbitrary byte value.
>
>In this patch I've introduced OP1DATA, which doesn't set last_bc and
>which is used whenever we're output data (as opposed to an opcode).
>
I can't say I like this patch, but it is hard for me to be specific without
understanding whatever problem the last_bc field is supposed to fix.
Alex added it 2000-10-18 and says it fixes "gcj/232", but I don't know
how to find that in the merged database.  On general principles, I dislike
adding a nop.



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