{"push_id":2139983648,"size":15,"distinct_size":15,"ref":"refs/heads/openj9","head":"7461a05d7cd08358fe4584882e001d026172e5ab","before":"b562087ed037c48169c352a3f58b0d8769527933","commits":[{"sha":"606b7986f8705e4a60220a82f33a3c9fa4480855","author":{"name":"Ray Banerjee","email":"889722df5e7170db2b63f892d14388263d019c4c@ca.ibm.com"},"message":"Correct ireturn child types in ShiftAndRotate tests\n\nThese tests fail on s390x due to the missing conversions before\nireturn.\n\nThe choice of doing a signed vs unsigned widening (before the `ireturn`)\nis based off of how the oracle is set up right now. From where our IL\nstands today, `ireturn` has no way of deciphering whether the left child\nof the `*shr` (which conventionally would dictate the signedness of the\nresulting shift) was signed or unsigned to begin with. So the “right choice”\nhere, is to mimic how the oracle interprets the resulting bits while\nproducing its result.\n\nSomething to keep mind, in this instance (and for future tests similar to this),\nthe goal is to validate whether the associated Opcode (`*shr` in this case)\nbehaves as expected. `ireturn` and the said conversion before it, are just\npart of \"extracting the computed result”. So the “extraction process” must\nfollow the same convention as the associated Test Oracle for the Test Case\nto be deemed valid.\n\nFixes Issue: #1903\n\nRelated Issues: #1901\n\nSigned-off-by: Rwitaban (Ray) Banerjee <rayb@ca.ibm.com>","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/606b7986f8705e4a60220a82f33a3c9fa4480855"},{"sha":"57318d42c581403c4323464cd598df65638d9dad","author":{"name":"Dhruv Chopra","email":"0bbdc7958ab423ce1d54837d0740b36b20cd9548@ibm.com"},"message":"Fix Compare and Branch helper for unsigned comparisons\n\nUnsigned comparison nodes such as ifbucmpgt can be\nevaluated incorrectly if the data type of the second\nchild is LoadConstant and 8 or 16 bits. We generate a\nsigned comparison instruction instead of an unsigned\ninstruction. This commit fixes that by correctly generating\nan unsigned comparison instruction if the comparison node\nis also unsigned.\n\nCloses: #1943\nSigned-off-by: Dhruv Chopra <Dhruv.C.Chopra@ibm.com>","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/57318d42c581403c4323464cd598df65638d9dad"},{"sha":"6d354e47f2f920725765b4a47855881d4c6d4b03","author":{"name":"Andrew Young","email":"8b9458315ee8f411bb7f4881004cc85233fba02e@gmail.com"},"message":"Set OMR_ENV_DATA64 when configuring build tools\n\nBuild tools require to be configured according to the current host\nplatform. This variable currently defaults to the target platform, which\ncauses cross compilation issues.\n\nissue: #1955\n\nSigned-off-by: Andrew Young <youngar17@gmail.com>","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/6d354e47f2f920725765b4a47855881d4c6d4b03"},{"sha":"7661d156cba8220c9d43fec19646add99535a240","author":{"name":"Shivam Mittal","email":"42e8fab2a77b3f3f466c970fe9d9f14bf9305a9f@gmail.com"},"message":"Remove and fold fixedPointOverflowExceptionEnabled\n\nCloses: #2024\n\nSigned-off-by: Shivam Mittal <shivammittal99@gmail.com>","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/7661d156cba8220c9d43fec19646add99535a240"},{"sha":"b36ee188daf103a3d01cd4da18bea1f0bffc69f6","author":{"name":"Ashutosh Mehra","email":"8e17fc7fbdf9c161cfc0aea10b53385338bec753@in.ibm.com"},"message":"Provide APIs for enabling cgroup subsystems\n\nInstead of providing APIs that enable cgroup limits for all\nsubsystems, a new set of APIs is added which allows to query\nwhich subsystems are available and enable subsystems individually.\nThis would allow runtimes to enable cgroup subsystems based on their\nrequirements.\nNote that there is another subtle change in the design of the\nport library APIs related to cgroup system.\nEnabling a cgroup subsystem in portlibrary (by calling\nomrsysinfo_cgroup_enable_subsystems) would make the port library APIs\nto use cgroup limits internally.\nAny port library API that directly returns cgroup limits,\neg omrsysinfo_cgroup_get_memlimit(), does not check whether the\ncorresponding subsystem (eg memory) is enabled or not.\nTherefore, when using omrsysinfo_cgroup_* APIs,\nthe caller doesn't need to explicitly enable the corresponding\nsubsystem in the port library.\n\nConsider the case of omrsysinfo_get_physical_memory() which returns\nphysical memory of the system.\nIf the system is running in a cgroup, and the cgroup's \"memory\"\nsubsystem is enabled, then omrsysinfo_get_physical_memory()\nwould internally call omrsysinfo_cgroup_get_memlimit(), and\nreturn memory limit imposed by the cgroup.\nHowever, if the user wants to just get the memory limit imposed by the\ncgroup, it can directly call omrsysinfo_cgroup_get_memlimit(),\nwithout enabling the \"memory\" subsystem.\n\nIssue: #1281\nSigned-off-by: Ashutosh Mehra <asmehra1@in.ibm.com>","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/b36ee188daf103a3d01cd4da18bea1f0bffc69f6"},{"sha":"fdb965da295a7619176b19771f0401585cbe5ff2","author":{"name":"Liqun Liu","email":"393864b6562a91991f8a898d0b494be6d31cd419@ca.ibm.com"},"message":"Support folding immutable array shadows\n\nIf we know an array is not going to be changed nor is its content, we\ncan fold the load of its element. This is done by adding a bit in\nKnownObjectTable to indicate whether a known object is an array with\nimmutable content. When VP sees an array shadow load from such array,\nit will improve the array shadow to an ImmutableArrayShadow, and ask\ntransformIndirectLoadChain to fold it. ImmutableArrayShadow is also\nserved as an indicator to transformIndirectLoadChain that it is safe\nto fold an array shadow with such symbol.\n\nSigned-off-by: Liqun Liu <liqunl@ca.ibm.com>","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/fdb965da295a7619176b19771f0401585cbe5ff2"},{"sha":"5f5b6e9b4c45cc427b523bfd442469688fc8bdc1","author":{"name":"Charlie Gracie","email":"0e02fed2408d1fa282d2fba36068cde7cf71e330@gmail.com"},"message":"Merge pull request #1963 from youngar/master\n\nSet OMR_ENV_DATA64 when configuring build tools","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/5f5b6e9b4c45cc427b523bfd442469688fc8bdc1"},{"sha":"8729c25e34dc81609d960f32d61d45f1f137ffff","author":{"name":"Vijay Sundaresan","email":"a38a7ea029bbe13a4f084700f66b52d19903308c@ca.ibm.com"},"message":"Merge pull request #1796 from liqunl/arrayshadow\n\nSupport folding immutable array shadows","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/8729c25e34dc81609d960f32d61d45f1f137ffff"},{"sha":"d94ef6f5164a1dc1145e45d8b5c7a5d4ee9b392c","author":{"name":"Ashutosh Mehra","email":"8e17fc7fbdf9c161cfc0aea10b53385338bec753@in.ibm.com"},"message":"Refactor addCgroupEntry, readCgroupFile and fix couple of issues\n\nList of changes done:\n1. Updated addCgroupEntry() to avoid using PPG_cgroupSubsystemsAvailable\n- Updated readCgroupFile() to compute available subsystems\n- Add check for PPG_cgroupEntryList before calling readCgroupFile()\n- Added comment to explain reason for using global variable\nattachedPortLibraries\n- Macros of type J9_ARE_*_BITS_SET have been replaced by\nOMR_ARE_*_BITS_SET\n- Moved declaration of variable 'rc' such that it is declared only\nfor LINUX platform.\n\nSigned-off-by: Ashutosh Mehra <asmehra1@in.ibm.com>","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/d94ef6f5164a1dc1145e45d8b5c7a5d4ee9b392c"},{"sha":"a36d65e786750b0354c7f7abe538e896a7b9f373","author":{"name":"Charlie Gracie","email":"0e02fed2408d1fa282d2fba36068cde7cf71e330@gmail.com"},"message":"Merge pull request #1424 from ashu-mehra/cgroup_resource_system\n\nProvide APIs for enabling cgroup subsystems","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/a36d65e786750b0354c7f7abe538e896a7b9f373"},{"sha":"cdbc2c37af2f7ac130be972f87e23c5a6e209b69","author":{"name":"Robert Young","email":"430bccaf572ac1c5a5068c1a3741600a2c9a65cb@gmail.com"},"message":"The std::map value pairs have const keys\n\nThe allocator must create std::pairs with constant keys. This is a\ncompilation failure on recent OSX versions.\n\nSigned-off-by: Robert Young <rwy0717@gmail.com>","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/cdbc2c37af2f7ac130be972f87e23c5a6e209b69"},{"sha":"336d09eb3fee4cbcb7f64048209cb5568f3f9f04","author":{"name":"Daryl Maier","email":"654c53488e76faec3e98e87271e048eabb668dc6@ca.ibm.com"},"message":"Merge pull request #2026 from shivammittal99/remove_and_fold\n\nRemove and fold fixedPointOverflowExceptionEnabled","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/336d09eb3fee4cbcb7f64048209cb5568f3f9f04"},{"sha":"02cd2278c9e789ff0a32b2eb5a5f09c2dda95164","author":{"name":"Daryl Maier","email":"654c53488e76faec3e98e87271e048eabb668dc6@ca.ibm.com"},"message":"Merge pull request #2031 from rwy0717/jit-fixes\n\nThe std::map value pairs have const keys","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/02cd2278c9e789ff0a32b2eb5a5f09c2dda95164"},{"sha":"8ec39a2122ce8724a7c72d7fb1acc2741ca3690a","author":{"name":"Daryl Maier","email":"654c53488e76faec3e98e87271e048eabb668dc6@ca.ibm.com"},"message":"Merge pull request #2014 from dchopra001/trilFailures\n\nFix Compare and Branch helper for unsigned comparisons","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/8ec39a2122ce8724a7c72d7fb1acc2741ca3690a"},{"sha":"7461a05d7cd08358fe4584882e001d026172e5ab","author":{"name":"Daryl Maier","email":"654c53488e76faec3e98e87271e048eabb668dc6@ca.ibm.com"},"message":"Merge pull request #1928 from rbanerjee/fix_ShiftAndRotateTest_failures_s390x\n\nCorrect ireturn child types in ShiftAndRotate tests","distinct":true,"url":"https://api.github.com/repos/keithc-ca/omr/commits/7461a05d7cd08358fe4584882e001d026172e5ab"}]}
{"actor":{"display_login":"keithc-ca"}}