{"push_id":2588418755,"size":3,"distinct_size":3,"ref":"refs/heads/master","head":"477e3f7e5b43e8ccec28bda9f203e05830fce512","before":"6759500b59b97495970089bf4038ef5bc40230fe","commits":[{"sha":"58f2c8bc5591284359a4ce49e3497c7058ad416e","author":{"name":"Michael Ferguson","email":"9efea9f3e732de9799f8e0cd6a4854f20f095f82@users.noreply.github.com"},"message":"Add config const doVerboseComm to aid debugging changes","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/58f2c8bc5591284359a4ce49e3497c7058ad416e"},{"sha":"715b28bf7c48b2a4c30c8d1caa87f9e8762f7d18","author":{"name":"Michael Ferguson","email":"9efea9f3e732de9799f8e0cd6a4854f20f095f82@users.noreply.github.com"},"message":"Fix comm count regressions from PR #9572.","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/715b28bf7c48b2a4c30c8d1caa87f9e8762f7d18"},{"sha":"477e3f7e5b43e8ccec28bda9f203e05830fce512","author":{"name":"Michael Ferguson","email":"9efea9f3e732de9799f8e0cd6a4854f20f095f82@users.noreply.github.com"},"message":"Merge pull request #9611 from mppf/fix-comm-counts\n\nFix comm count regression due to change in nested tuples\r\n\r\nPR #9572 changed the behavior of nested tuples so that they are captured by \r\nreference rather than by value. This had the impact of increasing comm counts \r\nfor privatization when the privatization state includes a tuple of other\r\nvalues. To address this problem, this PR modifies the distribution code where\r\nthe nested tuple causes new communication and replaces it with a record.\r\n\r\nIssue #9610 discusses to what extent nested tuples should be captured by \r\nreference.\r\n\r\n- [x] pass failing tests with replicated \r\n- [x] pass failing tests with block\r\n- [x] full gasnet testing \r\n\r\nReviewed by @benharsh - thanks!","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/477e3f7e5b43e8ccec28bda9f203e05830fce512"}]}
{"actor":{"display_login":"mppf"}}