Skip to content

Backport JDK-8280476 fix - #191

Open
Roger Hu (rogerhu) wants to merge 1 commit into
JetBrains:jbr17.469from
rogerhu:rogerh/jdk-8280476
Open

Roger Hu (rogerhu) wants to merge 1 commit into
JetBrains:jbr17.469from
rogerhu:rogerh/jdk-8280476

Conversation

@rogerhu

@rogerhu Roger Hu (rogerhu) commented Dec 5, 2022 •

Copy link
Copy Markdown

This bug (https://bugs.openjdk.org/browse/JDK-8280476) is triggered when building against Xcode 13.3. I see the exact same error message and realized that this commit needed to be added.

During or after building the image, I see:

/Users/rogerh/Development/JetBrainsRuntime/build/macosx-aarch64-server-release/jdk/bin/java --version
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  Internal Error (assembler_aarch64.hpp:248), pid=63683, tid=9475
#  guarantee(val < (1ULL << nbits)) failed: Field too big for insn
#
# JRE version:  (17.0.5) (build )
# Java VM: OpenJDK 64-Bit Server VM (17.0.5-internal+0-adhoc.rogerh.JetBrainsRuntime, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, g1 gc, bsd-aarch64)
# No core dump will be written. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /Users/rogerh/Development/JetBrainsRuntime/build/macosx-aarch64-server-release/make-support/failure-logs/hs_err_pid63683.log
#

There are also sprintf error warnings that I was able to suppress to get things to compile with Xcode 13:

+++ b/make/hotspot/gensrc/GensrcAdlc.gmk
@@ -58,6 +58,8 @@ ifeq ($(call check-jvm-feature, compiler2), true)
 
   ADLC_CFLAGS += -I$(TOPDIR)/src/hotspot/share
 
+  ADLC_CFLAGS += -Wno-deprecated-declarations
+
   # Add file macro mappings
   ADLC_CFLAGS += $(FILE_MACRO_CFLAGS)

Along with passing in --with-extra-cxxflags="-Wno-deprecated-declarations" when running the configure script.

@rogerhu Roger Hu (rogerhu) changed the title Allow JDK Backport JDK-8280476 fix Dec 5, 2022
@rogerhu
Roger Hu (rogerhu) changed the base branch from jbr17 to jbr17.469 December 5, 2022 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant