{"action":"created","issue":{"url":"https://api.github.com/repos/halide/Halide/issues/1616","repository_url":"https://api.github.com/repos/halide/Halide","labels_url":"https://api.github.com/repos/halide/Halide/issues/1616/labels{/name}","comments_url":"https://api.github.com/repos/halide/Halide/issues/1616/comments","events_url":"https://api.github.com/repos/halide/Halide/issues/1616/events","html_url":"https://github.com/halide/Halide/pull/1616","id":188102678,"number":1616,"title":"Support for AVX512","user":{"login":"abadams","id":515398,"avatar_url":"https://avatars.githubusercontent.com/u/515398?v=3","gravatar_id":"","url":"https://api.github.com/users/abadams","html_url":"https://github.com/abadams","followers_url":"https://api.github.com/users/abadams/followers","following_url":"https://api.github.com/users/abadams/following{/other_user}","gists_url":"https://api.github.com/users/abadams/gists{/gist_id}","starred_url":"https://api.github.com/users/abadams/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abadams/subscriptions","organizations_url":"https://api.github.com/users/abadams/orgs","repos_url":"https://api.github.com/users/abadams/repos","events_url":"https://api.github.com/users/abadams/events{/privacy}","received_events_url":"https://api.github.com/users/abadams/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":11,"created_at":"2016-11-08T21:03:59Z","updated_at":"2016-11-09T21:56:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/halide/Halide/pulls/1616","html_url":"https://github.com/halide/Halide/pull/1616","diff_url":"https://github.com/halide/Halide/pull/1616.diff","patch_url":"https://github.com/halide/Halide/pull/1616.patch"},"body":"Assumes the cannonlake / skylake server feature set. Relies on Intel's SDE emulator for testing.\r\n\r\nI only found one llvm bug doing this (https://llvm.org/bugs/show_bug.cgi?id=30947) so I think llvm is sufficiently mature for this now.\r\n\r\nSee issue #1608 for the feature request"},"comment":{"url":"https://api.github.com/repos/halide/Halide/issues/comments/259537940","html_url":"https://github.com/halide/Halide/pull/1616#issuecomment-259537940","issue_url":"https://api.github.com/repos/halide/Halide/issues/1616","id":259537940,"user":{"login":"zvookin","id":1501716,"avatar_url":"https://avatars.githubusercontent.com/u/1501716?v=3","gravatar_id":"","url":"https://api.github.com/users/zvookin","html_url":"https://github.com/zvookin","followers_url":"https://api.github.com/users/zvookin/followers","following_url":"https://api.github.com/users/zvookin/following{/other_user}","gists_url":"https://api.github.com/users/zvookin/gists{/gist_id}","starred_url":"https://api.github.com/users/zvookin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zvookin/subscriptions","organizations_url":"https://api.github.com/users/zvookin/orgs","repos_url":"https://api.github.com/users/zvookin/repos","events_url":"https://api.github.com/users/zvookin/events{/privacy}","received_events_url":"https://api.github.com/users/zvookin/received_events","type":"User","site_admin":false},"created_at":"2016-11-09T21:56:07Z","updated_at":"2016-11-09T21:56:07Z","body":"Fixing the one letter diff via spelling out the names as llvm does is certainly an improvement. Being consistent with llvm is likely also a win.\r\n\r\nIf we were to do feature flags, I have no issue with only providing one feature for combinations that only ever appear as a set together.\r\n\r\nPerhaps adding \"avx512_with_integer\" as an alias for the \"avx_skylake\" would be worthwhile. (The hypothesis is this is the only AVX512 target that most people will use for shipping widely deployed software.) To take that further we could do:\r\n    av512_float_only\r\n    avx512_knl // If you're using this, you probably know what you're doing\r\n    avx512_with_integer\r\n    avx512_with_integer_cnl // Idea is Intel will maintain proper subsets from here :-)\r\nthough I am well aware I am tilting at a windmill."}}
{"actor":{"display_login":"zvookin"}}