RFR: 8268119: Rename copy_os_cpu.inline.hpp files to copy_os_cpu.hpp

Stefan Karlsson stefank at openjdk.java.net
Thu Jun 3 09:06:41 UTC 2021


On Wed, 2 Jun 2021 14:28:57 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

> Today we transitively include the `copy_<os>_<cpu>.inline.hpp` files from `copy.hpp`. This is goes against the HotSpot Style Guide that states:
> 
>> .inline.hpp files should only be included in .cpp or .inline.hpp files.
> 
> The `copy_<os>_<cpu>.inline.hpp` don't include any other HotSpot files, so I propose that we simply rename them to `copy_<os>_<cpu>.hpp`.

I pulled in #4303 locally and cross-compiled to verify that this didn't break the build.

Thanks @kimbarrett and @coleenp for reviewing.

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

PR: https://git.openjdk.java.net/jdk/pull/4311


More information about the hotspot-dev mailing list