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

Add test for sign extension in jump operations #243

Merged
merged 4 commits into from
May 7, 2024

Conversation

Alan-Jowett
Copy link
Owner

Add test to check if immediate value is correctly not sign extended when doing a comparison.

Alan Jowett and others added 2 commits May 7, 2024 09:17
Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Signed-off-by: Alan Jowett <alanjo@microsoft.com>
@coveralls
Copy link

coveralls commented May 7, 2024

Coverage Status

coverage: 95.147%. remained the same
when pulling 354a0cd on non_sign_extend_jump
into de61836 on main.

Signed-off-by: Alan Jowett <alanjo@microsoft.com>
@Alan-Jowett Alan-Jowett changed the title Add test for non-sign extending JLE Add test for sign extension in jump operations May 7, 2024
@Alan-Jowett Alan-Jowett marked this pull request as ready for review May 7, 2024 17:07
@Alan-Jowett Alan-Jowett requested a review from hawkinsw May 7, 2024 17:08
Signed-off-by: Alan Jowett <alanjo@microsoft.com>
@Alan-Jowett Alan-Jowett merged commit 35201b4 into main May 7, 2024
19 checks passed
@Alan-Jowett Alan-Jowett deleted the non_sign_extend_jump branch May 7, 2024 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants