RFR: 8152311: allow JVMCI compiler to change the compilation policy for a method
doug.simon at oracle.com
Thu May 5 21:14:58 UTC 2016
> On 05 May 2016, at 20:06, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> I looked on VM changes. You put #include under #if INCLUDE_JVMCI but call to JVMCIRuntime::adjust_comp_level() method are guarded only by flag. Some compilers may complain.
Ok, I added the missing #if INCLUDE_JVMCI guards. Are there regular test builds with INCLUDE_JVMCI=0 that would have caught this?
> In thread.hpp is there a reason to use several bool fields instead of one field for flags?
Probably didn’t seem worth it when there were only 2 bool fields. Now that there will be three, should I convert to using a flags field?
Thanks for the review.
> On 5/5/16 5:39 AM, Doug Simon wrote:
>> Please read the bug for a full description of this proposed change.
More information about the hotspot-compiler-dev