RFR: 8288935: Remove excessive includes introduced in loom

Stefan Karlsson stefank at openjdk.org
Wed Jun 22 09:17:52 UTC 2022


On Wed, 22 Jun 2022 05:55:51 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> Some header files introduced in loom has unnecessary #includes. 
> 
> Also, some hpp files have complex inline functions that should be moved to either inline.hpp or cpp files.

Looks good. I saw one obvious missing include.

src/hotspot/share/runtime/continuationEntry.inline.hpp line 31:

> 29: 
> 30: #include "oops/access.hpp"
> 31: #include "runtime/frame.hpp"

Needs utilities/align.hpp

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

Marked as reviewed by stefank (Reviewer).

PR: https://git.openjdk.org/jdk/pull/9231


More information about the hotspot-dev mailing list