On 3/07/2021 7:58 am, Daniel D.Daugherty wrote:
On Fri, 2 Jul 2021 07:03:50 GMT, David Holmes <dhol...@openjdk.org> wrote:

Please review this simple refactoring to share the common code used to create 
j.l.Thread instances for the internal VM JavaThreads. (Also fix a missing space 
from my previous change in thread.cpp.)

It is all very straight-forward.

Compiler folk I can go one step further and remove 
CompileBroker::create_thread_oop if desired.

Testing: tiers 1-3, GHA

Thanks,
David

Very nice cleanup. Thumbs up!

Thanks for the review Dan.

David

-------------

Marked as reviewed by dcubed (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/4663

Reply via email to