Integrated: 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:42 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`.
This pull request has now been integrated.
Changeset: 1296a6c4
Author: Stefan Karlsson <stefank at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/1296a6c425e22e0fdc748a996b886923c602ab3f
Stats: 36 lines in 10 files changed: 0 ins; 12 del; 24 mod
8268119: Rename copy_os_cpu.inline.hpp files to copy_os_cpu.hpp
Reviewed-by: kbarrett, coleenp
-------------
PR: https://git.openjdk.java.net/jdk/pull/4311
More information about the hotspot-dev
mailing list