{"action":"created","issue":{"url":"https://api.github.com/repos/cea-sec/miasm/issues/674","repository_url":"https://api.github.com/repos/cea-sec/miasm","labels_url":"https://api.github.com/repos/cea-sec/miasm/issues/674/labels{/name}","comments_url":"https://api.github.com/repos/cea-sec/miasm/issues/674/comments","events_url":"https://api.github.com/repos/cea-sec/miasm/issues/674/events","html_url":"https://github.com/cea-sec/miasm/pull/674","id":295882732,"number":674,"title":"Add SSE instructions","user":{"login":"commial","id":4194483,"avatar_url":"https://avatars0.githubusercontent.com/u/4194483?v=4","gravatar_id":"","url":"https://api.github.com/users/commial","html_url":"https://github.com/commial","followers_url":"https://api.github.com/users/commial/followers","following_url":"https://api.github.com/users/commial/following{/other_user}","gists_url":"https://api.github.com/users/commial/gists{/gist_id}","starred_url":"https://api.github.com/users/commial/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/commial/subscriptions","organizations_url":"https://api.github.com/users/commial/orgs","repos_url":"https://api.github.com/users/commial/repos","events_url":"https://api.github.com/users/commial/events{/privacy}","received_events_url":"https://api.github.com/users/commial/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-02-09T14:08:47Z","updated_at":"2018-03-02T08:25:03Z","closed_at":"2018-02-10T14:52:28Z","author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/cea-sec/miasm/pulls/674","html_url":"https://github.com/cea-sec/miasm/pull/674","diff_url":"https://github.com/cea-sec/miasm/pull/674.diff","patch_url":"https://github.com/cea-sec/miasm/pull/674.patch"},"body":"This PR fixes and introduces new SSE instructions for x86.\r\nThese instructions have been tested with LLVM jitter against QEMU `test-i386`, function `test_sse`. The XMM (with 128 bits) patch is necessary to reproduce these tests.\r\n\r\nWithout the patch, the first 12 lines of the tests works.\r\nWith the patch, the first 224 lines are now working (next instruction to be implemented being `SHUFPS`)."},"comment":{"url":"https://api.github.com/repos/cea-sec/miasm/issues/comments/369854105","html_url":"https://github.com/cea-sec/miasm/pull/674#issuecomment-369854105","issue_url":"https://api.github.com/repos/cea-sec/miasm/issues/674","id":369854105,"user":{"login":"commial","id":4194483,"avatar_url":"https://avatars0.githubusercontent.com/u/4194483?v=4","gravatar_id":"","url":"https://api.github.com/users/commial","html_url":"https://github.com/commial","followers_url":"https://api.github.com/users/commial/followers","following_url":"https://api.github.com/users/commial/following{/other_user}","gists_url":"https://api.github.com/users/commial/gists{/gist_id}","starred_url":"https://api.github.com/users/commial/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/commial/subscriptions","organizations_url":"https://api.github.com/users/commial/orgs","repos_url":"https://api.github.com/users/commial/repos","events_url":"https://api.github.com/users/commial/events{/privacy}","received_events_url":"https://api.github.com/users/commial/received_events","type":"User","site_admin":false},"created_at":"2018-03-02T08:25:03Z","updated_at":"2018-03-02T08:25:03Z","author_association":"OWNER","body":"In the current version of Miasm (without the XMM dirty patch), you can't reach this instruction.\r\n\r\nWith the patch, I've been able to reproduce it, but it seems to be related to the patch, which is not yet officially supported. Indeed, gcc produces a warning during the translation of this instruction:\r\n```\r\n/tmp/tmpWXmkFY.c: In function ‘block_jitblock_8055695’:\r\n/tmp/tmpWXmkFY.c:435:36: warning: integer constant is too large for its type\r\n  var_128_00 = (((((((mycpu->XMM0)^(0xffffffffffffffffffffffffffffffff))))&(mycpu->XMM5))));\r\n                                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\r\n```\r\n\r\nIf you still want to run code with this patch, I can only recommend you to use LLVM."}}
{"actor":{"display_login":"commial"}}