{"action":"created","issue":{"url":"https://api.github.com/repos/rust-lang/rust/issues/37429","repository_url":"https://api.github.com/repos/rust-lang/rust","labels_url":"https://api.github.com/repos/rust-lang/rust/issues/37429/labels{/name}","comments_url":"https://api.github.com/repos/rust-lang/rust/issues/37429/comments","events_url":"https://api.github.com/repos/rust-lang/rust/issues/37429/events","html_url":"https://github.com/rust-lang/rust/pull/37429","id":185569516,"number":37429,"title":"struct field reordering and optimization","user":{"login":"camlorn","id":6968705,"avatar_url":"https://avatars.githubusercontent.com/u/6968705?v=3","gravatar_id":"","url":"https://api.github.com/users/camlorn","html_url":"https://github.com/camlorn","followers_url":"https://api.github.com/users/camlorn/followers","following_url":"https://api.github.com/users/camlorn/following{/other_user}","gists_url":"https://api.github.com/users/camlorn/gists{/gist_id}","starred_url":"https://api.github.com/users/camlorn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/camlorn/subscriptions","organizations_url":"https://api.github.com/users/camlorn/orgs","repos_url":"https://api.github.com/users/camlorn/repos","events_url":"https://api.github.com/users/camlorn/events{/privacy}","received_events_url":"https://api.github.com/users/camlorn/received_events","type":"User","site_admin":false},"labels":[{"id":223796632,"url":"https://api.github.com/repos/rust-lang/rust/labels/relnotes","name":"relnotes","color":"fad8c7","default":false},{"id":211668100,"url":"https://api.github.com/repos/rust-lang/rust/labels/T-compiler","name":"T-compiler","color":"bfd4f2","default":false},{"id":211668019,"url":"https://api.github.com/repos/rust-lang/rust/labels/T-lang","name":"T-lang","color":"bfd4f2","default":false}],"state":"closed","locked":false,"assignee":{"login":"eddyb","id":77424,"avatar_url":"https://avatars.githubusercontent.com/u/77424?v=3","gravatar_id":"","url":"https://api.github.com/users/eddyb","html_url":"https://github.com/eddyb","followers_url":"https://api.github.com/users/eddyb/followers","following_url":"https://api.github.com/users/eddyb/following{/other_user}","gists_url":"https://api.github.com/users/eddyb/gists{/gist_id}","starred_url":"https://api.github.com/users/eddyb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eddyb/subscriptions","organizations_url":"https://api.github.com/users/eddyb/orgs","repos_url":"https://api.github.com/users/eddyb/repos","events_url":"https://api.github.com/users/eddyb/events{/privacy}","received_events_url":"https://api.github.com/users/eddyb/received_events","type":"User","site_admin":false},"assignees":[{"login":"eddyb","id":77424,"avatar_url":"https://avatars.githubusercontent.com/u/77424?v=3","gravatar_id":"","url":"https://api.github.com/users/eddyb","html_url":"https://github.com/eddyb","followers_url":"https://api.github.com/users/eddyb/followers","following_url":"https://api.github.com/users/eddyb/following{/other_user}","gists_url":"https://api.github.com/users/eddyb/gists{/gist_id}","starred_url":"https://api.github.com/users/eddyb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eddyb/subscriptions","organizations_url":"https://api.github.com/users/eddyb/orgs","repos_url":"https://api.github.com/users/eddyb/repos","events_url":"https://api.github.com/users/eddyb/events{/privacy}","received_events_url":"https://api.github.com/users/eddyb/received_events","type":"User","site_admin":false}],"milestone":null,"comments":64,"created_at":"2016-10-27T04:14:02Z","updated_at":"2016-12-19T15:31:39Z","closed_at":"2016-12-18T11:16:50Z","pull_request":{"url":"https://api.github.com/repos/rust-lang/rust/pulls/37429","html_url":"https://github.com/rust-lang/rust/pull/37429","diff_url":"https://github.com/rust-lang/rust/pull/37429.diff","patch_url":"https://github.com/rust-lang/rust/pull/37429.patch"},"body":"This is work in progress.  The goal is to divorce the order of fields in source code from the order of fields in the LLVM IR, then optimize structs (and tuples/enum variants)by always ordering fields from least to most aligned.  It does not work yet.  I intend to check compiler memory usage as a benchmark, and a crater run will probably be required.\n\nI don't know enough of the compiler to complete this work unaided.  If you see places that still need updating, please mention them.  The only one I know of currently is debuginfo, which I'm putting off intentionally until a bit later.\n\nr? @eddyb\n"},"comment":{"url":"https://api.github.com/repos/rust-lang/rust/issues/comments/267993639","html_url":"https://github.com/rust-lang/rust/pull/37429#issuecomment-267993639","issue_url":"https://api.github.com/repos/rust-lang/rust/issues/37429","id":267993639,"user":{"login":"nikomatsakis","id":155238,"avatar_url":"https://avatars.githubusercontent.com/u/155238?v=3","gravatar_id":"","url":"https://api.github.com/users/nikomatsakis","html_url":"https://github.com/nikomatsakis","followers_url":"https://api.github.com/users/nikomatsakis/followers","following_url":"https://api.github.com/users/nikomatsakis/following{/other_user}","gists_url":"https://api.github.com/users/nikomatsakis/gists{/gist_id}","starred_url":"https://api.github.com/users/nikomatsakis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nikomatsakis/subscriptions","organizations_url":"https://api.github.com/users/nikomatsakis/orgs","repos_url":"https://api.github.com/users/nikomatsakis/repos","events_url":"https://api.github.com/users/nikomatsakis/events{/privacy}","received_events_url":"https://api.github.com/users/nikomatsakis/received_events","type":"User","site_admin":false},"created_at":"2016-12-19T15:31:39Z","updated_at":"2016-12-19T15:31:39Z","body":"@camlorn can't we add a compiler flag (`-Z reorder-layouts`) and then pass this flag to the affected tests?\r\n\r\nIn retrospect I wish I had raised this to the core team to discuss proper handling."}}
{"actor":{"display_login":"nikomatsakis"}}