)]}' { "log": [ { "commit": "1fbd4520f9067439f09b98753b0c7c221930aa76", "tree": "5118f170e867eae6625f18e2743e0e9497e43f06", "parents": [ "0339bee5b1be39669606c4ba57edd42a0bacf0ff" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Mar 22 15:52:22 2016 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Mar 22 15:52:22 2016 +0000" }, "message": "Fix missing header inclusion in art/runtime/thread.cc.\n\nThe file art/runtime/thread.cc was missing an #include of\nart/runtime/gc/accounting/heap_bitmap-inl.h, which is\nrequired by art::ReferenceMapVisitor::VisitDeclaringClass to\nfind the (inlined) implementation of\nart::gc::accounting::HeapBitmap::Test.\n\nThis fixes the MIPS32 build.\n\nChange-Id: I00a8bc85d8b32cfed02309e13e437f0cfd0d493e\n" }, { "commit": "d5f3f32bc0ca729cf4bbf7975525dc921ce94678", "tree": "6ba23a67137d031113e17bbfcacdf2d16fed7645", "parents": [ "9060376f38cb03353c82ddf15844156414249b19" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Mar 21 14:05:56 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Mar 21 20:08:01 2016 -0700" }, "message": "Add sanity checking to declaring class visiting\n\nWhen we have an unmarked declaring class in the image, it will\nSIGSEGV due to the bitmap being read-only. Print some useful\ninfo to try and debug this.\n\nThis probably hurts performance a bit, TODO disable when the bug is\nfixed.\n\nBug: 27493510\n\n(cherry picked from commit b33b1dc78d178060ff0f4327d448fd3a89df51de)\n\nChange-Id: I0c25afd5a6101a71fa1494e12dc019ba64bf43be\n" }, { "commit": "48a7bae220c823f03f53a58043845c2e58860b22", "tree": "c0eeabe3d13ebc6bc22211f110d11eb2775ef47c", "parents": [ "459898dc4470559ba1e1d578bc52a914d1f573f5" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Mar 21 14:09:54 2016 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Mar 21 14:09:54 2016 +0000" }, "message": "Pacify g++ about a set-but-unused variable.\n\nThis fixes the MIPS32 build.\n\nChange-Id: Ic1e6d4627d69a3701c2f3e8fdd784d97a4e41ba5\n" }, { "commit": "2c2d2a05eaf81d07df27418f8dfd68de6fa28ac1", "tree": "f7d49c6caf3e3ed1fa1d59699c00b4ede6921b2f", "parents": [ "db1f7dac02f6dcecac3e032f10abbcdbf3cf4331" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Mar 17 21:27:19 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Mar 18 10:32:25 2016 -0700" }, "message": "ART: Speed up stack guard page install\n\nOnly the main thread doesn\u0027t have its stack mapped in under normal\nconditions. Reading each page is a lot of overhead and we should\ntry to avoid it.\n\nRewrite to first try a (non-fatal) protect. If the outcome is a\nsuccess, finish. Otherwise do the stack mapping, and try again.\n\nBug: 27718174\nChange-Id: I16b214567585ed2f09970f618ccdec7eed219fd3\n" }, { "commit": "32ce2adefb8a3d0eda59a29f5e87c1eb43eef796", "tree": "50aaa40b31ede355359765259edc7989485cc7d5", "parents": [ "c71dc3852edfa8e0cd2083d12845ff8607e189ff" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 04 14:58:03 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 04 18:37:19 2016 -0800" }, "message": "Add more systracing everywhere\n\nAdded to:\nJIT\nDexFile functions\nOat file manager\n\nAdded helper ScopedTrace to prevent errors and reduce excess code.\n\nBug: 27502458\n\n(cherry picked from commit dabdc0fe183d4684f3cf4d70cb09d318cff81b42)\n\nChange-Id: Ifaeff8913d79eefc797380987d13cc00456266f8\n" }, { "commit": "a73280df8cac1279b6dea0424722f42ef0048613", "tree": "22adc28cc88bc440b8e28a13a389c3a3c2b22341", "parents": [ "47a2a45a6673ddf3322115ff5058763f82a9368f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 15 13:05:16 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 15 16:24:56 2016 +0000" }, "message": "Add an option to disable native stack dumping on SIGQUIT.\n\nSome of our (stress) run-tests do ANR dumping, which end up\nstressing libunwind, that has known problems. To avoid getting\nflakes due to libunwind, disable native stack dumping on SIGQUIT\nfor our run-tests.\n\nbug:27185632\nbug:24664297\nChange-Id: I69085e48db903d6240448d71666ae2dcd091922e\n" }, { "commit": "e6141d7aba1078c0465294408133aaf00ae47f1b", "tree": "159b05407974c46ac39f0be10912f16bc910aeff", "parents": [ "f3c35954350ab3386cff538a123bc1b8e78e89f0", "7ed9c561048d79083b6d0576c71a986a3123bca6" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Feb 05 22:35:25 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Feb 05 22:35:25 2016 +0000" }, "message": "Merge \"Use 8-byte increment bracket sizes for rosalloc thread local runs.\"" }, { "commit": "7ed9c561048d79083b6d0576c71a986a3123bca6", "tree": "8e352f05dc5e82126dd61e0e8e6ddb2a1003e16a", "parents": [ "c3cf1d2e35a45bd2e2c60fbc9c2a1a6a56dfc529" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Feb 02 15:22:09 2016 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Feb 05 12:12:44 2016 -0800" }, "message": "Use 8-byte increment bracket sizes for rosalloc thread local runs.\n\nVery small space savings (\u003c 1%) after device boot and up to 10%\nallocation speedup.\n\nSome minor cleanup.\n\nBug: 9986565\n\nChange-Id: I51d791c4674d6944fe9a7ee78537ac3490c1a02c\n" }, { "commit": "b331febbab8e916680faba722cc84b66b84218a3", "tree": "35f985b021e476914bfe91492da23fee218014a7", "parents": [ "586996afc905518ed926e4680aab67bedabec9b7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 05 16:51:53 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 05 17:11:54 2016 +0000" }, "message": "Revert \"Revert \"Implement on-stack replacement for arm/arm64/x86/x86_64.\"\"\n\nThis reverts commit bd89a5c556324062b7d841843b039392e84cfaf4.\n\nChange-Id: I08d190431520baa7fcec8fbdb444519f25ac8d44\n" }, { "commit": "e7f75f35371cf32954d94f9bb10fc96924f7edb4", "tree": "4968ea7f5dd02786e50bf4dc1873c30ff37cde8a", "parents": [ "c3cf1d2e35a45bd2e2c60fbc9c2a1a6a56dfc529" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 01 16:08:15 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 01 16:19:59 2016 -0800" }, "message": "Add read barrier option to UpdateEntrypoints\n\nAlso call this without read barrier for image relocation.\n\nBug: 26786304\nChange-Id: Ia718c1491b54cadb7283c62afc1eb9031a15b4ef\n" }, { "commit": "1452bee8f06b9f76a333ddf4760e4beaa82f8099", "tree": "8ffb80498cd8fdff66a4ab272723cf4d4e69e57b", "parents": [ "7f9ea1451bb74055ee5d51ae04fc869faa06cbdd" ], "author": { "name": "buzbee", "email": "buzbee@google.com", "time": "Fri Mar 06 14:43:04 2015 -0800" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Mon Jan 11 13:04:49 2016 -0800" }, "message": "Fast Art interpreter\n\nAdd a Dalvik-style fast interpreter to Art.\nThree primary deficiencies in the existing Art interpreter\nwill be addressed:\n\n1. Structural inefficiencies (primarily the bloated\n fetch/decode/execute overhead of the C++ interpreter\n implementation).\n2. Stack memory wastage. Each managed-language invoke\n adds a full copy of the interpreter\u0027s compiler-generated\n locals on the shared stack. We\u0027re at the mercy of\n the compiler now in how much memory is wasted here. An\n assembly based interpreter can manage memory usage more\n effectively.\n3. Shadow frame model, which not only spends twice the memory\n to store the Dalvik virtual registers, but causes vreg stores\n to happen twice.\n\nThis CL mostly deals with #1 (but does provide some stack memory\nsavings). Subsequent CLs will address the other issues.\n\nCurrent status:\n Passes all run-tests.\n Phone boots interpret-only.\n 2.5x faster than Clang-compiled Art goto interpreter on fetch/decode/execute\n microbenchmark, 5x faster than gcc-compiled goto interpreter.\n 1.6x faster than Clang goto on Caffeinemark overall\n 2.0x faster than Clang switch on Caffeinemark overall\n 68% of Dalvik interpreter performance on Caffeinemark (still much slower,\n primarily because of poor invoke performance and lack of execute-inline)\n Still nearly an order of magnitude slower than Dalvik on invokes\n (but slightly better than Art Clang goto interpreter.\n Importantly, saves ~200 bytes of stack memory per invoke (but still\n wastes ~400 relative to Dalvik).\n\nWhat\u0027s needed:\n Remove the (large quantity of) bring-up hackery in place.\n Integrate into the build mechanism. I\u0027m still using the old Dalvik manual\n build step to generate assembly code from the stub files.\n Remove the suspend check hack. For bring-up purposes, I\u0027m using an explicit\n suspend check (like the other Art interpreters). However, we should be\n doing a Dalvik style suspend check via the table base switch mechanism.\n This should be done during the alternative interpreter activation.\n General cleanup.\n Add CFI info.\n Update the new target bring-up README documentation.\n Add other targets.\n\nIn later CLs:\n Consolidate mterp handlers for expensive operations (such as new-instance) with\n the code used by the switch interpreter. No need to duplicate the code for\n heavyweight operations (but will need some refactoring to align).\n Tuning - some fast paths needs to be moved down to the assembly handlers,\n rather than being dealt with in the out-of-line code.\n JIT profiling. Currently, the fast interpreter is used only in the fast\n case - no instrumentation, no transactions and no access checks. We\n will want to implement fast + JIT-profiling as the alternate fast\n interpreter. All other cases can still fall back to the reference\n interpreter.\n Improve invoke performance. We\u0027re nearly an order of magnitude slower than\n Dalvik here. Some of that is unavoidable, but I suspect we can do\n better.\n Add support for our other targets.\n\nChange-Id: I43e25dc3d786fb87245705ac74a87274ad34fedc\n" }, { "commit": "7c1559a06041c9c299d5ab514d54b2102f204a84", "tree": "c50b54bf82f457f44dbf0741947d836749d4a96a", "parents": [ "7cd230c8f74a227ea04f1dd93c8d855aa81fc1fe" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Dec 15 10:55:36 2015 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Dec 15 11:45:56 2015 +0000" }, "message": "x86 Baker\u0027s read barrier fast path implementation.\n\nIntroduce an x86 fast path implementation in Optimizing for\nBaker\u0027s read barriers (for both heap reference loads and GC\nroot loads). The marking phase of the read barrier is\nperformed by a slow path, invoking a new runtime entry point\n(artReadBarrierMark).\n\nOther read barrier algorithms continue to use the original\nslow path based implementation, which has been renamed as\nGenerateReadBarrierSlow/GenerateReadBarrierForRootSlow.\n\nBug: 12687968\nChange-Id: Ie610c4befc19ff22378a8cba38b422dcacb54320\n" }, { "commit": "a4f1220c1518074db18ca1044e9201492975750b", "tree": "796f0dcd733ad5ffd7d8310af871f3e2c22ee0be", "parents": [ "763fd2d3d131898cad6295a19ae9a30e22ce5f2a" ], "author": { "name": "Mark Mendell", "email": "mark.p.mendell@intel.com", "time": "Thu Aug 06 15:23:34 2015 -0400" }, "committer": { "name": "Mark Mendell", "email": "mark.p.mendell@intel.com", "time": "Thu Dec 10 13:23:49 2015 -0500" }, "message": "Optimizing: Add direct calls to math intrinsics\n\nSupport the double forms of:\n cos, sin, acos, asin, atan, atan2, cbrt, cosh, exp, expm1,\n hypot, log, log10, nextAfter, sinh, tan, tanh\n\nAdd these entries to the vector addressed off the thread pointer. Call\nthe libc routines directly, which means that we have to implement the\nnative ABI, not the ART one. For x86_64, that includes saving XMM12-15\nas the native ABI considers them caller-save, while the ART ABI\nconsiders them callee-save. We save them by marking them as used by the\ncall to the math function. For x86, this is not an issue, as all the XMM\nregisters are caller-save.\n\nOther architectures will call Java as before until they are ready to\nimplement the new intrinsics.\n\nBump the OAT version since we are incompatible with old boot.oat files.\n\nChange-Id: Ic6332c3555c09393a17d1ad4daf62932488722fb\nSigned-off-by: Mark Mendell \u003cmark.p.mendell@intel.com\u003e\n" }, { "commit": "73be1e8f8609708f6624bb297c9628de44fd8b6f", "tree": "47a22745797a345dea8dd846aad00334da0d5bf1", "parents": [ "47229aa5848df7d45578dbdd9285f57dfa9399a8" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Sep 17 15:22:56 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Dec 04 12:08:02 2015 +0000" }, "message": "Inline monomorphic calls.\n\nChange-Id: If38171c2dc7d4a4378df5d050afc4fff4499c98f\n" }, { "commit": "0d5a281c671444bfa75d63caf1427a8c0e6e1177", "tree": "fd9bbe0f1c581bcc7c05bbfb2643ffe0b1fb014e", "parents": [ "dd4cbcc924c8ba2a578914a4a366996693bdcd74" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Nov 13 10:07:31 2015 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Sun Nov 15 12:16:41 2015 +0000" }, "message": "x86/x86-64 read barrier support for concurrent GC in Optimizing.\n\nThis first implementation uses slow paths to instrument heap\nreference loads and GC root loads for the concurrent copying\ncollector, respectively calling the artReadBarrierSlow and\nartReadBarrierForRootSlow (new) runtime entry points.\n\nNotes:\n- This implementation does not instrument HInvokeVirtual\n nor HInvokeInterface instructions (for class reference\n loads), as the corresponding read barriers are not stricly\n required with the current concurrent copying collector.\n- Intrinsics which may eventually call (on slow path) are\n disabled when read barriers are enabled, as the current\n slow path infrastructure does not support this case.\n- When read barriers are enabled, the code generated for a\n HArraySet instruction always go into the array set slow\n path for object arrays (delegating the operation to the\n runtime), as we are lacking a mechanism to keep a\n temporary register live accross a runtime call (needed for\n the instrumentation of type checking code, which requires\n two successive read barriers).\n\nBug: 12687968\nChange-Id: I14cd6107233c326389120336f93955b28ffbb329\n" }, { "commit": "04302dbb106d590ff72c0dfecda23d85b6565059", "tree": "2fe66e258de36bcebc913d48e04468b7c639578f", "parents": [ "0e06a0989f4fc53e0d281a5a2cb5b5a17feaea0c" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Nov 11 23:45:34 2015 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Nov 13 15:21:29 2015 -0800" }, "message": "Fix class unloading with the CC collector.\n\nAvoid unnecessarily decoding dex cache and class loader weak roots,\nwhich would trigger read barriers.\n\nRe-enable 141-class-unload with the CC collector.\n\nBug: 12687968\nBug: 24468364\nChange-Id: Ib4c19f25000873cab0e06047040442d135285745\n" }, { "commit": "6cff48f05894a60aa4742472983a75504536ca15", "tree": "bd399d46ef08cd836010f3119c8c5db054561267", "parents": [ "823e693aa946ba75cd047429e1290011a2ed8729" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Sun Jan 26 21:36:13 2014 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 23 11:16:11 2015 -0700" }, "message": "ART: Speed up thread stack tracing for ANRs\n\nShare a BacktraceMap for all threads being dumped, which speeds up\nANR dumping. Results from Nexus 9 (average of five):\n\nBefore: 0.587s\nAfter: 0.206s\n\nChange-Id: Ia70e0dbd39049318c02de561e7b95258d4849467\n" }, { "commit": "524e7ea8cd17bad17bd9f3e0ccbb19ad0d4d9c02", "tree": "ad07cc96f633bdae839ff2f1553d84b9c864a930", "parents": [ "b697c447eb61c2e14315166ec3b0d16375ae403c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Oct 16 17:13:34 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 20 11:52:11 2015 +0100" }, "message": "Remove ArtCode.\n\n- Instead use OatQuickMethodHeader.\n- Various cleanups now that we don\u0027t have all those\n ArtMethod -\u003e ArtCode -\u003e OatQuickMethodHeader indirections.\n\nAs a consequence of this cleanup, exception handling got a bit\nfaster.\n\nParserCombinators benchmark (exception intensive) on x64: (lower is better)\nBefore:\nParserCombinators(RunTime): 1062500.0 us.\nAfter:\nParserCombinators(RunTime): 833000.0 us.\n\nChange-Id: Idac917b6f1b0dc254ad68fb3781cd61bccadb0f3\n" }, { "commit": "6bc4374e3fa00e3ee5e832e1761c43e0b8a71558", "tree": "38118523aa6b9a92fbdcd7bf1d32ca6185a3d3f0", "parents": [ "16065ce56394c73c87dcb78ead4164ddc80fddb3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Oct 12 18:11:10 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 13 12:05:27 2015 +0100" }, "message": "Add an abstraction over a compiled code.\n\nThat\u0027s just step 1, moving code-related functions of ArtMethod to\nanother class. That class is only a wrapper on an ArtMethod, but will\nbe changed to be a wrapper around compiled code.\n\nChange-Id: I6f35fc06d37220558dff61691e51ae20066b0dd6\n" }, { "commit": "da458691d0d8023dee973f5b04633e243219c311", "tree": "d5776d071548ab4ec4322f4315d9c513cadf7d90", "parents": [ "02280e6d97e4ed51f149b9863a5e382111559442", "03ec930faded5bbfa096533ce60b6893847922db" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Oct 12 18:02:51 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Oct 12 18:02:51 2015 +0000" }, "message": "Merge \"ART: Lock counting in the interpreter\"" }, { "commit": "03ec930faded5bbfa096533ce60b6893847922db", "tree": "085ec76235f72647cd2e083801f15d82394e038b", "parents": [ "14412079e6eba4d16f40c0bdc5e94812e1c7f5bc" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Aug 27 17:41:47 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 09 16:32:13 2015 -0700" }, "message": "ART: Lock counting in the interpreter\n\nTo support structured locking when balanced-locking verification\nfailed, count lock and unlock operations in the verifier.\n\nBug: 23502994\nChange-Id: I2bb915da6e3d43c49723a943b42d4d5a7c939aa1\n" }, { "commit": "4b1d4ec525448302556e4c2b601b4dbb8e5e294e", "tree": "5f87db96e8eb205b4880a923b7cd8c27a8073ea8", "parents": [ "526ec078c7e4ace31193d7f49ded661ea0929292" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Oct 01 16:47:26 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Oct 08 11:16:06 2015 -0700" }, "message": "ART: Do not abort on exception in CreatePeer\n\nDifferent parts of CreatePeer may throw an exception, especially\nthe Thread constructor. Do not abort in such a case, but return\nand report a failure to attach/create a thread.\n\nBug: 24200698\n\n(cherry picked from commit 2a196784553f4fd0c0f7d4b8aac87281db3a4748)\n\nChange-Id: I06f2c997f0451c71f791d1f12bea6f8ee65e8ab2\n" }, { "commit": "3cfa4d05afa76e19ca99ec964b535a15c73683f0", "tree": "28c4950dff3700f69e3d32fc496890a722cfc379", "parents": [ "f4156986d1ba5b6dd7dfe89754c8562557cd1bc5" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 06 17:04:01 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 07 09:50:25 2015 -0700" }, "message": "ART: Remove interpreter entrypoints\n\nThese are no longer used as entrypoints. Make them proper runtime\nfunctions local to the interpreter.\n\nBump oat version.\n\nChange-Id: Icdd92e20eafc5668b68eeebf55cf624560f462b3\n" }, { "commit": "910e8275440dd3b70f2e8f6c473ba1707c00261d", "tree": "42b98611f84698fefd954d7b60e7370a0521d047", "parents": [ "6387821209a03c5d873cf9dc6fd11434918bbdf4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 30 09:24:22 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 30 10:16:56 2015 -0700" }, "message": "Revert \"Revert \"Keep internal stack trace methods declaring classes live\"\"\n\nFixed some code which used old stack trace representation in throwable.cc.\n\nBug: 22720414\n\nThis reverts commit 45a8522898702f6a725ae19d97bceedc8fc609a6.\n\nChange-Id: I7385075caf649ce76da5d8866fd0031b9e8e5ef4\n" }, { "commit": "45a8522898702f6a725ae19d97bceedc8fc609a6", "tree": "e37a48f0f06a60271c02c4175cfbe8d1b409e71d", "parents": [ "566fc926942ed57925627d1bd43e099938084f1e" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Sep 30 07:20:13 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Sep 30 07:20:13 2015 +0000" }, "message": "Revert \"Keep internal stack trace methods declaring classes live\"\n\nBreaks jdwp tests.\n\nBug: 22720414\n\nThis reverts commit 566fc926942ed57925627d1bd43e099938084f1e.\n\nChange-Id: Ic3544e86af35bf748f9d47e0a4077dcdc206a821\n" }, { "commit": "566fc926942ed57925627d1bd43e099938084f1e", "tree": "c7551ecbfbaa82ec37a755b3aff53fa24253d64a", "parents": [ "0766e3ff5638b14d6b3ea6157c800eed11be5b98" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 29 10:21:55 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 29 18:11:12 2015 -0700" }, "message": "Keep internal stack trace methods declaring classes live\n\nWe now store the declaring classes in the internal stack trace to\nmake sure class unloading doesn\u0027t unload any classes owning methods\nin the stack trace.\n\nThis fixes DexClassLoaderTest in libcore. Added regression test.\n\nBug: 22720414\n\nChange-Id: I185f87c8ec0807e83f4661bd5bb5652dba6fc281\n" }, { "commit": "639bdd13993644a267f177f8f5936496bda65e2b", "tree": "fc68e344355ab1c0cfef4d47dd8f2cbc9fc7e022", "parents": [ "b175a45be78152606a1db1eac0b79ef8777644f5" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 03 11:22:45 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 28 15:37:28 2015 -0700" }, "message": "ART: Single-frame deopt\n\nAdd deoptimization of a single frame. Works by removing the managed\ncode frame and jumping into the quick-to-interpreter bridge, and\nthe bridge understanding a stored ShadowFrame.\n\nWe need a separate fixup pass. For x86, we leave the return address\non the stack so we don\u0027t need to push it there.\n\nBug: 21611912\nChange-Id: I06625685ced8b054244f8685ab50b238a705b9d2\n" }, { "commit": "99170c636dfae4908b102347cfe9f92bad1881cc", "tree": "1ecff5dd69ea20b09e622cb88e7835a55dfa4be8", "parents": [ "5836f8465ddd0a7f5cbeec9c30d739bd0619867f" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Jul 06 11:10:37 2015 -0700" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Sep 17 17:07:22 2015 +0200" }, "message": "Deoptimization support in optimizing compiler for setting local values\n\nDue to compiler optimizations, we may not always be able to update\nthe value of a local variable in a compiled frame (like a variable\nseen as constant by the compiler). To avoid that situation, we simply\ndeoptimize compiled frames updated by the debugger so they are\nexecuted by the interpreter with the updated value.\n\nWhen the debugger attempts to set a local variable (actually a DEX\nregister or a pair of registers) in a compiled frame, we allocate a\nShadowFrame associated to that frame (using its frame id) and set the\nnew value in that ShadowFrame. When we know we are about to continue\nthe execution of the compiled frame, we deoptimize the stack using\nthe preallocated ShadowFrame (instead of creating a new one). We\ninitialize it with the current value of all DEX registers except\nthe ones that have been set by the debugger. Therefore, the\nShadowFrame represent the runtime context modified by the debugger.\n\nBumps oat version to force recompilation.\n\nBug: 19944235\nChange-Id: I0ebe6241264f7a3be0f14ee4516c1f7436e04da6\n" }, { "commit": "26f728661a08062a373a3203b72dc2555c2aed2d", "tree": "4f6e8168ac7ca422c7de3c7cdda6db21958358fd", "parents": [ "b6f7dd330f115fc977d1d5a10122c41c9dd7c210" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 15 09:52:07 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 15 11:15:12 2015 +0200" }, "message": "Cleanup thread access in StackVisitor\n\nAdds method StackVisitor::GetThread to give access to the visited\nThread* so we no longer need to copy that pointer in subclasses.\n\nAlso adds a few missing const and DISALLOW_COPY_AND_ASSIGN.\n\nChange-Id: I57649ee7742ef4ef1e01447ac2fbb66f977b22eb\n" }, { "commit": "520633bebd2bf4d70884d30f179dbde9f275aac6", "tree": "0006fb9db71545d9aaa88df3952a7488b30b2ae6", "parents": [ "e0a49e3a93aa54b9e603d797d9e925a98742753a" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 08 17:03:36 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Sep 10 17:39:56 2015 +0200" }, "message": "Support deoptimization on exception\n\nAllows to deoptimize when an exception is being thrown. We only\ndeoptimize if an executable frame (starting from the catch handler)\nneeds to be executed with the interpreter.\n\nBefore executing deoptimized frames, the exception is restored. The\ninterpreter starts by handling this exception at the point of the\nthrowing instruction.\n\nBug: 23714835\nChange-Id: I0c5f7d4b257644acf12210aae8e5b6bb0f4af1f7\n" }, { "commit": "f1d666e1b48f8070ef1177fce156c08827f08eb8", "tree": "772cbbe6652b32ea2be31532156295db27579390", "parents": [ "897ce64153e4758663e464fb5fb339f9a7b496c8" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 03 16:13:34 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 03 17:33:19 2015 -0700" }, "message": "Add ScopedThreadSuspension\n\nFixes the TransitionFromRunnableToSuspended and\nTransitionFromSuspendedToRunnable pattern that was prone to errors.\n\nChange-Id: Ie6ae9c0357c83b4fc4899d05dfa0975553170267\n" }, { "commit": "30b5e27083913bb711fca0ca89a941797fcf3d5d", "tree": "c0b5229955f580d8ef928433c5e7bc372c70c3aa", "parents": [ "b52cfcb3868d08102b4c971974a7b1443861a90c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 01 11:14:34 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 01 19:46:40 2015 -0700" }, "message": "Enable lockless decoding of weak globals\n\nWill help speed up decoding weak DexCache roots.\n\nChange-Id: I9a68beb4106cbd383111a30e249c9b0149064e78\n" }, { "commit": "0747466fca310eedea5fc49e37d54f240a0b3c0f", "tree": "6d27922fa35330dd8d863ea56c82c5f04b711822", "parents": [ "9ee5d6cdc14ac94b64ea1961bf221bad48746929" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:12:33 2015 +0000" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Aug 26 16:54:32 2015 +0200" }, "message": "Revert \"Revert \"Fix deoptimization with pending exception\"\"\n\nThis reverts commit 6e2d5747d00697a25251d25dd33b953e54709507.\n\nFixes the deoptimization path from compiled code (generated by the\nOptimizing compiler) by adding wrapper artDeoptimizeFromCompiledCode.\nThis wrapper, called through the matching assembler stub\nart_quick_deoptimize_from_compiled_code, pushes the deoptimization\ncontext just before deoptimizing the stack.\n\nBug: 23371176\nBug: 19944235\nChange-Id: Ia7082656998aebdd0157438f7e6504c120e10d3e\n" }, { "commit": "6e2d5747d00697a25251d25dd33b953e54709507", "tree": "b68a418e93780ed55abd79dc8e76f3459e884254", "parents": [ "54b62480636ae846d705fc180c7bd6cd08ec1e42" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:05:17 2015 +0000" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:05:17 2015 +0000" }, "message": "Revert \"Fix deoptimization with pending exception\"\n\nThis reverts commit 54b62480636ae846d705fc180c7bd6cd08ec1e42.\n\nThis is causing test failures with Optimizing compiler.\n\nBug: 23371176\nBug: 19944235\nChange-Id: Ie3ffbcf2b6d2ca8bc93cb008a4e29a7567d04a7c\n" }, { "commit": "54b62480636ae846d705fc180c7bd6cd08ec1e42", "tree": "d46a6c7bebb45ddbbd5b0a6818b3bb2ce49f48ed", "parents": [ "f0170ded84fbcd3d9152cdbcf9ffbf74c71b03ba" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Aug 20 12:07:57 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:13:49 2015 +0200" }, "message": "Fix deoptimization with pending exception\n\nWhen deoptimizing the stack, we set a fake exception in the current\nThread* (see method Thread::GetDeoptimizationException). On the next\nexception check, the QuickExceptionHandler will deoptimize the stack.\n\nThe issue is when we deoptimize while an exception is already pending\nin the current Thread*: setting the fake exception will clobber the\npending exception which is not correct. This happens in the\nartQuickToInterpreterBridge when returning from the interpreter and\nwe want to deoptimize the stack for debugging (like single-stepping).\n\nThis CL saves the pending exception before asking for deoptimization.\nThen the exception is restored just before executing the deoptimized\nframes with the interpreter.\n\nAlso cleans up the way we save deoptimization context (return value\nand pending exception).\n\nBug: 23371176\nBug: 19944235\nChange-Id: I7f4c8347b328817c452beda3399e210eba3a88a4\n" }, { "commit": "0e780e48aca3f6b5aef12659362d6f6adb3e2da0", "tree": "cd7f7ae78c186cb4e6b6eae0b987178b8cce7381", "parents": [ "060841a3966afc7a5251134d1f1ef100849b1468", "10e5ea90e20375e5ad2ff26cb760c0e7c586cc16" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 14 01:37:58 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Aug 14 01:37:58 2015 +0000" }, "message": "Merge \"Visit declaring class of methods on call stack\"" }, { "commit": "10e5ea90e20375e5ad2ff26cb760c0e7c586cc16", "tree": "45cc204db4d61e729cb8bcea681e7efa758d7bfb", "parents": [ "b923298c9b6e47b4110dd0cb23cb6a4e98b8552b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Aug 13 12:56:31 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Aug 13 17:59:00 2015 -0700" }, "message": "Visit declaring class of methods on call stack\n\nBug: 22720414\nChange-Id: Iab9727dde243d76fd9dfdb1db73899fedd1ab6ea\n" }, { "commit": "3887c468d731420e929e6ad3acf190d5431e94fc", "tree": "67dacb849e722e33e118b97714a48e467c06cbd5", "parents": [ "6a5037eb3340e4c981fd7de3ff45167ee5b7fc82" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Aug 12 18:15:42 2015 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Aug 12 18:15:42 2015 +0100" }, "message": "Remove unnecessary `explicit` qualifiers on constructors.\n\nChange-Id: Id12e392ad50f66a6e2251a68662b7959315dc567\n" }, { "commit": "94800f46da0aded5152545322ad842f89e8a21d8", "tree": "fa17558ade66567e1a6b512db578344731dd146c", "parents": [ "46e0f460aa233a4a0ac161bfafde164357fe2b26", "90ef3db4bd1d4865f5f9cb95c8e7d9afb46994f9" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 05 18:02:53 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Aug 05 18:02:53 2015 +0000" }, "message": "Merge \"Address some comments and clean up\"" }, { "commit": "4382f1ef6ab2d6992d330cc8b43057d9139a1243", "tree": "dbea9b43e11a0253a1eae59b1a6db21c5cd6d9ea", "parents": [ "1cf85515600d298a2c9e4de137c0926949205097" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Aug 05 01:08:53 2015 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 04 19:11:29 2015 -0700" }, "message": "Revert \"Revert \"ART: Use bionic TLS slot for thread-self\"\"\n\nThis reverts commit 50832b6804d15ea5ddfe99a2753a09111b5bfe37.\n\nFix clearing the TLS on detach.\n\nChange-Id: I75ed5ccccb4f31800e5f893055e96067a9695258\n" }, { "commit": "90ef3db4bd1d4865f5f9cb95c8e7d9afb46994f9", "tree": "3c6f8989e0c6e027382a3c08d221ba97afd72291", "parents": [ "1cf85515600d298a2c9e4de137c0926949205097" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Aug 04 15:19:41 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Aug 04 18:56:54 2015 -0700" }, "message": "Address some comments and clean up\n\nChange-Id: I538cf204f1c89d5fc81f8fc5e5800fcf1cf87359\n" }, { "commit": "1cf85515600d298a2c9e4de137c0926949205097", "tree": "4c0c6d8b111219dfde3cfddecbf1763037c64ad0", "parents": [ "d2606b1b922573d45707931bc387cbf990771238", "50832b6804d15ea5ddfe99a2753a09111b5bfe37" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Aug 05 00:36:05 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Aug 05 00:36:05 2015 +0000" }, "message": "Merge \"Revert \"ART: Use bionic TLS slot for thread-self\"\"" }, { "commit": "50832b6804d15ea5ddfe99a2753a09111b5bfe37", "tree": "ab80ccb63697ee973fd50c5083d7b56838b45e0b", "parents": [ "b81535875ff46d1630c76091fe969a9eb7fc4d16" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Aug 05 00:31:02 2015 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Aug 05 00:31:02 2015 +0000" }, "message": "Revert \"ART: Use bionic TLS slot for thread-self\"\n\nThis reverts commit b81535875ff46d1630c76091fe969a9eb7fc4d16.\n\nBreaking tests, need to investigate.\n\nChange-Id: Iab8c4f6136e6cb9bd6cf3c3495aa33962ef890ee\n" }, { "commit": "bab71e2a51834dd6986f2166d332f540824687f9", "tree": "20090c63521589061bdc9f87aeb07aa2b4a1b377", "parents": [ "fd38927d03a5fdf4ccfce8b9e0ffb1a99c4f0b37", "b81535875ff46d1630c76091fe969a9eb7fc4d16" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 04 22:35:34 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Aug 04 22:35:34 2015 +0000" }, "message": "Merge \"ART: Use bionic TLS slot for thread-self\"" }, { "commit": "c60e1b755c5632dfeb04c333489ede52ee5c945f", "tree": "9582a0ffc99e4ad11dcd5d95dd97b09bc6acc5bf", "parents": [ "7b926cdacc2b67241bc9cb5f2d4b04b13ca79d0e" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 30 08:57:50 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 30 08:57:50 2015 -0700" }, "message": "ART: Use __ANDROID__ instead of HAVE_ANDROID_OS\n\nUse the proper define.\n\nChange-Id: I71e291ac25f5d5f0187ac9b6ef2d6872f19e6085\n" }, { "commit": "b81535875ff46d1630c76091fe969a9eb7fc4d16", "tree": "0a3713c652c9db1afe8276c621107d90caa15149", "parents": [ "595335100a947693b9af5fb6c0b5b3c1f0b91788" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 28 19:51:37 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 28 19:51:37 2015 -0700" }, "message": "ART: Use bionic TLS slot for thread-self\n\nUse a private bionic TLS slot to store self instead of using\npthreads.\n\nChange-Id: Icc86a2b7590734637366f9d5e41a5c6d18cc5772\n" }, { "commit": "1aee900d5a0b3a8d78725a7551356bda0d8554e1", "tree": "325cb4ce99cf7824eb56dda1d85f23ad9c74f10a", "parents": [ "a09d5df5128c5c51226028d4a82ed461cf993024" ], "author": { "name": "Man Cao", "email": "manc@google.com", "time": "Tue Jul 14 22:31:42 2015 -0700" }, "committer": { "name": "Man Cao", "email": "manc@google.com", "time": "Tue Jul 28 10:45:02 2015 -0700" }, "message": "Add read barrier support to the entrypoints.\n\nAlso remove \"THIS_LOAD_REQUIRES_READ_BARRIER\" since reading\nan ArtMethod* no longer needs read barrier.\n\nstub_test should also work with read barriers now.\n\nChange-Id: I3fba18042de2f867a18dbdc38519986212bd9769\n" }, { "commit": "4e2cb098017bf073335ebb02b1bc0a36828cd720", "tree": "7dfb6c7b71a747f31d2903c383423aac96a11591", "parents": [ "f64914d7392b7d35c7f35bc917b0797b9c4605ac" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 22 16:17:51 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 22 17:26:05 2015 -0700" }, "message": "Add uninterruptible role\n\nExample error:\ncannot call function \u0027CopyOf\u0027 while mutex \u0027uninterruptible_\u0027 is held\n\nTODO: Add annotation to more locations.\n\nBug: 20072211\n\nChange-Id: I1bbf5a77e3deeafa5898df529cb7cb53a6d010d2\n" }, { "commit": "90443477f9a0061581c420775ce3b7eeae7468bc", "tree": "8c74b81dfae162e0fd0ccf8d5ac50827ba815174", "parents": [ "6078aec213dfaf111c29969706e8e5967cfc9bea" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 16 20:32:27 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 22 15:13:56 2015 -0700" }, "message": "Move to newer clang annotations\n\nAlso enable -Wthread-safety-negative.\n\nChanges:\nSwitch to capabilities and negative capabilities.\n\nFuture work:\nUse capabilities to implement uninterruptible annotations to work\nwith AssertNoThreadSuspension.\n\nBug: 20072211\n\nChange-Id: I42fcbe0300d98a831c89d1eff3ecd5a7e99ebf33\n" }, { "commit": "ef76c3fa57ddf3f7ca32e118f599da8cb4eb1560", "tree": "360c0fc574cb37d7af59f217880673d3b8fe86c5", "parents": [ "11237947eefd3ec3167d9df41f2f3a041e717b6f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jul 17 18:46:32 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jul 17 18:49:18 2015 -0700" }, "message": "Get non proxy method for StackDumpVisitor\n\nFixes failing DCHECK.\n\nBug: 22553459\nChange-Id: I6fe1012f9a5f8d29cb98bf25e3a279da51c5e1c0\n" }, { "commit": "14c3bf91b2ec434295ec84d6446f495fb7de6d5c", "tree": "6eaa51c263b84b7a11f40f21be993987cdca80f3", "parents": [ "6d3d1e3d866a880b4df95ba96ed126c1723e3dd6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jul 13 14:35:43 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jul 17 15:34:40 2015 -0700" }, "message": "Change intern table to not use WaitHoldingLocks\n\nBug: 22423014\nChange-Id: I9e16b8cb4def72fff73f1783a182877105feb7aa\n" }, { "commit": "c6f3439b67962584cfb764133e192c3317f0e1fc", "tree": "58f31ef9ed7155cd386f139be230383d943a1c72", "parents": [ "e978aba24094cce90453ad3294b8c4536ee8e4b4", "eac4424b3420c280f97ff2f815b5dedd8dac9801" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jul 14 01:47:39 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Jul 14 01:47:39 2015 +0000" }, "message": "Merge \"ART: JNI thread state transition optimization\"" }, { "commit": "4abf45148c643f2f2ac0e177bd3d394abd9d1945", "tree": "fe9257aa3dcadca7ddf0ea96e4f0251a43a36cb2", "parents": [ "0541502e6d9d0efa50aa333db32898a0664512c3" ], "author": { "name": "Evgenii Stepanov", "email": "eugenis@google.com", "time": "Mon Jul 13 10:41:57 2015 -0700" }, "committer": { "name": "Evgenii Stepanov", "email": "eugenis@google.com", "time": "Mon Jul 13 18:02:40 2015 -0700" }, "message": "Disable ASan on Thread::InstallImplicitProtection.\n\nThis function reads random stack locations and confuses ASan.\n\nBug: 21785137\nChange-Id: Iaaee6032a651618e5275ab46e6e8f886df60a5d0\n" }, { "commit": "0541502e6d9d0efa50aa333db32898a0664512c3", "tree": "1c8c2adc8e70bee2c2e972ac84f0726e19d5a348", "parents": [ "5b1eea0a6c90e6e82393cdb7337f0c2bb05637d9", "1e13374baf7dfaf442ffbf9809c37c131d681eaf" ], "author": { "name": "Evgenii Stepanov", "email": "eugenis@google.com", "time": "Tue Jul 14 00:47:06 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Jul 14 00:48:54 2015 +0000" }, "message": "Merge \"Generalize Valgrind annotations in ART to support ASan.\"" }, { "commit": "1e13374baf7dfaf442ffbf9809c37c131d681eaf", "tree": "9a8a53295a8389d7ec10accf3efb8785a97a3599", "parents": [ "f68c8545382925062da2b87169ca2b5314f0b431" ], "author": { "name": "Evgenii Stepanov", "email": "eugenis@google.com", "time": "Wed May 20 12:30:59 2015 -0700" }, "committer": { "name": "Evgenii Stepanov", "email": "eugenis@google.com", "time": "Mon Jul 13 17:38:40 2015 -0700" }, "message": "Generalize Valgrind annotations in ART to support ASan.\n\nAlso add redzones around non-fixed mem_map(s).\nAlso extend -Wframe-larger-than limit to enable arm64 ASan build.\n\nChange-Id: Ie572481a25fead59fc8978d2c317a33ac418516c\n" }, { "commit": "eac4424b3420c280f97ff2f815b5dedd8dac9801", "tree": "4e7c31b4312aa000bd3d2250df6a05daf61523d7", "parents": [ "ec97825f8553b3a99b6dfbbb90a50fe65301ea94" ], "author": { "name": "Yu Li", "email": "yu.l.li@intel.com", "time": "Mon Jun 29 10:50:03 2015 +0800" }, "committer": { "name": "Man Cao", "email": "manc@google.com", "time": "Mon Jul 13 15:56:19 2015 -0700" }, "message": "ART: JNI thread state transition optimization\n\nThis patch improves the JNI performance by removing the explicit acquiring and\nreleasing the mutator lock when a thread state transits between suspended and\nrunnable states.\n\nThe functions responsible for changing the state were found to be the costliest\npart of the JNI. Originally, a thread needs to acquire a shared mutator lock by\na CAS instruction when entering the runnable state and also needs to release\nthe lock by a CAS when entering the native state from runnable. This patch\nremoves these CAS operations when a thread state transits between suspended and\nrunnable. A thread in the runnable state is considered to have shared ownership\nof the mutator lock and therefore transitions in and out of the runnable state\nhave associated implication on the mutator lock ownership. Meanwhile, a barrier\nis added to control suspending all threads from running.\n\nJNI transition overhead was reduced by 25% on IA platform and by 17% on ARM\nplatform by this patch, while it has little impact on GC pause time (measured\nwith \"suspend all histogram\").\n\nChange-Id: Icee95d8ffff1bbfc95309a41cc48836536fec689\nSigned-off-by: Yu, Li \u003cyu.l.li@intel.com\u003e\nSigned-off-by: Haitao, Feng \u003chaitao.feng@intel.com\u003e\nSigned-off-by: Lei, Li \u003clei.l.li@intel.com\u003e\n" }, { "commit": "5b1eea0a6c90e6e82393cdb7337f0c2bb05637d9", "tree": "3b8bddd878f07ee418a83a09f229ea44f2a20a05", "parents": [ "24db1bc55bf8fb3240690436a229352453ae75e1", "0b71357fb52be9bb06d35396a3042b4381b01041" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Jul 13 22:51:39 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jul 13 22:51:40 2015 +0000" }, "message": "Merge \"Thread-local mark stacks for the CC collector.\"" }, { "commit": "0b71357fb52be9bb06d35396a3042b4381b01041", "tree": "44b8d6b0ca4a71f33cda55068fa9e14864c55625", "parents": [ "0e87d1809443d41b431ce3a686b4e074df2d7b8b" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Jun 16 18:29:23 2015 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Jul 13 15:50:20 2015 -0700" }, "message": "Thread-local mark stacks for the CC collector.\n\nThread-local mark stacks are assigned to mutators where they push\nreferences in read barriers to reduce the (CAS) synchronization cost\nin a global mark stack/queue.\n\nWe step through three mark stack modes (thread-local, shared,\nGC-exclusive) and use per-thread flags to disable/enable system weak\naccesses (only for the CC collector) instead of the existing global\none to safely perform the marking phase. The reasons are 1)\nthread-local mark stacks for mutators need to be revoked using a\ncheckpoint to avoid races (incorrectly leaving a reference on mark\nstacks) when terminating marking, and 2) we can’t use a checkpoint\nwhile system weak accesses are disabled (or a deadlock would\nhappen). More details are described in the code comments.\n\nPerformance improvements in Ritzperf EAAC: a ~2.8% improvement\n(13290-\u003e12918) in run time and a ~23% improvement (51.6s-\u003e39.8s) in\nthe total GC time on N5.\n\nBug: 12687968\nChange-Id: I5d234d7e48bf115cd773d38bdb62ad24ce9116c7\n" }, { "commit": "5662383511a550e428bcdee0cc1be28e464ceed4", "tree": "bce155444896a9ec790b17e9de1ec6fa1d95805b", "parents": [ "f470eb88cc3d3654f0fb1ca38622fa3a35fbd8a3" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 09 21:53:15 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jul 10 21:09:37 2015 -0700" }, "message": "ART: Add Thread start VLOG\n\nAdd a stack trace at the time of thread creation or attaching to\nthe verbose:threads logging.\n\nChange-Id: I5e07c81ce24f145c37ab172eda7bb4c54dad4445\n" }, { "commit": "d4a02cfc59a449d216faa687cfdaa4252b4a433f", "tree": "dc0f23e71354403028b30f1592637e2d66db6210", "parents": [ "0022bdf71533a300deebe6b9cf287eaceeb0620c", "33d6903e570daf8f3cf7c1f6ebd9a6dd22c7c23c" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Jun 23 13:53:54 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Jun 23 13:53:55 2015 +0000" }, "message": "Merge \"Replace some run-time assertions with compile-time ones in ART.\"" }, { "commit": "33d6903e570daf8f3cf7c1f6ebd9a6dd22c7c23c", "tree": "8069008292fe085d051300f1a871392dc413f820", "parents": [ "822c00d0fe3b70d1f1fe5bac03713674d45f8173" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Jun 18 18:20:59 2015 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Jun 18 18:20:59 2015 +0100" }, "message": "Replace some run-time assertions with compile-time ones in ART.\n\nChange-Id: I16c3fad45c4b98b94b7c83d071374096e81d407a\n" }, { "commit": "41f9cc28f2c9edd3903ba6ca1c75b022445552ad", "tree": "92e485e7bbf4e43ef4e40fd2824610074888a8ce", "parents": [ "c4977a58b1feca22a83fd5239aa266c7ce16ab87" ], "author": { "name": "Pavel Vyssotski", "email": "pavel.n.vyssotski@intel.com", "time": "Tue Jun 16 17:57:37 2015 +0600" }, "committer": { "name": "Pavel Vyssotski", "email": "pavel.n.vyssotski@intel.com", "time": "Wed Jun 17 16:28:57 2015 +0600" }, "message": "ART: Compiler generated GC map should take care of temp registers.\n\nIf the compiler use a temp register as a reference it should be\nadded to generated GC map along with other SSA regs.\n\nChange-Id: I3c79896dcd72228c9cf2bbbe737642d9ffb7684d\nSigned-off-by: Pavel Vyssotski \u003cpavel.n.vyssotski@intel.com\u003e\n" }, { "commit": "cbc5064ff05179b97b416f00ca579c55e38cd7d9", "tree": "9ce221e6644ff770b8484ba8cb5581e538b88eef", "parents": [ "864a2d955aa85ab989c86d7f1eeacbe0b11f8b0f" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Jun 01 17:33:12 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Jun 10 09:35:08 2015 +0200" }, "message": "JDWP: asynchronous invoke command handling\n\nThe JDWP thread used to wait for the result of a method invocation\nrunning in an event thread. But doing that prevents the JDWP thread\nfrom processing incoming commands from the debugger if the event\nthread gets suspended by a debug event occurring in another thread.\nIn Android Studio (or another IDE), this leads to the debugger being\nblocked (with the famous message \"Waiting until last debugger command\ncompletes\" of Android Studio / IntelliJ) because it is actually\nwaiting for the reply of its latest command while the JDWP thread\ncannot process it.\n\nThis CL changes the way invoke commands (ClassType.InvokeCommand,\nClassType.NewInstance and ObjectReference.InvokeCommand) are handled\nin the ART runtime.\nThe JDWP thread no longer waits for the event thread to complete the\nmethod invocation. It now simply waits for the next JDWP command to\nprocess. This means it does not send any reply for invoke commands,\nexcept if the information given by the debugger is wrong. In this\ncase, it still sends a reply with the appropriate error code.\nThe event thread is now responsible for sending the reply (containing\nthe result and the exception object of the invoked method) before\ngoing back to the suspended state.\n\nIn other words, we add special handling for invoke commands so they\nare handled asynchronously while other commands remained handled\nsynchronously. In the future, we may want to handle all commands\nasynchronously (using a queue of reply/event for instance) to remove\nthe special handling code this CL is adding.\n\nNow the JDWP thread can process commands while a thread is invoking\na method, it is possible for the debugger to detach (by sending a\nVirtualMachine.Dispose command) before the invocation completes. In\nthat situation, we must not suspend threads again (including the\nevent thread that executed the method) because they would all remain\nsuspended forever.\n\nAlso minor cleanup of the use of JDWP constants and update comments.\n\nBug: 21515842\nBug: 18899981\nChange-Id: I15e00fb068340f3d69dc9225d8d2065246e68c58\n" }, { "commit": "f795869da0a1fa006fdcdacd8afb6149a63fc1a7", "tree": "b4cd23012e62b80788870f2c85c30fc6032299fa", "parents": [ "430afefe1cced35cf5eb12ce7a6dcbfc5007c97e" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jun 09 14:09:14 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jun 09 17:17:09 2015 +0200" }, "message": "Follow up on CL 151605\n\n- Fixes return type of StackedShadowFrameRecord::GetType\n- Makes StackedShadowFrameType an enum class (scoped enum)\n- Moves DeoptimizationReturnValueRecord and StackedShadowFrameRecord\n to thread.cc file and use forward declaration in thread.h header\n- Fixes tools/generate-operator-out.py for scoped enum classes.\n\nBug: 20845490\nChange-Id: I6b67e288b1db563699161e58ec2e2330d42dd8f5\n" }, { "commit": "1f2d3ba6af52cf6f566deb38b7e07735c9a08fb6", "tree": "af6ae612d43c50703292d39a8a0b37dda3f919a6", "parents": [ "ccd6337f31d13706c602f3d9436e9b4025075b63" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon May 18 12:12:50 2015 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Jun 08 00:53:55 2015 -0700" }, "message": "Fix nested deoptimization.\n\nHandle nested deoptimization cases. Create a stacked shadow frame\nrecords to keep track of deoptimization shadow frames. Shadow frames\nunder construction can be tracked in the same stack.\n\nBug: 20845490\nChange-Id: I768285792c29e7c3cfcd21e7a2600802506024d8\n" }, { "commit": "f677ebfd832c9c614fea5e6735725fec2f7a3f2a", "tree": "051b3ba6343bdbca4929d728dd5c0341e4383a08", "parents": [ "c47908e8c32fd58bc4dc75998a80f706954db1dc" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri May 29 16:29:43 2015 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jun 04 17:25:02 2015 +0100" }, "message": "Cache stack map encoding\n\nOperations on CodeInfo and StackMap objects repeatedly read encoding\ninformation from the MemoryRegion. Since these are 3-bit-loads of\nvalues that never change, caching them can measurably reduce compile\ntimes.\n\nAccording to benchmarks, this patch saves 1-3% on armv7, 2-4% on x86,\nand 0-1% on x64.\n\nChange-Id: I46b197513601325d8bab562cc80100c00ec28a3b\n" }, { "commit": "449357d3f61bdd134967ecdacc3516e1707cf882", "tree": "650ef203d23926485398200888cadfc9aa7608fc", "parents": [ "2519fc40d4ae89322d28d1ff610fe81bb90fb564" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 01 22:29:51 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 03 16:10:33 2015 -0700" }, "message": "ART: Refactor Thread::Init\n\nThis refactor allows the parent thread to allocate the JNIEnvExt\nfor the child (with a fallback in place in Init). This allows to\nthrow an OOME in CreateNativeThread instead of aborting in the\nchild.\n\nBug: 21291279\n\n(cherry picked from commit 520abbd0edcf333f07164539620ce65258c72383)\n\nChange-Id: Iccc1a5c202999f5bfacec706d9833e53135ba2fa\n" }, { "commit": "415fd0833fe44b99ed57353f4bd7572681506daa", "tree": "5c798549945457a86faf181495337722335ae4b0", "parents": [ "a96bea54b7e3def5490d169f2bf92be2e68dd001" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Jun 01 11:42:27 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Jun 01 12:27:32 2015 +0200" }, "message": "Fix JDWP tests after ArtMethod change\n\nFixes Throwable::GetStackDepth for exception event detection after\ninternal stack trace representation change.\n\nAdds missing ArtMethod::GetInterfaceMethodIfProxy call in case of\nproxy method.\n\nBug: 19264997\nChange-Id: I363e293796848c3ec491c963813f62d868da44d2\n" }, { "commit": "e401d146407d61eeb99f8d6176b2ac13c4df1e33", "tree": "17927f9bfe7d2041b5942c89832d55f9dedb24c5", "parents": [ "2006b7b9b8e32722bd0d640c62549d8a0ac624b6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 13:56:20 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 29 18:45:49 2015 -0700" }, "message": "Move mirror::ArtMethod to native\n\nOptimizing + quick tests are passing, devices boot.\n\nTODO: Test and fix bugs in mips64.\n\nSaves 16 bytes per most ArtMethod, 7.5MB reduction in system PSS.\nSome of the savings are from removal of virtual methods and direct\nmethods object arrays.\n\nBug: 19264997\nChange-Id: I622469a0cfa0e7082a2119f3d6a9491eb61e3f3d\n" }, { "commit": "dc31313c4577bb4c00d90774159c84e738ed348d", "tree": "5f850028ebc8843cdcaf56c5fc122bd660937603", "parents": [ "03b5a398a96ea29c39ddfe3d810245b868d2871f", "80afd02024d20e60b197d3adfbb43cc303cf29e0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 26 17:42:31 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue May 26 17:42:31 2015 +0000" }, "message": "Merge \"ART: Clean up arm64 kNumberOfXRegisters usage.\"" }, { "commit": "80afd02024d20e60b197d3adfbb43cc303cf29e0", "tree": "ef054c7b4f2a739f7cf063e0bc4c501c2c7e41b5", "parents": [ "559b178e34c5d92e7932f92e5d8a981ac334606f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 19 18:08:00 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 26 15:59:02 2015 +0100" }, "message": "ART: Clean up arm64 kNumberOfXRegisters usage.\n\nAvoid undefined behavior for arm64 stemming from 1u \u003c\u003c 32 in\nloops with upper bound kNumberOfXRegisters.\n\nCreate iterators for enumerating bits in an integer either\nfrom high to low or from low to high and use them for\n\u003carch\u003eContext::FillCalleeSaves() on all architectures.\n\nRefactor runtime/utils.{h,cc} by moving all bit-fiddling\nfunctions to runtime/base/bit_utils.{h,cc} (together with\nthe new bit iterators) and all time-related functions to\nruntime/base/time_utils.{h,cc}. Improve test coverage and\nfix some corner cases for the bit-fiddling functions.\n\nBug: 13925192\nChange-Id: I704884dab15b41ecf7a1c47d397ab1c3fc7ee0f7\n" }, { "commit": "e12997fbce8e22431be58cac9db2535f7b4a7ac3", "tree": "ae167aa833e1ef249d121b179c95827fa4e7ec6c", "parents": [ "aa49c23d47e5fdfcf51380550ee864e9d30d082b" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri May 22 14:01:33 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri May 22 14:01:33 2015 +0100" }, "message": "Return an invalid StackMap when one cannot be found.\n\nThis avoids aborting when handling a crash.\n\nChange-Id: Ie5b5d48061fa9258b349b0284f7b00c5855d9fbd\n" }, { "commit": "39b6c24ce68cf05db0f82f454b4401cfb03d675f", "tree": "29e15d2597cd05419a1c0faedfe6977af9a7dfa3", "parents": [ "768b19b6bbddaa2262f6532e7fcb96fecbfee981" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue May 19 20:30:23 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed May 20 18:40:43 2015 -0700" }, "message": "Intercept JNI invocation of String.\u003cinit\u003e methods.\n\nlibmono uses JNI AllocObject and CallNonvirtualVoidMethod to create and\ninitialize a string instead of using the recommended NewObject. This\nchange adds an intercept to change the String.\u003cinit\u003e call to a\nStringFactory call instead. Then, it uses the object id of the original\nstring object referrer and maps it to the result of the StringFactory.\n\nBug: 21288130\n\n(cherry picked from commit 15e9ad1d028d7f12cb598b075453173532a00d91)\n\nChange-Id: I3421c43722c07397da4a398c2ca9110e1d40bcfa\n" }, { "commit": "1d316adafda81cb2f30e210d3d108b97c7c11f15", "tree": "57aa3ebf536ce612281523ce385cedf583afa93a", "parents": [ "bd0cfef567012fa6c75243d9157ddceaab7cdaac", "1cc71ebf333ca323ae0e130fefbce4593e385c10" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon May 11 18:04:09 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon May 11 18:04:10 2015 +0000" }, "message": "Merge \"Insert a read barrier in the JNI bridge.\"" }, { "commit": "8e5bd18fc665d7ec5461ea068e98740a65da754c", "tree": "83441cdfdab06709b573aad2ab731cc65c10b9f1", "parents": [ "cdeb0b5fede4c06488f43a212591e661d946bc78" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed May 06 11:34:34 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 11 15:23:25 2015 +0100" }, "message": "Add a flag to StackVisitor for inlining.\n\nThe flag tells whether the stack walk needs to include inlined\nJava frames.\n\nThis does not do anything just yet, as we\u0027re not inlining anyways.\n\nChange-Id: I716e25094fe56fa335ca1f9a398c1bcdba478e73\n" }, { "commit": "50030ef998be09789da4a9a56738362852068f12", "tree": "02df1bd0954f9b970434d1363466b0cfe182bc12", "parents": [ "6727a48193db2a0cf01af971cccffe1a6518c247" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 08 14:19:26 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 08 15:09:17 2015 -0700" }, "message": "Check IsReferenceVReg during deopt\n\nRequired since the quick GC maps may not agree with the verifier\nones. Without this check we may copy stale object references into\nthe shadow frame.\n\n(cherry picked from commit f00baf56ef647684888a407dbb6adadd704a2039)\n\nBug: 20736048\nChange-Id: I7783c8a8ee45cf601b08b4c38f1dec7f7d11380c\n" }, { "commit": "1cc71ebf333ca323ae0e130fefbce4593e385c10", "tree": "e94897ebb36a21f1c276fc69c44f6a3522cbe093", "parents": [ "35b6625c21e9304dd79700f4ff1c26a7e30d2b2d" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu May 07 10:47:27 2015 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu May 07 11:28:08 2015 -0700" }, "message": "Insert a read barrier in the JNI bridge.\n\nBug: 20720510\nBug: 12687968\nChange-Id: Ia9edb614853bad45ec25d6e2142361a8bda1eccf\n" }, { "commit": "848f70a3d73833fc1bf3032a9ff6812e429661d9", "tree": "b0349b3a40aab5a915af491b100659a5ca9fbbf6", "parents": [ "d14438f0c5071962be7fab572b54687d32d9d087" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Jan 15 13:49:50 2014 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Apr 27 18:54:52 2015 -0700" }, "message": "Replace String CharArray with internal uint16_t array.\n\nSummary of high level changes:\n - Adds compiler inliner support to identify string init methods\n - Adds compiler support (quick \u0026 optimizing) with new invoke code path\n that calls method off the thread pointer\n - Adds thread entrypoints for all string init methods\n - Adds map to verifier to log when receiver of string init has been\n copied to other registers. used by compiler and interpreter\n\nChange-Id: I797b992a8feb566f9ad73060011ab6f51eb7ce01\n" }, { "commit": "b28412e3fcc97fbcd0a11d25703a4ad64c3d9552", "tree": "9e4df825a6ef666ee714c782421dfb826e2703df", "parents": [ "76f1413492c228bfa710e1eaa4c60370eaffbb8a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 24 15:28:36 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 24 15:28:36 2015 -0700" }, "message": "Guard against name \u003d\u003d null in ShortDump\n\nBug: 20561804\nChange-Id: If1e4adb74d5b936cf171f2402bf4525bf8c2d209\n" }, { "commit": "a61894d88fabe45677f491c9f6bde30059a49026", "tree": "b9d8b09e5f90792867b6720a1fb4ab5c76cdfa5f", "parents": [ "4ceed922d44b68c3fa7cbe670014c9e2e003b92b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 23 16:32:54 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 23 17:21:26 2015 -0700" }, "message": "Fix reflection handling and test flakiness\n\nFixed reflection invoke to handle exceptions which occur from\nFindClass or NewObject by throwing these instead of\nthe expected InvocationTargetException.\n\nAdded test case to 080 for this reflection invoke.\n\nFixed println throwing OOM in 104-growth-limit.\n\nChange-Id: I65766e7c3478e299da06fdc3a521fe3f3e8fdba9\n" }, { "commit": "2cebb24bfc3247d3e9be138a3350106737455918", "tree": "d04d27d21b3c7733d784e303f01f873bb99e7770", "parents": [ "1f02f1a7b3073b8fef07770a67fbf94afad317f0" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Apr 21 16:50:40 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 12:44:27 2015 -0700" }, "message": "Replace NULL with nullptr\n\nAlso fixed some lines that were too long, and a few other minor\ndetails.\n\nChange-Id: I6efba5fb6e03eb5d0a300fddb2a75bf8e2f175cb\n" }, { "commit": "9086b65b2ad35dd39a8afc62d535be8217208d08", "tree": "3abc6e3297b840289bc240e3a9bc7e61b7a99cbc", "parents": [ "8d20011a9de7cd94bee59db3ae8c0cbbf55911d9" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Apr 14 09:35:18 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Apr 14 11:06:34 2015 -0700" }, "message": "Fix valgrind tests\n\nDelete large objects in space destructor. Also some cleanup.\n\nChange-Id: I4c4e90149841a156b7a3236201b37683e14890fb\n" }, { "commit": "1561e1b4029cc25a653e17b36663ae047345fad0", "tree": "dfcb3db9060f9e05be6109616811bbe97f20b40c", "parents": [ "6c7bd25b2b2a07740bcd838c2f4024e67be83fd1" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Mon Apr 13 23:18:02 2015 +0000" }, "committer": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Mon Apr 13 16:22:01 2015 -0700" }, "message": "Revert \"ART: Turn of native stack dump on x86 on-device\"\n\nThis reverts commit 2f0751ea75e5f093d9ce7b8587e65a2ca3de0855.\n\nBug: 20040863\n\nChange-Id: Ibd40916c0f84512b0ba13af09d4fd345be39e227\n" }, { "commit": "c785344b87221f5e4e6473e5b762e4e61fe65dcf", "tree": "cd32ad2c2604596a18926f04d4c313dab255ecfd", "parents": [ "a29d93b380c9aeb8270e281aefbdd0c77a430d43" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 27 14:35:38 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 10 12:57:27 2015 -0700" }, "message": "Move ArtField to native\n\nAdd linear alloc. Moved ArtField to be native object. Changed image\nwriter to put ArtFields after the mirror section.\n\nSavings:\n2MB on low ram devices\n4MB on normal devices\n\nTotal PSS measurements before (normal N5, 95s after shell start):\nImage size: 7729152 bytes\n23112 kB: .NonMoving\n23212 kB: .NonMoving\n22868 kB: .NonMoving\n23072 kB: .NonMoving\n22836 kB: .NonMoving\n19618 kB: .Zygote\n19850 kB: .Zygote\n19623 kB: .Zygote\n19924 kB: .Zygote\n19612 kB: .Zygote\nAvg: 42745.4 kB\n\nAfter:\nImage size: 7462912 bytes\n17440 kB: .NonMoving\n16776 kB: .NonMoving\n16804 kB: .NonMoving\n17812 kB: .NonMoving\n16820 kB: .NonMoving\n18788 kB: .Zygote\n18856 kB: .Zygote\n19064 kB: .Zygote\n18841 kB: .Zygote\n18629 kB: .Zygote\n3499 kB: .LinearAlloc\n3408 kB: .LinearAlloc\n3424 kB: .LinearAlloc\n3600 kB: .LinearAlloc\n3436 kB: .LinearAlloc\nAvg: 39439.4 kB\n\nNo reflection performance changes.\n\nBug: 19264997\nBug: 17643507\n\nChange-Id: I10c73a37913332080aeb978c7c94713bdfe4fe1c\n" }, { "commit": "d0ad2eea51850ed5972c23d03380b2305cdf7cb7", "tree": "533baced17c6dc3860e0e2121ce06acde57b8f4d", "parents": [ "917f0866f66d3465fa41ddcf9ab09eba2fa78dd3" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 31 14:59:59 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 09 15:18:14 2015 -0700" }, "message": "Add support for nested method verifiers\n\nCan occur in the following scenario:\nMethodVerifier::Verify -\u003e MethodVerifier::GetStaticField -\u003e\nResolveFieldJLS -\u003e ThrowNoSuchFieldError -\u003e EnsureInitialized -\u003e\nVerifyClass\n\nAlso fixed another case where we can be requested to dump for ANR\nwhile we are suspended in one of the AllowSuspension points.\n\nBug: 20140397\n\nChange-Id: Ib17f6b98954caa5d1ea1c1dcde66091cc6d11c25\n" }, { "commit": "3f5881fda3606b27e30bf903052c73b03910f90b", "tree": "0c60e00a3923d47658e2d224997ab2742c113877", "parents": [ "1576be32be4a99a1cffdaaf209a3cd67e8b2f88a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Apr 08 10:26:16 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Apr 09 09:09:11 2015 -0700" }, "message": "ART: IRT refactor\n\nIRT creation might fail. Add a path that allows to bypass the aborts\nand instead signal validity. Hide this path with a private constructor,\nrewrite users to use a static Create method.\n\nBug: 20110201\n\nChange-Id: I440499c3372cd7557eb970b70ce2c4543da520e4\n" }, { "commit": "4809d0a8a5fca85a67dd0588ead5dfbd0f1acf96", "tree": "12853aa2c6a6238cac7c66b5d5610735a2f31591", "parents": [ "63a99fce3ceac3c9de1f47b88a75094e3e1ffba3" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Apr 07 10:39:04 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Apr 07 10:42:46 2015 -0700" }, "message": "Fix CC root visiting bug\n\nAlso some cleanup.\n\nChange-Id: Ia3de8f2d409770be3619ec116e8b06ecd82338fe\n" }, { "commit": "bb87e0f1a52de656bc77cb01cb887e51a0e5198b", "tree": "113f014c6e20fab3e936a3ac05f9f738639541f6", "parents": [ "e57fc0f0260fcb1d08cbb720ec95c04c0f394b91" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 03 11:21:55 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Apr 06 10:44:37 2015 -0700" }, "message": "Refactor and improve GC root handling\n\nChanged GcRoot to use compressed references. Changed root visiting to\nuse virtual functions instead of function pointers. Changed root visting\ninterface to be an array of roots instead of a single root at a time.\nAdded buffered root marking helper to avoid dispatch overhead.\n\nRoot marking seems a bit faster on EvaluateAndApplyChanges due to batch\nmarking. Pause times unaffected.\n\nMips64 is untested but might work, maybe.\n\nBefore:\nMarkConcurrentRoots: Sum: 67.678ms 99% C.I. 2us-664.999us Avg: 161.138us Max: 671us\n\nAfter:\nMarkConcurrentRoots: Sum: 54.806ms 99% C.I. 2us-499.986us Avg: 136.333us Max: 602us\n\nBug: 19264997\n\nChange-Id: I0a71ebb5928f205b9b3f7945b25db6489d5657ca\n" }, { "commit": "2f0751ea75e5f093d9ce7b8587e65a2ca3de0855", "tree": "df99d69761aa9c5ace190f4970c0952009c1713d", "parents": [ "698385411e5db8c7ac9bd4471f152b8c6ea7f7bf" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Apr 03 14:36:07 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Apr 03 14:36:07 2015 -0700" }, "message": "ART: Turn of native stack dump on x86 on-device\n\nTemporary workaround for bug.\n\nBug: 20040863\nChange-Id: Ifdde68c66b94ebc2f13c7983f38700c8bcf41559\n" }, { "commit": "d43b3ac88cd46b8815890188c9c2b9a3f1564648", "tree": "6c599c3f40d57e92786bd7f41c0541d9eaa2643b", "parents": [ "a109632b240f3c9355ca95500f6f48e4478e3c51" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Apr 01 14:03:04 2015 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Apr 01 14:05:13 2015 -0700" }, "message": "Revert \"Revert \"Deoptimization-based bce.\"\"\n\nThis reverts commit 0ba627337274ccfb8c9cb9bf23fffb1e1b9d1430.\n\nChange-Id: I1ca10d15bbb49897a0cf541ab160431ec180a006\n" }, { "commit": "0ba627337274ccfb8c9cb9bf23fffb1e1b9d1430", "tree": "0e1d0813c1d8d1c7239a900c1653296975713df0", "parents": [ "e295e6ec5beaea31be5d7d3c996cd8cfa2053129" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Mar 24 02:39:46 2015 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Mar 24 02:39:46 2015 +0000" }, "message": "Revert \"Deoptimization-based bce.\"\n\nThis breaks compiling the core image:\n\n Error after BCE: art::SSAChecker: Instruction 219 in block 1 does not dominate use 221 in block 1.\n\nThis reverts commit e295e6ec5beaea31be5d7d3c996cd8cfa2053129.\n\nChange-Id: Ieeb48797d451836ed506ccb940872f1443942e4e\n" }, { "commit": "e295e6ec5beaea31be5d7d3c996cd8cfa2053129", "tree": "4d8a657d23d511743ce35bee596544d7f652efdb", "parents": [ "d24ba2c44c76a2b2dd13aafe8f7981c15be31a98" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Sat Mar 07 06:37:59 2015 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Mar 23 16:39:37 2015 -0700" }, "message": "Deoptimization-based bce.\n\nA mechanism is introduced that a runtime method can be called\nfrom code compiled with optimizing compiler to deoptimize into\ninterpreter. This can be used to establish invariants in the managed code\nIf the invariant does not hold at runtime, we will deoptimize and continue\nexecution in the interpreter. This allows to optimize the managed code as\nif the invariant was proven during compile time. However, the exception\nwill be thrown according to the semantics demanded by the spec.\n\nThe invariant and optimization included in this patch are based on the\nlength of an array. Given a set of array accesses with constant indices\n{c1, ..., cn}, we can optimize away all bounds checks iff all 0 \u003c\u003d min(ci) and\nmax(ci) \u003c array-length. The first can be proven statically. The second can be\nestablished with a deoptimization-based invariant. This replaces n bounds\nchecks with one invariant check (plus slow-path code).\n\nChange-Id: I8c6e34b56c85d25b91074832d13dba1db0a81569\n" }, { "commit": "004c230b4cfc856690c61faabc41864061813c88", "tree": "40d95445eaa2eb248fa639755689c797a5e0c2ae", "parents": [ "735dc87c92bee338d0638b3290c2b93a122429f2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 20 10:06:38 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 20 14:55:15 2015 +0000" }, "message": "Compress the StackMaps.\n\nFirst step towards the compression of the StackMap (not\nthe DexRegisterMap). Next step will be to just use what is\nneeded (instead of byte -\u003e word).\n\nChange-Id: I4f81b2d05bf5cc508585e16fbbed1bafbc850e2e\n" }, { "commit": "12d625f87bcd6c4059a205bb39007a255f57f382", "tree": "b69f6f80f56977c176e0c441869d2f1de06ede70", "parents": [ "b871a15f1f662c8efe10d41199180bd00b6decd1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 13 11:33:37 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 13 16:38:25 2015 -0700" }, "message": "Change method verifiers to be thread local roots\n\nBug: 10921004\n\nChange-Id: I3a1425d76cd380587ba67ef6d5ec1227564e55bf\n" }, { "commit": "15b9d5274399736ac09705f0507df24fac4f00c1", "tree": "04564a9265f5dccefdd32ea7bdd25adc0267f80b", "parents": [ "8bc616d09f93523f4bc982cc60c377b00161522a" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Mar 12 15:05:13 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 13 10:26:47 2015 +0000" }, "message": "API change in StackVisitor::GetVReg*.\n\n- Remove GetVReg() and SetVReg() that were expecting to always succeed.\n- Change Quick-only methods to take a FromQuickCode suffix.\n- Change deopt to use dead values when GetVReg does not succeed:\n the optimizing compiler will not have a location for uninitialized\n Dex registers and potentially dead registers.\n\nChange-Id: Ida05773a97aff8aa69e0caf42ea961f80f854b77\n" }, { "commit": "0aa50ce2fb75bfc2e815a0c33adf9b049561923b", "tree": "9a3f9603ab30d5cbc7fc21aee0ceb48bbb0dd25a", "parents": [ "e8e42f3548fd894f860912bb1b71ce6fa2d7daf3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 10 11:03:29 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 10 14:51:11 2015 +0000" }, "message": "Remove ThrowLocation.\n\nNote that this is a cleanup change, and has no functionality change.\nThe ThrowLocation had no use anymore.\n\nChange-Id: I3d2126af1dc673cec3a0453ff3d56a172663a5f6\n" }, { "commit": "e8e42f3548fd894f860912bb1b71ce6fa2d7daf3", "tree": "ea27dd630728cfa994ce2c5da522538744a7a8a9", "parents": [ "54a8cc689dc23f346c9aac0d5fc6f47e61df8cad", "1558b577907b613864e98f05862543557263e864" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Mar 10 08:48:46 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Mar 10 08:48:47 2015 +0000" }, "message": "Merge \"JDWP: allocate DebugInvokeReq only when requested\"" }, { "commit": "ee0770379ab3e328fdd5a733e62f05b9f5216e38", "tree": "e37dfb0e4530395abad5a6c344651645b56e3082", "parents": [ "2cfdabd2bb4833d7092819d27ef08a9e1cdffead" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 09 14:56:37 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 09 17:23:01 2015 +0000" }, "message": "Use the current Java stack instead of the ThrowLocation.\n\nThe ThrowLocation contains information we can actually retrieve\nfrom the current stack.\n\nChange-Id: I7e5b2f519f95830c457cb31ea10a581aa9c973ee\n" }, { "commit": "1558b577907b613864e98f05862543557263e864", "tree": "5498d8d15f198341fe46a8badc7e7591611a09b5", "parents": [ "2cfdabd2bb4833d7092819d27ef08a9e1cdffead" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Feb 25 15:05:59 2015 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Mar 09 15:19:49 2015 +0100" }, "message": "JDWP: allocate DebugInvokeReq only when requested\n\nOnly allocates thread-local DebugInvokeReq when the debugger requests\na thread to invoke a method. The JDWP thread allocates that structure\nthen attaches it to the target thread. When the thread is resumed, it\nexecutes the method. Once the invocation completes, the thread\ndetaches the DebugInvokeReq, signals the JDWP thread then suspends.\nFinally, the JDWP thread wakes up, prepares the reply with the invoke\nresult (or exception) and deallocates the DebugInvokeReq.\n\nAlso ensures GC safety for object returned by the invoke. We add the\nobject to the JDWP object registry right after the invoke. We now\nreference that object with a JDWP ObjectID instead of an Object* in\nthe DebugInvokeReq struct. This prevent from accessing a stale\nreference if the GC runs and moves the Object*.\n\nThis CL includes the following changes:\n- Move former DebugInvokeReq::ready flag to\n Thread::tls_32bit_sized_values::ready_for_debug_invoke. It\u0027s needed\n to know whether a thread has been suspended by an event, thus ready\n to invoke a method from the debugger.\n- Remove DebugInvokeReq::invoke_needed: we now test if we attached a\n DebugInvokeReq* to the thread.\n- Rename misleading FinishMethod function to RequestMethod.\n\nBug: 19142632\nBug: 18166750\nChange-Id: I351fb4eb94bfe69fcafb544d21d55ff35a033000\n" }, { "commit": "14691c5e786e8c2c5734f687e4c96217340771be", "tree": "7c2156671b323c70ffdd1d48d5e2f1d1de79c5fc", "parents": [ "3d7d2af4c6502b771b032ee9bf3ab30e78f9c60d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Mar 05 10:40:17 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 09 10:09:33 2015 +0000" }, "message": "Compute the right catch location for the debugger.\n\nAlso remove tls ThrowLocation, it is not needed anymore.\n\nChange-Id: I78fddf09ce968ca475e39c17fa76d699c589c8d9\n" } ], "next": "7642cfc90fc9c3ebfd8e3b5041915705c93b5cf0" }