{"action":"created","issue":{"url":"https://api.github.com/repos/ldc-developers/ldc/issues/1856","repository_url":"https://api.github.com/repos/ldc-developers/ldc","labels_url":"https://api.github.com/repos/ldc-developers/ldc/issues/1856/labels{/name}","comments_url":"https://api.github.com/repos/ldc-developers/ldc/issues/1856/comments","events_url":"https://api.github.com/repos/ldc-developers/ldc/issues/1856/events","html_url":"https://github.com/ldc-developers/ldc/pull/1856","id":185394833,"number":1856,"title":"WIP: Map `export` visibility to LLVM DLL storage classes","user":{"login":"kinke","id":1889215,"avatar_url":"https://avatars.githubusercontent.com/u/1889215?v=3","gravatar_id":"","url":"https://api.github.com/users/kinke","html_url":"https://github.com/kinke","followers_url":"https://api.github.com/users/kinke/followers","following_url":"https://api.github.com/users/kinke/following{/other_user}","gists_url":"https://api.github.com/users/kinke/gists{/gist_id}","starred_url":"https://api.github.com/users/kinke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kinke/subscriptions","organizations_url":"https://api.github.com/users/kinke/orgs","repos_url":"https://api.github.com/users/kinke/repos","events_url":"https://api.github.com/users/kinke/events{/privacy}","received_events_url":"https://api.github.com/users/kinke/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2016-10-26T13:33:51Z","updated_at":"2016-10-26T14:37:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ldc-developers/ldc/pulls/1856","html_url":"https://github.com/ldc-developers/ldc/pull/1856","diff_url":"https://github.com/ldc-developers/ldc/pull/1856.diff","patch_url":"https://github.com/ldc-developers/ldc/pull/1856.patch"},"body":"Trying to fix #1745."},"comment":{"url":"https://api.github.com/repos/ldc-developers/ldc/issues/comments/256367196","html_url":"https://github.com/ldc-developers/ldc/pull/1856#issuecomment-256367196","issue_url":"https://api.github.com/repos/ldc-developers/ldc/issues/1856","id":256367196,"user":{"login":"kinke","id":1889215,"avatar_url":"https://avatars.githubusercontent.com/u/1889215?v=3","gravatar_id":"","url":"https://api.github.com/users/kinke","html_url":"https://github.com/kinke","followers_url":"https://api.github.com/users/kinke/followers","following_url":"https://api.github.com/users/kinke/following{/other_user}","gists_url":"https://api.github.com/users/kinke/gists{/gist_id}","starred_url":"https://api.github.com/users/kinke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kinke/subscriptions","organizations_url":"https://api.github.com/users/kinke/orgs","repos_url":"https://api.github.com/users/kinke/repos","events_url":"https://api.github.com/users/kinke/events{/privacy}","received_events_url":"https://api.github.com/users/kinke/received_events","type":"User","site_admin":false},"created_at":"2016-10-26T14:37:22Z","updated_at":"2016-10-26T14:37:22Z","body":"With the current logic, all modules of a binary should be compiled with a single ldc2/ldmd2 call. Otherwise, other objects (in the same binary) import the symbol via `dllimport`, i.e., access it via an unnecessary additional indirection (at least on Windows). I've just tested lnking independently compiled objects, and the MS linker emits a warning in this case:\r\n\r\n```\r\ncurrent.obj : warning LNK4217: locally defined symbol _D8current24foo2FZv imported in function _Dmain\r\n```"}}
{"actor":{"display_login":"kinke"}}