Code Review fix for 8005044 remove crufty '_g' support from HS runtime code
coleen.phillimore at oracle.com
Tue Dec 18 13:29:05 PST 2012
This looks good. Thank you for doing this.
On 12/18/2012 03:46 PM, Daniel D. Daugherty wrote:
> I'm sponsoring this code review request from Ron Durbin. This change
> is targeted at JDK8/HSX-25 in the RT_Baseline repo.
> Sending again with correct subject line, bug URLs and webrev URL.
> This set of changes removes the runtimesupport for generation of debug
> versions that follow _g semantics.
> JDK-8005044 remove crufty '_g' support from HS runtime code
> Files have been modified to remove all reference and support for
> debug versions that follow _g semantics.
> Passed JPRT last night:
> Additional Testing In process: (suggested by Dan):
> - test with shared archive creation and use; see the e-mail
> from Coleen
> - just a usage message; visual inspection of the code
> - comments only; no testing needed
> - the only code changes come into play when the "gamma"
> launcher is used
> - and when JAVA_HOME refers to a valid JDK, the function
> fakes up a JVM path so that callers using the JVM path
> to find other things in the JDK will work.
> - I can't find any way that the actual JVM path value
> that is returned is exposed
> - I don't see a way to test this other than have a debug
> printf() or manual code inspection.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the hotspot-runtime-dev