Re: RFA: Changes to interpreter to avoid non-debugging slowdown

[Tom Tromey <tromey at redhat dot com>]

>>>>> "Kyle" == Kyle Galloway <kgallowa@redhat.com> writes:

Kyle> There are other reasons for this as well, in addition to what Keith 
Kyle> said.  If we want the full suite of event notifications, we need to add 
Kyle> code to check for them so it could mean a slew of conditional checks to 
Kyle> see if we are debugging.  If we instead can just #ifdef this code into 
Kyle> the debugging version, it will mean we don't have to do conditional 
Kyle> checks in either case

Sounds good.

Kyle> Small oversight on my part, I've fixed it.  I had also forgot
Kyle> about the change in ncode() in the changelog.  New changelog and
Kyle> new patch attached.

The patch wasn't there... the changelog entry looks good though.

Kyle>    * /include/java-interp.h (_Jv_InterpMethod::run_debug): New method.

One nit: we don't put a '/' before the file name.

Tom