{"action":"created","issue":{"url":"https://api.github.com/repos/JuliaLang/julia/issues/16128","repository_url":"https://api.github.com/repos/JuliaLang/julia","labels_url":"https://api.github.com/repos/JuliaLang/julia/issues/16128/labels{/name}","comments_url":"https://api.github.com/repos/JuliaLang/julia/issues/16128/comments","events_url":"https://api.github.com/repos/JuliaLang/julia/issues/16128/events","html_url":"https://github.com/JuliaLang/julia/issues/16128","id":151959188,"number":16128,"title":"misc. benchmark regressions since 0.4","user":{"login":"JeffBezanson","id":744556,"avatar_url":"https://avatars.githubusercontent.com/u/744556?v=3","gravatar_id":"","url":"https://api.github.com/users/JeffBezanson","html_url":"https://github.com/JeffBezanson","followers_url":"https://api.github.com/users/JeffBezanson/followers","following_url":"https://api.github.com/users/JeffBezanson/following{/other_user}","gists_url":"https://api.github.com/users/JeffBezanson/gists{/gist_id}","starred_url":"https://api.github.com/users/JeffBezanson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JeffBezanson/subscriptions","organizations_url":"https://api.github.com/users/JeffBezanson/orgs","repos_url":"https://api.github.com/users/JeffBezanson/repos","events_url":"https://api.github.com/users/JeffBezanson/events{/privacy}","received_events_url":"https://api.github.com/users/JeffBezanson/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/JuliaLang/julia/labels/performance","name":"performance","color":"d7e102"},{"url":"https://api.github.com/repos/JuliaLang/julia/labels/regression","name":"regression","color":"ff2510"}],"state":"open","locked":false,"assignee":{"login":"JeffBezanson","id":744556,"avatar_url":"https://avatars.githubusercontent.com/u/744556?v=3","gravatar_id":"","url":"https://api.github.com/users/JeffBezanson","html_url":"https://github.com/JeffBezanson","followers_url":"https://api.github.com/users/JeffBezanson/followers","following_url":"https://api.github.com/users/JeffBezanson/following{/other_user}","gists_url":"https://api.github.com/users/JeffBezanson/gists{/gist_id}","starred_url":"https://api.github.com/users/JeffBezanson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JeffBezanson/subscriptions","organizations_url":"https://api.github.com/users/JeffBezanson/orgs","repos_url":"https://api.github.com/users/JeffBezanson/repos","events_url":"https://api.github.com/users/JeffBezanson/events{/privacy}","received_events_url":"https://api.github.com/users/JeffBezanson/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/JuliaLang/julia/milestones/14","html_url":"https://github.com/JuliaLang/julia/milestones/0.5.0","labels_url":"https://api.github.com/repos/JuliaLang/julia/milestones/14/labels","id":868319,"number":14,"title":"0.5.0","description":"Release-defining items for v0.5.","creator":{"login":"vtjnash","id":330950,"avatar_url":"https://avatars.githubusercontent.com/u/330950?v=3","gravatar_id":"","url":"https://api.github.com/users/vtjnash","html_url":"https://github.com/vtjnash","followers_url":"https://api.github.com/users/vtjnash/followers","following_url":"https://api.github.com/users/vtjnash/following{/other_user}","gists_url":"https://api.github.com/users/vtjnash/gists{/gist_id}","starred_url":"https://api.github.com/users/vtjnash/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vtjnash/subscriptions","organizations_url":"https://api.github.com/users/vtjnash/orgs","repos_url":"https://api.github.com/users/vtjnash/repos","events_url":"https://api.github.com/users/vtjnash/events{/privacy}","received_events_url":"https://api.github.com/users/vtjnash/received_events","type":"User","site_admin":false},"open_issues":18,"closed_issues":72,"state":"open","created_at":"2014-11-16T16:16:12Z","updated_at":"2016-06-13T20:49:07Z","due_on":"2016-07-31T04:00:00Z","closed_at":null},"comments":50,"created_at":"2016-04-29T20:14:47Z","updated_at":"2016-06-14T01:19:45Z","closed_at":null,"body":"Just ran `make test-perf` and observed this. Most results are the same, and we've improved on about 6 tests, but unfortunately there are some significant regressions:\r\n\r\ntest                |  0.4 | after #16236  | 320b2ee7cb |  4f65737874  |  4f65737874 -O3  |\r\n------------------|--------|------------------|----------------|-------------------|---------------------------|\r\nparse_int       |  0.190  |                  |                    |      0.315       |                    |\r\nrand_mat_stat     |   15.311  |  19.42  |    **47.312**  |     11.45         |          |\r\ncons               |   80.837    |  94.14  |   86.229           |     91.86        |\r\nrandn_zig       |  23.74     |              |                        |    27.84         |  24.39       |\r\ngk                |   52.035     |  68.4  |   79.365        |  62.92     |         |\r\nsparsemul         |   40.171  | 44.6  |   44.436        |    44.45    |    36.49    |\r\nsparsemul2        |   62.836  | 66  |    66.982         |    65.73     |   63.23     |\r\nactorgraph        |  633.873 | 643  |   709.688         |    685       |   650      |\r\nsimplex           |   39.209  |  46  |   55.224            |     55.02   |   39.90     |\r\nsplitline           |    49.86      |            |   69.56      |  56.87     |    60.86    |\r\njson               |   14.113  |  19.904  |  23.864     |   21.76      |   19.56   |\r\nadd1              |     33.63 |               |                 |     44.09     |    37.82    |\r\nadd1_logical      |   41.797  |  90.927  |  **102.121**  |   **105.3**   |   60.12   |\r\nbinary_trees        |  30.694  |  67.287  |  44.508    |    42.76    |         |\r\nk_nucleotide       |  71.174  |  113.44  |                 |   82.74  |     78.35    |\r\n"},"comment":{"url":"https://api.github.com/repos/JuliaLang/julia/issues/comments/225754939","html_url":"https://github.com/JuliaLang/julia/issues/16128#issuecomment-225754939","issue_url":"https://api.github.com/repos/JuliaLang/julia/issues/16128","id":225754939,"user":{"login":"vtjnash","id":330950,"avatar_url":"https://avatars.githubusercontent.com/u/330950?v=3","gravatar_id":"","url":"https://api.github.com/users/vtjnash","html_url":"https://github.com/vtjnash","followers_url":"https://api.github.com/users/vtjnash/followers","following_url":"https://api.github.com/users/vtjnash/following{/other_user}","gists_url":"https://api.github.com/users/vtjnash/gists{/gist_id}","starred_url":"https://api.github.com/users/vtjnash/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vtjnash/subscriptions","organizations_url":"https://api.github.com/users/vtjnash/orgs","repos_url":"https://api.github.com/users/vtjnash/repos","events_url":"https://api.github.com/users/vtjnash/events{/privacy}","received_events_url":"https://api.github.com/users/vtjnash/received_events","type":"User","site_admin":false},"created_at":"2016-06-14T01:19:45Z","updated_at":"2016-06-14T01:19:45Z","body":"Looking into the `add1_logical` regression, it looks like LLVM 3.7 (SROA?) is not able to optimize `countnz(logical_y)` correctly anymore. As written (and if we make for-loops to have implicit `@inbounds`), then it should be vectorizable now (as llvm 3.3 can do on current master), and thus much faster than 0.4 (but llvm 3.7 fails to do this).\r\n\r\naddendum: for some reason, putting `@inbounds` in front of the for-loop elids bounds checking inside `next` since it isn't marked, but I think that's an inliner bug for the effect-free computation of getindex"}}
{"actor":{"display_login":"vtjnash"}}