Integrated: 8257238: Cleanup include directives for precompiled.hpp

David Holmes dholmes at openjdk.java.net
Mon Nov 30 03:40:55 UTC 2020


On Mon, 30 Nov 2020 01:12:32 GMT, David Holmes <dholmes at openjdk.org> wrote:

> The precompiled header file is:
> 
> src/hotspot/share/precompiled/precompiled.hpp
> 
> The normal way to include precompiled headers is to just have:
> 
> #include "precompiled.hpp"
> 
> as the first directive in a file. This works because the makefiles set:
> 
>  -I$(TOPDIR)/src/hotspot/share/precompiled
> 
>  Some files instead have:
> 
> #include "precompiled/precompiled.hpp"
> 
> ./os/bsd/semaphore_bsd.cpp
> ./os/linux/waitBarrier_linux.cpp
> ./os/posix/semaphore_posix.cpp
> ./os/posix/signals_posix.cpp
> 
> This should be trivially cleaned up.

This pull request has now been integrated.

Changeset: 222e9430
Author:    David Holmes <dholmes at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/222e9430
Stats:     6 lines in 4 files changed: 0 ins; 0 del; 6 mod

8257238: Cleanup include directives for precompiled.hpp

Reviewed-by: kbarrett

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

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


More information about the hotspot-runtime-dev mailing list