{"push_id":2104243078,"size":9,"distinct_size":0,"ref":"refs/heads/openj9","head":"c7f82df5a5f6c82c03b53e0015be58a3464cec5d","before":"6abf600ca9219a2d26e679c4e1eee21c2cdb9173","commits":[{"sha":"9a2a33de93f66bcd420b403e951e471d1714d2b0","author":{"name":"Devin Papineau","email":"6afdd962f27299d2def60e4995a1acf15a126efb@ca.ibm.com"},"message":"Return TR_RegionStructure* from getContainingLoop\n\nWhen it exists, the containing loop is always a region. The previous\ntype signature makes it inconvenient to get the containing loop as\nTR_RegionStructure* in cases where it might not exist, because it isn't\npossible to call asRegion on NULL, necessitating awkward code like this:\n\n    s->getContainingLoop() ? s->getContainingLoop()->asRegion() : NULL\n\nSigned-off-by: Devin Papineau <devinmp@ca.ibm.com>","distinct":false,"url":"https://api.github.com/repos/keithc-ca/omr/commits/9a2a33de93f66bcd420b403e951e471d1714d2b0"},{"sha":"547e63e338d6c4472ef09f164f5d776c92aba58c","author":{"name":"Devin Papineau","email":"6afdd962f27299d2def60e4995a1acf15a126efb@ca.ibm.com"},"message":"Stop versioning unimportant checks in outer loops\n\nUpon entry into a loop, a very low frequency check may not run at all\nbefore control exits the loop. If a check isn't going to run, the loop\ncan complete normally even if that check would fail. As a result,\nversioning the check is likely to spuriously send execution to the cold\nloop. Furthermore, there is little benefit to be had from removing a\ncheck that seldom runs. For these reasons, versioner tries to avoid\nversioning \"unimportant\" checks.\n\nUnimportant checks have so far been identified based on the frequency\nratio between the check and the header of the loop under consideration.\nBut this can cause an inconsistency when an outer loop contains an inner\nloop that contains a check, and the check is invariant w.r.t. to the\nouter loop.\n\nIf the inner loop iterates more frequently than the outer loop, as is\noften the case, then it's possible that an unimportant check within the\ninner loop is too frequent to look unimportant at the level of the outer\nloop. In this case, the entire outer loop, including the inner loop! can\nbe versioned based on the check, even though it would not be considered\nimportant enough to version the inner loop only.\n\nThis commit avoids the inconsistency by considering the headers of any\ninner loops within which a check is nested. To determine whether the\ncheck is unimportant, its frequency is compared to the frequency of the\nhottest header. The original importance test is available under the\nenvironment variable TR_loopVersionerUseOuterHeaderForImportance.\n\nSigned-off-by: Devin Papineau <devinmp@ca.ibm.com>","distinct":false,"url":"https://api.github.com/repos/keithc-ca/omr/commits/547e63e338d6c4472ef09f164f5d776c92aba58c"},{"sha":"24ad22e7c4fce8cfd59ea4777de0e4182862d4a1","author":{"name":"Jon Oommen","email":"435f12a07995dee594701dda95a720c2934a64ae@gmail.com"},"message":"Add CompactScheme delegate to Example/glue\n\nRefactor MM_CollectorLanguageInterface (CLI) methods into a new\nMM_CompactingDelegate (CD) class.\n\nSigned-off-by: Jonathan Oommen <jon.oommen@gmail.com>","distinct":false,"url":"https://api.github.com/repos/keithc-ca/omr/commits/24ad22e7c4fce8cfd59ea4777de0e4182862d4a1"},{"sha":"c25bf2c65a52d8166addf2b5945bd2fc63fac96e","author":{"name":"Dan Heidinga","email":"1b53ca17e96b3d7d7d6d1da523257c93b123ffe0@ca.ibm.com"},"message":"Replace 'J9CONST_TABLE' with 'const'\n\nJ9CONST_TABLE is defined in OMR to be 'const' and only exists for\nhistorical reasons. This makes the code clearer and move obvious\nabout what is occuring.\n\nThis is a mechanical conversion from J9CONST_TABLE -> const.\n\nSigned-off-by: Dan Heidinga <daniel_heidinga@ca.ibm.com>","distinct":false,"url":"https://api.github.com/repos/keithc-ca/omr/commits/c25bf2c65a52d8166addf2b5945bd2fc63fac96e"},{"sha":"699668a9cf9bac6442892d1b323fbb846f746e31","author":{"name":"Andrew Craik","email":"bcb59e842745b45505c664d88bb63e811534eaab@ca.ibm.com"},"message":"arraytranslate StringCoding.implEncodeISOArray\n\nHandling of arraytranslate for ISO-8859-1 encoding from UTF-16 is currently in the\nValuePropagationCommon file. This change adds another recognized method\nto the list of methods supported by this code.\n\nSigned-off-by: Andrew Craik <ajcraik@ca.ibm.com>","distinct":false,"url":"https://api.github.com/repos/keithc-ca/omr/commits/699668a9cf9bac6442892d1b323fbb846f746e31"},{"sha":"2cd23cf0133249de4e7681a27766c45775bd7565","author":{"name":"Vijay Sundaresan","email":"a38a7ea029bbe13a4f084700f66b52d19903308c@ca.ibm.com"},"message":"Merge pull request #1833 from andrewcraik/improveStringCoding\n\narraytranslate StringCoding.implEncodeISOArray","distinct":false,"url":"https://api.github.com/repos/keithc-ca/omr/commits/2cd23cf0133249de4e7681a27766c45775bd7565"},{"sha":"a8ba02e87275df19657768107e49c0b7abe3d011","author":{"name":"Charlie Gracie","email":"0e02fed2408d1fa282d2fba36068cde7cf71e330@gmail.com"},"message":"Merge pull request #1951 from DanHeidinga/djh/const_table\n\nReplace 'J9CONST_TABLE' with 'const'","distinct":false,"url":"https://api.github.com/repos/keithc-ca/omr/commits/a8ba02e87275df19657768107e49c0b7abe3d011"},{"sha":"43b91a0569fbb5248b6a83f5da883786e2d93c04","author":{"name":"Charlie Gracie","email":"0e02fed2408d1fa282d2fba36068cde7cf71e330@gmail.com"},"message":"Merge pull request #1590 from jonoommen/issue-1329-final\n\nAdd CompactScheme delegate to Example/glue","distinct":false,"url":"https://api.github.com/repos/keithc-ca/omr/commits/43b91a0569fbb5248b6a83f5da883786e2d93c04"},{"sha":"c7f82df5a5f6c82c03b53e0015be58a3464cec5d","author":{"name":"Vijay Sundaresan","email":"a38a7ea029bbe13a4f084700f66b52d19903308c@ca.ibm.com"},"message":"Merge pull request #1908 from jdmpapin/versioner-inner-loop-freq\n\nStop versioning unimportant checks in outer loops","distinct":false,"url":"https://api.github.com/repos/keithc-ca/omr/commits/c7f82df5a5f6c82c03b53e0015be58a3464cec5d"}]}
{"actor":{"display_login":"keithc-ca"}}