RFR: 8257238: Cleanup include directives for precompiled.hpp
Kim Barrett
kbarrett at openjdk.java.net
Mon Nov 30 03:35: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.
Looks good, and trivial.
-------------
Marked as reviewed by kbarrett (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/1510
More information about the hotspot-runtime-dev
mailing list