{"push_id":1441399833,"size":94,"distinct_size":91,"ref":"refs/heads/master","head":"1f67c238f7022b1512793ef3b3a75554e6b49898","before":"c666f255b56b73ff75b5ed4ef3eda1663abddda0","commits":[{"sha":"e03b86d35ce0ffec031bc1a59b1b4d9375ea97e9","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Merge remote-tracking branch 'origin/improve-optimize-on-clauses' into no-heap-alloc-bundles","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/e03b86d35ce0ffec031bc1a59b1b4d9375ea97e9"},{"sha":"b715e5b9cf76d9146d65f3416f327dcf5a60d7ea","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Stack allocate argument bundles","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/b715e5b9cf76d9146d65f3416f327dcf5a60d7ea"},{"sha":"dd6b010b7a6b82a77186f9319c08ec7167ff8b5a","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Add commented out estimateSizeInBytes for history","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/dd6b010b7a6b82a77186f9319c08ec7167ff8b5a"},{"sha":"2011ec1596b9a1f8df10e82db479184cb386e90e","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Change compiler-runtime interface to include size\n\nTask spawning needs to include the argument bundle size so that the\ntasking layer can be responsible for copying it if necessary.\n\nAdditionally, update chpl-comm.h and chpl-tasks.h to indicate that the\nargument bundles may be reused after the runtime call returns.\n\nNote that the runtime will not compile with this change\nwithout following changes updating the comm and tasking\nimplementations.","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/2011ec1596b9a1f8df10e82db479184cb386e90e"},{"sha":"9c8afa66b0ef6498a9c1289c56bbf50477cd9787","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Move round_up_to_mask and friends to own header\n\nI want to use these functions in tasks-fifo.c, so I moved them to a\nruntime header where they are more available.","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/9c8afa66b0ef6498a9c1289c56bbf50477cd9787"},{"sha":"af7bef27c9ecef384499a467e44c3dd0abd2fd0a","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"1st changes to comm layer to remove allocations","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/af7bef27c9ecef384499a467e44c3dd0abd2fd0a"},{"sha":"af78e25ba5bb6f0f1b1ec208e1f3264bc1a320b7","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"1st try removing allocations from tasks-fifo.c","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/af78e25ba5bb6f0f1b1ec208e1f3264bc1a320b7"},{"sha":"107bdbfa57d92f98ede54e3984904760acca3910","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"1st try updating tasks-qthreads","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/107bdbfa57d92f98ede54e3984904760acca3910"},{"sha":"842025799ab872e7057ef0cd9bf6c8d7a14e31de","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Adjusted gasnet and fifo so hellos pass with gasnet+fifo","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/842025799ab872e7057ef0cd9bf6c8d7a14e31de"},{"sha":"aa6291102d433b6ae59e8b061af516e4ca9a2529","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Add note about chpl_task_createCommTask's argument","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/aa6291102d433b6ae59e8b061af516e4ca9a2529"},{"sha":"a49f0f8430e4925a70c98cfe666bce72e7268ca2","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Compiler changes to add a runtime header to arg bundles","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/a49f0f8430e4925a70c98cfe666bce72e7268ca2"},{"sha":"8e26367efb47d18402b9fdde6386e233dca399cb","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Runtime header/interface updates for typed bundle args","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/8e26367efb47d18402b9fdde6386e233dca399cb"},{"sha":"fee626a2d8a574e0e3bb4a5b9b25ea50c59010d8","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Module updates for type bundle args","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/fee626a2d8a574e0e3bb4a5b9b25ea50c59010d8"},{"sha":"2d860dff746637016d82cf40a9137a77d8da174f","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"fifo, comm none and gasnet updates for rt headers on arg bundles\n\nAfter this commit, hellos pass with fifo+gasnet.","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/2d860dff746637016d82cf40a9137a77d8da174f"},{"sha":"8f1f048f7c43673d20a06b225a3c38dbad6e8d50","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Adjust tasks-qthreads for new arg bundle strategy","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/8f1f048f7c43673d20a06b225a3c38dbad6e8d50"},{"sha":"6144ca54a6a0b9cb6491a339226a69b39615abae","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Add a comment","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/6144ca54a6a0b9cb6491a339226a69b39615abae"},{"sha":"87c90b600707813ac1a7f976e3937291353c9ded","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Never heap allocate argument bundles\n\nThe brave new world is that the runtime is responsible for copying\nargument bundles. There does not seem to be a benefit to heap allocating\nthem when the runtime will just copy them from the stack or the heap\nequally well. (Even GASNet active messages can be stack-allocated...).\n\nOne might argue that heap allocating the argument bundles saves valuable\nstack space, but as much stack space would be used in the case that the\nfunction is called directly (as with direct on optimization). Additionally,\nwe could structure the C code so that it is clear that this stack space\ncan be freed once the runtime call returns. (However it seems moot because\nthe arguments we are putting into the arg bundle are probably already\non the stack. E.g. if they are function arguments).\n\nThis resolves an issue I was seeing where a task function tried\nto free its argument bundle, but the bundle the task fn gets is\na copy, not the original, so it core dumped. If we needed to heap\nallocate some argument bundles for task functions, we would need\nto include a pointer to the original arg bundle in the arg bundle\npassed to the runtime.","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/87c90b600707813ac1a7f976e3937291353c9ded"},{"sha":"75f35d9df87230ce22014f49d963bd5961e649da","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Merge remote-tracking branch 'upstream/master' into no-heap-alloc-bundles","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/75f35d9df87230ce22014f49d963bd5961e649da"},{"sha":"59be51d66161942e3174e3042617690b7dbefeca","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Remove commented-out estimateSizeInBytes","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/59be51d66161942e3174e3042617690b7dbefeca"},{"sha":"491031200b6c02b14d58447f954d944f79556a46","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Adjust tasks-massivethreads.c for filename->int change","distinct":true,"url":"https://api.github.com/repos/noakesmichael/chapel/commits/491031200b6c02b14d58447f954d944f79556a46"}]}
{"actor":{"display_login":"noakesmichael"}}