{"action":"created","issue":{"url":"https://api.github.com/repos/apple/swift/issues/7349","repository_url":"https://api.github.com/repos/apple/swift","labels_url":"https://api.github.com/repos/apple/swift/issues/7349/labels{/name}","comments_url":"https://api.github.com/repos/apple/swift/issues/7349/comments","events_url":"https://api.github.com/repos/apple/swift/issues/7349/events","html_url":"https://github.com/apple/swift/pull/7349","id":206373655,"number":7349,"title":"Support for Open Existentials that do no take an address","user":{"login":"shajrawi","id":17753158,"avatar_url":"https://avatars.githubusercontent.com/u/17753158?v=3","gravatar_id":"","url":"https://api.github.com/users/shajrawi","html_url":"https://github.com/shajrawi","followers_url":"https://api.github.com/users/shajrawi/followers","following_url":"https://api.github.com/users/shajrawi/following{/other_user}","gists_url":"https://api.github.com/users/shajrawi/gists{/gist_id}","starred_url":"https://api.github.com/users/shajrawi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shajrawi/subscriptions","organizations_url":"https://api.github.com/users/shajrawi/orgs","repos_url":"https://api.github.com/users/shajrawi/repos","events_url":"https://api.github.com/users/shajrawi/events{/privacy}","received_events_url":"https://api.github.com/users/shajrawi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2017-02-09T00:59:48Z","updated_at":"2017-02-09T01:31:34Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apple/swift/pulls/7349","html_url":"https://github.com/apple/swift/pull/7349","diff_url":"https://github.com/apple/swift/pull/7349.diff","patch_url":"https://github.com/apple/swift/pull/7349.patch"},"body":"Small part of radar rdar://problem/30080769\r\n\r\nBootstrapping SILGen for opaque values - This PR adds support for open_existential instruction that takes an opaque value + moves the corresponding test-case from the todo file into a completed one.\r\n\r\nWorth noting: while support has been added (almost) everywhere, including our utilities, The SIL Optimizer passes have not been changed / asked to make any special considerations for this new instruction."},"comment":{"url":"https://api.github.com/repos/apple/swift/issues/comments/278519288","html_url":"https://github.com/apple/swift/pull/7349#issuecomment-278519288","issue_url":"https://api.github.com/repos/apple/swift/issues/7349","id":278519288,"user":{"login":"atrick","id":2165663,"avatar_url":"https://avatars.githubusercontent.com/u/2165663?v=3","gravatar_id":"","url":"https://api.github.com/users/atrick","html_url":"https://github.com/atrick","followers_url":"https://api.github.com/users/atrick/followers","following_url":"https://api.github.com/users/atrick/following{/other_user}","gists_url":"https://api.github.com/users/atrick/gists{/gist_id}","starred_url":"https://api.github.com/users/atrick/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/atrick/subscriptions","organizations_url":"https://api.github.com/users/atrick/orgs","repos_url":"https://api.github.com/users/atrick/repos","events_url":"https://api.github.com/users/atrick/events{/privacy}","received_events_url":"https://api.github.com/users/atrick/received_events","type":"User","site_admin":false},"created_at":"2017-02-09T01:31:34Z","updated_at":"2017-02-09T01:31:34Z","body":"@gottesmm I'm on the fence now regarding the suffix.\r\n\r\nAfter looking at the patch, I agree this could be called `open_existential_value` because the returned value always has an opaque type--it can't be substituted with a concrete type. I thought the `_opaque` suffix was a little strange just be cause we don't use it anywhere else and won't need it for any other instructions.\r\n\r\nBut if we do want `open_existential_value` then we would also want `init_existential_value` and `deinit_existential_value` to keep them in the same family, and those could work on any type, not just opaque types."}}
{"actor":{"display_login":"atrick"}}