Skip to content
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

Verify type of MethodHandle.catchException()'s exType argument #4171

Merged
merged 1 commit into from
Jan 7, 2019

Conversation

pdbain-ibm
Copy link
Contributor

Verify that a Throwable class is provided.

Fixes #3271

Signed-off-by: Peter Bain peter_bain@ca.ibm.com

Copy link
Member

@DanHeidinga DanHeidinga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@DanHeidinga
Copy link
Member

Jenkins test all xlinux jdk8,jdk11

Verify that a Throwable class is provided.

Fixes eclipse-openj9#3271

Signed-off-by: Peter Bain <peter_bain@ca.ibm.com>
@pdbain-ibm
Copy link
Contributor Author

@DanHeidinga
Thank you for the review.
The sanity check failed compiling DDR:

23:37:52 Unhandled structure type: _IO_FILEPointer->_lock _IO_lock_t*
23:37:52 Unhandled structure type: big_int_unionPointer->i128 __int128
23:37:52 Exception in thread "main" java.lang.RuntimeException: Unexpected SRP reference type: -1 from J9SRP(J9ROMClassTOCEntry)
23:37:52 	at com.ibm.j9ddr.tools.PointerGenerator.writeSRPMethod(PointerGenerator.java:1215)

which should be completely unrelated to this change. I have rebased the change and updated the pull request in case my branch was stale.

Would you kindly resubmit the sanity build?

@DanHeidinga
Copy link
Member

Jenkins test all xlinux jdk8,jdk11

@pdbain
Copy link

pdbain commented Jan 7, 2019

Still seeing the same failure in the test builds: compilation failure in DDR:

superset directory name : /home/jenkins/workspace/PullRequest-Extended-JDK8-linux_x86-64_cmprssptrs-OpenJ9/build/linux-x86_64-normal-server-release/vm/
superset file name : superset.dat
superset directory name : /home/jenkins/workspace/PullRequest-Extended-JDK8-linux_x86-64_cmprssptrs-OpenJ9/build/linux-x86_64-normal-server-release/vm/
superset file name : superset.dat
Writing generated classes to /home/jenkins/workspace/PullRequest-Extended-JDK8-linux_x86-64_cmprssptrs-OpenJ9/openj9/debugtools/DDR_VM/src/com/ibm/j9ddr/vm29/pointer/generated
Writing generated classes to /home/jenkins/workspace/PullRequest-Extended-JDK8-linux_x86-64_cmprssptrs-OpenJ9/openj9/debugtools/DDR_VM/src/com/ibm/j9ddr/vm29/structure
Unhandled structure type: _IO_FILEPointer->_lock _IO_lock_t*
Unhandled structure type: big_int_unionPointer->i128 __int128
Processing complete
Exception in thread "main" java.lang.RuntimeException: Unexpected SRP reference type: -1 from J9SRP(J9ROMClassTOCEntry)
	at com.ibm.j9ddr.tools.PointerGenerator.writeSRPMethod(PointerGenerator.java:1215)
	at com.ibm.j9ddr.tools.PointerGenerator.generateImplementationMethods(PointerGenerator.java:567)
	at com.ibm.j9ddr.tools.PointerGenerator.generateClass(PointerGenerator.java:329)
	at com.ibm.j9ddr.tools.PointerGenerator.generateClasses(PointerGenerator.java:157)
	at com.ibm.j9ddr.tools.PointerGenerator.main(PointerGenerator.java:117)

@keithc-ca could you help here? This changes only Java, which should not affect DDR, correct?
Thank you
-p

@keithc-ca
Copy link
Contributor

This may be due to the compiler in use (to my knowledge, we've never fully tested gcc 5.x):

00:05:16.775 * Toolchain:      gcc (GNU Compiler Collection)
00:05:16.775 * C Compiler:     Version 5.5.0 (at /usr/bin/gcc)
00:05:16.775 * C++ Compiler:   Version 5.5.0 (at /usr/bin/g++)

@DanHeidinga
Copy link
Member

Jenkins test all xlinux jdk8

@pdbain-ibm
Copy link
Contributor Author

The Java 8 sanity tests are proceeding much better. Would you like an issue opened for the DDR issue?

@keithc-ca
Copy link
Contributor

Unless we intend to support gcc 5.x, there's no need for a DDR issue.

@DanHeidinga
Copy link
Member

Unless we intend to support gcc 5.x

Agreed. We're not planning to support gcc 5.x (moving to gcc 7.3!) so if this just a compiler issue, then no new issue is needed.

@DanHeidinga DanHeidinga merged commit 93c622c into eclipse-openj9:master Jan 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants