RFR(L): JEP165: Compiler Control

Roland Westrelin roland.westrelin at oracle.com
Mon Sep 28 10:02:33 UTC 2015


Hi Nils,

> http://cr.openjdk.java.net/~neliasso/8046155/webrev.03/

What does CompileCommandCompatibility do?

vm_operations.cpp

Is that extra include really needed?

CompilerQueueTest.java

Is that extra import really needed?

block.hpp

You don’t need that ciEnv here, right?

compileBroker.cpp

1717   should_break = dirset->BreakAtExecuteOption || task->check_break_at_flags();

doesn’t seem to be used

Except for those minor issues, looks good to me.


> - Won't build c1 only or shark only.

You can’t push if c1 only doesn’t build, right?

Roland.


More information about the hotspot-compiler-dev mailing list