{"push_id":928307028,"size":22,"distinct_size":0,"ref":"refs/heads/master","head":"81c9c6e39a7182da47339b25d543692daa6f4d17","before":"a1291429e718578cf73b34ad16d685d72ea0efcb","commits":[{"sha":"21109417b19937e3577b46819a00ea834997c000","author":{"email":"ef88cdf223e82d966d05894857ed10061f9c24e7@microsoft.com","name":"bartonjs"},"message":"Update OS X instructions to include upgrading OpenSSL","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/21109417b19937e3577b46819a00ea834997c000"},{"sha":"5687289d8560e61cfaa6aa39ae02aea435a47cae","author":{"email":"04b66ffb2360c42ea91fdf2fb99d31cafab10efd@microsoft.com","name":"Andy Ayers"},"message":"Add note about attributions and licenses for 3rd-party originated tests","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/5687289d8560e61cfaa6aa39ae02aea435a47cae"},{"sha":"b6ed20b4c0ba3bf877a305c44864a32786e905ea","author":{"email":"04b66ffb2360c42ea91fdf2fb99d31cafab10efd@microsoft.com","name":"Andy Ayers"},"message":"Initial version of the inline tree\n\nExtend `inlExpLst` to to describe the full inlining tree in a method, with a\ncorresponding dump method.\n\nAdd a top-level tree node to serve as the root of the tree. Set this as the\ninline context for all top-level expressions. Update the code that links in new\nnodes when there is a successful inline to build the tree links. Since the\nchild list is extended by a prepend and candidates are currently visited in\nlexical (increasing \"local\" IL offset) order, the child list ends up reversed.\nThe dumper prints the list entries from back to front to compensate so the tree\ndisplay reflects lexical order.\n\nCapture IL offset to help in identifying call sites. Note this is incomplete and\noffets are often missing copies of the parent's offset. Will work on improving\nthis subsequently.\n\nUpdate the depth check to return the depth. This is currently only used in\ndiagnostic messages.\n\nRemove some unused inlining-related data members from the compiler class. Cache\nthe `JitPrintInlinedMethods` config value and use it to dump the tree when\ninlining is done, if there were any inlines.\n\nThe jit traditionally (and probably inadvertently) allowed one level of\nrecursive inlining. Added a test case showing how this can happen.\n\nBecause there is now a top-level inlining context, the recursion check can now\npotentially detect and block these recursive inlines. Support legacy behavior\nfor now by setting `ixlCode` to `nullptr` in the top-level record.\n\nThis change should be no diff.\n\nSample output:\n```\nInlines into Enumerable:Where(ref,ref):ref\n  [IL=3] Error:ArgumentNull(ref):ref\n  [IL=17] Error:ArgumentNull(ref):ref\n  [IL=?] WhereArrayIterator`1:.ctor(ref,ref):this\n    [IL=?] Iterator`1:.ctor():this\n      [IL=?] Object:.ctor():this\n      [IL=?] Environment:get_CurrentManagedThreadId():int\n        [IL=?] Thread:get_CurrentThread():ref\n  [IL=?] WhereListIterator`1:.ctor(ref,ref):this\n    [IL=?] Iterator`1:.ctor():this\n      [IL=?] Object:.ctor():this\n      [IL=?] Environment:get_CurrentManagedThreadId():int\n        [IL=?] Thread:get_CurrentThread():ref\n  [IL=?] WhereEnumerableIterator`1:.ctor(ref,ref):this\n    [IL=?] Iterator`1:.ctor():this\n      [IL=?] Object:.ctor():this\n      [IL=?] Environment:get_CurrentManagedThreadId():int\n        [IL=?] Thread:get_CurrentThread():ref\n```\nSample showing the recursive inline:\n```\nInlines into Fact:factRx(int,int,int,int):int\n  [IL=6] Fact:factRx(int,int,int,int):int\n```","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/b6ed20b4c0ba3bf877a305c44864a32786e905ea"},{"sha":"e7467226a0c552cea5c431233e2a117253d6d22c","author":{"email":"fa77123c115913ca9e97c82fc48a0b47e477b7b4@microsoft.com","name":"Mike McLaughlin"},"message":"Fixed issue #5109.  Console stops echoing typed keys after running a coreclr program.\n\nThe DbgTransportSession/TwoWayPipe code was closing handle 0 (stdout) because the DbgTransportSession class wasn't being zero init'ed via a memset. The TwoWayPipe code initialized the pipe handles to -1.  The fix is to reinitialized the TwoWayPipe by calling the constructor. Even though it is overkill also checked for 0 in the Disconnect code that closes the pipes.","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/e7467226a0c552cea5c431233e2a117253d6d22c"},{"sha":"bfe57c6f88442b4c4b4bc0754443ce61035d24c8","author":{"email":"aa34124753128a778aeec27fbba9844a037e1b54@microsoft.com","name":"Sergiy Kuryata"},"message":"Fix incorrect merge in gc.cpp\n\n@Maoni0 noticed that there was one place in gc.cpp that was incorrectly\nmerged with code from other branches. This change fixes the problem.","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/bfe57c6f88442b4c4b4bc0754443ce61035d24c8"},{"sha":"237af3f0d84dc45ef038b557269c0cb5b8619393","author":{"email":"7d64366f37ef860577e4066c4e0d9bb2c4eaca54@microsoft.com","name":"Hongtao Yu"},"message":"Fix a test bug where dllexport qualifier was missing.","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/237af3f0d84dc45ef038b557269c0cb5b8619393"},{"sha":"d6738b0c6c90efcd6da1a47f85e73378e7caf3b1","author":{"email":"81ee8470e98a47362ed5f670b9c71e9136d75203@microsoft.com","name":"Lubomir Litchev"},"message":"Fix ARM build break (Issue 2362).\n\nRemove unnecessary __cdecl modifier.","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/d6738b0c6c90efcd6da1a47f85e73378e7caf3b1"},{"sha":"be97df9df7a05376aac92d0c034756c73a7b744e","author":{"email":"2ff937dcffb25c52286c693dfb53fb39bfb3b804@microsoft.com","name":"Bruce Forstall"},"message":"Improve buildtest.cmd, runtest.cmd\n\nApply the same kind of changes that were applied to build.cmd:\n1. Improved readability and structure of the code.\n2. Make all three look similar.\n3. Add \"sequential\" and \"msbuildargs\" options\n4. Add better logging and less verbose console output","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/be97df9df7a05376aac92d0c034756c73a7b744e"},{"sha":"1e5934c1d949a93902ac8a00f312020470f709b7","author":{"email":"94274d0b635748ad8df96c0fc4f19e1ecd3f67e2@microsoft.com","name":"Stephen Toub"},"message":"Merge pull request #2561 from mikem8361/fixtransport\n\nFixed issue #5109.  Console stops echoing typed keys after running coreclr.","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/1e5934c1d949a93902ac8a00f312020470f709b7"},{"sha":"71ad0d56e5e59c53840b9137457bc8f962a568ac","author":{"email":"94274d0b635748ad8df96c0fc4f19e1ecd3f67e2@microsoft.com","name":"Stephen Toub"},"message":"Merge pull request #2559 from bartonjs/osx_openssl_requirements\n\nUpdate OS X instructions to include upgrading OpenSSL","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/71ad0d56e5e59c53840b9137457bc8f962a568ac"},{"sha":"d86e600cae07387942c6185dfb77e77938368c28","author":{"email":"202bfdaf89d4929b26015b6e3335e051707b8d9a@microsoft.com","name":"Jan Kotas"},"message":"Merge pull request #2562 from sergiy-k/gccodefix\n\nFix incorrect merge in gc.cpp","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/d86e600cae07387942c6185dfb77e77938368c28"},{"sha":"cdaaabe12a631cb45b86ad304376c560a8d8b4f2","author":{"email":"202bfdaf89d4929b26015b6e3335e051707b8d9a@microsoft.com","name":"Jan Kotas"},"message":"Merge pull request #2560 from AndyAyersMS/TestAttributionNote\n\nAdd note about attributions and licenses for 3rd-party originated tests","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/cdaaabe12a631cb45b86ad304376c560a8d8b4f2"},{"sha":"1061685666b78eecfc3db61d4dafd2b07d221b28","author":{"email":"202bfdaf89d4929b26015b6e3335e051707b8d9a@microsoft.com","name":"Jan Kotas"},"message":"Merge pull request #2567 from LLITCHEV/GetTypeIntrinsic\n\nFix ARM build break (Issue 2362).","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/1061685666b78eecfc3db61d4dafd2b07d221b28"},{"sha":"b6fa55d1e8521b7715a6936243cadb832c58de7c","author":{"email":"c457b3e5195ce3a91c472dc392bcc90fb508bb62@microsoft.com","name":"Jan Vorlicek"},"message":"Fix issue with unlimited stack setting\n\nThis change fixes a problem when a process stack size is set to unlimited.\nIn such case, the main thread stack memory range can shrink during the\nprocess run time. But CoreCLR caches the initial stack limit value and\nuses it to detect whether an object is on stack. Since the stack range is\nnot correct and there can be allocations in the original range of the stack,\nthis check sometimes fails and can cause issues.\n\nThe fix is to remember the stack top at the beginning of the scan in the\nScanContext and use it to check the object address too.","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/b6fa55d1e8521b7715a6936243cadb832c58de7c"},{"sha":"1592b657f495e8948e6642be3df6c0f36c64f77a","author":{"email":"202bfdaf89d4929b26015b6e3335e051707b8d9a@microsoft.com","name":"Jan Kotas"},"message":"Merge pull request #2573 from janvorli/fix-unlimited-stack-issue\n\nFix issue with unlimited stack setting","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/1592b657f495e8948e6642be3df6c0f36c64f77a"},{"sha":"f7a10e319c3323f8dcb89172ffee7dda8cfb365f","author":{"email":"b0f3f51d8d106eafa68beb64af628906d14dc803@gmail.com","name":"Pat Gavlin"},"message":"Merge pull request #2563 from hoyMS/Bug2452\n\nFix for bug 2452.","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/f7a10e319c3323f8dcb89172ffee7dda8cfb365f"},{"sha":"2a52c91242ca3010803e85f5857b977f470a0510","author":{"email":"04b66ffb2360c42ea91fdf2fb99d31cafab10efd@microsoft.com","name":"Andy Ayers"},"message":"Merge pull request #2550 from AndyAyersMS/InlineTree\n\nInitial version of the inline tree","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/2a52c91242ca3010803e85f5857b977f470a0510"},{"sha":"214eef76c70a0b8dcabb4b31f25a080c0633ef2a","author":{"email":"0b25057bc7b68bd418a09f6f1a8c477e8d42892f@microsoft.com","name":"Brian Sullivan"},"message":"Fix Issure 2286 - RyuJIT needs to honor volitile reads/writes and avoid making CSE's across them\n\nIn Liveness.cpp we will now treat a volatile accesses as definitions of the global heap\nAssertions are also added in ValueNumber.cpp such that a block that defines or mutates the global heap\nmust have been marked as cotailing a definition of the global heap using the bbHeapDef flag.","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/214eef76c70a0b8dcabb4b31f25a080c0633ef2a"},{"sha":"680c1f68c44068f181bc65b290454ccfb14f7c16","author":{"email":"2ff937dcffb25c52286c693dfb53fb39bfb3b804@microsoft.com","name":"Bruce Forstall"},"message":"Merge pull request #2570 from BruceForstall/BuildCmdCleanup\n\nImprove buildtest.cmd, runtest.cmd","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/680c1f68c44068f181bc65b290454ccfb14f7c16"},{"sha":"ef74f53fe9ff2a6e563c52b63f7a9ec8e0dd95bd","author":{"email":"202bfdaf89d4929b26015b6e3335e051707b8d9a@microsoft.com","name":"Jan Kotas"},"message":"Enable tailcallopt for CoreRT\n\ntailcallopt was disabled for ReadyToRun compilation mode because of zapper limitation. Moved the block to the zapper so that CoreRT can take advantage of it.","distinct":false,"url":"https://api.github.com/repos/RussKeldorph/coreclr/commits/ef74f53fe9ff2a6e563c52b63f7a9ec8e0dd95bd"}]}