-
Notifications
You must be signed in to change notification settings - Fork 729
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.
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
jdk11 plinux compiled with gcc 7.3 doesn't work with NativeImageBuffer #2788
Comments
Notes from an old investigation of a similar problem that ended up being a compiler mis-match.
|
Investigating. Downloaded and built Java 11 and looked at the compilation log per the previous investigation: |
Further to above comment:
env->NewObject() in C++ calls newObjectV(): |
Bug in gcc version 7.3.0. I instrumented
and
Running with the
Running with the
Relevant files attached. |
Opened a Gnu bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87289 |
The compiler tripped over a
Replaced the This code is dubious too, though I don't plan to fix it in this changeset:
|
Comment on bug:
|
Pretty sure we build with that option on other platforms. Can you confirm that and then update the makefiles to include the option? We'll need to do some perf testing on JDK8 (as it has the best perf test coverage) to validate that the adding the option doesn't negatively effect startup, footprint, throughput. |
LinuxX86 does have
I have a pull request |
It might fix this particular issue, which doesn't mean we won't find more issues afterwards. How much testing have you done on a ppc le JVM, compiled with gcc 7.3, with the fix applied? |
I ran a build with sanity and extended testing enabled on all platforms, but the PPCLE builds failed to run due to machine issues. GAC started an X86 build, but we should run a plinux test build. |
ppc le is the platform which isn't working with gcc 7.3. There may be more aliasing problems lurking before its working. "Running a build" needs to be done with gcc 7.3 enabled in order to test this. |
As per #2718, jdk11 plinux compiled with gcc 7.3 cannot load classes from the jimage file, unless -Djdk.image.use.jvm.map=false is specified.
The text was updated successfully, but these errors were encountered: