-
Notifications
You must be signed in to change notification settings - Fork 392
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
Z: Fix arraycmplen opcode error #7140
Z: Fix arraycmplen opcode error #7140
Conversation
isArrayCmpSign method has an assertion to make sure that the opcode is |
4b67650
to
962a934
Compare
Testing:
|
962a934
to
eed07fd
Compare
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, @Spencer-Comin would appreciate, if you also review the change given that your recent work in this area.
@ehsankianifar , Can you rebase your change ?
LGTM |
Jenkins build zos,zlinux |
Just realized that couple of different commits are added in this PR. |
The isArrayCmpSign function accept only arraycmp opcode Removed from arraycmplen since sign doesnt mean anything there Fix: eclipse#7136 Signed-off-by: ehsan kiani far ehsan.kianifar@gmail.com
7b6dac9
to
c17d4fc
Compare
fixed in #7228 |
The isArrayCmpSign function accept only arraycmp opcode.
Removed from arraycmplen since sign doesnt mean anything there
Fix: #7136
Signed-off-by: ehsan kiani far ehsan.kianifar@gmail.com