{"action":"created","issue":{"url":"https://api.github.com/repos/JuliaLang/julia/issues/17826","repository_url":"https://api.github.com/repos/JuliaLang/julia","labels_url":"https://api.github.com/repos/JuliaLang/julia/issues/17826/labels{/name}","comments_url":"https://api.github.com/repos/JuliaLang/julia/issues/17826/comments","events_url":"https://api.github.com/repos/JuliaLang/julia/issues/17826/events","html_url":"https://github.com/JuliaLang/julia/pull/17826","id":169514192,"number":17826,"title":"RFC: Use LLVM_VERSION instead of LLVMxx","user":{"login":"ararslan","id":6396159,"avatar_url":"https://avatars.githubusercontent.com/u/6396159?v=3","gravatar_id":"","url":"https://api.github.com/users/ararslan","html_url":"https://github.com/ararslan","followers_url":"https://api.github.com/users/ararslan/followers","following_url":"https://api.github.com/users/ararslan/following{/other_user}","gists_url":"https://api.github.com/users/ararslan/gists{/gist_id}","starred_url":"https://api.github.com/users/ararslan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ararslan/subscriptions","organizations_url":"https://api.github.com/users/ararslan/orgs","repos_url":"https://api.github.com/users/ararslan/repos","events_url":"https://api.github.com/users/ararslan/events{/privacy}","received_events_url":"https://api.github.com/users/ararslan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":21,"created_at":"2016-08-05T02:08:58Z","updated_at":"2016-08-05T05:18:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/JuliaLang/julia/pulls/17826","html_url":"https://github.com/JuliaLang/julia/pull/17826","diff_url":"https://github.com/JuliaLang/julia/pull/17826.diff","patch_url":"https://github.com/JuliaLang/julia/pull/17826.patch"},"body":"Fixes #9860.\r\n\r\nAs noted in the linked issue, the names `LLVM37`, `LLVM34`, etc. aren't immediately descriptive because they don't refer to specific versions but rather to ranges of versions. This implements the proposed fix of having a single `LLVM_VERSION` macro that holds the LLVM version as a two-digit number and instead of checking whether `LLVMxx` is defined, we check whether `LLVM_VERSION >= xx`. Similarly, checking whether `LLVMxx` is _not_ defined equates to `LLVM_VERSION < xx`.\r\n\r\nThis works locally, so fingers crossed that it works everywhere else..."},"comment":{"url":"https://api.github.com/repos/JuliaLang/julia/issues/comments/237757604","html_url":"https://github.com/JuliaLang/julia/pull/17826#issuecomment-237757604","issue_url":"https://api.github.com/repos/JuliaLang/julia/issues/17826","id":237757604,"user":{"login":"ararslan","id":6396159,"avatar_url":"https://avatars.githubusercontent.com/u/6396159?v=3","gravatar_id":"","url":"https://api.github.com/users/ararslan","html_url":"https://github.com/ararslan","followers_url":"https://api.github.com/users/ararslan/followers","following_url":"https://api.github.com/users/ararslan/following{/other_user}","gists_url":"https://api.github.com/users/ararslan/gists{/gist_id}","starred_url":"https://api.github.com/users/ararslan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ararslan/subscriptions","organizations_url":"https://api.github.com/users/ararslan/orgs","repos_url":"https://api.github.com/users/ararslan/repos","events_url":"https://api.github.com/users/ararslan/events{/privacy}","received_events_url":"https://api.github.com/users/ararslan/received_events","type":"User","site_admin":false},"created_at":"2016-08-05T05:18:55Z","updated_at":"2016-08-05T05:18:55Z","body":"Okay, added the `JL_` prefix. I had meant to add it when you had first mentioned it but forgot. Thanks so much for all of your help and patience!"}}
{"actor":{"display_login":"ararslan"}}