RFR: 8213224: Move code related to GC threads calculation out of AdaptiveSizePolicy
Roman Kennke
rkennke at redhat.com
Fri Dec 7 20:00:21 UTC 2018
It is already pushed, but I noticed you have removed the definition of
Abstract_VM_Version::parallel_worker_threads() but the declaration is
still there. One can code against this but then get linker failure.
Maybe also remove:
static unsigned int parallel_worker_threads();
in vm_version.hpp?
Roman
> Could anyone review this cleanup change?
> In addition to straight code movement, it removes unnecessary dependency
> on adaptiveSizePolicy.hpp in G1 code, and removed unnecessary include
> statements in adaptiveSizePolicy.hpp/cpp.
>
> Webrev: https://cr.openjdk.java.net/~manc/8213224/webrev.00/
> RFE: https://bugs.openjdk.java.net/browse/JDK-8213224
> Tested on submit repo with the help from JC (CCed).
>
> Thanks,
> Man
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20181207/712f06f7/signature.asc>
More information about the hotspot-gc-dev
mailing list