Integrated: JDK-8260404: jvm_io.h include missing in a number of files
Thomas Stuefe
stuefe at openjdk.java.net
Wed Jan 27 07:37:41 UTC 2021
On Tue, 26 Jan 2021 10:22:09 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
> One of the last include corrections caused a build error on our AIX machines because compile.cpp uses jio_snprintf() without including jvm_io.h.
>
> I did not deeply follow up on the why, but "8260307: Do not include method.hpp in frame.hpp" is a possible candidate. AIX builds without JFR and hence miss a number of include chains other platforms have.
>
> I scanned hotspot sources for files using jio_snprintf but not including jvm_io.h. There are some hits, which should be fixed.
>
> All this is trivial, right?
This pull request has now been integrated.
Changeset: 1c770468
Author: Thomas Stuefe <stuefe at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/1c770468
Stats: 26 lines in 16 files changed: 16 ins; 0 del; 10 mod
8260404: jvm_io.h include missing in a number of files
Reviewed-by: shade, iklam, dholmes
-------------
PR: https://git.openjdk.java.net/jdk/pull/2232
More information about the hotspot-dev
mailing list