Integrated: 8268118: Rename bytes_os_cpu.inline.hpp files to bytes_os_cpu.hpp

Stefan Karlsson stefank at openjdk.java.net
Thu Jun 3 09:07:43 UTC 2021


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

> Today we transitively include the `bytes_<os>_<cpu>.inline.hpp` files from `bytes.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 `bytes_<os>_<cpu>.inline.hpp` don't include any other HotSpot files, so I propose that we simply rename them to `bytes_<os>_<cpu>.hpp`.

This pull request has now been integrated.

Changeset: c8f4c02b
Author:    Stefan Karlsson <stefank at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/c8f4c02bf005ee1531193535632a5ece768916d0
Stats:     540 lines in 24 files changed: 258 ins; 258 del; 24 mod

8268118: Rename bytes_os_cpu.inline.hpp files to bytes_os_cpu.hpp

Reviewed-by: coleenp, dholmes

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

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


More information about the hotspot-dev mailing list