{"action":"created","issue":{"url":"https://api.github.com/repos/servo/servo/issues/10916","repository_url":"https://api.github.com/repos/servo/servo","labels_url":"https://api.github.com/repos/servo/servo/issues/10916/labels{/name}","comments_url":"https://api.github.com/repos/servo/servo/issues/10916/comments","events_url":"https://api.github.com/repos/servo/servo/issues/10916/events","html_url":"https://github.com/servo/servo/pull/10916","id":151836532,"number":10916,"title":"Use NEON build flag on ARM and AArch64","user":{"login":"mmatyas","id":4354863,"avatar_url":"https://avatars.githubusercontent.com/u/4354863?v=3","gravatar_id":"","url":"https://api.github.com/users/mmatyas","html_url":"https://github.com/mmatyas","followers_url":"https://api.github.com/users/mmatyas/followers","following_url":"https://api.github.com/users/mmatyas/following{/other_user}","gists_url":"https://api.github.com/users/mmatyas/gists{/gist_id}","starred_url":"https://api.github.com/users/mmatyas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mmatyas/subscriptions","organizations_url":"https://api.github.com/users/mmatyas/orgs","repos_url":"https://api.github.com/users/mmatyas/repos","events_url":"https://api.github.com/users/mmatyas/events{/privacy}","received_events_url":"https://api.github.com/users/mmatyas/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/servo/servo/labels/S-awaiting-review","name":"S-awaiting-review","color":"d4c5f9"}],"state":"open","locked":false,"assignee":{"login":"nox","id":123095,"avatar_url":"https://avatars.githubusercontent.com/u/123095?v=3","gravatar_id":"","url":"https://api.github.com/users/nox","html_url":"https://github.com/nox","followers_url":"https://api.github.com/users/nox/followers","following_url":"https://api.github.com/users/nox/following{/other_user}","gists_url":"https://api.github.com/users/nox/gists{/gist_id}","starred_url":"https://api.github.com/users/nox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nox/subscriptions","organizations_url":"https://api.github.com/users/nox/orgs","repos_url":"https://api.github.com/users/nox/repos","events_url":"https://api.github.com/users/nox/events{/privacy}","received_events_url":"https://api.github.com/users/nox/received_events","type":"User","site_admin":false},"milestone":null,"comments":7,"created_at":"2016-04-29T10:25:52Z","updated_at":"2016-04-29T16:55:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/servo/servo/pulls/10916","html_url":"https://github.com/servo/servo/pull/10916","diff_url":"https://github.com/servo/servo/pull/10916.diff","patch_url":"https://github.com/servo/servo/pull/10916.patch"},"body":"The NEON flag is already used when building for Android, this patch enables it on other ARM devices too.\r\n\r\nNote that this patch just adds the build flag to the compilation, for actually enabling the SIMD code in Servo, we'll also need #10900 (but it's not a dependency).\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"35\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/servo/servo/10916)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/servo/servo/issues/comments/215807400","html_url":"https://github.com/servo/servo/pull/10916#issuecomment-215807400","issue_url":"https://api.github.com/repos/servo/servo/issues/10916","id":215807400,"user":{"login":"aneeshusa","id":2085567,"avatar_url":"https://avatars.githubusercontent.com/u/2085567?v=3","gravatar_id":"","url":"https://api.github.com/users/aneeshusa","html_url":"https://github.com/aneeshusa","followers_url":"https://api.github.com/users/aneeshusa/followers","following_url":"https://api.github.com/users/aneeshusa/following{/other_user}","gists_url":"https://api.github.com/users/aneeshusa/gists{/gist_id}","starred_url":"https://api.github.com/users/aneeshusa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aneeshusa/subscriptions","organizations_url":"https://api.github.com/users/aneeshusa/orgs","repos_url":"https://api.github.com/users/aneeshusa/repos","events_url":"https://api.github.com/users/aneeshusa/events{/privacy}","received_events_url":"https://api.github.com/users/aneeshusa/received_events","type":"User","site_admin":false},"created_at":"2016-04-29T16:55:33Z","updated_at":"2016-04-29T16:55:33Z","body":"I don't see any places where the host target is added to the `targets` list; it's just the one cross-compile target (e.g. `[\"arm-linux-androideabi\"]` or an explicit target). It would make sense that you need the host target to actually compile, but I don't see it being used in the `targets` variable."}}