On Wed, 16 Sep 2020 11:12:04 GMT, Kim Barrett <kbarrett@openjdk.org> wrote:
Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
Reinstate allocation.hpp include
Marked as reviewed by kbarrett (Reviewer).
_Mailing list message from [Thomas Schatzl](mailto:thomas.schatzl@oracle.com) on [hotspot-gc-dev](mailto:hotspot-gc-dev@openjdk.java.net):_ I tend to always prefer the .inline.hpp if available as something might be defined in the .inline.hpp that is used in the .hpp file. I.e. as a rule include to .inline.hpp if available. It does not seem to be the case here so just the .hpp should be good.
Right. I do as crystal^W static analyzer guides: include only what is necessary. Integrating now. ------------- PR: https://git.openjdk.java.net/jdk/pull/198