Skip to content

Backport JDK-8280476 fix #191

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
rogerhu wants to merge 1 commit into
base: jbr17.469
Choose a base branch
from
Open

Backport JDK-8280476 fix #191

rogerhu wants to merge 1 commit into from

Conversation

@rogerhu
Copy link

@rogerhu rogerhu commented Dec 5, 2022 •
edited
Loading

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 rogerhu changed the title Allow JDK Backport JDK-8280476 fix Dec 5, 2022
@rogerhu 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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@rogerhu