{"action":"opened","issue":{"url":"https://api.github.com/repos/bos/aeson/issues/424","repository_url":"https://api.github.com/repos/bos/aeson","labels_url":"https://api.github.com/repos/bos/aeson/issues/424/labels{/name}","comments_url":"https://api.github.com/repos/bos/aeson/issues/424/comments","events_url":"https://api.github.com/repos/bos/aeson/issues/424/events","html_url":"https://github.com/bos/aeson/issues/424","id":159232497,"number":424,"title":"Are INLINE pragmas worth the compilation time?","user":{"login":"phadej","id":51087,"avatar_url":"https://avatars.githubusercontent.com/u/51087?v=3","gravatar_id":"","url":"https://api.github.com/users/phadej","html_url":"https://github.com/phadej","followers_url":"https://api.github.com/users/phadej/followers","following_url":"https://api.github.com/users/phadej/following{/other_user}","gists_url":"https://api.github.com/users/phadej/gists{/gist_id}","starred_url":"https://api.github.com/users/phadej/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phadej/subscriptions","organizations_url":"https://api.github.com/users/phadej/orgs","repos_url":"https://api.github.com/users/phadej/repos","events_url":"https://api.github.com/users/phadej/events{/privacy}","received_events_url":"https://api.github.com/users/phadej/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-06-08T17:58:39Z","updated_at":"2016-06-08T17:58:39Z","closed_at":null,"body":"### Without (after removal of pragmas)\r\n\r\nINLINE pragmas in `FromJSON.hs` and `ToJSON.hs`\r\n\r\nSecond line ais nother clean build with aeson, criterion, statistics and aeson-benchmarks:\r\nIt seems that INLINE pragmas slowdown the downstream package compilation as well?\r\n\r\n```\r\nstack build aeson  69.23s user 2.65s system 99% cpu 1:12.35 total\r\nstack build       313.90s user 12.86s system 100% cpu 5:26.45 total\r\n```\r\n\r\n```\r\nbenchmarking generic/direct/twitter100\r\ntime                 405.4 μs   (402.9 μs .. 407.9 μs)\r\n                     1.000 R²   (0.999 R² .. 1.000 R²)\r\nmean                 405.0 μs   (402.6 μs .. 407.8 μs)\r\nstd dev              8.039 μs   (6.568 μs .. 10.82 μs)\r\nvariance introduced by outliers: 12% (moderately inflated)\r\n\r\nbenchmarking manual/direct/twitter100\r\ntime                 387.0 μs   (383.2 μs .. 391.5 μs)\r\n                     0.999 R²   (0.999 R² .. 1.000 R²)\r\nmean                 385.2 μs   (383.0 μs .. 387.6 μs)\r\nstd dev              8.185 μs   (7.162 μs .. 9.457 μs)\r\nvariance introduced by outliers: 14% (moderately inflated)\r\n\r\nbenchmarking th/direct/twitter100\r\ntime                 406.7 μs   (402.7 μs .. 409.9 μs)\r\n                     0.999 R²   (0.999 R² .. 1.000 R²)\r\nmean                 404.4 μs   (401.7 μs .. 407.7 μs)\r\nstd dev              9.895 μs   (8.084 μs .. 12.91 μs)\r\nvariance introduced by outliers: 16% (moderately inflated)\r\n```\r\n\r\n\r\n### With:\r\n\r\n```\r\nstack build aeson  179.39s user 4.51s system 97% cpu 3:09.30 total\r\nstack build        567.26s user 16.09s system 100% cpu 9:41.51 total\r\n```\r\n\r\n```\r\nbenchmarking generic/direct/twitter100\r\ntime                 399.7 μs   (395.2 μs .. 405.4 μs)\r\n                     0.999 R²   (0.999 R² .. 1.000 R²)\r\nmean                 400.3 μs   (397.5 μs .. 403.5 μs)\r\nstd dev              9.911 μs   (7.580 μs .. 12.47 μs)\r\nvariance introduced by outliers: 17% (moderately inflated)\r\n\r\nbenchmarking manual/direct/twitter100\r\ntime                 385.4 μs   (383.1 μs .. 387.9 μs)\r\n                     1.000 R²   (0.999 R² .. 1.000 R²)\r\nmean                 386.9 μs   (384.4 μs .. 390.0 μs)\r\nstd dev              9.782 μs   (7.519 μs .. 15.14 μs)\r\nvariance introduced by outliers: 18% (moderately inflated)\r\n\r\nbenchmarking th/direct/twitter100\r\ntime                 404.5 μs   (400.5 μs .. 407.8 μs)\r\n                     0.999 R²   (0.999 R² .. 1.000 R²)\r\nmean                 402.1 μs   (399.1 μs .. 406.3 μs)\r\nstd dev              12.35 μs   (9.032 μs .. 18.06 μs)\r\nvariance introduced by outliers: 24% (moderately inflated)\r\n```"}}
{"actor":{"display_login":"phadej"}}