{"action":"created","issue":{"url":"https://api.github.com/repos/WebAssembly/design/issues/350","labels_url":"https://api.github.com/repos/WebAssembly/design/issues/350/labels{/name}","comments_url":"https://api.github.com/repos/WebAssembly/design/issues/350/comments","events_url":"https://api.github.com/repos/WebAssembly/design/issues/350/events","html_url":"https://github.com/WebAssembly/design/issues/350","id":106443338,"number":350,"title":"Left/right shift shift amount","user":{"login":"kg","id":198130,"avatar_url":"https://avatars.githubusercontent.com/u/198130?v=3","gravatar_id":"","url":"https://api.github.com/users/kg","html_url":"https://github.com/kg","followers_url":"https://api.github.com/users/kg/followers","following_url":"https://api.github.com/users/kg/following{/other_user}","gists_url":"https://api.github.com/users/kg/gists{/gist_id}","starred_url":"https://api.github.com/users/kg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kg/subscriptions","organizations_url":"https://api.github.com/users/kg/orgs","repos_url":"https://api.github.com/users/kg/repos","events_url":"https://api.github.com/users/kg/events{/privacy}","received_events_url":"https://api.github.com/users/kg/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2015-09-14T22:48:01Z","updated_at":"2015-09-22T13:52:47Z","closed_at":null,"body":"Right now the int64 left/right shift operations take int64 for both operands in the ml prototype, and that seems to be what's implied in the spec.\r\n\r\nIs that actually what we want? I think the rhs should be int32, at most. On x86 I believe the shift amount is stored in 8 bits of the count register, so we're definitely not getting anything out of the right-hand operand being 64-bit.\r\n\r\nIt would make sense (I think?) for the right hand operand of shift operations to always be int32. Ideally it would be an int8, but we don't have that type :-)"},"comment":{"url":"https://api.github.com/repos/WebAssembly/design/issues/comments/142295582","html_url":"https://github.com/WebAssembly/design/issues/350#issuecomment-142295582","issue_url":"https://api.github.com/repos/WebAssembly/design/issues/350","id":142295582,"user":{"login":"sunfishcode","id":4503403,"avatar_url":"https://avatars.githubusercontent.com/u/4503403?v=3","gravatar_id":"","url":"https://api.github.com/users/sunfishcode","html_url":"https://github.com/sunfishcode","followers_url":"https://api.github.com/users/sunfishcode/followers","following_url":"https://api.github.com/users/sunfishcode/following{/other_user}","gists_url":"https://api.github.com/users/sunfishcode/gists{/gist_id}","starred_url":"https://api.github.com/users/sunfishcode/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sunfishcode/subscriptions","organizations_url":"https://api.github.com/users/sunfishcode/orgs","repos_url":"https://api.github.com/users/sunfishcode/repos","events_url":"https://api.github.com/users/sunfishcode/events{/privacy}","received_events_url":"https://api.github.com/users/sunfishcode/received_events","type":"User","site_admin":false},"created_at":"2015-09-22T13:52:47Z","updated_at":"2015-09-22T13:52:47Z","body":"No, `SHRL` and `SHRX` are the same as x86's other shifts; the only difference is that they don't set FLAGS."}}