RFR: 8293188: x86_64: Introduce stubGenerator_x86_64.hpp
David Holmes
dholmes at openjdk.org
Thu Sep 1 05:03:10 UTC 2022
On Wed, 31 Aug 2022 23:07:24 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
> Introduce stubGenerator_x86_64.hpp and extract StubGenerator class declaration there.
>
> Testing: hs-tier1 - hs-tier4
Rather tricky to review but as long as things build okay this should be fine. I'm assuming the motivation here was just to get smaller files to work with?
A couple of minor nits.
Thanks.
src/hotspot/cpu/x86/stubGenerator_x86_64.cpp line 51:
> 49: #include "jfr/support/jfrIntrinsics.hpp"
> 50: #endif
> 51: #include "stubGenerator_x86_64.hpp"
This should be in alphabetical order prior to the conditional includes.
src/hotspot/cpu/x86/stubGenerator_x86_64.hpp line 2:
> 1: /*
> 2: * Copyright (c) 2022, Oracle and/or its affiliates. All rights reserved.
As the content of this file has been moved from another, the original file's copyright years should be preserved. Thanks.
src/hotspot/cpu/x86/stubGenerator_x86_64.hpp line 24:
> 22: *
> 23: */
> 24:
Do we need the usual header file include guard?
-------------
Marked as reviewed by dholmes (Reviewer).
PR: https://git.openjdk.org/jdk/pull/10111
More information about the hotspot-dev
mailing list