RFR: 8199264: Remove universe.inline.hpp to simplify include dependencies
harold.seigel at oracle.com
Wed Mar 7 22:04:13 UTC 2018
This looks good. Thanks for doing it.
On 3/7/2018 4:59 PM, Stefan Karlsson wrote:
> Hi all,
> Please review this patch to remove universe.inline.hpp.
> The main motivation for this patch is to lower the number of
> .inline.hpp files included from .hpp files. My initial though was to
> just move the two trivial functions from the universe.inline.hpp into
> universe.hpp, and get rid of universe.inline.hpp. However,
> field_type_should_be_aligned evaluates to true at the single location
> it is used, and element_type_should_be_aligned is only used in
> arrayOop.hpp, so it was moved there instead.
> There are a number of places that used to include universe.inline.hpp
> that now include universe.hpp. I left them all by purpose, to minimize
> the potential for build breakages. Even removing the universe.hpp
> include in arrayOop.hpp breaks the build, so I'd like to defer that
> cleaning to another patch.
More information about the hotspot-dev