)]}' { "log": [ { "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" }, { "commit": "7642cfc90fc9c3ebfd8e3b5041915705c93b5cf0", "tree": "807b3c797483310ac23954c5eddb7441b91749c5", "parents": [ "dc47e986941b1a3754447fabea272485f3f0f382" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 26 10:56:09 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 03 17:50:49 2015 +0000" }, "message": "Change how we report exceptions to the debugger.\n\nThis is only a refactoring/cleanup. Bug fixes with respect\nto catch location, and more cleanups will follow.\n\nChange-Id: I30d3c6260b0c8f8115a811621397225b88f2063a\n" }, { "commit": "e5f13e57ff8fa36342beb33830b3ec5942a61cca", "tree": "02e370ecc4e0d955f28bfc71a41015f94fbb19b7", "parents": [ "354d58ba776866ea7b1c71f0d0848d5aaa013ae3" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 24 09:37:21 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 24 10:47:02 2015 -0800" }, "message": "Revert \"Revert \"Add JIT\"\"\n\nAdded missing EntryPointToCodePointer.\n\nThis reverts commit a5ca888d715cd0c6c421313211caa1928be3e399.\n\nChange-Id: Ia74df0ef3a7babbdcb0466fd24da28e304e3f5af\n" }, { "commit": "a5ca888d715cd0c6c421313211caa1928be3e399", "tree": "bdb08a2cbcf277ab7f02626a23b52a3fdf272ffe", "parents": [ "2535abe7d1fcdd0e6aca782b1f1932a703ed50a4" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 24 08:10:57 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 24 08:10:57 2015 +0000" }, "message": "Revert \"Add JIT\"\n\nSorry, run-test crashes on target:\n0-05 12:15:51.633 I/DEBUG (27995): Abort message: \u0027art/runtime/mirror/art_method.cc:349] Check failed: PcIsWithinQuickCode(reinterpret_cast\u003cuintptr_t\u003e(code), pc) java.lang.Throwable java.lang.Throwable.fillInStackTrace() pc\u003d71e3366b code\u003d0x71e3362d size\u003dad000000\u0027\n10-05 12:15:51.633 I/DEBUG (27995): r0 00000000 r1 0000542b r2 00000006 r3 00000000\n10-05 12:15:51.633 I/DEBUG (27995): r4 00000006 r5 b6f9addc r6 00000002 r7 0000010c\n10-05 12:15:51.633 I/DEBUG (27995): r8 b63fe1e8 r9 be8e1418 sl b6427400 fp b63fcce0\n10-05 12:15:51.633 I/DEBUG (27995): ip 0000542b sp be8e1358 lr b6e9a27b pc b6e9c280 cpsr 40070010\n10-05 12:15:51.633 I/DEBUG (27995): \n\nBug: 17950037\nThis reverts commit 2535abe7d1fcdd0e6aca782b1f1932a703ed50a4.\n\nChange-Id: I6f88849bc6f2befed0c0aaa0b7b2a08c967a83c3\n" }, { "commit": "2535abe7d1fcdd0e6aca782b1f1932a703ed50a4", "tree": "140026ff9638ff34050680b6c706b82fa1740b56", "parents": [ "38fee8ef4bc0f4dbe2c6d1f5585895f0c4d16984" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 17 10:38:49 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 23 16:45:49 2015 -0800" }, "message": "Add JIT\n\nCurrently disabled by default unless -Xjit is passed in.\n\nThe proposed JIT is a method JIT which works by utilizing interpreter\ninstrumentation to request compilation of hot methods async during\nruntime.\n\nJIT options:\n-Xjit / -Xnojit\n-Xjitcodecachesize:N\n-Xjitthreshold:integervalue\n\nThe JIT has a shared copy of a compiler driver which is accessed\nby worker threads to compile individual methods.\n\nAdded JIT code cache and data cache, currently sized at 2 MB\ncapacity by default. Most apps will only fill a small fraction of\nthis cache however.\n\nAdded support to the compiler for compiling interpreter quickened\nbyte codes.\n\nAdded test target ART_TEST_JIT\u003dTRUE and --jit for run-test.\n\nTODO:\nClean up code cache.\nDelete compiled methods after they are added to code cache.\nAdd more optimizations related to runtime checks e.g. direct pointers\nfor invokes.\nAdd method recompilation.\nMove instrumentation to DexFile to improve performance and reduce\nmemory usage.\n\nBug: 17950037\n\nChange-Id: Ifa5b2684a2d5059ec5a5210733900aafa3c51bca\n" }, { "commit": "966c3ae95d3c699ee9fbdbccc1acdaaf02325faf", "tree": "7a9bd5dbfb8b02f8bb7e3387876be0c1f7844063", "parents": [ "85ed6bdd890c08f50c205d7f0604b5a35603b13e" ], "author": { "name": "Mark P Mendell", "email": "mark.p.mendell@intel.com", "time": "Tue Jan 27 15:45:27 2015 +0000" }, "committer": { "name": "Mark Mendell", "email": "mark.p.mendell@intel.com", "time": "Tue Jan 27 17:00:21 2015 -0500" }, "message": "Revert \"Revert \"ART: Implement X86 hard float (Quick/JNI/Baseline)\"\"\n\nThis reverts commit 949c91fb91f40a4a80b2b492913cf8541008975e.\n\nThis time, don\u0027t clobber EBX before saving it.\n\nRedo some of the macros to make register usage explicit.\n\nChange-Id: I8db8662877cd006816e16a28f42444ab7c36bfef\n" }, { "commit": "949c91fb91f40a4a80b2b492913cf8541008975e", "tree": "45c840d1d6fd0ab71d96cb6c61931f468b3a0adf", "parents": [ "aeb47bb12420e65b4b5f61164e6396ea93734a0a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jan 27 10:48:44 2015 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jan 27 13:41:29 2015 +0000" }, "message": "Revert \"ART: Implement X86 hard float (Quick/JNI/Baseline)\"\n\nAnd the 3 Mac build fixes. Fix conflicts in context_x86.* .\n\nThis reverts commits\n 3d2c8e74c27efee58e24ec31441124f3f21384b9 ,\n 34eda1dd66b92a361797c63d57fa19e83c08a1b4 ,\n f601d1954348b71186fa160a0ae6a1f4f1c5aee6 ,\n bc503348a1da573488503cc2819c9e30807bea31 .\n\nBug: 19150481\nChange-Id: I6650ee30a7d261159380fe2119e14379e4dc9970\n" }, { "commit": "3d2c8e74c27efee58e24ec31441124f3f21384b9", "tree": "416a60f70414b026395e3660edeee5e1cb10b6f7", "parents": [ "d834380c94af85b498560f3b5feae21ef7fab1ed" ], "author": { "name": "Mark Mendell", "email": "mark.p.mendell@intel.com", "time": "Tue Jan 13 17:32:55 2015 -0500" }, "committer": { "name": "Mark Mendell", "email": "mark.p.mendell@intel.com", "time": "Fri Jan 23 12:53:45 2015 -0500" }, "message": "ART: Implement X86 hard float (Quick/JNI/Baseline)\n\nUse XMM0-XMM3 as parameter registers for float/double on X86. X86_64\nalready uses XMM0-XMM7 for parameters.\n\nChange the \u0027hidden\u0027 argument register from XMM0 to XMM7 to avoid a\nconflict.\n\nAdd support for FPR save/restore in runtime/arch/x86.\n\nMinimal support for Optimizing baseline compiler.\n\nBump the version in runtime/oat.h because this is an ABI change.\n\nChange-Id: Ia6fe150e8488b9e582b0178c0dda65fc81d5a8ba\nSigned-off-by: Mark Mendell \u003cmark.p.mendell@intel.com\u003e\n" }, { "commit": "e34fa1df67fbe0173b4ea9abddcc3ae3d0537037", "tree": "a5148f079b5671a95f60910c41981ebf91db3a02", "parents": [ "9f06b1946ae3ef1d2fd75bbf7f0a288bc611fe58" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jan 14 14:55:47 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 15 12:23:28 2015 -0800" }, "message": "Print more info in MarkSweep::VerifyRoot\n\nRefactored old root callback to use a new class called RootInfo.\nRootInfo contains all the relevant info related to the root\nassociated with the callback. The MarkSweep::VerifyRoot function\nnow uses this info to print the StackVisitor\u0027s described location\nif the GC root is of the type kRootJavaFrame.\n\nSome other cleanup.\n\nExample output:\nE/art (12167): Tried to mark 0x123 not contained by any spaces\nE/art (12167): Attempting see if it\u0027s a bad root\nE/art (12167): Found invalid root: 0x123 with type RootJavaFrame\nE/art (12167): Location\u003dVisiting method\n\u0027void java.lang.Runtime.gc()\u0027 at dex PC 0xffffffff (native PC 0x0)\nvreg\u003d0\n\n(cherry picked from commit 12f7423a2bb4bfab76700d84eb6d4338d211983a)\n\nBug: 18588862\nChange-Id: Ic5a2781f704e931265ffb3621c2eab4b2e25f60f\n" }, { "commit": "0f9b03c65e0ee8bdc5ddf58af100f5fc356cc98b", "tree": "7cb2063f05a857a18e9ff0b7ffed7f741270cfff", "parents": [ "59b9cf7ec0ccc13df91be0bd5c723b8c52410739" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 12 18:21:07 2015 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 12 18:21:07 2015 +0000" }, "message": "Revert \"ART: Implement hard float for X86\"\n\nThis reverts commit 59b9cf7ec0ccc13df91be0bd5c723b8c52410739.\n\nChange-Id: I08333b528032480def474286dc368d916a07e17f\n" }, { "commit": "59b9cf7ec0ccc13df91be0bd5c723b8c52410739", "tree": "ee929a74e0bb5b308134393e5f0011bdbf6e8f51", "parents": [ "893e8881e31180721512c1b9e5ffacb03aad2e45" ], "author": { "name": "Mark Mendell", "email": "mark.p.mendell@intel.com", "time": "Fri Jan 09 14:44:36 2015 -0500" }, "committer": { "name": "Mark Mendell", "email": "mark.p.mendell@intel.com", "time": "Mon Jan 12 10:36:55 2015 -0500" }, "message": "ART: Implement hard float for X86\n\nUse XMM0-XMM3 as parameter registers for float/double on X86. X86_64\nalready uses XMM0-XMM7 for parameters.\n\nChange the \u0027hidden\u0027 argument register from XMM0 to XMM7 to avoid a\nconflict.\n\nThis change was requested to simplify the Optimizing compiler\nimplementation.\n\nChange-Id: I89ba8ade99b9a8a5b1ad1ee5f5cbfd33d656bfaa\nSigned-off-by: Mark Mendell \u003cmark.p.mendell@intel.com\u003e\n" }, { "commit": "956af0f0cb05422e38c1d22cbef309d16b8a1a12", "tree": "b558c804d206dad8da648b815750f1b3c97610ae", "parents": [ "407d77f344cfbdbbfb50531c5f0766bc0892e2fe" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Dec 11 14:34:28 2014 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Dec 12 09:33:34 2014 -0800" }, "message": "Remove portable.\n\nChange-Id: I3bf3250fa866fd2265f1b115d52fa5dedc48a7fc\n" }, { "commit": "376b2bbf7c39108223a7a01568a7b4b04d84eeac", "tree": "f6fdbe6180c3d330bc24aa35a7d9fa51f3eb1c55", "parents": [ "b1a38e246cfbfb21100d9c9e57f33970e824f075" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 09 14:26:32 2014 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 09 14:49:02 2014 +0000" }, "message": "Ensure stack maps are 4 byte aligned.\n\nWith the recent move to gcc 4.9, we are hitting alignment\nSIGBUS on ARM. The reason is that gcc will optimize two consecutive\n32bits loads into one 64bits load, and the instruction (ldrd)\nwill fail if the data is not aligned.\n\nAlso removed the emission of mapping table when a method is optimized.\nThe information can be found in the StackMap itself.\n\nChange-Id: Icf79406c18a3f4db3c05d52fc2c0dd2e35bf0f8f\n" }, { "commit": "1a102189c1782e003f27e95ce093e854408767f3", "tree": "36b874dcd5d5c8bb7a2c30aa70cff58faaed19a3", "parents": [ "b5fb207ed5e3570ec14e18811b3fa066168fe493" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 17:49:19 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 18:00:07 2014 -0800" }, "message": "Remove MethodHelper.\n\nMove use as a shorty processor to only use in portable.\nMove GetNumberOfReferenceArgsWithoutReceiver to mirror::ArtMethod.\n\nChange-Id: I7ded3d05315c84bce4ab19cb330ef74289da4bb3\n" }, { "commit": "f2247513a54300cfa6a2f23fdbbafead115c99b2", "tree": "16d40b270b8a248ae9464d9aca9cd432c284b135", "parents": [ "384f729a465d0ab3a76faa7ec8fa05ced82b4a85" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 16:17:08 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 16:27:15 2014 -0800" }, "message": "Remove MethodHelper::HasSameNameAndSignature.\n\nMove sole use to a static method with art_method.cc.\n\nChange-Id: I2e7994cc1c31b5ca74df5d7be5538003d4ed0150\n" }, { "commit": "a0485607a4a4d8c683a9849f6f20902c4e1da7a4", "tree": "bbfa8e6182b702f161ea07efd53975fab1b6b736", "parents": [ "08f1f50d6c2e8f247b8f5f19711d75a792851c7a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 15:48:04 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 15:57:13 2014 -0800" }, "message": "Move GetClassFromTypeIdx to ArtMethod.\n\nMove GetClassFromTypeIdx out of MethodHelper into ArtMethod in\npreparation for the removal of MethodHelper.\n\nChange-Id: I9c03dd8c821944c606ea08cdf92afc80c4275247\n" }, { "commit": "e0a02dabedd7db2c511a513fb48d7e39ed3dd9c0", "tree": "70ee0e4caa2cc26c661b6cb0231b8423506e50fa", "parents": [ "e94652f1e321b2c8b71acbe5b07d2ebf69fbdb99" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 14:10:53 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 14:21:56 2014 -0800" }, "message": "Move FindDexMethodIndexInOtherDexFile into ArtMethod.\n\nMove FindDexMethodIndexInOtherDexFile out of MethodHelper into ArtMethod in\npreparation for the removal of MethodHelper.\nTidy ClassLinker::ResolveMethod so that all exception paths flow through the\nexception pending assertion.\nTidy artQuickResolutionTrampoline to be more explicit about variable names and\nonly update the dex cache if necessary.\n\nChange-Id: I3e48eb4f6c1291533067c1b53efe90c53bfcaea8\n" }, { "commit": "022b46cb09a6189797fb3e77dbe7673a3d59249f", "tree": "f3440544c000174e72508ff3561e0e984f9f09f9", "parents": [ "124cd8f768436e082ca0dfed8f1e4f4713220e44", "e14f2b3047fd310c5df65cf13470aa2786522a3b" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Nov 22 02:13:20 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Sat Nov 22 02:13:21 2014 +0000" }, "message": "Merge \"Fixed ArtMethod::GetQuickFrameInfo() for proxy methods\"" }, { "commit": "a7dd0386f35c0ba4aef3f5b16bc84c6f4e2fc702", "tree": "86bcc8c05fb3f868dbdae3dcacbf95de880d4d64", "parents": [ "af6dad6cbb6685f92d7274f9e8c45baa98232c79" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Nov 20 17:08:58 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Nov 21 10:11:41 2014 -0800" }, "message": "Fix oatdump to use OatHeader pointer size\n\nBug: 18473190\nChange-Id: If505b4f62105899f4f1257d3bccda3e6eb0dcd7c\n\n(cherry picked from commit c934e483ceabbd589422beea1fa35f5182ecfa99)\n" }, { "commit": "2d7210188805292e463be4bcf7a133b654d7e0ea", "tree": "7705a3bf841ae44b2396728fa22ed0b5dcb44dbf", "parents": [ "e0491682d101c69bf88c3c24a965312129cbfa38" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 10 11:08:06 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 18 12:27:37 2014 -0800" }, "message": "Change 64 bit ArtMethod fields to be pointer sized\n\nChanged the 64 bit entrypoint and gc map fields in ArtMethod to be\npointer sized. This saves a large amount of memory on 32 bit systems.\nReduces ArtMethod size by 16 bytes on 32 bit.\n\nTotal number of ArtMethod on low memory mako: 169957\nImage size: 49203 methods -\u003e 787248 image size reduction.\nZygote space size: 1070 methods -\u003e 17120 size reduction.\nApp methods: ~120k -\u003e 2 MB savings.\n\nSavings per app on low memory mako: 125K+ per app\n(less active apps -\u003e more image methods per app).\n\nSavings depend on how often the shared methods are on dirty pages vs\nshared.\n\nTODO in another CL, delete gc map field from ArtMethod since we\nshould be able to get it from the Oat method header.\n\nBug: 17643507\n\nChange-Id: Ie9508f05907a9f693882d4d32a564460bf273ee8\n\n(cherry picked from commit e832e64a7e82d7f72aedbd7d798fb929d458ee8f)\n" }, { "commit": "de58ab2c03ff8112b07ab827c8fa38f670dfc656", "tree": "c872bfbcad1e90845008140bbddcc43e56dc19d2", "parents": [ "3ed86e4e98dfe1b05c9a03aa2aee42c145a018c3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 05 12:46:03 2014 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Nov 06 16:49:52 2014 +0000" }, "message": "Implement try/catch/throw in optimizing.\n\n- We currently don\u0027t run optimizations in the presence of a try/catch.\n- We therefore implement Quick\u0027s mapping table.\n- Also fix a missing null check on array-length.\n\nChange-Id: I6917dfcb868e75c1cf6eff32b7cbb60b6cfbd68f\n" }, { "commit": "6b14d55c812ee3438e87db249d5a144e0397fb3b", "tree": "6ed2472193e930269a33fbc50ccb1e4d4c2d43a7", "parents": [ "ded66a01f81812e0129d17c3d08d5eda18433062" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 28 21:50:58 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 28 21:50:58 2014 -0700" }, "message": "Move GetNameAsString out of MethodHelper into ArtMethod.\n\nChange-Id: I98b8ed9d91741267659616cb76ce5f6fec4feae8\n" }, { "commit": "03b6eafba8ace9a9c4d5ee9c47723d1910ccd7a8", "tree": "3451b85f23b14cd78ab6bd1257528ba2eb6769e5", "parents": [ "4816ecfc1b2d544685ec5edcdeaad6870f6bfd7e" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 28 09:34:57 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 28 11:04:43 2014 -0700" }, "message": "Tidy MethodProtoHelper.\n\nMove to place of only use, class_linker.cc. Be lazy in computing the name.\n\nChange-Id: I1438efbda58369ddd0ac36eda8a5a0a6c6fdff77\n" }, { "commit": "2d2621a1463d2f3f03fa73503fa42e43657cdcfc", "tree": "1f7cf393693d72db5c186f99b04ac511958c6418", "parents": [ "ac293277b69882105810fabd6c53f55de58823fe" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 23 16:48:06 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 27 16:49:43 2014 -0700" }, "message": "Optimize method linking\n\nAdded more inlining, removed imt array allocation and replaced it\nwith a handle scope. Removed some un-necessary handle scopes.\n\nAdded logic to base interface method tables from the superclass so\nthat we dont need to reconstruct for every interface (large win).\n\nFacebook launch Dalvik KK MR2:\nTotalTime: 3165\nTotalTime: 3652\nTotalTime: 3143\nTotalTime: 3298\nTotalTime: 3212\nTotalTime: 3211\n\nFacebook launch TOT before:\nWaitTime: 3702\nWaitTime: 3616\nWaitTime: 3616\nWaitTime: 3687\nWaitTime: 3742\nWaitTime: 3767\n\nAfter optimizations:\nWaitTime: 2903\nWaitTime: 2953\nWaitTime: 2918\nWaitTime: 2940\nWaitTime: 2879\nWaitTime: 2792\n\nLinkInterfaceMethods no longer one of the hottest methods, new list:\n4.73% art::ClassLinker::LinkVirtualMethods(art::Thread*, art::Handle\u003cart::mirror::Class\u003e)\n3.07% art::DexFile::FindClassDef(char const*) const\n2.94% art::mirror::Class::FindDeclaredStaticField(art::mirror::DexCache const*, unsigned int)\n2.90% art::DexFile::FindStringId(char const*) const\n\nBug: 18054905\nBug: 16828525\n\n(cherry picked from commit 1fb463e42cf1d67595cff66d19c0f99e3046f4c4)\n\nChange-Id: I27cc70178fd3655fbe5a3178887fcba189d21321\n" }, { "commit": "5667fdbb6e441dee7534ade18b628ed396daf593", "tree": "a06fe0a79b3465571556d13f509daf1f664fc614", "parents": [ "b62ff579cd870b0bf213765b07d7b404d15ece7b" ], "author": { "name": "Zheng Xu", "email": "zheng.xu@arm.com", "time": "Thu Oct 23 18:29:55 2014 +0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 24 16:27:22 2014 -0700" }, "message": "ARM: Use hardfp calling convention between java to java call.\n\nThis patch default to use hardfp calling convention. Softfp can be enabled\nby setting kArm32QuickCodeUseSoftFloat to true.\n\nWe get about -1 ~ +5% performance improvement with different benchmark\ntests. Hopefully, we should be able to get more performance by address the left\nTODOs, as some part of the code takes the original assumption which is not\noptimal.\n\nDONE:\n1. Interpreter to quick code\n2. Quick code to interpreter\n3. Transition assembly and callee-saves\n4. Trampoline(generic jni, resolution, invoke with access check and etc.)\n5. Pass fp arg reg following aapcs(gpr and stack do not follow aapcs)\n6. Quick helper assembly routines to handle ABI differences\n7. Quick code method entry\n8. Quick code method invocation\n9. JNI compiler\n\nTODO:\n10. Rework ArgMap, FlushIn, GenDalvikArgs and affected common code.\n11. Rework CallRuntimeHelperXXX().\n\nChange-Id: I9965d8a007f4829f2560b63bcbbde271bdcf6ec2\n" }, { "commit": "1d8cdbc5202378a5f1a4b3a1fba610675ed4dcd5", "tree": "6a1b0f49aee5a97b513bd0becc734d284aa7fb65", "parents": [ "1c1786f193323d3bd706463894001117f3471595" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Sep 22 22:51:09 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Oct 20 16:01:28 2014 -0700" }, "message": "Refactor quick entrypoints\n\nRemove FinishCalleeSaveFrameSetup.\nAssembly routines write down anchor into TLS as well as placing runtime\nmethod in callee save frame.\nSimplify artSet64InstanceFromCode by not computing the referrer from the\nstack in the C++ code.\nMove assembly offset tests next to constant declaration and tidy arch_test.\n\nChange-Id: Iededeebc05e54a1e2bb7bb3572b8ba012cffa1c8\n" }, { "commit": "e14f2b3047fd310c5df65cf13470aa2786522a3b", "tree": "1bb4ae00f0fd9b1057e3af550d28dcf86a9992c7", "parents": [ "8946e41dcce414dc3359c23d93e001f91d186612" ], "author": { "name": "Daniel Mihalyi", "email": "daniel.mihalyi@mattakis.com", "time": "Fri Oct 10 18:24:11 2014 +0200" }, "committer": { "name": "Daniel Mihalyi", "email": "daniel.mihalyi@mattakis.com", "time": "Mon Oct 20 12:37:36 2014 +0200" }, "message": "Fixed ArtMethod::GetQuickFrameInfo() for proxy methods\n\nIf we call ArtMethod::GetQuickFrameInfo() on a proxy constructor when\ninstrumentation stubs are installed, the entry point returned by\nInstrumentation::GetQuickCodeFor() would be a artQuickProxyInvokeHandler,\nand not a regular quick oat code pointer. This patch adds special handling\nfor proxy constructors to always compute frame info from the regular quick\ncode entry point. (These entry points are never updated by instrumentation\nfor proxy methods.)\n\nChange-Id: I8e51f5622b6d327037d5c968b07505b2bf4ac737\nSigned-off-by: Daniel Mihalyi \u003cdaniel.mihalyi@mattakis.com\u003e\n" }, { "commit": "6f3dbbadf4ce66982eb3d400e0a74cb73eb034f3", "tree": "f7a20779e4d665f948c5fbcd26dac0071dafb8d4", "parents": [ "2df6840f68dd18d7dd8dbf53f8b6181bbfdc4fc4" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 14 17:41:57 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 16 19:27:28 2014 -0700" }, "message": "Make ART compile with GCC -O0 again.\n\nTidy up InstructionSetFeatures so that it has a type hierarchy dependent on\narchitecture.\nAdd to instruction_set_test to warn when InstructionSetFeatures don\u0027t agree\nwith ones from system properties, AT_HWCAP and /proc/cpuinfo.\nClean-up class linker entry point logic to not return entry points but to\ntest whether the passed code is the particular entrypoint. This works around\nimage trampolines that replicate entrypoints.\nBug: 17993736\n\nChange-Id: I5f4b49e88c3b02a79f9bee04f83395146ed7be23\n" }, { "commit": "6b604a1b0289e5e7211c2e5f8c4f395f51de7c3d", "tree": "2789123796907a61bdfbc9c6f5727ac6cda8f559", "parents": [ "e7e34d08cffbd9a43ca9b097f8c7fdd54fbeaa27" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 25 15:35:37 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 25 16:51:29 2014 -0700" }, "message": "Fix Class::IsInSamePackage to not read beyond the end of a StringPiece.\n\nFix length of string piece to be size_type rather than int because expecting\nnegatively sized strings is a bad idea and we should use unsigned types to\ndefensively guard against mistakes.\nRemove max_size and capacity since the return type is inconsistent between\nGoogle and the STL, and we don\u0027t need the functions.\nAdd a bound check in libartd to operator[] accesses.\n\nChange-Id: I1b87a03d8fbd95e7dbb106745e304d1083898075\n" }, { "commit": "5a4b8a236030460651a3136397d23ca6744e7eb7", "tree": "0e43891398e416d3fa77c7de391bf4db4408e8ee", "parents": [ "19f7c95491a053b818f914137fa73df0517b8792" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 11 08:30:08 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 15 19:50:12 2014 -0700" }, "message": "ART: Rename Handle hierarchy\n\nBring the names in line with normal OO principles: ConstHandle\nbecomes Handle, and Handle becomes MutableHandle.\n\nChange-Id: I0f018eb7ba28bc422e3a23dd73a6cbe6fc2d2044\n" }, { "commit": "9bdec8839c1a34d09ec3e8916a4bd3a022ac45b6", "tree": "bd757f04809fdbfebd501594731e3eac9e5fd487", "parents": [ "ee736b29675455ab0be615581145aa239ce7a6b3" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Aug 15 17:11:12 2014 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Aug 18 11:25:13 2014 -0700" }, "message": "Check that we don\u0027t accidentally invoke compiled code when -Xint.\n\nThe heap poisoning breakge (b/17018234) would have been detected with\nthis check.\n\nBug: 17018234\nChange-Id: If4827ea1b02396d41012f0955e55c887387a0565\n" }, { "commit": "b74ff132f217776c1ab698712bc2dbc526297288", "tree": "86da3e98cbd942bb4c2fda6f0f50fd44c1c0a101", "parents": [ "826f1038187e00399a8c6e0044ad0ea29c0ba000", "648d7112609dd19c38131b3e71c37bcbbd19d11e" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Thu Aug 14 00:01:12 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Aug 12 19:23:43 2014 +0000" }, "message": "Merge \"Reduce stack usage for overflow checks\"" }, { "commit": "648d7112609dd19c38131b3e71c37bcbbd19d11e", "tree": "54062831327c660acb309e877e8d8df9ba0c2d5d", "parents": [ "99c251bbd225dd97d0deece29559a430b12a0b66" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Fri Jul 25 16:15:27 2014 -0700" }, "committer": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Wed Aug 13 09:01:41 2014 -0700" }, "message": "Reduce stack usage for overflow checks\n\nThis reduces the stack space reserved for overflow checks to 12K, split\ninto an 8K gap and a 4K protected region. GC needs over 8K when running\nin a stack overflow situation.\n\nAlso prevents signal runaway by detecting a signal inside code that\nresulted from a signal handler invokation. And adds a max signal count to\nthe SignalTest to prevent it running forever.\n\nAlso reduces the number of iterations for the InterfaceTest as this was\ntaking (almost) forever with the --trace option on run-test.\n\nBug: 15435566\n\nChange-Id: Id4fd46f22d52d42a9eb431ca07948673e8fda694\n" }, { "commit": "1ff3c98775a4577cf053dba9a0c2d5c21c07b298", "tree": "2d09c27c69678b53b4c9dc486024f3547efd4bca", "parents": [ "99c251bbd225dd97d0deece29559a430b12a0b66" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 12 02:30:58 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 12 16:08:05 2014 -0700" }, "message": "Avoid use of std::string where we have const char*.\n\nRemoving the ClassHelper caused std::string creation for all calls to\nClass::GetDescriptor and a significant performance regression. Make the\nstd::string an out argument so the caller can maintain it and its life time\nwhile allowing GetDescriptor to return the common const char* case.\n\nDon\u0027t generate GC maps when compilation is disabled.\n\nRemove other uses of std::string that are occuring on critical paths.\nUse the cheaper SkipClass in CompileMethod in CompilerDriver.\nSpecialize the utf8 as utf16 comparison code for the common shorter byte\nencoding.\nForce a bit of inlining, remove some UNLIKELYs (they are prone to pessimizing\ncode), add some LIKELYs.\n\nx86-64 host 1-thread interpret-only of 57 apks:\nBefore: 29.539s\nAfter: 23.467s\n\nRegular compile:\nBefore: 1m35.347s\nAfter: 1m20.056s\n\nBug: 16853450\nChange-Id: Ic705ea24784bee24ab80084d06174cbf87d557ad\n" }, { "commit": "f0a3f09c3d54646166a55c05a6b39c7dd504129c", "tree": "35a76479f1b774ae100727547a48da5bc0878e66", "parents": [ "484e2c2d3531e5bb36f0e1e12f26c708939c6579" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Jul 24 16:26:09 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Aug 06 12:49:57 2014 -0700" }, "message": "Fix proxy tracing and enable tests that now work with tracing.\n\nAlso updates proxy_test to generate an image for GetQuickOatCodeFor.\n\nBug: 16386215\nChange-Id: Ie7daad3d73ea7b60187bc1e7037ade0df8277107\n" }, { "commit": "94f7b49578b6aaa80de8ffed230648d601393905", "tree": "cfc69e453faefee38178ceb85378e1f0f1e17812", "parents": [ "8df73882c60451e7f789bf9b1f3db2d7dc228640" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Jul 22 18:08:23 2014 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Jul 29 13:30:46 2014 -0700" }, "message": "Add GcRoot to clean up and enforce read barriers.\n\nIntroduce a value-type wrapper around Object* for GC roots so that 1)\nwe won\u0027t have to directly add the read barrier code in many places and\n2) we can avoid accidentally bypassing/missing read barriers on GC\nroots (the GcRoot interface ensures that the read barrier is executed\non a read).\n\nThe jdwp test passed.\n\nBug: 12687968\nChange-Id: Ib167c7c325b3c7e3900133578815f04d219972a1\n" }, { "commit": "167cc7c33f7100e3f7acc1594c066daa0122e27a", "tree": "d265344c32c5bf37d356fcb4871f57a1250d16b6", "parents": [ "0398e171f206cd3b140a358ac31b0a3760380df1" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jul 29 08:33:44 2014 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jul 29 08:33:44 2014 +0000" }, "message": "Revert \"Fix proxy tracing and enable tests that now work with tracing.\"\n\nThis reverts commit 0398e171f206cd3b140a358ac31b0a3760380df1.\n\nChange-Id: I1346ab01485cc7207be0ecb4d8788c500c0df903\n" }, { "commit": "0398e171f206cd3b140a358ac31b0a3760380df1", "tree": "db5b34142d812f95581a864b681940efc94c6945", "parents": [ "e5647289ccc7dafd8b91771a7bd1cd19416aed66" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Jul 24 16:26:09 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Jul 28 18:33:04 2014 -0700" }, "message": "Fix proxy tracing and enable tests that now work with tracing.\n\nBug: 16386215\nChange-Id: Iec2a372c921caceb050c6baf72d48b3d822899a4\n" }, { "commit": "2cdbad7c62f126581ec5177104de961c4d71adaa", "tree": "6283b87bc9b65249decac9b0e38fe396782b8d7e", "parents": [ "c62944adcd427dca55489d234428da47c43a0aab" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Jul 16 10:44:41 2014 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Tue Jul 22 15:03:50 2014 -0700" }, "message": "Set vtable in class object to null after linking.\n\nThis is follow-up work of embedding imt and vtable for\nfaster interface/virtual call dispatching.\nOnce vtable becomes embedded, the original vtable is nulled.\n\nChange-Id: I307696657d1e283654169dbecb8f7815c42bbabc\n" }, { "commit": "22d5e735f403c57525fe868304c7123f0ce66399", "tree": "2458684efa56f0b800dd75a9dedd0449f76f581f", "parents": [ "fbde4dd1cb6db729e3f3ee5bdae0cdd824d73054" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 22:23:51 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jul 16 06:13:46 2014 -0700" }, "message": "Remove object_utils.h.\n\nBreak into object_lock, field_helper and method_helper.\nClean up header files following this.\nAlso tidy some of the Handle code in response to compiler errors when resolving\nthe changes in this CL.\n\nChange-Id: I73e63015a0f02a754d0866bfaf58208aebcaa295\n" }, { "commit": "e63db27db913f1a88e2095a1ee8239b2bb9124e8", "tree": "893dee6783bca6717259321a6e4ba029c9c123e2", "parents": [ "07b8441303ea82fca3cb85d71ecf8752d73cedd7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 15:36:11 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 17:07:49 2014 -0700" }, "message": "Break apart header files.\n\nCreate libart-gtest for common runtime and compiler gtest routines.\nRename CompilerCallbacksImpl that is quick compiler specific.\nRename trace clock source constants to not use the overloaded profiler term.\n\nChange-Id: I4aac4bdc7e7850c68335f81e59a390133b54e933\n" }, { "commit": "98d1cc8033251c93786e2fa8c59a2e555a9493be", "tree": "f0a76b8fff060ee484af09028da65a8339d57057", "parents": [ "aebf3cda094f34cf846d19a7724bdc8005267c95" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu May 15 17:02:16 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jul 11 17:17:10 2014 -0700" }, "message": "Improve performance of invokevirtual/invokeinterface with embedded imt/vtable\n\nAdd an embedded version of imt/vtable into class object. Both tables start at\nfixed offset within class object so method/entry point can be loaded directly\nfrom class object for invokeinterface/invokevirtual.\n\nBug: 8142917\nChange-Id: I4240d58cfbe9250107c95c0708c036854c455968\n" }, { "commit": "9f1020305292a21fd14a402b189c765a125226ab", "tree": "6b730cbe56ded370d1b4293629826ad2c7b06f7f", "parents": [ "bc72903b909f5147b8cb207f3e5d02a8ef85e4e7" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri May 23 08:59:42 2014 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Jun 11 14:29:00 2014 +0200" }, "message": "Fix exception reporting from interpreter\n\nTo comply with JDWP exception report rules, we must report an exception at the\nlocation of the throw (or the first instruction encountered after a native\ncall). To do this, we use the CatchLocationFinder visitor to look for a catch\nhandler until we reach a native frame or the top frame.\n\nBecause interpreter handles pending exception on a method-by-method basis, we\nneed a flag to remember we already reported the exception and avoid reporting\nit multiple times when unwinding methods. The drawback is we need to maintain\nthe state of this flag. We clear it when the exception is cleared. In the case\nwe temporarily clear the exception (when finding a catch handler for instance),\nwe restore the flag to its previous value at the same time we restore the\npending exception.\n\nBump oat version to force recompilation because we modify Thread offsets.\n\nBug: 14402770\nChange-Id: Ic059c58f80b2023b118038301f8f0a24f1e18241\n" }, { "commit": "bfd9a4378eacaf2dc2bbe05ad48c5164fc93c9fe", "tree": "3d3f667c8232a9c1bb6fe9daea0d364f9ae01d8c", "parents": [ "2e1ca953c7fb165da36cc26ea74d3045d7e272c8" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 21 17:43:44 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 09 12:46:32 2014 -0700" }, "message": "Change MethodHelper to use a Handle.\n\nAdded ConstHandle to help prevent errors where you modify the value\nstored in the handle of the caller. Also fixed compaction bugs\nrelated to not knowing MethodHelper::GetReturnType can resolve types.\nThis bug was present in interpreter RETURN_OBJECT.\n\nBug: 13077697\n\nChange-Id: I71f964d4d810ab4debda1a09bc968af8f3c874a3\n" }, { "commit": "822266b9dc7d8dc9e084192ae0f4bc95af4e8cf8", "tree": "6f9b2470d9f1bfa74bf53da3b3794ec8e6bf8d6f", "parents": [ "31e7fcb904f03a504f082d25814ac4644b5073e4" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 29 16:55:06 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 29 16:55:06 2014 -0700" }, "message": "Ignore catch blocks whose type can\u0027t be resolved.\n\nReverts change 72b3e430d880ef57eaa6a34a0822165994052202 but keeps unit test and\nmissing delete that would fail assertions on long jump context recycling.\n\nChange-Id: I926755e8b831b208aa7e1ce46421bef3793a1441\n" }, { "commit": "0cd81352a7c06e381951cea1b104fd73516f4341", "tree": "fcc8ff9cdeb608d0913097aa1fb64d1879f11ffa", "parents": [ "dfd301f19800a4f168283cf2d0b15b2c09071955" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 22 16:48:55 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 23 12:55:45 2014 -0700" }, "message": "Revert \"Revert \"Fix an outstanding compaction bug in interpreter.\"\"\n\nFixed the generic trampoline to not use ToJObject when unnecessary.\n\nBug: 15167269\n\nThis reverts commit 3bdb873122964da7937eb070cbcf2ef638a8e459.\n\nChange-Id: I0525d0e0f3afb753c770e1572070a0fa22b02271\n" }, { "commit": "3bdb873122964da7937eb070cbcf2ef638a8e459", "tree": "3d29123f33853e11adb67bdd6ac56f19ee798a1f", "parents": [ "e09ae0920be57760fb390b6944bce420fa0b5582" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 22 22:06:51 2014 +0000" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 22 22:06:51 2014 +0000" }, "message": "Revert \"Fix an outstanding compaction bug in interpreter.\"\n\nThis reverts commit e09ae0920be57760fb390b6944bce420fa0b5582.\n\nChange-Id: I48036306130d5ccfec683d0dc3e9a642a02ee9c1\n" }, { "commit": "e09ae0920be57760fb390b6944bce420fa0b5582", "tree": "acc40266093df4289ffb6728c979cafd6b5114d2", "parents": [ "b8033db2a8dc6f7c7e29b1552177542964f56e44" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 15 12:39:19 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 22 10:47:44 2014 -0700" }, "message": "Fix an outstanding compaction bug in interpreter.\n\nFixed a bug in DoFieldPut where the FieldHelper GetType could cause\nthread suspension which would result in a stale obj.\n\nAdded more handles in the class linker to facilitate moving fiels\nand methods in the future.\n\nRemoved un-necessarly passing handle references since these are value\ntypes and don\u0027t need to be passed by reference.\n\nAdded a special NullHandle type which allows null handles without a\nhandle scope.\n\nChange-Id: I1b51723920a2e4f4f8b2907066f578a3e879fd5b\n" }, { "commit": "4c1c510bea6f20f4d8b09e15547cd2967ad51c88", "tree": "f3d15233294e46db0f43e83997e71a8bb990745e", "parents": [ "fdfe733d8fad10fb8370f468fbd5905416f338fa" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 14 16:51:16 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 20 14:48:56 2014 +0100" }, "message": "Improve stack walk performance.\n\nMove a few functions from art_method.cc to art_method-inl.h\nand introduce new overloads that take already known partial\nresults; do not rely on the compiler to magically merge the\nidentical but non-trivial paths. The partial results are\nDCHECKed to be correct.\n\nChange-Id: I342c3001bbff08a2bbbb9a7b62ae67188ad8cffc\n" }, { "commit": "2b7c4d196c8abe32f4ca633534917da9de53c359", "tree": "f14dc45f53d5681a5c0e57b8a8cc6b35eee6e896", "parents": [ "84e524207b23d58a1b1e5f4443000ccac97c4184" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon May 19 10:52:16 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon May 19 15:19:28 2014 -0700" }, "message": "Don\u0027t get and restore thread state for ScopedFastNativeObjectAccess.\n\nBefore we would ensure that we were runnable for fast native object\naccess. However, these are done when you are already runnable.\n\nChange-Id: Ia4c6e4c83d146fe2a988b37b3133ca46b0f0fa42\n" }, { "commit": "72b3e430d880ef57eaa6a34a0822165994052202", "tree": "b102b12d67e6cbc5cf0e6dc4e4ca147325264a43", "parents": [ "d0916f36d27b643bca970f3645c38f44270c74ef" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue May 13 21:42:05 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 14 17:33:52 2014 -0700" }, "message": "ART: Fix typo in ArtMethod::FindCatchBlock\n\nThe thrown exception is always resolved, as we have an instance of\nit. What is potentially not resolved is the catch handler\u0027s exception\ntype.\n\nThe resolution failure will trigger a NoClassDefFoundError, which\nshould replace the original exception. For this, the API has to be\nchanged a little bit to tell callers that there was this change.\n\nChange-Id: Id51d54a15c732ed175eb617b3b0331b89cbb2051\n" }, { "commit": "eb8167a4f4d27fce0530f6724ab8032610cd146b", "tree": "bcfeaf13ad78f2dd68466bbd0e20c71944f7e854", "parents": [ "6fb66a2bc4e1c0b7931101153e58714991237af7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 07 15:43:14 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue May 13 14:45:54 2014 -0700" }, "message": "Add Handle/HandleScope and delete SirtRef.\n\nDelete SirtRef and replaced it with Handle. Handles are value types\nwhich wrap around StackReference*.\n\nRenamed StackIndirectReferenceTable to HandleScope.\n\nAdded a scoped handle wrapper which wraps around an Object** and\nrestores it in its destructor.\n\nRenamed Handle::get -\u003e Get.\n\nBug: 8473721\n\nChange-Id: Idbfebd4f35af629f0f43931b7c5184b334822c7a\n" }, { "commit": "7624d25dad2d1ba25969ae704fccf68649103ae5", "tree": "de72194b76a4e23e0b15ec4085447ae7e4425815", "parents": [ "e1910f1d802dff79bba5ef61e1c4fd0b95f6e5b0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 02 14:40:15 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 13 11:43:22 2014 +0100" }, "message": "Move quick frame info to OatQuickMethodHeader.\n\nRename OatMethodHeader to OatQuickMethodHeader, move frame\ninfo from OatMethodOffsets to OatQuickMethodHeader. Retrieve\nthe info from other places for non-quick methods (portable\ncompiled bytecode or jni stub, generic jni, runtime,\nabstract and proxy).\n\nThis change has a libcore/ companion CL\n \"Remove ArtMethod\u0027s quick fields for frame size and spills.\"\n https://android-review.googlesource.com/94164\n\nBug: 11767815\nChange-Id: I0e31a7875d76732e1ec479c86b9b5ca01203507f\n" }, { "commit": "5d27fafdf03f259e92eaee9f6319b9349cc8d62e", "tree": "53afb1174be2ae235e41c5ac9eeb1e8d67e3c4e1", "parents": [ "a54c22c0041306f6881694f4c305e655b10e37be" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri May 02 17:17:18 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri May 02 23:30:03 2014 -0700" }, "message": "Allow ArtMethod::Invoke in unstarted runtimes.\n\nChange-Id: I0141f4daef4751589d03d27484eb65c811b14f27\n" }, { "commit": "b0fa5dc7769c1e054032f39de0a3f6d6dd06f8cf", "tree": "839d13ebfa7170967dd9b4abd434b7abda53da99", "parents": [ "948740c1938860df055ddc801f20fd1707331e38" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Apr 28 16:47:08 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Apr 29 14:36:28 2014 -0700" }, "message": "Force inlining on trivial accessors.\n\nMake volatility for GetFieldObject a template parameter.\nMove some trivial mirror::String routines to a -inl.h.\n\nBug: 14285442\n\nChange-Id: Ie23b11d4f18cb15a62c3bbb42837a8aaf6b68f92\n" }, { "commit": "990d46f3333dce06a640ced697ee069330a73d7c", "tree": "7caf896723665029823c73e2bec87e6db97ec428", "parents": [ "bf25f7e3a007ecfe4b2bcfa0a9abcb784ff54e26", "8a630577ed2d9e9571c3434c505e5de223b23c07" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Apr 28 09:09:06 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Apr 28 09:09:07 2014 +0000" }, "message": "Merge \"Move mapping table and vmap table offsets to OatMethodHeader.\"" }, { "commit": "8a630577ed2d9e9571c3434c505e5de223b23c07", "tree": "106367100c639011f0abb72b3b0e227c0764e8e0", "parents": [ "96c6ab93336b972a38bd2448bcccf19188b8389b" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Apr 09 18:45:35 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Apr 25 12:24:46 2014 +0100" }, "message": "Move mapping table and vmap table offsets to OatMethodHeader.\n\nThis change has a libcore/ companion CL\n \"Remove ArtMethod\u0027s quick fields mapping table and vmap table.\"\n https://android-review.googlesource.com/91254\n\nBug: 11767815\nChange-Id: I46ce2067e1ecd915da3890606498e31ffc332813\n" }, { "commit": "fd3077e4b9ebadd281777310d26e64443858f653", "tree": "c63911af29580ab8c0fc8dd29171e626871449aa", "parents": [ "a08ec9b372d4f5e918b3d68499fbd1731180cd98" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Apr 23 10:32:43 2014 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Fri Apr 25 10:36:05 2014 +0200" }, "message": "Refactor exception handling for deoptimization\n\nThis CL refactors the exception handling (on the quick side) by isolating the\nsearch of catch handler and the preparation of deoptimization.\n\nWe rename the CatchFinder class to QuickExceptionHandler so it\u0027s less specific\nto catch handler search.\n\nFinding catch handler happens in QuickExceptionHandler::FindCatch. Since the\nCatchBlockStackVisitor resolves exception types, it may cause thread suspension\nand breaks the assertion current thread can\u0027t be suspended. Therefore, we place\nthe exception in a SirtRef (while it is detached from the current thread) and\nremove the thread suspension assertion.\n\nDeoptimization now happens in QuickExceptionHandler::DeoptimizeStack. It uses\nthe new DeoptimizeStackVisitor class to create shadow frames.\n\nWe also add the Thread::GetDeoptimizationException method to get the definition\nof the fake exception in only one place.\n\nChange-Id: I01b19fa72af64329b5c3b6c7f0c3339d2d724978\n" }, { "commit": "a08ec9b372d4f5e918b3d68499fbd1731180cd98", "tree": "4c21e507ec2b2ca36011313d9a40d151e23f6193", "parents": [ "4af159bfcc66c6e90a2aee4a2035a996a399e7ed", "aa961918da5142220029da2809287e0dd537a5d7" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Apr 23 00:16:15 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Apr 23 00:16:15 2014 +0000" }, "message": "Merge \"Fix FindCatchBlock to work in -Xverify:none mode.\"" }, { "commit": "aa961918da5142220029da2809287e0dd537a5d7", "tree": "93cdad41650f353e1c7d82aaa8894037b12bb2a0", "parents": [ "82b1a81890970a8b07f9132aeae537a6c43df6b0" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Apr 22 13:54:32 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Apr 22 17:03:21 2014 -0700" }, "message": "Fix FindCatchBlock to work in -Xverify:none mode.\n\nFindCatchBlock now uses ResolveType to get the exception type,\nsince it might not be able to find it in the dex cache.\n\nBug: 13948502\nChange-Id: Ia6f1c7dc743206ae1c8551bf6239f48ee4d3a784\n" }, { "commit": "987560fee798e48fb725c44b796d8ca7a5872ad6", "tree": "d61f35a129847f3f5fbed1436649e75acd940b3f", "parents": [ "44b0053fdb7ad8a30138d29f714172a7dc69efb8" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Apr 22 11:42:59 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Apr 22 14:52:19 2014 -0700" }, "message": "Remove support for app JNI workarounds.\n\nChange-Id: I4396df7e93fcace4b5b19c2c387e5c30089182a6\n" }, { "commit": "62f0512bf6d9bc6141358bf22e93afa70dc58b1a", "tree": "7fdc0b9be0df2da5ff2575862a24db5c10eae9dc", "parents": [ "215f3e60cd97471c0843eae22a041c335e89071c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 21 11:21:29 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 21 11:25:56 2014 -0700" }, "message": "Improvements to Field.get/set.\n\nAvoid unnecessary repeated computation in Field.get/set.\nRefactor FromReflectedField and FromReflectedMethod into common helpers in\nmirror::ArtField and mirror::ArtMethod, and make use of thereby avoiding\ntransitions through JNI.\nAvoid JNI use from within FromReflectedField and FromReflectedMethod.\nTidy up Field.get/set wrt moving collector support.\nBug: 12189533\n\nChange-Id: I643ab3474bade4abac3a3ae2b6e373b2bb0891c8\n" }, { "commit": "b95a5345ae4217b70ca36f0cced92f68dda7caf5", "tree": "d9c76db71eb8dd3a668643781eefe6152888fd1d", "parents": [ "66e4c3e96dccdec7423d673ad6bbf7821a776651" ], "author": { "name": "Stuart Monteith", "email": "stuart.monteith@arm.com", "time": "Wed Mar 12 13:32:32 2014 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Mar 19 17:17:55 2014 -0700" }, "message": "AArch64: Add arm64 runtime support.\n\nAdds support for arm64 to ART. Assembler stubs are sufficient for\ndown calls into interpreter. JNI compiler and generics are not finished.\n\nBasic Generic JNI functionality.\n\nChange-Id: I4a07c79d1e037b9f5746673480e32cf456867b82\n" }, { "commit": "b373e091eac39b1a79c11f2dcbd610af01e9e8a9", "tree": "034d820c4829e0dcf6161473cc39f7250123bfaa", "parents": [ "9545a446e99b22248099fe66f5f9431530c20851" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Thu Feb 20 16:06:36 2014 -0800" }, "committer": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Thu Mar 13 12:21:15 2014 -0700" }, "message": "Implicit null/suspend checks (oat version bump)\n\nThis adds the ability to use SEGV signals\nto throw NullPointerException exceptions from Java code rather\nthan having the compiler generate explicit comparisons and\nbranches. It does this by using sigaction to trap SIGSEGV and when triggered\nmakes sure it\u0027s in compiled code and if so, sets the return\naddress to the entry point to throw the exception.\n\nIt also uses this signal mechanism to determine whether to check\nfor thread suspension. Instead of the compiler generating calls\nto a function to check for threads being suspended, the compiler\nwill now load indirect via an address in the TLS area. To trigger\na suspend, the contents of this address are changed from something\nvalid to 0. A SIGSEGV will occur and the handler will check\nfor a valid instruction pattern before invoking the thread\nsuspension check code.\n\nIf a user program taps SIGSEGV it will prevent our signal handler\nworking. This will cause a failure in the runtime.\n\nThere are two signal handlers at present. You can control them\nindividually using the flags -implicit-checks: on the runtime\ncommand line. This takes a string parameter, a comma\nseparated set of strings. Each can be one of:\n\nnone switch off\nnull null pointer checks\nsuspend suspend checks\nall all checks\n\nSo to switch only suspend checks on, pass:\n-implicit-checks:suspend\n\nThere is also -explicit-checks to provide the reverse once\nwe change the default.\n\nFor dalvikvm, pass --runtime-arg -implicit-checks:foo,bar\n\nThe default is -implicit-checks:none\n\nThere is also a property \u0027dalvik.vm.implicit_checks\u0027 whose value is the same\nstring as the command option. The default is \u0027none\u0027. For example to switch on\nnull checks using the option:\n\nsetprop dalvik.vm.implicit_checks null\n\nIt only works for ARM right now.\n\nBumps OAT version number due to change to Thread offsets.\n\nBug: 13121132\nChange-Id: If743849138162f3c7c44a523247e413785677370\n" }, { "commit": "36fea8dd490ab6439f391b8cd7f366c59f026fd2", "tree": "6720290f54717f7cee1116103cd1839147936146", "parents": [ "63dcdff34c40a64df6939ea5bba71813abd2acd3" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Mar 10 13:37:40 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Mar 12 15:16:12 2014 -0700" }, "message": "Fixing structure of native frame for Generic JNI\n\nThis changes the layout of the callee-save frame used in generic\nJNI to be consistent with the JNI compiler, that is, the SIRT is\ninline (above the method reference). Now the location of the\n\"this\" object is consistent.\n\nChange-Id: Ibad0882680712cb640b4c70ada0229ef7cf4e62c\n" }, { "commit": "e0dcd46314d07eeb332edea292f5110178e4e3d2", "tree": "d1060bc54e1ad26bad69a6e06662e643e75ce484", "parents": [ "80a250f8862096b72e7357c12b40dc9d5310a475" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Mar 08 15:21:04 2014 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Sun Mar 09 02:17:46 2014 +0000" }, "message": "JNI down call fixes.\n\nEnsure SIRT isn\u0027t accessed via quick callee save frame.\nSome tidying of code.\n\nChange-Id: I8fec3e89aa6d2e86789c60a07550db2e92478ca7\n" }, { "commit": "4e30541a92381fb280cd0be9a1763b713ee4d64c", "tree": "84093651bbf0ad95b66b846c4f4cf4101994037b", "parents": [ "e266ba9935bd12d685d83f73cd8d759e46c3014d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 19 10:54:44 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Feb 21 15:24:04 2014 -0800" }, "message": "Fix and optimize verify object.\n\nVerifyObject no longer resides in heap. You can now enable\nVerifyObject for non-debug builds. VerifyStack is still slow, so it\nis now guarded by its own flag.\n\nFixed the image writer to not use verification at places where\nverification fails due to invalid reads.\n\nFixed RosAlloc to use SizeOf which doesn\u0027t call verify object.\n\nAdded a flag paremeter to some of the mirror getters / setters to\nbe able to selectively disable VerifyObject on certain calls.\n\nOptimized the GC to not verify each object multiple times during\nobject scanning if verify object is enabled.\n\nAdded 3 verification options: verify reads, verify this, and verify\nwrites so that you can select how much verification you want for\nmirror getters and setters.\n\nRemoved some useless DCHECKs which would slow debug builds without\nproviding any benefits.\n\nTODO: RosAlloc verification doesn\u0027t currently work with verify\nobjects.\n\nBug: 12934910\nBug: 12879358\n\nChange-Id: Ic61033104dfc334543f89b0fc0ad8cd4f4015d69\n" }, { "commit": "936b37f3a7f224d990a36b2ec66782a4462180d6", "tree": "94de34072e8ce0a2a251ed8d5ccc7d87709db750", "parents": [ "fd80b5717c0cdd10ef2caabf4291415a52fcc874" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Feb 14 00:52:24 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Feb 20 14:46:02 2014 -0800" }, "message": "Upcall support for x86-64.\n\nSufficient to pass jni_internal_test.\n\nChange-Id: Ia0d9b8241ab8450e04765b9c32eb6dc8fc1a8733\n" }, { "commit": "815873ecc312b1d231acce71e1a16f42cdaf09f2", "tree": "18ba2fa951775e60b240271bfe975e6e2cfc654c", "parents": [ "2befd09cf4fe89a18a655f3e1dd310831bfa769f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Feb 13 18:02:13 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 18 10:45:12 2014 -0800" }, "message": "Change root visitor to use Object**.\n\nSimplifies code and improves the performance of root visiting since\nwe usually don\u0027t need to check to see if the object moved.\n\nChange-Id: Iba998f5a15ae1fa1b53ca5226dd2168a411196cf\n" }, { "commit": "d2fe10a3a34af171bf1631219cd2d6ff6b7778b5", "tree": "b6b7eb8eba23a5c2723518da99c03bf47b97f58a", "parents": [ "5a3f55ad9519e87c0d3bbddaf3d8a186a887a79b" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Jan 15 10:20:56 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Feb 17 11:32:15 2014 +0100" }, "message": "Remove blacklist\n\nRemoves the class initialization blacklist and use transaction to detect and\nrevert class initialization attempting to invoke native method. This only\nconcerns class initialization happening at compilation time when generating an\nimage (like boot.art for the system).\n\nIn transactional mode, we log every object\u0027s field assignment and array update.\nTherefore we\u0027re able to abort a transaction to restore values of fields and\narray as they were before the transaction starts. We also log changes to the\nintern string table so we can restore its state prior to transaction start.\n\nSince transactional mode only happens at compilation time, we don\u0027t need to log\nall these changes at runtime. In order to reduce the overhead of testing if\ntransactional mode is on/off, we templatize interfaces of mirror::Object and\nmirror::Array, respectively responsible for setting a field and setting an\narray element.\n\nFor various reasons, we skip some specific fields from transaction:\n- Object\u0027s class and array\u0027s length must remain unchanged so garbage collector\ncan compute object\u0027s size.\n- Immutable fields only set during class loading: list of fields, method,\ndex caches, vtables, ... as all classes have been loaded and verified before a\ntransaction occurs.\n- Object\u0027s monitor for performance reason.\n\nBefore generating the image, we browse the heap to collect objects that need to\nbe written into it. Since the heap may still holds references to unreachable\nobjects due to aborted transactions, we trigger one collection at the end of\nthe class preinitialization phase.\n\nSince the transaction is held by the runtime and all compilation threads share\nthe same runtime, we need to ensure only one compilation thread has exclusive\naccess to the runtime. To workaround this issue, we force class initialization\nphase to run with only one thread. Note this is only done when generating image\nso application compilation is not impacted. This issue will be addressed in a\nseparate CL.\n\nBug: 9676614\nChange-Id: I221910a9183a5ba6c2b99a277f5a5a68bc69b5f9\n" }, { "commit": "fd14755c09ada4769b5b6658e484441ca3f628b5", "tree": "c70c10b6e319f44dadb56c78acaf7ce89b502d82", "parents": [ "3c976a8f82007285316dd9d1bbccb659b0668a06", "0177e53ea521ad58b70c305700dab32f1ac773b7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 12 01:16:29 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Feb 12 01:16:30 2014 +0000" }, "message": "Merge \"Work in the direction of hard float quick ABIs.\"" }, { "commit": "0177e53ea521ad58b70c305700dab32f1ac773b7", "tree": "db15627a19a04634cf84cecd15b813319d80d225", "parents": [ "abaf927f29f6feceb3df3e6ced7d01970ba0dbe9" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 11 16:30:46 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 11 17:02:21 2014 -0800" }, "message": "Work in the direction of hard float quick ABIs.\n\nPass a shorty to ArtMethod::Invoke so that register setup can use it.\nDocument x86-64 ABI.\nAdd extra debug output for one JNI native method registration fails, namely a\ndump of the Class and its dex file\u0027s location.\nAdd hack to get testing of OatMethod\u0027s without GC maps working in 64bit.\n\nChange-Id: Ic06b68e18eac33637df2caf5e7e775ff95ae70f3\n" }, { "commit": "83c8ee000d525017ead8753fce6bc1020249b96a", "tree": "d5167ed15dee2629905ac3640b6ea0578d4ae312", "parents": [ "7cba217ab0661d74deccbb97160cdf60b74d4ea3" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 28 14:50:23 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 11 10:40:10 2014 -0800" }, "message": "Add root types and thread id to root visiting.\n\nEnables us to pass the root type and thread id to hprof.\n\nBug: 12680863\nChange-Id: I6a0f1f9e3aa8f9b4033d695818ae7ca3460d67cb\n" }, { "commit": "ef7d42fca18c16fbaf103822ad16f23246e2905d", "tree": "c67eea52a349c2ea7f2c3bdda8e73933c05531a8", "parents": [ "822115a225185d2896607eb08d70ce5c7099adef" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Jan 06 12:55:46 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Feb 06 23:20:27 2014 -0800" }, "message": "Object model changes to support 64bit.\n\nModify mirror objects so that references between them use an ObjectReference\nvalue type rather than an Object* so that functionality to compress larger\nreferences can be captured in the ObjectRefererence implementation.\nObjectReferences are 32bit and all other aspects of object layout remain as\nthey are currently.\n\nExpand fields in objects holding pointers so they can hold 64bit pointers. Its\nexpected the size of these will come down by improving where we hold compiler\nmeta-data.\nStub out x86_64 architecture specific runtime implementation.\nModify OutputStream so that reads and writes are of unsigned quantities.\nMake the use of portable or quick code more explicit.\nTemplatize AtomicInteger to support more than just int32_t as a type.\nAdd missing, and fix issues relating to, missing annotalysis information on the\nmutator lock.\nRefactor and share implementations for array copy between System and uses\nelsewhere in the runtime.\nFix numerous 64bit build issues.\n\nChange-Id: I1a5694c251a42c9eff71084dfdd4b51fff716822\n" }, { "commit": "5ddb4104ac605d66693b55b79f26f8b8a5505e63", "tree": "9231f1e54e9a1aaca640c590d5b4f318720e263b", "parents": [ "5a2ced515a456f15dcf194843c024e835eda7dbe" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jan 07 08:58:46 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 08 09:15:46 2014 -0800" }, "message": "Remove intialized static storage from dex cache.\n\nThe initialized static storage array is used by compiled code to determine if\nfor a sget/sput class initialization is necessary. The compiled code typically\ndoesn\u0027t require this test as the class is pre-initialized or the class being\naccessed is the same as the current method.\n\nChange-Id: Icbc45e692b3d0ac61e559e69edb6c9b29439e571\n" }, { "commit": "c528dba35b5faece51ca658fc008b688f8b690ad", "tree": "d95c113a639d07f7e3b88e8a6caa9b5f6e044801", "parents": [ "76f552307ee980221626d1dff0ada56c989d24ca" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 26 12:00:11 2013 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Dec 10 16:00:58 2013 -0800" }, "message": "Enable moving classes.\n\nSlight reduction in Zygote size, memory savings are in the noise.\nBefore: Zygote size: 8739224\nAfter: Zygote size: 8733568\n\nFixed a bug where we didn\u0027t set the concurrent start bytes after\nswitching the allocator from bump pointer to ROSAlloc in the\nzygote. This caused excessive memory usage.\n\nAdded the method verifiers as roots to fix an issue caused by\nRegTypes holding a Class*.\n\nAdded logic to clear card table in the SemiSpace collector, this\nreduces DalvikOther from ~2400k -\u003e ~1760k when using the SemiSpace\ncollector.\n\nAdded a missing lock to the timing loggers which caused a rare\none time crash in std::set.\n\nBug: 11771255\nBug: 8499494\nBug: 10802951\n\nChange-Id: I99d2b528cd51c1c5ed7012e3220b3aefded680ae\n" }, { "commit": "1eb512d33f94d1dd7ea38263307ba0f7a0dfa653", "tree": "b4d4d9b16013ab90fb4b40d23013d7ef44bb5852", "parents": [ "b917ea1a62aa0ab8eca3f689ef64b5be34e11abb" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 18 15:42:20 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sun Oct 20 14:55:26 2013 -0700" }, "message": "Fast JNI support.\n\nUse a modifier to signal a native method is a fast JNI method. If the\nmodifier is set then don\u0027t perform runnable transitions.\n\nChange-Id: I7835b4d837bfdd1cb8e2d54b919c0d5e6cf90499\n" }, { "commit": "ea46f950e7a51585db293cd7f047de190a482414", "tree": "9dddc8073547a2dcb58a19e1728932a89cb149c3", "parents": [ "5e3572709b5a5d59957f835db4f73760ecef08da" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Jul 30 01:26:50 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 13 18:09:46 2013 -0700" }, "message": "Refactor java.lang.reflect implementation\n\nCherry-picked from commit ed41d5c44299ec5d44b8514f6e17f802f48094d1.\n\nMove to ArtMethod/Field instead of AbstractMethod/Field and have\njava.lang.reflect APIs delegate to ArtMethod/ArtField.\n\nBug: 10014286.\n\nChange-Id: Iafc1d8c5b62562c9af8fb9fd8c5e1d61270536e7\n" }, { "commit": "1809a72a66d245ae598582d658b93a24ac3bf01e", "tree": "f864ff869185b2873637ca70584c66fd6a1e899d", "parents": [ "a9a50926963b5093fb851ed966d201f3e95f72d3" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Aug 09 22:05:32 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Aug 09 22:23:50 2013 -0700" }, "message": "Uleb128 compression of vmap and mapping table.\n\nBug 9437697.\n\nChange-Id: I30bcb97d12cd8b46d3b2cdcbdd358f08fbb9947a\n" }, { "commit": "848871b4d8481229c32e0d048a9856e5a9a17ef9", "tree": "4be4602e3becc899f76a29a87618801458fe04b9", "parents": [ "423fb4d70f2ac36bf9f630146b4150771a8e7e76" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Aug 05 10:56:33 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Aug 07 22:17:00 2013 -0700" }, "message": "Entry point clean up.\n\nCreate set of entry points needed for image methods to avoid fix-up at load time:\n - interpreter - bridge to interpreter, bridge to compiled code\n - jni - dlsym lookup\n - quick - resolution and bridge to interpreter\n - portable - resolution and bridge to interpreter\n\nFix JNI work around to use JNI work around argument rewriting code that\u0027d been\naccidentally disabled.\nRemove abstact method error stub, use interpreter bridge instead.\nConsolidate trampoline (previously stub) generation in generic helper.\nSimplify trampolines to jump directly into assembly code, keeps stack crawlable.\nDex: replace use of int with ThreadOffset for values that are thread offsets.\nTidy entry point routines between interpreter, jni, quick and portable.\n\nChange-Id: I52a7c2bbb1b7e0ff8a3c3100b774212309d0828e\n" }, { "commit": "9e8f45e269fba18f89489e57922481ba08a624eb", "tree": "02d6eb67a07256ced9891b5f7d4cb8fade857942", "parents": [ "f133479fa1d7e41bc9e63f50088dbcb8592d074c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jul 31 10:58:53 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jul 31 10:58:53 2013 -0700" }, "message": "Ensure has_no_move_exception is initialized on all return paths.\n\nThe evils of early returns.\n\nChange-Id: I3b53b7cbd8ddffb5c66f2187911d8cf5d5d4afdf\n" }, { "commit": "c449aa8151bf81d93d22ff24677ccf09a6da218e", "tree": "343274bb1e635bc503ae0ffc44c68ec44f6b69e5", "parents": [ "c95d6b1cb35b382804b9a310d32f66059eca65d9" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Jul 29 14:35:46 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Jul 29 14:45:07 2013 -0700" }, "message": "Clear exception if catch block doesn\u0027t have move-exception.\n\nBug: 10040419\n\nChange-Id: Icc7a55cb3cdfbc3efd2b161bbe22b3e5007de35f\n" }, { "commit": "2ce745c06271d5223d57dbf08117b20d5b60694a", "tree": "08ca327d1f1c8266fafb11422cfad6982f21e0bb", "parents": [ "fc0e3219edc9a5bf81b166e82fd5db2796eb6a0d" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 17:44:30 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 21:46:11 2013 -0700" }, "message": "Fix cpplint whitespace/braces issues\n\nChange-Id: Ide80939faf8e8690d8842dde8133902ac725ed1a\n" }, { "commit": "7940e44f4517de5e2634a7e07d58d0fb26160513", "tree": "ac90242d96229a6942f6e24ab137bc1f8f2e0025", "parents": [ "5cd9e3b122f276f610980cbaf0d2ad6ed4cd9088" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 13:46:57 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 17:49:01 2013 -0700" }, "message": "Create separate Android.mk for main build targets\n\nThe runtime, compiler, dex2oat, and oatdump now are in seperate trees\nto prevent dependency creep. They can now be individually built\nwithout rebuilding the rest of the art projects. dalvikvm and jdwpspy\nwere already this way. Builds in the art directory should behave as\nbefore, building everything including tests.\n\nChange-Id: Ic6b1151e5ed0f823c3dd301afd2b13eb2d8feb81\n" }, { "commit": "1d54e73444e017d3a65234e0f193846f3e27472b", "tree": "1de93661e95a0ce6fa78fdfc23d0cfd3dd2a06f7", "parents": [ "4c22e7eabef3f815841dfc6e0d5bbead96150752" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 02 21:10:01 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jun 21 11:10:42 2013 -0700" }, "message": "GC clean up.\n\nGreater use of directories and namespaces.\nFix bugs that cause verify options to fail.\nAddress numerous other issues:\n\nGC barrier wait occurring holding locks:\nGC barrier waits occur when we wait for threads to run the check point function\non themselves. This is happening with the heap bitmap and mutator lock held\nmeaning that a thread that tries to take either lock exclusively will block\nwaiting on a thread that is waiting. If this thread is the thread we\u0027re waiting\nto run the check point then the VM will deadlock.\nThis deadlock occurred unnoticed as the call to check for wait safety was\nremoved in: https://googleplex-android-review.googlesource.com/#/c/249423/1.\n\nNewTimingLogger:\nExisting timing log states when a split ends but not when it begins. This isn\u0027t\ngood for systrace, in the context of GC it means that races between mutators\nand the GC are hard to discover what phase the GC is in, we know what phase it\njust finished and derive but that\u0027s not ideal.\n\nSupport for only 1 discontinuous space:\nCode special cases continuous and large object space, rather than assuming we\ncan have a collection of both.\n\nSorted atomic stacks:\nUsed to improve verification performance. Simplify their use and add extra\nchecks.\n\nSimplify mod-union table abstractions.\n\nReduce use of std::strings and their associated overhead in hot code.\n\nMake time units of fields explicit.\n\nReduce confusion that IsAllocSpace is really IsDlMallocSpace.\n\nMake GetTotalMemory (exposed via System) equal to the footprint (as in Dalvik)\nrather than the max memory footprint.\n\nChange-Id: Ie87067140fa4499b15edab691fe6565d79599812\n" }, { "commit": "790ad90ccd2f3e297e2cf0a47adbf4f9ffb10149", "tree": "719a141f77194d4eeab03bfb1e7385799e001263", "parents": [ "ca6ad221d1bd857e26817f6b63d4ad9f3fe7ca40" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed May 22 15:02:08 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu May 23 10:17:09 2013 -0700" }, "message": "Some more interpreter cleanup.\n\n- Pass method helper and code item to entry point so they don\u0027t\n have to be regenerated.\n- Removed interpreter only check from invoke, changing entry\n points instead.\n\nChange-Id: Ib0ea83dcffcdb295d3a48d92ad8a93ac59f9932e\n" }, { "commit": "aa4a793d7175ceeb1efa215c2af5e52e42b22f0a", "tree": "9c3db54a108d4327471c6d4890d28cdfe1f030ff", "parents": [ "167436311a08a65dea28dda079a137893821c9c7" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon May 13 11:28:27 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon May 13 13:14:19 2013 -0700" }, "message": "Rename abstract method code_ to entry_point_from_compiled_code_.\n\nChange-Id: I9b02d2df95bbeafa6e6387b461f574c57337a61e\n" }, { "commit": "167436311a08a65dea28dda079a137893821c9c7", "tree": "4754f47306a0ffa0ad97635015d9596b427dd425", "parents": [ "c44f68fb6a6d55c0df28dd05b93d8906ec20bd08" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed May 08 10:59:04 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri May 10 17:16:00 2013 -0700" }, "message": "Improve interpreter to interpreter invokes.\n\nThe interpreter constructs a shadow frame instead of arg array to make\ninterpreter to interpreter transitions faster. This adds a pointer to\nan entry for the interpreter to each method.\n\nChange-Id: If48911d3aa3470847b8548a9e92090b829f4f254\n" }, { "commit": "79fe539587d4c09244172d0168eeed0ec9770466", "tree": "4fb8fbd03727a74acb4dfde7e969adf32ee32750", "parents": [ "09b55e1df3b0bcecee25882e45e882887cc8654f" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Apr 24 18:41:58 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Apr 26 14:50:52 2013 -0700" }, "message": "Removing remaining compiled stubs from image.\n\nAbstract method error stub and jni dlsym lookup stubs are gone.\nAfter this change, the image no longer needs to be executable.\n\nChange-Id: Ic75d72bf7e76e3b8ecc596e82af68ab592dde15e\n" }, { "commit": "62d6c772205b8859f0ebf7ad105402ec4c3e2e01", "tree": "e2f2ba6d71ed5a39c9f6909e3f7c08e998053315", "parents": [ "c9b17c7ee96cd04fac9048aab624ed554fe260bf" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 27 08:32:07 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Apr 08 14:24:13 2013 -0700" }, "message": "Interpreter entries and instrumentation as a listener.\n\nMake the instrumentation responsible for whether we want method entry/exit\nstubs, and allow it to use interpreter entry stubs when instruction by\ninstruction instrumentation is required. Improve deoptimization so more JDWP\ntest cases are passing.\n\nRefactor exception debug posting, in particular improve reporting in the\ninterpreter. Improve class linker exception throwing so that broken dex files\nare more likely to be reported. Fixes the performance issue Bug: 8410519.\n\nFix some error reporting lock level errors for the large object space. Make\nfast object verification faster.\n\nAdd some debug mode robustness to finding dex PCs in GC maps.\n\nAdd printf attributes to JniAbortF and fix errors.\n\nExpand run-test 044 to test return behaviors and fix issues with not throwing\nappropriate exceptions for proxies.\n\nEnsure causes are reported with a class linker NoClassDefFoundError and JNI\nNoSuchFieldError.\n\nRemove unused debugMe and updateDebuggerFromCode.\n\nThere\u0027s a minor sizing tweak to the arg array builder, and an extra reference\narray check in the interpreter.\n\nSome clean-up of trace code.\n\nFix reg type cache destructor if it is called after the reg type cache is\nshutdown (as is the case in oatdump).\n\nChange-Id: I6519c7b35df77f978d011999354c864f4918e8ce\n" }, { "commit": "74180cad94848107cf297d37e72437c5a6eecf1b", "tree": "9dfa6f7183bc7d6085a2753483b0f7a93c3ce486", "parents": [ "857fe960a02834c0d6b8792dcc0af8143995cb1f" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Mar 27 15:29:11 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Mar 27 15:29:11 2013 -0700" }, "message": "Remove code related to compiled invoke stubs.\n\nNote that the oat file version is now bumped to 004. A clean-oat will be\nnecessary after syncing this change.\n\nChange-Id: If8875335b7fcc39b6b40c6f95de07da50da7b6b8\n" }, { "commit": "6474d190d5604898354ebf767f1944b6e3e9b445", "tree": "ac4be675c62fd8129fe85f740dcd2e3bfee424e2", "parents": [ "52003fe9e2020e9d68cee1300fdb4a3f55be85a5" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Mar 26 14:08:09 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Mar 26 16:03:53 2013 -0700" }, "message": "Remove use of compiled invoke stubs from portable.\n\nNow the invoke stubs can be safely removed. Tested and working on ARM,\nbasic testing done on x86/MIPS, but portable is currently broken for them\neven without the change.\n\nChange-Id: Ib73f2b7aa9d81f5f0e1e817d16b9bec464c5a5aa\n" }, { "commit": "4f6ad8ab428038129b2d0d6c40b7fd625cca15e1", "tree": "d1025ed9203d0cdf7aa959d9cecd8dc1f0c0bfeb", "parents": [ "aabe8adacc7bf3a8ab6f6f3dd37fc839b6fd762f" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Mar 18 15:27:28 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Mar 18 15:29:59 2013 -0700" }, "message": "Various performance improvements.\n\nPerformance had regressed due to verify object and method invocation changes.\nAvoid trampolines for static calls in same class.\nVarious inlining changes.\nMake verify object something that\u0027s only compiled-in in debug builds.\n\nChange-Id: Ia261a52232c3b10667c668f8adfadc0da3048bc5\n" }, { "commit": "5d9173014c1ca09f7249a6b07629aa37778b5f8f", "tree": "80543dbe9af6795212373bb12532b638ff3b22f3", "parents": [ "aed0716b9592bb3095cdfc4b111011f9ed74877d" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Feb 27 17:57:33 2013 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 15 14:18:28 2013 -0700" }, "message": "Changes to remove need for compiled invoke stubs for quick.\n\nARM, x86, and MIPS implementation complete, though MIPS is untested.\n\nThe ArgArray is changed to be a uint32_t array instead of a JValue array.\nAlso, a separate result for float/double was needed for x86/MIPS. The invoke\nstubs are currently still there, but only used for portable.\n\nChange-Id: I0647f8d5d420cea61370e662e85bdc0c13b5e378\n" }, { "commit": "c928de90ad22ecdf83c18a07008409595f13d3b1", "tree": "065a08f379960f7ba946318aaa742d7ddd0c94bd", "parents": [ "0f827169742aad6209d830db773a101849c32a83" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 27 14:30:44 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 27 14:30:44 2013 -0800" }, "message": "Remove Iceland.\n\nART_USE_LLVM_COMPILER is removed and when necessary ART_USE_PORTABLE_COMPILER\nis used in #ifdefs.\n\nChange-Id: Iffa9ce5b0246c7c427ccc4e67ecc134624632e55\n" }, { "commit": "11ffc2d02b7d979f26a8a04013fce3eb4f9a2816", "tree": "a54cdfe70f2bd45ea36bfd6cf9e7ad8783a29234", "parents": [ "e9de2dd6bd51f9be3457b5e2ffdd97f457ddebcb" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Feb 01 11:52:17 2013 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Feb 01 14:45:01 2013 -0800" }, "message": "Changes to LLVM to support deoptimization.\n\nAdded a magic exception value (-1) and a handler to transition\nto the interpreter. This is currently untested.\n\nChange-Id: I2f53135e7505c54355ecf7c579897f68bbdcbda3\n" }, { "commit": "2dd0e2cea360bc9206eb88ecc40d259e796c239d", "tree": "9d619dc9508cbe73e4793bf6f08cbc761abfb48a", "parents": [ "ac21b797b3a425975d656d6b84a7b24401d35f42" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jan 24 12:42:14 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 30 16:48:31 2013 -0800" }, "message": "Directory restructuring of object.h\n\nBreak object.h into constituent files.\nReduce number of #includes in other GC header files.\nIntroduce -inl.h files to avoid mirror files #include-ing each other.\nCheck invariants of verifier RegTypes for all constructors.\n\nChange-Id: Iecf1171c02910ac152d52947330ef456df4043bc\n" } ] }