{"push_id":4409630128,"size":33,"distinct_size":31,"ref":"refs/heads/master","head":"fbcfb367b627a1cdbde913b1ab251473d307a17e","before":"507f8cc82a848d7740c979ef1d504ff1421c65fb","commits":[{"sha":"c3ba2f8ae42a33ebc329969606015d36d7cf15d9","author":{"name":"Greg Titus","email":"b3066f53d7a0fd3eaa1b657a44ad0856312d2203@users.noreply.github.com"},"message":"Support multiple system prediff scripts in one test run.\n\nThis allows for running more than one system prediff script.  The\nstart_test -sysprediff option can be given more than once and is\nadditive, and CHPL_SYSTEM_PREDIFF is now a comma-separated list.\n\nWhile here I'm also adding a slurm-srun specific system prediff.  This\nfilters out the meta-messages slurm produces when jobs or job instances\nexit erroneously.","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/c3ba2f8ae42a33ebc329969606015d36d7cf15d9"},{"sha":"7e375cd52329e39ff9ad5a7b9bec95b1be227934","author":{"name":"Michael Ferguson","email":"9efea9f3e732de9799f8e0cd6a4854f20f095f82@users.noreply.github.com"},"message":"Use PRIM_INIT for typed param initialization\n\nTo prepare for split init on params.","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/7e375cd52329e39ff9ad5a7b9bec95b1be227934"},{"sha":"3eb551bea49eda3a975b750fc4996751e74629a8","author":{"name":"Michael Ferguson","email":"9efea9f3e732de9799f8e0cd6a4854f20f095f82@users.noreply.github.com"},"message":"Enable split init for params, add tests","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/3eb551bea49eda3a975b750fc4996751e74629a8"},{"sha":"e36fbd4be9e6bbea74ed0bf6f464289f03209b8a","author":{"name":"Greg Titus","email":"b3066f53d7a0fd3eaa1b657a44ad0856312d2203@users.noreply.github.com"},"message":"Don't unload cray-libsci in the Chapel module for Shasta any more.\n\nWe've been unloading the cray-libsci module in the Chapel module for\nShasta systems, because cray-libsci caused the addition of -lhugetlbfs\nin linker command lines and there wasn't a libhugetlbfs on at least some\nof the Shasta systems we used.  This led to link errors.\n\nHere, stop doing this.  The unload is now failing because of certain\nchanges in how the PE modules are integrated with each other.  And in\naddition, based on some experimentation it appears that now even with\nthe cray-libsci module loaded we only get a -lhugetlbfs in the linker\ncommand if we also have a craype-hugepages module loaded.","distinct":false,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/e36fbd4be9e6bbea74ed0bf6f464289f03209b8a"},{"sha":"dc2b85fde8def26a1847ca1e745acfedcb7bc81c","author":{"name":"Greg Titus","email":"b3066f53d7a0fd3eaa1b657a44ad0856312d2203@users.noreply.github.com"},"message":"Reflecting review feedback, change var name 'pdf' to 'prediff'.","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/dc2b85fde8def26a1847ca1e745acfedcb7bc81c"},{"sha":"1fb76621f84511baa48efbef0de7725174803ed7","author":{"name":"Engin Kayraklioglu","email":"c31ea34e2db42d6392821a26e45cb8a6424e83c9@gmail.com"},"message":"Fix memcpy size for large misaligned copies in ugni","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/1fb76621f84511baa48efbef0de7725174803ed7"},{"sha":"f28920570aa58df0ce96ccb897fb9411afd4b2d5","author":{"name":"Engin Kayraklioglu","email":"c31ea34e2db42d6392821a26e45cb8a6424e83c9@gmail.com"},"message":"Add a test that used to segfault with ugni","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/f28920570aa58df0ce96ccb897fb9411afd4b2d5"},{"sha":"2ecefd0dcce6ea2f856260a87c4efd4473f34aae","author":{"name":"Engin Kayraklioglu","email":"c31ea34e2db42d6392821a26e45cb8a6424e83c9@gmail.com"},"message":"Move test to a proper location","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/2ecefd0dcce6ea2f856260a87c4efd4473f34aae"},{"sha":"cce4f4e3a135eb25082ddda4a1bb00bd76b1e605","author":{"name":"Greg Titus","email":"b3066f53d7a0fd3eaa1b657a44ad0856312d2203@users.noreply.github.com"},"message":"Merge pull request #14654 from gbtitus/shasta-dont-remove-cray_libsci-any-more\n\nDon't unload cray-libsci in the Chapel module for Shasta any more.\r\n\r\n(Reviewed by @ronawho.)\r\n\r\nWe've been unloading the cray-libsci module in the Chapel module for\r\nShasta systems, because cray-libsci caused the addition of -lhugetlbfs\r\nin linker command lines and there wasn't a libhugetlbfs on at least some\r\nof the Shasta systems we used. This led to link errors.\r\n\r\nHere, stop doing this. The unload is now failing because of certain\r\nchanges in how the PE modules are integrated with each other. And in\r\naddition, based on some experimentation it appears that now even with\r\nthe cray-libsci module loaded we only get a -lhugetlbfs in the linker\r\ncommand if we also have a craype-hugepages module loaded.\r\n\r\nThis closes https://github.com/Cray/chapel-private/issues/638.","distinct":false,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/cce4f4e3a135eb25082ddda4a1bb00bd76b1e605"},{"sha":"8e67e4fc4b79e1595890c4b38e4dd96ca627ba67","author":{"name":"Michael Ferguson","email":"9efea9f3e732de9799f8e0cd6a4854f20f095f82@users.noreply.github.com"},"message":"Fix some param issues uncovered","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/8e67e4fc4b79e1595890c4b38e4dd96ca627ba67"},{"sha":"2825c006296a1396793dca634708b59a8e1d0a58","author":{"name":"Engin Kayraklioglu","email":"481f309f266a75d2b81e63f6cac02538db9eddee@users.noreply.github.com"},"message":"Merge pull request #14652 from e-kayrakli/ugni-comm-wrong-memcpy-size\n\nFix memcpy size for large misaligned gets with ugni\r\n\r\nThis PR fixes a wrong size used in a memcpy in ugni layer.\r\n\r\nGETs larger than the \"trampoline buffer\" are handled piece by piece in ugni. The\r\ncode goes through several branches and the one where large, misaligned transfers\r\nbetween registered memory are handled, the memcpy from NIC buffer to local\r\nmemory used the full message size, instead of size of a piece. This resulted in\r\nsegfaults in some cases.\r\n\r\nThis is the cause of a bug reported by Arkouda devs:\r\n\r\n  https://github.com/mhmerrill/arkouda/issues/236\r\n\r\nProbably we didn't have any test that hit that branch (or maybe got lucky that\r\nthe sizes weren't so huge?). This also adds a test that uses arrays of same type\r\nand size reported in the Arkouda bug report.\r\n\r\nTest:\r\n- [x] added test fails on master and passes on this branch\r\n- [x] `test/release/examples/` and `test/runtime/configMatters` on XC\r\n\r\n[Reviewed by @ronawho and @gbtitus]","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/2825c006296a1396793dca634708b59a8e1d0a58"},{"sha":"594977cc6c9a280ed10f81fdb00adb4b1b6293d7","author":{"name":"Greg Titus","email":"b3066f53d7a0fd3eaa1b657a44ad0856312d2203@users.noreply.github.com"},"message":"Fix multiple system prediffs, add multiple system preexec support.\n\nThere were several flaws in the existing code for multiple system\nprediffs, which I'd failed to notice before.  Here, fix those and also\nadd support for multiple system preexecs.","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/594977cc6c9a280ed10f81fdb00adb4b1b6293d7"},{"sha":"020cb13d16db3c947dfb131101357f241b42db27","author":{"name":"Bradford Chamberlain","email":"9233b6f5388d6867a2a7be14d8b4ba53c86cfde2@MacBook-Pro.hitronhub.home"},"message":"New version of revcomp that follows the gcc #6 version and uses OOP","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/020cb13d16db3c947dfb131101357f241b42db27"},{"sha":"a7c2bd70456ec0120c32c238f54ded0f820ce981","author":{"name":"Bradford Chamberlain","email":"9233b6f5388d6867a2a7be14d8b4ba53c86cfde2@MacBook-Pro.hitronhub.home"},"message":"Send debug to stderr to avoid messing up the output","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/a7c2bd70456ec0120c32c238f54ded0f820ce981"},{"sha":"abe101750d5828c3adfa9f1bee17fbf50abaf6aa","author":{"name":"Bradford Chamberlain","email":"9233b6f5388d6867a2a7be14d8b4ba53c86cfde2@MacBook-Pro.hitronhub.home"},"message":"Clean-up","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/abe101750d5828c3adfa9f1bee17fbf50abaf6aa"},{"sha":"9b4b2c1e51f8278de891089488f9f10cd222b291","author":{"name":"Bradford Chamberlain","email":"9233b6f5388d6867a2a7be14d8b4ba53c86cfde2@MacBook-Pro.hitronhub.home"},"message":"Improve the attributions","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/9b4b2c1e51f8278de891089488f9f10cd222b291"},{"sha":"95e41f3d4d748e1e29b06f229b3bb706291bb66f","author":{"name":"Brad Chamberlain","email":"76b11d3d92bad7a642f7c1b15d8a810ff35b7019@users.noreply.github.com"},"message":"Merge pull request #14661 from bradcray/revcomp-oop\n\nNew chunk-by-chunk revcomp version with dynamic resizing and OOP\r\n\r\n[trivial, not reviewed]\r\n\r\nThis is a new version of revcomp that does dynamic resizing of the buffer (correctly,\r\nthis time) and which reads the data in chunk-by-chunk. It takes more of an OOP\r\norganization than our previous versions, simply because it was easier for me to\r\nreason about it that way, and I think the result ends up cleaner.","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/95e41f3d4d748e1e29b06f229b3bb706291bb66f"},{"sha":"7cad6a8881192f48e58143f114b58a60d6d057b4","author":{"name":"Michael Ferguson","email":"9efea9f3e732de9799f8e0cd6a4854f20f095f82@users.noreply.github.com"},"message":"Merge pull request #14657 from mppf/split-init-param\n\nEnable split init for params\r\n\r\nPR #14564 added split initialization for `var` / `const` variable\r\ndeclarations. This PR continues the effort by enabling the feature for\r\n`param` variable declarations.\r\n\r\n* updated isPrimitiveScalar to include complex (because there are complex\r\n  params and so I'd think of complex as a \"primitive scalar\" type) to\r\n  resolve a function resolution issue. The call site to this function\r\n  could alternatively be adjusted.\r\n* updated the normalizer to allow split init for `param` and in general\r\n  to use `PRIM_INIT_VAR` instead of `PRIM_MOVE` for initializing a\r\n  `param`\r\n* updated `fits_in_mantissa` in function resolution to consider\r\n  non-default-sized integral values\r\n* updated `resolveInitVar` in function resolution to handle initializing\r\n  params and to check that a param is only ever initialized to one value\r\n* fixed a bug in `resolveMoveForRhsCallExpr` with minimal modules and\r\n  removed a no-longer-necessary `USR_FATAL` (that interfered with other\r\n  efforts here)\r\n* updated `resolvePrimInit` to not initialize numeric variables for\r\n  split-init scenarios (for these, init is now handled at\r\n  `PRIM_INIT_VAR_SPLIT_INIT` rather than at `PRIM_INIT_VAR_SPLIT_DECL`)\r\n* updated `postFoldMoveUpdateForParam` to avoid emitting a confusing\r\n  error when initializing a param from an un-coercible type (using some\r\n  less-than-ideal AST matching, but I had trouble coming up with a better\r\n  solution)\r\n* add tests of split init of params and also of some param cases that\r\n  were not working correctly previously\r\n\r\nReviewed by @vasslitvinov - thanks!\r\n\r\n- [x] full local testing","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/7cad6a8881192f48e58143f114b58a60d6d057b4"},{"sha":"c611c04104851dd9af86a5c1ebd6d4c62d6643d9","author":{"name":"Engin Kayraklioglu","email":"c31ea34e2db42d6392821a26e45cb8a6424e83c9@gmail.com"},"message":"Add a needed header for C++ cygwin compilation","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/c611c04104851dd9af86a5c1ebd6d4c62d6643d9"},{"sha":"75231daac15be828fe90eb42b51aa2b6bf1ec899","author":{"name":"Brad Chamberlain","email":"a739c4f8e636da25cffcf7ae77a3ad0067722ef5@cray.com"},"message":"Remove need to suppress slurm/aprun revcomp failures\n\nOur revcomp versions have historically relied on querying the size\nof stdin, which hasn't been supported for slurm/aprun, so those\nconfigurations were suppressed.  In my latest version, I removed\nthe need to query the length of the file, so the suppress should\nhave been removed as well.","distinct":true,"url":"https://api.github.com/repos/e-kayrakli/chapel/commits/75231daac15be828fe90eb42b51aa2b6bf1ec899"}]}
{"actor":{"display_login":"e-kayrakli"}}