{"push_id":2363647486,"size":37,"distinct_size":37,"ref":"refs/heads/master","head":"d9902140fd199765e479840ab81dbcc7acf7474e","before":"2da2709e4a8df211c360005798161779e4fd00e1","commits":[{"sha":"d04afb354ab1ab413b9d4ef41056ef6921248a77","author":{"name":"Lydia Duncan","email":"de4dee48938b6ca38d3dfde4f7329b7853655052@users.noreply.github.com"},"message":"Add two tests of default values using \"new\" on records with initializers\n\nOur default argument handling doesn't respond appropriately to records that\ndefine initializers.  Add two tests that demonstrate this: one for explicit\ninitializers with this problem, and one for default initializers when a field\nis declared in this manner.","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/d04afb354ab1ab413b9d4ef41056ef6921248a77"},{"sha":"c6ce99fa33662d1d95f6784dccc2ae44b1db6df8","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Ignore erroneous auto-copy calls in auto-copy fns","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/c6ce99fa33662d1d95f6784dccc2ae44b1db6df8"},{"sha":"646e4bbbb6350781aff1627b0074bdb1b279ecec","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Add comparison operators for astloc","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/646e4bbbb6350781aff1627b0074bdb1b279ecec"},{"sha":"c9dd63c67295b4109839cb33921d566f9c779863","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Handle dead code / type iteration\n\nGets these tests working:\n types/enum/ldelaney/enum-iterator\n arrays/userAPI/sliceOps2D.chpl\n\nThe dead code problem could alternatively be solved by reverting\nPR #1162.","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/c9dd63c67295b4109839cb33921d566f9c779863"},{"sha":"16929af9c93f2b5e6b48c1edbaf9c21ce851dbf9","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Improve gatherLoopDetails to work during callDestructors","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/16929af9c93f2b5e6b48c1edbaf9c21ce851dbf9"},{"sha":"eff8dd963746f054d4cabcd30be99606679da282","author":{"name":"Michael Ferguson","email":"23ba8fd1dec0341f3388da35d9cd16cd43645b13@users.noreply.github.com"},"message":"Improve gatherLoopDetails more for use after ReturnByRef\n\nRelevant test:\n test/chplvis/benchmarks-hpcc/hpl-vdb.chpl","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/eff8dd963746f054d4cabcd30be99606679da282"},{"sha":"87e5d8160e754c2d1687b398fc98aa0d9639747a","author":{"name":"Lydia Duncan","email":"de4dee48938b6ca38d3dfde4f7329b7853655052@users.noreply.github.com"},"message":"Needed a where clause on the record initializer to avoid use as copy init\n\nFor some reason, the test avoided generating that error message.  Need to\ninvestigate that, but in a separate PR.","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/87e5d8160e754c2d1687b398fc98aa0d9639747a"},{"sha":"b91c7a131f8c9a8a00d817389b0adb5d703a65b5","author":{"name":"Lydia Duncan","email":"de4dee48938b6ca38d3dfde4f7329b7853655052@users.noreply.github.com"},"message":"Add a variant of the defaultValueIsRecordWithInit case for regular procs\n\nSeems to behave in the same way as initializers (thankfully)","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/b91c7a131f8c9a8a00d817389b0adb5d703a65b5"},{"sha":"20fb99c8749f3b045b023924a1c875c647cb4f63","author":{"name":"Lydia Duncan","email":"de4dee48938b6ca38d3dfde4f7329b7853655052@users.noreply.github.com"},"message":"Modify the handling of new expressions for argument default values\n\nWhen the type being created is a record with default initializers, the return\nvalue is void but that's not what we want to use to determine the type of the\nargument.  So, when handling resolveNew for a concrete record, if we know we're\nthe last statement of a block for an ArgSymbol, add a SymExpr to the temporary\nwe send in to the initializer, so that we can get the right type back.\n\nThis is a hack and I am not proud of it, but I'm not sure there's a better way,\naside from redoing argument handling.\n\nAlso, currently only applies to concrete records.  Need to expand to generic\nrecords and generic record instantiations, but I'm going to test this version\nfirst as a proof of concept.","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/20fb99c8749f3b045b023924a1c875c647cb4f63"},{"sha":"f173bd7ce8659ce2f6b632bf417647dddbaac369","author":{"name":"Lydia Duncan","email":"de4dee48938b6ca38d3dfde4f7329b7853655052@users.noreply.github.com"},"message":"Add two variations of the default value with initializers for generics\n\nThese cover the case where the type used for the default value is a generic\nrecord with initializers.  The earlier commits did not handle this, but the\nextension is fairly similar.","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/f173bd7ce8659ce2f6b632bf417647dddbaac369"},{"sha":"943fc6af9d593cf357d55feee9f7f118252e24c5","author":{"name":"Lydia Duncan","email":"de4dee48938b6ca38d3dfde4f7329b7853655052@users.noreply.github.com"},"message":"Fix failure for generic records with initializers.\n\nThere's a different code pathway for generic records instead of concrete ones.\nThis commit applies the same changes to that pathway.","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/943fc6af9d593cf357d55feee9f7f118252e24c5"},{"sha":"0b4f4e7202245815d9668e34d256432581fda07a","author":{"name":"Lydia Duncan","email":"de4dee48938b6ca38d3dfde4f7329b7853655052@users.noreply.github.com"},"message":"Merge branch 'master' into ofTypeVoid","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/0b4f4e7202245815d9668e34d256432581fda07a"},{"sha":"cf19a3cc637960746b8988e1cf638de5efd32ff1","author":{"name":"Michael Noakes","email":"64347b73ba5ec8fb944dede5a8d7f9d3ac0a277c@cray.com"},"message":"parent param :- Fetch substitutions from parent classes","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/cf19a3cc637960746b8988e1cf638de5efd32ff1"},{"sha":"dce7475a19bdf8f332194e643ad9971c67ba4d92","author":{"name":"Michael Noakes","email":"64347b73ba5ec8fb944dede5a8d7f9d3ac0a277c@cray.com"},"message":"parent param :- Add support to postFold \".v\"","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/dce7475a19bdf8f332194e643ad9971c67ba4d92"},{"sha":"a981198516789d237c1b0b63398dcd5bbe1829b4","author":{"name":"Michael Noakes","email":"64347b73ba5ec8fb944dede5a8d7f9d3ac0a277c@cray.com"},"message":"parent param :- Add a test that failed without this work","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/a981198516789d237c1b0b63398dcd5bbe1829b4"},{"sha":"160adebf4a309929a7197ed875c91f643f3a11ec","author":{"name":"Elliot Ronaghan","email":"77fc313decc7150b7d7b80308a16c0ce425c75a9@gmail.com"},"message":"Directly create aBarrier classes in the AllLocalesBarrier\n\nInstead of using the record wrapper, directly create the atomic barrier class.\nThis should avoid any potential dynamic dispatch, which was causing an issue\nfor --baseline testing.\n\nAvoiding dynamic dispatch also slightly improves the performance of this\nbarrier, though not significantly since a majority of the time is spent\ncommunicating. For the 16-node micro-benchmark that does 100,000 barriers this\nthis shaves a 1/3 of a second off (6.5s -> 6.2s)","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/160adebf4a309929a7197ed875c91f643f3a11ec"},{"sha":"4b83be007f05e72571822467a2f912f459401b65","author":{"name":"Lydia Duncan","email":"de4dee48938b6ca38d3dfde4f7329b7853655052@users.noreply.github.com"},"message":"Merge branch 'master' into ofTypeVoid","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/4b83be007f05e72571822467a2f912f459401b65"},{"sha":"db63d62ceac6346a650fedc9cb963e6b28614819","author":{"name":"Elliot Ronaghan","email":"77fc313decc7150b7d7b80308a16c0ce425c75a9@gmail.com"},"message":"Merge pull request #8593 from ronawho/fix-isx-opt-baseline\n\nDirectly create aBarrier classes in the AllLocalesBarrier\r\n\r\n[reviewed by @benharsh]\r\n\r\nInstead of using the record wrapper, directly create the atomic barrier class.\r\nThis should avoid any potential dynamic dispatch, which was causing an issue\r\nfor --baseline testing.\r\n\r\nAvoiding dynamic dispatch also slightly improves the performance of this\r\nbarrier, though not significantly since a majority of the time is spent\r\ncommunicating. For the 16-node micro-benchmark that does 100,000 barriers this\r\nthis shaves a 1/3 of a second off (6.5s -> 6.2s)","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/db63d62ceac6346a650fedc9cb963e6b28614819"},{"sha":"f43c0a2e02235866302b870719dbb029fe22de7e","author":{"name":"Elliot Ronaghan","email":"77fc313decc7150b7d7b80308a16c0ce425c75a9@gmail.com"},"message":"Reduce contention from ugni's polling thread in isx-hand-optimized\n\nThis just enables the opt-in mechanism added in #8562 for isx-hand-optimized\n\nFor ISx we ideally want all tasks to be operating in lockstep so that they all\nhit the exchange barrier at the same time so we don't waste time waiting for\ntasks to catch up. However, the progress thread creates contention with tasks\nrunning on the same core, so those tasks are noticeably slower even for parts\nof the computation that are completely local. Prior to this change for the\ninput stage (which just creates an array and fills it with random values) we'd\nsee timings like:\n\n    input = 0.9 avg (0.81 min .. 1.28 max)\n\nwhere the max value is from the task sharing a core with the progress thread.\nWith this change we see much tighter timings:\n\n    input = 0.89 avg (0.81 min .. 0.95 max)\n\nI think the remaining variability in timings is because of the serialized\ncomponent of dynamic array registration. We have other work to reduce the\ncritical section for array registration (#8177), but prior to this change (and\nthe scalable barrier) we had a really hard time evaluating the impact of that\nwork since timings were so variable to begin with.","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/f43c0a2e02235866302b870719dbb029fe22de7e"},{"sha":"81e4cacee08145baef48faa8070dd0845c06614e","author":{"name":"Deeksha Kaurav","email":"a581a3eaa99b539fab5ff438f59ebdafd599a711@itbhu.ac.in"},"message":"string max min","distinct":true,"url":"https://api.github.com/repos/daviditen/chapel/commits/81e4cacee08145baef48faa8070dd0845c06614e"}]}
{"actor":{"display_login":"daviditen"}}