RFR: 8325517: Shenandoah: Reduce unnecessary includes from shenandoahControlThread.cpp

Y. Srinivas Ramakrishna ysr at openjdk.org
Thu Feb 8 21:14:53 UTC 2024


On Thu, 8 Feb 2024 18:32:46 GMT, William Kemper <wkemper at openjdk.org> wrote:

> The control thread used to run much more of the cycle directly. This code was all factored out into different classes, but many of the vestigial headers remained. Removing these improves compilation times and makes maintenance easier.

Looks good modulo Aleksey's suggestion on checking use of header file's content in compilation unit rather than relying on serendipitous (and fragile) transitive dependencies.

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

Marked as reviewed by ysr (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/17778#pullrequestreview-1871265523


More information about the shenandoah-dev mailing list