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

Fix suggestion on https://github.com/StanfordPL/stoke/issues/983 #984

Merged
merged 2 commits into from
Apr 4, 2018

Conversation

sdasgup3
Copy link
Contributor

@sdasgup3 sdasgup3 commented Apr 3, 2018

Hello Team,
Here is a suggustion on possible fix of bugs reported #983.
I have added few simple test cases but the following is the testing involved in the backroud.

Testing Strategy

  • All the implementations are tested with Strata's final emiited testsuite.
  • The immediate versions of the all the instructions are tested using the above testsuite on each of the 256 immediate values.

@sdasgup3 sdasgup3 mentioned this pull request Apr 3, 2018
@bchurchill
Copy link
Member

Thanks for reporting these issues and offering a fix! We'll try merging these for now, and we'll open a new ticket if our CI system complains.

@bchurchill bchurchill merged commit 57550e7 into StanfordPL:develop Apr 4, 2018
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.

2 participants