{"push_id":1411313478,"size":6,"distinct_size":6,"ref":"refs/heads/master","head":"c2c3db3785470c503d9f899ee82d5c29c7171382","before":"c5000812f3ddad8de67c6d56883f896b54680c45","commits":[{"sha":"ba7d34a129374ebf459881f13dda43ca6d0f7956","author":{"name":"Luc des Trois Maisons","email":"2b0ee36cd0c26969b2989b638e7431937d3d24e5@ca.ibm.com"},"message":"Remove Dominators ctor for custom method symbols\n\nThis (mostly duplicated) version of the constructor was used exclusively\nby CFG frequency completion, which was itself removed in b1af0e4 (merge)\n/ 9ac3b56.\n\nSigned-off-by: Luc des Trois Maisons <lmaisons@ca.ibm.com>","distinct":true,"url":"https://api.github.com/repos/eclipse/omr/commits/ba7d34a129374ebf459881f13dda43ca6d0f7956"},{"sha":"f286d4f48af758107b1a8a7a173565da2476a0e4","author":{"name":"Luc des Trois Maisons","email":"2b0ee36cd0c26969b2989b638e7431937d3d24e5@ca.ibm.com"},"message":"Use a deque for _dfNumbers representation\n\n_dfNumbers is currently represented using a CS2::StaticArrayOf. This\ncommit changes its implementation to a TR::deque.\n\nIt is probably worth noting that the existing implementation makes\n(inconsistent) use of the ValueAt indexing function rather than using\nthe indexing operator (operator []) excusively.  After reviewing the\ncode, it appears that this distinction is without material difference,\nand the revised implementation uses deque's indexing operator\nexclusively.\n\nSigned-off-by: Luc des Trois Maisons <lmaisons@ca.ibm.com>","distinct":true,"url":"https://api.github.com/repos/eclipse/omr/commits/f286d4f48af758107b1a8a7a173565da2476a0e4"},{"sha":"cf19e1d2a8d773567ddb918f4cacbc28ff5fce54","author":{"name":"Luc des Trois Maisons","email":"2b0ee36cd0c26969b2989b638e7431937d3d24e5@ca.ibm.com"},"message":"Refactor dominators init using deque for the stack\n\nDominators' initialization does a depth-first walk of successor edges\nusing a local stack to keep track of which edges still need to be\nconsidered.  Currently the stack is implemented using a CS2::ArrayOf\nalong with a top-of-stack index variable.\n\nIn this change set, we replace the ArrayOf with a TR::deque, and in the\nprocess eliminate the index variable in favour of using facilities like\nsize() and empty(). Furthermore, to clarify this functions intended\nstrategy, this change set significantly reduces the redundancy in\ntracing.\n\nSigned-off-by: Luc des Trois Maisons <lmaisons@ca.ibm.com>","distinct":true,"url":"https://api.github.com/repos/eclipse/omr/commits/cf19e1d2a8d773567ddb918f4cacbc28ff5fce54"},{"sha":"63cc05b4ec9353b5525c9c07182e53fd911cd10b","author":{"name":"Luc des Trois Maisons","email":"2b0ee36cd0c26969b2989b638e7431937d3d24e5@ca.ibm.com"},"message":"Use a deque to keep track of BBInfo objects\n\nThe current implementation of the _info container that holds\nTR_Dominator's BBInfo objects uses a CS2::StaticArrayOf.  This\nimplementation replaces the representation with one using a TR::deque.\n\nAdditionally, this change redefines TR_Dominators::BBInfo to be a struct\nrather than a class with all members being public.  This better\nrepresents how this data type is used and should have no effect (beyond\ninconsequential name-mangling changes on platforms that encode the\nclass-key type in mangled names, like MSVC++)\n\nUnlike _dfNumbers, all indexing operations of _info use the indexing\noperator (operator []) and thus, beyond reordering the constructor\nparameters, and changing which function gets called to clear the\ncontainer, this was essentially a drop-in replacement.\n\nSigned-off-by: Luc des Trois Maisons <lmaisons@ca.ibm.com>","distinct":true,"url":"https://api.github.com/repos/eclipse/omr/commits/63cc05b4ec9353b5525c9c07182e53fd911cd10b"},{"sha":"bed8064cfcb1332fcf84b12cd589bc737c6e17e7","author":{"name":"Luc des Trois Maisons","email":"2b0ee36cd0c26969b2989b638e7431937d3d24e5@ca.ibm.com"},"message":"Use a deque for the _dominators container\n\n_dominators is currently represented as a CS2::StaticArrayOf.  This\nchange set replaces its container implmentation with a TR::deque.\n\nThe original implementation makes a semantically consequential use of\nthe CS2::ArrayOf::ValueAt function (which returns a default value for\nindexes that are outside the bounds of the container).  Where that is\nused, this change provides a local implementation to maintain\ncompatibility with its users.\n\nSigned-off-by: Luc des Trois Maisons <lmaisons@ca.ibm.com>","distinct":true,"url":"https://api.github.com/repos/eclipse/omr/commits/bed8064cfcb1332fcf84b12cd589bc737c6e17e7"},{"sha":"c2c3db3785470c503d9f899ee82d5c29c7171382","author":{"name":"vijaysun-omr","email":"a38a7ea029bbe13a4f084700f66b52d19903308c@ca.ibm.com"},"message":"Merge pull request #441 from lmaisons/arrayOf\n\nReplace uses of CS2::ArrayOf and CS2::StaticArrayOf from TR_Dominators","distinct":true,"url":"https://api.github.com/repos/eclipse/omr/commits/c2c3db3785470c503d9f899ee82d5c29c7171382"}]}
{"actor":{"display_login":"vijaysun-omr"}}