RFR(S): 8016277: Crash in nmethod::is_compiled_by_c1() on x86

Roland Westrelin roland.westrelin at oracle.com
Mon Aug 26 07:08:50 PDT 2013


> I would appreciate if you could add brackets to the simple if statements:
> if (compiler() == NULL) {
>  return false;
> }
> 
> And also, if you could put the comment above the line instead of to the right:
> 
> // the Method may be reclaimed by class unloading now that the nmethod is in zombie state
> _method = NULL;

I'll make those changes and use set_method() rather than _method = . Thanks for the review.

Roland.


More information about the hotspot-compiler-dev mailing list