-
Notifications
You must be signed in to change notification settings - Fork 738
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
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>
@DanHeidinga
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? |
Jenkins test all xlinux jdk8,jdk11 |
Still seeing the same failure in the test builds: compilation failure in DDR:
@keithc-ca could you help here? This changes only Java, which should not affect DDR, correct? |
This may be due to the compiler in use (to my knowledge, we've never fully tested gcc 5.x):
|
Jenkins test all xlinux jdk8 |
The Java 8 sanity tests are proceeding much better. Would you like an issue opened for the DDR issue? |
Unless we intend to support gcc 5.x, there's no need for a DDR issue. |
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. |
Verify that a Throwable class is provided.
Fixes #3271
Signed-off-by: Peter Bain peter_bain@ca.ibm.com