RFR (XS) 8202134: Non-PCH build for arm32 fails (original) (raw)

Stefan Karlsson stefan.karlsson at oracle.com
Mon Apr 23 09:15:50 UTC 2018


Looks good.

StefanK

On 2018-04-23 11:13, Aleksey Shipilev wrote:

On 04/23/2018 11:06 AM, Aleksey Shipilev wrote:

Current arm32 jdk/jdk fails to build without PCH.

Bug (with sample build failures): https://bugs.openjdk.java.net/browse/JDK-8202134 Fix: diff -r ac761df837c7 src/hotspot/cpu/arm/methodHandlesarm.cpp --- a/src/hotspot/cpu/arm/methodHandlesarm.cpp Thu Apr 19 12:02:38 2018 +0200 +++ b/src/hotspot/cpu/arm/methodHandlesarm.cpp Mon Apr 23 10:55:07 2018 +0200 @@ -35,6 +35,7 @@ #include "memory/resourceArea.hpp" #include "prims/methodHandles.hpp" #include "runtime/frame.inline.hpp" +#include "utilities/preserveException.hpp" #define _ masm-> diff -r ac761df837c7 src/hotspot/cpu/arm/nativeInstarm32.hpp --- a/src/hotspot/cpu/arm/nativeInstarm32.hpp Thu Apr 19 12:02:38 2018 +0200 +++ b/src/hotspot/cpu/arm/nativeInstarm32.hpp Mon Apr 23 10:55:07 2018 +0200 @@ -28,6 +28,7 @@ #include "asm/macroAssembler.hpp" #include "code/codeCache.hpp" #include "runtime/icache.hpp" +#include "runtime/orderAccess.inline.hpp" #include "runtime/os.hpp" #include "runtime/thread.hpp" #include "registerarm.hpp" Actually, .inline.hpp is better be in the .cpp that failed to build, not in the header itself: diff -r fcd5df7aa235 src/hotspot/cpu/arm/methodHandlesarm.cpp --- a/src/hotspot/cpu/arm/methodHandlesarm.cpp Wed Apr 18 11:19:32 2018 +0200 +++ b/src/hotspot/cpu/arm/methodHandlesarm.cpp Mon Apr 23 11:13:32 2018 +0200 @@ -35,6 +35,7 @@ #include "memory/resourceArea.hpp" #include "prims/methodHandles.hpp" #include "runtime/frame.inline.hpp" +#include "utilities/preserveException.hpp" #define _ masm-> diff -r fcd5df7aa235 src/hotspot/cpu/arm/relocInfoarm.cpp --- a/src/hotspot/cpu/arm/relocInfoarm.cpp Wed Apr 18 11:19:32 2018 +0200 +++ b/src/hotspot/cpu/arm/relocInfoarm.cpp Mon Apr 23 11:13:32 2018 +0200 @@ -29,6 +29,7 @@ #include "nativeInstarm.hpp" #include "oops/compressedOops.inline.hpp" #include "oops/oop.hpp" +#include "runtime/orderAccess.inline.hpp" #include "runtime/safepoint.hpp" void Relocation::pdsetdatavalue(address x, intptrt o, bool verifyonly) { Thanks, -Aleksey



More information about the hotspot-dev mailing list