{"push_id":3025853192,"size":2,"distinct_size":2,"ref":"refs/heads/master","head":"d8b2a96f800c75a2b2ff9f8589835fa398608d51","before":"0380bd6d71f585df94458860b57842fcb454e275","commits":[{"sha":"f3f642d7312e0decd8784fc693e4b4bd00854cba","author":{"name":"Elliot Ronaghan","email":"a0eed57aaedf007a763cbac2dd65f90f9fe7ec18@gmail.com"},"message":"Explicitly pass the address of _v for network atomics\n\nUse `__primitive(\"_wide_get_addr\", _v)` instead of passing `_v` by ref\nfor network atomics. Previously, we were passing a potentially remote\nchapel variable by reference to an extern proc and relying on the\ncompiler stripping away the wideness and just giving us the remote\naddress. Now, do this more explicitly since #11543 makes it an error to\npass remote variables to extern procs. IMO this also makes it much more\nclear what the code is trying to do.","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/f3f642d7312e0decd8784fc693e4b4bd00854cba"},{"sha":"d8b2a96f800c75a2b2ff9f8589835fa398608d51","author":{"name":"Elliot Ronaghan","email":"a0eed57aaedf007a763cbac2dd65f90f9fe7ec18@gmail.com"},"message":"Merge pull request #11579 from ronawho/net-atomic-wide-addr\n\nExplicitly pass the address of _v for network atomics\r\n\r\n[reviewed by @bradcray and @gbtitus]\r\n\r\nUse `__primitive(\"_wide_get_addr\", _v)` instead of passing `_v` by ref\r\nfor network atomics. Previously, we were passing a potentially remote\r\nchapel variable by reference to an extern proc and relying on the\r\ncompiler stripping away the wideness and just giving us the remote\r\naddress. Now, do this more explicitly since #11543 made it an error to\r\npass remote variables to extern procs. IMO this also makes it much more\r\nclear what the code is trying to do.","distinct":true,"url":"https://api.github.com/repos/chapel-lang/chapel/commits/d8b2a96f800c75a2b2ff9f8589835fa398608d51"}]}
{"actor":{"display_login":"ronawho"}}