{"push_id":1402677875,"size":3,"distinct_size":3,"ref":"refs/heads/master","head":"57c3a67ab8cdbd120fc2608a7e7481d4f10cba33","before":"a28da268c9ed6e10b9e98aa3c9d06ae7873dde4d","commits":[{"sha":"b1552c63cac9e94b729afc4d647f9c327eefc93b","author":{"name":"Ben Harshbarger","email":"b99bc1caf928277c7294b20ca475aa7bec2ad1d0@gmail.com"},"message":"Fix an issue where coforall index variables were autoCopied for each iteration","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/b1552c63cac9e94b729afc4d647f9c327eefc93b"},{"sha":"4ffaacf7733fde389143cfe54d4829b0df55a1fa","author":{"name":"Ben Harshbarger","email":"b99bc1caf928277c7294b20ca475aa7bec2ad1d0@gmail.com"},"message":"Return true for passByRef if the sym is already a ref.","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/4ffaacf7733fde389143cfe54d4829b0df55a1fa"},{"sha":"57c3a67ab8cdbd120fc2608a7e7481d4f10cba33","author":{"name":"Ben Harshbarger","email":"9c0877e4c29847f257b58964937e82f678f55169@users.noreply.github.com"},"message":"Merge pull request #4869 from benharsh/pass-index-by-ref\n\n flattenFunctions should pass _ref types by ref intent\r\n\r\nPrior to this change, flattenFunctions was passing things with the _ref type by \"const in\", because the _ref type was technically an aggregate class. With this modification, we should pass things by INTENT_REF if they already have the \"_ref type . This fixes an issue where coforall index variables were autoCopied for each iteration, which could lead to both incorrect and slow programs. The autoCopy was inserted in parallel.cpp, which erroneously decided an autoCopy was needed despite having the _ref type.\r\n\r\nTesting:\r\n- [x] full local\r\n- [x] full no-local\r\n- [x] gasnet for distributions/, multilocale/, release/examples/\r\n\r\n[ r @noakesmichael ]","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/57c3a67ab8cdbd120fc2608a7e7481d4f10cba33"}]}
{"actor":{"display_login":"daviditen"}}