{"action":"created","issue":{"url":"https://api.github.com/repos/cornell-brg/pydgin/issues/22","labels_url":"https://api.github.com/repos/cornell-brg/pydgin/issues/22/labels{/name}","comments_url":"https://api.github.com/repos/cornell-brg/pydgin/issues/22/comments","events_url":"https://api.github.com/repos/cornell-brg/pydgin/issues/22/events","html_url":"https://github.com/cornell-brg/pydgin/issues/22","id":83149773,"number":22,"title":"Possible bug with EORS","user":{"login":"salmanarif","id":8473947,"avatar_url":"https://avatars.githubusercontent.com/u/8473947?v=3","gravatar_id":"","url":"https://api.github.com/users/salmanarif","html_url":"https://github.com/salmanarif","followers_url":"https://api.github.com/users/salmanarif/followers","following_url":"https://api.github.com/users/salmanarif/following{/other_user}","gists_url":"https://api.github.com/users/salmanarif/gists{/gist_id}","starred_url":"https://api.github.com/users/salmanarif/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/salmanarif/subscriptions","organizations_url":"https://api.github.com/users/salmanarif/orgs","repos_url":"https://api.github.com/users/salmanarif/repos","events_url":"https://api.github.com/users/salmanarif/events{/privacy}","received_events_url":"https://api.github.com/users/salmanarif/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-05-31T18:07:53Z","updated_at":"2015-06-04T17:19:47Z","closed_at":"2015-06-03T04:30:13Z","body":"I have been running some tests using `arm-sim.py`, and it appears that an instruction of the form:\r\n`EORS Rd, Rm, #n` always sets the carry bit. According to [documentation on the EOR instruction](http://www.keil.com/support/man/docs/armasm/armasm_dom1361289871065.htm), the carry bit should only be set if `n` can be created by shifting an 8-bit value."},"comment":{"url":"https://api.github.com/repos/cornell-brg/pydgin/issues/comments/108980706","html_url":"https://github.com/cornell-brg/pydgin/issues/22#issuecomment-108980706","issue_url":"https://api.github.com/repos/cornell-brg/pydgin/issues/22","id":108980706,"user":{"login":"salmanarif","id":8473947,"avatar_url":"https://avatars.githubusercontent.com/u/8473947?v=3","gravatar_id":"","url":"https://api.github.com/users/salmanarif","html_url":"https://github.com/salmanarif","followers_url":"https://api.github.com/users/salmanarif/followers","following_url":"https://api.github.com/users/salmanarif/following{/other_user}","gists_url":"https://api.github.com/users/salmanarif/gists{/gist_id}","starred_url":"https://api.github.com/users/salmanarif/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/salmanarif/subscriptions","organizations_url":"https://api.github.com/users/salmanarif/orgs","repos_url":"https://api.github.com/users/salmanarif/repos","events_url":"https://api.github.com/users/salmanarif/events{/privacy}","received_events_url":"https://api.github.com/users/salmanarif/received_events","type":"User","site_admin":false},"created_at":"2015-06-04T17:19:47Z","updated_at":"2015-06-04T17:19:47Z","body":"It does not seem to have broken anything else, however, `TEQ` and `TST` instructions are setting the carry bit even if the second operand does not have a rotate or `bit[31]` as `1`. Should these not have the same behaviour as other instructions, now that you have fixed the issue with 32-bit immediates?"}}