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]
Other format: [Raw text]

Re: [RFA] _Jv_CompileMethod


Bryce McKinlay wrote:

Why call _Jv_InterpMethod::run() instead of _Jv_InterpMethod::compile() directly? That would make more logical sense I think, and the compile_mutex locking could be moved into compile() in order to handle the case where a request to compile were made simultaneously by both the interpreter and the JDWP.

Tom's already answered that. I'll mention that I tried REALLY hard to do what you mention in your follow-up (which I just saw), but it's incredibly, frustratingly difficult. And very ugly. Very, very ugly.


Also, for the current patch, it looks like the "When args is NULL, don't run anything -- just compile it." comment is on the wrong method.

Yes, you are right. Grr. I've fixed that in the patch I'm checking in.


Do you have commit access for gcc/libgcj?

I believe I still do, but if the commit fails, I'll let you know! ;-)


Keith


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