)]}' { "log": [ { "commit": "9e6dee2569ba3ec393b91ee4afe0f63a4daecfa9", "tree": "a4349d6188ed7ad2a4c35bb1cc3be7a50c74df68", "parents": [ "bec07a05dd1aa564895f0d9c58c8277c576dc9bb" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Apr 11 13:50:23 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Apr 11 13:50:23 2017 -0700" }, "message": "ART: Move trampoline logging to -verbose:dex\n\nMove invoke logging to VLOG(dex).\n\nBug: 37158297\nTest: m test-art-host\nChange-Id: Ib29b3920e8f227395b99a9ed65de5ca10c737cc0\n" }, { "commit": "db01a091aefbd78b56777f4c2e8c5e3f2d8c2712", "tree": "62dbbd082c5a4cfc122830a4df0a2ced219d8d6d", "parents": [ "432fccc4c001fcd822f401aea1a4214b713bd896" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Apr 03 15:39:55 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Apr 04 11:44:35 2017 -0700" }, "message": "Ensure invoking obsolete methods throws errors.\n\nThis sets the entrypoint for obsolete methods to a special stub that\nwill ensure that calling them results in an Error being thrown.\nPreviously we were allowing obsolete methods to be run if they could\nreach the appropriate places in the runtime.\n\nGetting into the state where this is possible is extremely difficult\nsince one can only get an jmethodID to an obsolete method by snatching\nit off the stack (or by inspecting internal runtime data). From there\nnormally invoking it will do lookup on the receiver which will get you\nthe original version.\n\nBug: 36867251\nBug: 31455788\n\nTest: ./test.py --host -j40\nTest: (with aosp_marlin-userdebug device) ./test.py --target -j4\n\nChange-Id: I2ca0503966a4e3de18dd89cb7ff224eba1459b49\n" }, { "commit": "8f2b925473cfdc7650cef407102957befe0c6bb5", "tree": "4af3395adaf03cb4a60cd40e7b3ee4196645eb9e", "parents": [ "de9b8d63ef548d56de9c9d4657a7b916050d81ad", "6b2dc3156a2140a5bfd9cbbf5d7dad332ab5f5bd" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Fri Mar 17 09:05:55 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Mar 17 09:05:56 2017 +0000" }, "message": "Merge \"ART: Get rid of most of java.lang.DexCache.\"" }, { "commit": "6b2dc3156a2140a5bfd9cbbf5d7dad332ab5f5bd", "tree": "a335cae93c0eabae63e39b8bc13ef3e15f4f7e50", "parents": [ "f83f3f6ecb1153d96cc8007e8a0d1e35af4d3f38" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Tue Mar 14 13:26:12 2017 +0000" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Mar 16 17:56:04 2017 +0000" }, "message": "ART: Get rid of most of java.lang.DexCache.\n\nAll remaining functionality is moved over to native. The DexCache\nobject itself is allocated in the Java heap, even though there\u0027s no\nlonger much of a reason to do so. It can be changed in a future change\nif needed.\n\nThis also renames mirror::Class:GetInterfaces to GetProxyInterfaces\nsince it\u0027s supposed to be called only for proxies.\n\nTest: test-art-host, cts -m CtsLibcoreTestCases\n\nChange-Id: Ie261f22a9f80c929f01d2b456f170c7a464ba21c\n" }, { "commit": "cbcedbf9382bc773713cd3552ed96f417bf1daeb", "tree": "58eed766a69ddddbc41d21624244e14547fa1dff", "parents": [ "ef62210a1466c133056c4bfdae9f8f5417065eff" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Mar 12 22:24:50 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Mar 16 10:20:34 2017 -0700" }, "message": "Add method info to oat files\n\nThe method info data is stored separately from the code info to\nreduce oat size by improving deduplication of stack maps.\n\nTo reduce code size, this moves the invoke info and inline info\nmethod indices to this table.\n\nOat size for a large app (arm64): 77746816 -\u003e 74023552 (-4.8%)\nAverage oat size reduction for golem (arm64): 2%\n\nRepurposed unused SrcMapElem deduping to be for MethodInfo.\nTODO: Delete SrcMapElem in a follow up CL.\n\nBug: 36124906\n\nTest: clean-oat-host \u0026\u0026 test-art-host-run-test\n\nChange-Id: I2241362e728389030b959f42161ce817cf6e2009\n" }, { "commit": "8d6768d47b66a688d35399d524ad5a5450e9d9d4", "tree": "0b8733baef378d9e060c8e74319a3846489590b2", "parents": [ "fbe1516b91f2117bb9c7282aaea5537572f62dd7" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 14 10:13:21 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 14 13:55:13 2017 +0000" }, "message": "Revert^6 \"Hash-based dex cache type array.\"\n\nFixed ImageWriter to write class table also if it contains\nonly boot class loader classes. Added a regression test and\nadded extra checks for debug-build to verify that dex cache\ntypes from app image are also in the class table. Removed\nsome unnecessary debug output.\n\nTest: 158-app-image-class-table\nBug: 34839984\nBug: 30627598\nBug: 34659969\n\nThis reverts commit 0b66d6174bf1f6023f9d36dda8538490b79c2e9f.\n\nChange-Id: I6a747904940c6ebc297f4946feef99dc0adf930c\n" }, { "commit": "0b66d6174bf1f6023f9d36dda8538490b79c2e9f", "tree": "1cc4d2ae868745a65fd0489a6fb2f5f2fc9e880f", "parents": [ "6374c58f2ea403b3a05fb27376110fe4d0fc8e3f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Mar 13 14:50:04 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Mar 13 16:08:01 2017 +0000" }, "message": "Revert^5 \"Hash-based dex cache type array.\"\n\nFor app images, ImageWriter does not add boot image\nclasses to the app image class table even though it\nkeeps them in the dex caches. The reason for that is\nunknown, the code looks OK.\n\nBug: 34839984\nBug: 30627598\nBug: 34659969\n\nAlso reverts \"Improve debugging output for a crash.\"\n\nThis reverts commits\n bfb80d25eaeb7a604d5dd25a370e3869e96a33ab,\n 8dd56fcb3196f466ecaffd445397cb11ef85f89f.\n\nTest: testrunner.py --host\nChange-Id: Ic8db128207c07588c7f11563208ae1e85c8b0e84\n" }, { "commit": "fe102795cf1526e8b48c564a91d81ec218f3387a", "tree": "9d0a220268768ea55d121da86cd1fa39df674e69", "parents": [ "63badfb7f155c6561ff3dd9aadd410e42836149d", "efd20cb2f63cf647c7d947d00e8987affefeb177" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Mar 10 02:49:48 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Mar 10 02:49:49 2017 +0000" }, "message": "Merge \"Catch signals that happen inside a fault handler.\"" }, { "commit": "5667f56867383fc4113aa4a6551efdf9f48ee5e7", "tree": "fed4de9fbacff02d86afdeb93cee54a7174d52b0", "parents": [ "68948e01f56ad1996af77f4c0aab721940b0e18d" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Feb 27 19:32:01 2017 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Mar 08 16:46:37 2017 -0800" }, "message": "Modify invoke interface trampoline to pass interface method.\n\nThis avoids touching the code item in the case that the interface method\nis resolved and in the dex cache. If it\u0027s not, the trampoline will still\nhave to go to the code item to try to resolve it.\n\nBug: 35800981\nTest: mm test-art-host\nChange-Id: Ia0e1d23429b3b928bcec727f4f2f16b7834408e5\n" }, { "commit": "efd20cb2f63cf647c7d947d00e8987affefeb177", "tree": "666ea53832f1085378bcc75620f314ba1f46a330", "parents": [ "caa348cee1312150a957bd308ce77ad406f3eb64" ], "author": { "name": "Josh Gao", "email": "jmgao@google.com", "time": "Tue Feb 28 16:53:59 2017 -0800" }, "committer": { "name": "Josh Gao", "email": "jmgao@google.com", "time": "Wed Mar 08 15:07:49 2017 -0800" }, "message": "Catch signals that happen inside a fault handler.\n\nUnblock some signals (SIGABRT, SIGBUS, SIGSEGV) that could happen\ninside of the ART internal fault handlers, to report crashes inside of\nthe signal handler. Because we can\u0027t use sigaction to change the\nhandler when this happens, because it modifies global state, add a new\nmember variable in Thread to track whether a call to the fault handler\nis reentrant or not.\n\nRemove the old nested signal implementation that attempted to do this.\n\nBug: http://b/35853436\nTest: changed the #if 0 to #if 1, ran a dummy process that\n threw a NullPointerException, inspected logcat\nChange-Id: I04bb4a09433c6817933d64ec681ec433b528f2a5\n" }, { "commit": "a1467d07c4c1e838ff5c07a4ff4ec35aab6a701f", "tree": "469b914b5bbb98264b7f70b601bff23801aaf201", "parents": [ "fc5d8c28e5265634aeeb2f255dc8d5747e2919ee" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 22 09:22:50 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 22 11:45:35 2017 -0800" }, "message": "Revert \"Revert \"Add missing card mark verification to CC\"\"\n\nAdd missing write barrier for AddStrongRoot on the dex cache.\n\nTest: test-art-host-run-test ART_TEST_INTERPRETER\u003dtrue ART_TEST_OPTIMIZING\u003dtrue ART_TEST_GC_STRESS\u003dtrue\n\nBug: 12687968\n\nThis reverts commit 50805e747cbb7e9c8d30bd3b49e27ab0741f3cf8.\n\nChange-Id: I72c6de2120d8e0ddc2512dd41010776aecfc9e2c\n" }, { "commit": "50805e747cbb7e9c8d30bd3b49e27ab0741f3cf8", "tree": "c71324e48fe026cbd11119ef43f44da7ae134093", "parents": [ "49ba69667ce70f8efbed7d68814ab184bee53486" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Feb 22 10:11:12 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Feb 22 10:11:12 2017 +0000" }, "message": "Revert \"Add missing card mark verification to CC\"\n\nFails in read-barrier-gcstress for 944-transform-classloaders\n\n\nBug: 12687968\n\nThis reverts commit 49ba69667ce70f8efbed7d68814ab184bee53486.\n\nChange-Id: Ie91eaa034cea77918235766983661efa14fb1a14\n" }, { "commit": "49ba69667ce70f8efbed7d68814ab184bee53486", "tree": "65f632ff9f9c73574a410308a6ce1b3fd117b7e0", "parents": [ "30e015c442c8033390c30d2f293604723c29bc75" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 14 19:24:22 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Feb 21 12:43:06 2017 -0800" }, "message": "Add missing card mark verification to CC\n\nEasier than adapting the code in heap.cc to do this. The verification\nensures that objects on clean cards never reference objects in newly\nallocated regions.\n\nRevert some changes from aog/341344 that caused the verification to\nfail.\n\nBug: 12687968\n\nTest: test-art-host with CC\n\nChange-Id: Iad583644bb76633ccea0dba87cb383f30adaa80b\n" }, { "commit": "bfb80d25eaeb7a604d5dd25a370e3869e96a33ab", "tree": "96068dd9b9b3bc2e1bcc25d38bbd2420d5b0f1bb", "parents": [ "7a49d93ee265b978ab29cae659e46d41da78459a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Feb 14 14:08:12 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Feb 20 14:13:14 2017 +0000" }, "message": "Revert^4 \"Hash-based dex cache type array.\"\n\nAdded extra output to the abort message to collect more data\nwhen we hit the crash. Added extra check when loading an app\nimage to verify that the class table isn\u0027t already broken.\n\nTest: testrunner.py --host\nBug: 34839984\nBug: 30627598\nBug: 34659969\n\nThis reverts commit 5812e20ff7cbc8efa0b8d7486ada2f58840a6ad5.\n\nChange-Id: I9bb442a184c236dcb75b3e42a095f39cd6bee59d\n" }, { "commit": "9b03cb4a14beb4997ce891d86166cdac0fe25fa5", "tree": "34a7c06c8e9c3830e755622e7a022492074f51d9", "parents": [ "1fed1dc7b1ea75b0465c0b2b3457718aab5a0f34" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 16 16:37:03 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Feb 17 15:05:16 2017 +0000" }, "message": "Clean up tracking new roots, do not track them for CC.\n\nThis addresses comments from\n https://android-review.googlesource.com/321552\n\nTest: ART_USE_READ_BARRIER\u003dfalse testrunner.py -b --host\nTest: ART_USE_READ_BARRIER\u003dtrue testrunner.py -b --host\nBug: 30627598\nChange-Id: I9740e599fe8170201a3b5f10113bbeb0dee500fe\n" }, { "commit": "d776ff08e07494327716f0d2ea1a774b2ebfbca9", "tree": "cedf874dd494d881adc572a10a9d14bca852add6", "parents": [ "3fb852a88d2a8ffaa87089752f4b1d5f9d6ce3c1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 17 09:32:18 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 15 14:46:15 2017 -0800" }, "message": "Add invoke infos to stack maps\n\nInvoke info records the invoke type and dex method index for invokes\nthat may reach artQuickResolutionTrampoline. Having this information\nrecorded allows the runtime to avoid reading the dex code and pulling\nin extra pages.\n\nCode size increase for a large app:\n93886360 -\u003e 95811480 (2.05% increase)\n\n1/2 of the code size increase is from making less stack maps deduped.\nI suspect there is less deduping because of the invoke info method\nindex.\n\nMerged disabled until we measure the RAM savings.\n\nTest: test-art-host, N6P boots\n\nBug: 34109702\n\nChange-Id: I6c5e4a60675a1d7c76dee0561a12909e4ab6d5d9\n" }, { "commit": "fe35fe172a7d3469ebf1dae9cca7e1ac8aefcba7", "tree": "3bf964f7786227f8104eeba08fe2e1768109a764", "parents": [ "a0637543dc82dfd54feda7c80da017bd39b1d2d5", "fa4333dcb481e564f54726b4e6f8153612df835e" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Feb 14 22:11:34 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Feb 14 22:11:35 2017 +0000" }, "message": "Merge \"ART: Add operator \u003d\u003d and !\u003d with nullptr to Handle\"" }, { "commit": "fa4333dcb481e564f54726b4e6f8153612df835e", "tree": "ae597c7587dc214434a180962c4373d3748f51ab", "parents": [ "2d98ba68f13dc219c088a12f369c5778bf398f14" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Feb 14 11:10:34 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Feb 14 11:44:48 2017 -0800" }, "message": "ART: Add operator \u003d\u003d and !\u003d with nullptr to Handle\n\nGet it in line with ObjPtr and prettify our code.\n\nTest: m\nChange-Id: I1322e2a9bc7a85d7f2441034a19bf4d807b81a0e\n" }, { "commit": "c069a30d42aefd902c20e8bc09dfad1683f07ded", "tree": "8bbf72bea7ea5d243b57f8e0ab64b687a9f60e4b", "parents": [ "3f38398380b80d1ded078ebed1211b7e4f51460f" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 18 09:23:12 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Tue Feb 14 14:04:33 2017 +0000" }, "message": "ART: invoke-custom support\n\nAdds invoke-custom instruction to the interpreter.\n\nBug: 33191717,30550796\nTest: art/test/run-test --host 952\nChange-Id: I3b754128649a8b3a00ade79ba2518d0e377f3a1e\n" }, { "commit": "5812e20ff7cbc8efa0b8d7486ada2f58840a6ad5", "tree": "be0e9fdeb90bcbad76b17b3aae4e746d977d80ee", "parents": [ "518f373324f355f8e58440dfbc3f5a1f2244bde7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 13 18:32:04 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 13 19:34:54 2017 -0800" }, "message": "Revert^3 \"Hash-based dex cache type array.\"\n\nAssert failing for \"earchbox:search\":\nF zygote64: class_linker.cc:4612] Check failed: handle_scope_iface.Get() !\u003d nullptr\n\nTest: m test-art-host\nBug: 34839984\nBug: 30627598\nBug: 34659969\n\nThis reverts commit 85c0f2ac03417f5125bc2ff1dab8109859c67d5c.\n\nChange-Id: I39846c20295af5875b0f945be7035c73ded23135\n" }, { "commit": "71953d82e802d8e8e1619562aea26ed3204bafd7", "tree": "2e06989273e3a8e98ccf6c3a40b431d5e05f20f8", "parents": [ "ab58a629d6cfe75f0ed34326c2ca6364b2203885", "85c0f2ac03417f5125bc2ff1dab8109859c67d5c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Feb 13 16:53:03 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Feb 13 16:53:04 2017 +0000" }, "message": "Merge \"Revert^2 \"Hash-based dex cache type array.\"\"" }, { "commit": "1c66504e0613bdf02dd15e15cd7e5167a4569d19", "tree": "ab01d21a1ab960e02e09ba040ee52ed0d3dc3a80", "parents": [ "11be69af1546bf66fa63ec13c250e4d28087b64e", "ea4c126a0165c5a4b997986e6e01c7f975642167" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Feb 10 13:11:42 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Feb 10 13:11:42 2017 +0000" }, "message": "Merge \"Change type initialization entrypoints to kSaveEverything.\"" }, { "commit": "85c0f2ac03417f5125bc2ff1dab8109859c67d5c", "tree": "844952d237e35faa5c5a2357ab037366c8397115", "parents": [ "8d4b1189639f0d8982bde681ccbdd7c03fe6ddbf" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 02 16:42:38 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Feb 10 11:23:11 2017 +0000" }, "message": "Revert^2 \"Hash-based dex cache type array.\"\n\nThe reason for the revert was fixed by\n https://android-review.googlesource.com/332666 .\nWe now enable clearing dex cache types in test 155 from that\nCL. Also avoid an unnecessary store in LookupResolvedTypes()\nand prevent verifier from messing up the dex cache types.\n\nTest: m test-art-host\nBug: 34839984\nBug: 30627598\nBug: 34659969\n\nThis reverts commit d16363a93053de0f32252c7897d839a46aff14ae.\n\nChange-Id: Ie8603cfa772e78e648d005b0b6eae59062ae729d\n" }, { "commit": "ea4c126a0165c5a4b997986e6e01c7f975642167", "tree": "0fdfd53472db379a1702846a89933ff1be7a4137", "parents": [ "f8512f8515f7568984e1ca209929262ea88e4b59" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Feb 06 19:59:33 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 08 21:24:01 2017 +0000" }, "message": "Change type initialization entrypoints to kSaveEverything.\n\nAlso avoid the unnecessary read barriers for boot image\nclasses with kBssEntry or kJitTableAddress (the kBssEntry\nand JIT work missed the `read_barrier_option` flag), fix\nbit-rotten non-Baker read barriers on ARM and ARM64 and\nfix bit-rotten ARM64 relative patcher\u0027s IsAdrpPatch() used\nfor erratum 843419 workaround.\n\naosp_angler-userdebug with CC:\n before:\n arm boot*.oat: 35440420\n arm64 boot*.oat: 43504952\n after:\n arm boot*.oat: 35222292 (-218128, -0.62%)\n arm64 boot*.oat: 43389048 (-115904, -0.26%)\n\naosp_angler-userdebug without CC:\n before:\n arm boot*.oat: 31927412\n arm64 boot*.oat: 39340512\n after:\n arm boot*.oat: 31708736 (-218676, -0.68%)\n arm64 boot*.oat: 39211768 (-128744, -0.33%)\n\nTest: m test-art-host (non-CC, Baker CC, table lookup CC)\nTest: m test-art-target on Nexus 6P (non-CC, Baker CC, table lookup CC)\nTest: Nexus 6P boots (non-CC, Baker CC, table lookup CC)\nBug: 30627598\nChange-Id: Ida5bbce414844de9e4273e40334165d4494230d4\n" }, { "commit": "575d3e60c68b5cf481b615dde4a16283507b19ed", "tree": "4718f2747233e74268de21d804df9bfe0b7e4362", "parents": [ "857acf51fbc5a931939c20e9e299c69676baf654" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 06 11:00:40 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 08 09:51:27 2017 -0800" }, "message": "Clean up code info table layout\n\nPreviously:\nTable layout was computed multiple places like stack_map_stream,\nand getters. This made it difficult to add new stack map tables and\nmade the code hard to understand.\n\nThis change makes the table layout specified all inside of the code\ninfo. Updating the layout only requires changing ComputeTableOffsets.\n\nChanged the stack map inline info offset to be an index, so that it is\nnot require the inline infos are directly after the dex register table.\n\nOat file size for a large app: 94459576 -\u003e 93882040 (-0.61%)\n\nUpdated oatdump and fixed a bug that was incorrectly computing the\nregister mask bytes.\n\nBug: 34621054\n\nTest: test-art-host\n\nChange-Id: I3a7f141e09d5a18bce2bc6c9439835244a22016e\n" }, { "commit": "90b936ddda63139ff46a6755c3b83ad6e4ab4ac5", "tree": "c7ce2c3004eecc16ab41ed7cde105c3019638d4b", "parents": [ "b78a8af993e877d74c5938f65f95feaf2fa01321" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 31 08:58:55 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Feb 03 10:03:52 2017 -0800" }, "message": "ART: Refactor verify_object.h\n\nMove the actual VerifyObject check into a new cc file, as we\ncommonly don\u0027t enable the check at all. This allows to cut the\n-inl include from almost all current users.\n\nThis also exposes missing -inl includes. Also fix up some of our old\nmess where .h defined functions require -inl.h defined functions.\n\nTest: m\n\nChange-Id: I3dd821bbe2015564a29bf1ed9be00f7a7276ad61\n" }, { "commit": "20eb58ef94db18de9aa3b0404aa65d958025f80b", "tree": "96394afcc45cbd72ccdcf5a1b7c02c164a24f5a9", "parents": [ "3cb871ab1af47576959fd24a99d370381b8f193e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 01 15:50:54 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 01 17:09:59 2017 -0800" }, "message": "Use dex cache for GetResolvedMethod\n\nUsing the dex cache is faster than going through the class linker.\nThis reverts to behavior from before aog/321573.\n\nSpeeds up pmd benchmark by ~50%.\n\nTest: test-art-host\n\nChange-Id: I58403aec03e2b7e7a3d3e108319cfb4c75a680cb\n" }, { "commit": "d16363a93053de0f32252c7897d839a46aff14ae", "tree": "8823c6dd4641eae333aa7001203460757a515010", "parents": [ "ac240396125911b52437a460a400c8b38be43a96" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 01 14:09:13 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 01 15:27:37 2017 +0000" }, "message": "Revert \"Hash-based dex cache type array.\"\n\nReverting to work around some programs crashing with\n Check failed: handle_scope_iface.Get() !\u003d nullptr.\nthough the reason for the failure not yet understood.\n\nTest: m test-art-host\nBug: 34839984\nBug: 30627598\nBug: 34659969\nBug: 30419309\n\nThis reverts commit ec7862283dd49f5a58d0ac45960ce27c2f7671b8.\n\nChange-Id: Ifded663633082f1e59e5b6ff2e026dc559bd6b82\n" }, { "commit": "500d4918e5fb6eafb76b36c4ec736f974835b94c", "tree": "5c6be88b473e60296ae9df22bfc55cfca85a3d28", "parents": [ "fdd869faca8dff54a456ccb5f1a68ecb44143721", "433b79a9130df5f2375b9d3d7518c1f3fb84a467" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 20:56:11 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jan 30 20:56:11 2017 +0000" }, "message": "Merge \"Revert \"Revert \"Make --debuggable rely on JIT code.\"\"\"" }, { "commit": "433b79a9130df5f2375b9d3d7518c1f3fb84a467", "tree": "dddc053886464b99be29b9ca69039d74d49605ca", "parents": [ "81356645157af44152c7b7db383596b5cf3479b5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 20:54:45 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 20:54:45 2017 +0000" }, "message": "Revert \"Revert \"Make --debuggable rely on JIT code.\"\"\n\nFixed by https://android-review.googlesource.com/#/c/330165/\n\nThis reverts commit 81356645157af44152c7b7db383596b5cf3479b5.\n\nChange-Id: Ifb74e1cc90ab6dea621f7f54a00b540d6ccd0cf6\n" }, { "commit": "d7c4ad030c2c316f5e1b2049ff8fa925b56bee2b", "tree": "fee68e3431cd62d6f5d6da89c1e105993bfb5528", "parents": [ "f22bf9f809def57c96a19d932467a175bbd9c08c", "81356645157af44152c7b7db383596b5cf3479b5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 20:20:19 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jan 30 20:20:20 2017 +0000" }, "message": "Merge \"Revert \"Make --debuggable rely on JIT code.\"\"" }, { "commit": "81356645157af44152c7b7db383596b5cf3479b5", "tree": "59aca3792abfa47c7424e4dac13248651e4662b2", "parents": [ "a0619e25aacf8b8074132f4951f75fdbcfd42925" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 20:18:02 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 20:19:28 2017 +0000" }, "message": "Revert \"Make --debuggable rely on JIT code.\"\n\nBreaks 137-cfi with ART_TEST_RUN_TEST_DEBUGGABLE\u003dtrue\n\nThis reverts commit a0619e25aacf8b8074132f4951f75fdbcfd42925.\n\nbug: 28769520\nChange-Id: Ifd7b8fc7c9b72c0a523fd57c4b1b80edd3547caa\n" }, { "commit": "8f28bd479af445293474867191a0bf1cf0f08a4f", "tree": "2bf92d368a6ad5b766bef70a3ee9c35d7af6db71", "parents": [ "ab5afeedfcb496026d8fe0774de99ee895888119", "a0619e25aacf8b8074132f4951f75fdbcfd42925" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 15:09:44 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jan 30 15:09:45 2017 +0000" }, "message": "Merge \"Make --debuggable rely on JIT code.\"" }, { "commit": "a0619e25aacf8b8074132f4951f75fdbcfd42925", "tree": "dddc053886464b99be29b9ca69039d74d49605ca", "parents": [ "ca21dc47adeed92a15a9d3fd090bdd0e6654679c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 20 13:57:43 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 12:41:52 2017 +0000" }, "message": "Make --debuggable rely on JIT code.\n\nRemoves -Xfully-deoptable in the process, which was added as a\ntemporary workaround until this CL.\nPartial revert of https://android-review.googlesource.com/#/c/302232/\n\nMakes things consistent with existing infrastructure:\n\n- Parse the --debuggable from the compiler options, just like\n --compiler-filter.\n\n- Add DEBUG_JAVA_DEBUGGABLE, passed by the zygote, for debuggable apps.\n\n- Java debuggable now solely relies on JIT for simplicity.\n\n- Debugging under userdebug for non-java-debuggable apps is still\n best effort.\n\nTest: test-art-host, jdwp\n\nbug: 28769520\nChange-Id: Id0593aacd85b9780da97f20914a50943957c858f\n" }, { "commit": "ec7862283dd49f5a58d0ac45960ce27c2f7671b8", "tree": "26d6dcc1d5ed4f0ba5ac15f17ef7377215684bf6", "parents": [ "c01d49091f4588777db5bf45345f388058caa99f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Dec 20 16:24:13 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 10:21:16 2017 +0000" }, "message": "Hash-based dex cache type array.\n\nTest: m test-art-host (Interpreter, Optimizing, JIT)\nTest: m test-art-target on Nexus 6P (Interpreter, Optimizing, JIT)\nTest: Nexus 6P boots\nTest: m valgrind-test-art-host\nBug: 30627598\nBug: 34659969\nBug: 30419309\nChange-Id: Ic00eda89e58088a3573fc9ec0ad04c0e69e161d1\n" }, { "commit": "055e7beb8a652d2600a19135d4d1ad0fa3e8d10d", "tree": "8ab012bf97857ca0b0aa952493abc1087c00e80f", "parents": [ "cb7b5dfa0522cb234b6f2e4893ce21fd9c1a6cda", "b048cb74b742b03eb6dd5f1d6dd49e559f730b36" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 27 09:31:36 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jan 27 09:31:37 2017 +0000" }, "message": "Merge \"Add per array size allocation entrypoints.\"" }, { "commit": "5b3c6c0fcca76d82a4c9acb03f7714457ae53dd9", "tree": "e1678dcd577abb63391a0cfcbb27a69a0ac4e769", "parents": [ "c9569731061e560cb56116255b00a49d6a5daeb8" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 19 14:22:26 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 26 10:37:58 2017 +0000" }, "message": "Refactor code for unresolved field entrypoint.\n\n- Do macro magic to avoid source code duplication.\n- Do not fetch the referrer from the assembly, but\n from the C entrypoint instead.\n\nTest: test-art-host test-art-target\n\nChange-Id: Ib139c94bc8f74686640cad538ba75dc56fa00e1d\n" }, { "commit": "b048cb74b742b03eb6dd5f1d6dd49e559f730b36", "tree": "b1f663cbb343488a548cce4db352dbc4af720a89", "parents": [ "f34077c96af3389e8eae65252d4c5d51cf630039" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 23 22:50:24 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 26 09:56:36 2017 +0000" }, "message": "Add per array size allocation entrypoints.\n\n- Update architectures that have fast paths for\n array allocation to use it.\n- Will add more fast paths in follow-up CLs.\n\nTest: test-art-target test-art-host.\nChange-Id: I138cccd16464a85de22a8ed31c915f876e78fb04\n" }, { "commit": "8d91ac31ccb92557e434d89ffade3372466e1af5", "tree": "37fd364ba6c9a6cf5e6a60a00c2542c5ffb12528", "parents": [ "2f670ccba022fe557c637571ac781519f0e84463" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 18 18:07:15 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 19 11:56:43 2017 +0000" }, "message": "Remove unused array entrypoints.\n\nTest: test-art-host test-art-target\nChange-Id: I910d1c912c7c9056ecea0e1e7da7afb2a7220dfa\n" }, { "commit": "e761bccf9f0d884cc4d4ec104568cef968296492", "tree": "05a2d20d61c0e91270df2747f0c242433b5ce62b", "parents": [ "b0355130e38034db6b904783a00f74a3524e1881" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 19 08:59:37 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 19 09:32:17 2017 +0000" }, "message": "Revert \"Revert \"Load the array class in the compiler for allocations.\"\"\n\nThis reverts commit fee255039e30c1c3dfc70c426c3d176221c3cdf9.\n\nChange-Id: I02b45f9a659d872feeb35df40b42c1be9878413a\n" }, { "commit": "fee255039e30c1c3dfc70c426c3d176221c3cdf9", "tree": "8207b72cc76513fed9f7b3c01aaa32cd54a87f1c", "parents": [ "cc99df230feb46ba717252f002d0cc2da6828421" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Jan 19 02:11:15 2017 +0000" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Jan 19 02:11:15 2017 +0000" }, "message": "Revert \"Load the array class in the compiler for allocations.\"\n\nlibcore test fails.\n\nThis reverts commit cc99df230feb46ba717252f002d0cc2da6828421.\n\nChange-Id: I5bac595acd2b240886062e8c1f11f9095ff6a9ed\n" }, { "commit": "cc99df230feb46ba717252f002d0cc2da6828421", "tree": "73ac045673e150fa367a8da4d46874f28e928491", "parents": [ "4507fdcb70bd570d5f3968061bf991f0a1233a93" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 17 23:00:24 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 18 15:16:32 2017 +0000" }, "message": "Load the array class in the compiler for allocations.\n\nRemoving one other dependency for needing to pass\nthe current method, and having dex_cache_resolved_types_\nin ArtMethod.\n\noat file increase:\n- x64: 0.25%\n- arm32: 0.30%\n- x86: 0.28%\n\ntest: test-art-host, test-art-target\nChange-Id: Ibca4fa00d3e31954db2ccb1f65a584b8c67cb230\n" }, { "commit": "39cee66a8ddf0254626c9591662cf87e4a1cedc4", "tree": "be25df71e51ce03a8847c23934322b8f282a291b", "parents": [ "a3974581751cd73a896f7c4fcab71beb17c4f9dc" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 13 16:04:53 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 18 11:12:33 2017 +0000" }, "message": "Entrypoints cleanup.\n\nRemove unused ones to facilitate the transition to compressed\ndex caches.\n\ntest: test-art-host, test-art-target\nChange-Id: I1d1cb0daffa86dd9dda2eaa3c1ea3650a5c8d9d0\n" }, { "commit": "942fd3130254d8276cbfe8e6201825e9c49e9b2c", "tree": "62dbcd85ab150604e2d72a3c2d0c3639a28c9b7a", "parents": [ "8bd59a0fd46db83616785168231e09fb95ed2ead" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 20:52:19 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jan 17 16:12:15 2017 +0000" }, "message": "Reduce using ArtMethod\u0027s dex_cache_resolved_types_.\n\nAvoid using the ArtMethod\u0027s dex cache type array shortcut\nin runtime, preparing for its removal. We do not completely\nremove the shortcut yet because it is still used by array\nallocation entrypoints.\n\nFix ArgArray::BuildArgArrayFromObjectArray in reflection.cc\nto not ask for the parameter type to be resolved. It should\nhave been previously resolved when retrieving the Method.\n\nAlso partially revert\n https://android-review.googlesource.com/310717\nbecause it relied on the removed AIOOBE check in the removed\nArtMethod::GetDexCacheResolvedType(). The removed check was\nsimply defensive but it could not be triggered without some\nmemory corruption.\n\nTest: m test-art-host\nBug: 30627598\nChange-Id: Ic45a5ff8c66b79429e440cbc08d67bf22a083682\n" }, { "commit": "e037a736be43c1e8ba9340dcbf1d17722356a37b", "tree": "f0bc7b4a78d04376b042046afbfd0677d766b523", "parents": [ "db54cc42859a8cc24ed1cda7f9b2f64b27dcb34a", "5d37c152f21a0807459c6f53bc25e2d84f56d259" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 17 09:16:31 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Jan 17 09:16:31 2017 +0000" }, "message": "Merge \"Put inlined ArtMethod pointer in stack maps.\"" }, { "commit": "5d37c152f21a0807459c6f53bc25e2d84f56d259", "tree": "7d8cbce0a55f258150a047def70244f79afc866d", "parents": [ "aa89a4c6fca095904521842c018399f1e3501a45" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 12 13:25:19 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 16 22:56:56 2017 +0000" }, "message": "Put inlined ArtMethod pointer in stack maps.\n\nCurrently done for JIT. Can be extended for AOT and inlined boot\nimage methods.\n\nAlso refactor the lookup of a inlined method at runtime to not\nrely on the dex cache, but look at the class loader tables.\n\nbug: 30933338\ntest: test-art-host, test-art-target\nChange-Id: I58bd4d763b82ab8ca3023742835ac388671d1794\n" }, { "commit": "5e01df491f371cdafd5c348511c373b527329a01", "tree": "22573d6cd092b9a7246ae12d8c83f9dc2f228400", "parents": [ "d9a9d44ae018826a380b62cdc09d536f8ce30208", "1998cd02603197f2acdc0734397a6d48b2f59b80" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Mon Jan 16 15:50:46 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jan 16 15:50:47 2017 +0000" }, "message": "Merge changes I168f24de,I8ec60a98,I3c426ed7,Ic809b0f3\n\n* changes:\n Implement HLoadClass/kBssEntry for boot image.\n Store resolved types for AOT code in .bss.\n Make runtime call on main for HLoadClass/kDexCacheViaMethod.\n Remove HLoadClass::LoadKind::kDexCachePcRelative.\n" }, { "commit": "fea84ddbccae1c1242d6a1815f6580d758bae8c7", "tree": "68ac300b7c09e3aaadc7e103b1bd03f5f1985304", "parents": [ "a48d74a59af68d45e5c9b9273c2017ae2470d58a" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Jan 16 13:52:20 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Jan 16 13:52:20 2017 +0000" }, "message": "ART: Fix artInvokePolymorphic.\n\nartInvokePolymorphic called the wrong visit method on\nRememberForGcArgumentVisitor. It should have called VisitArguments()\nrather than Visit().\n\nBug: 30550796,33191393\nTest: art/test/run-test 953\nTest: m test-art-run-test\nChange-Id: I46442203e8dc7000efd2cd64b2dcd5ed83163be6\n" }, { "commit": "1998cd02603197f2acdc0734397a6d48b2f59b80", "tree": "aa639c7ec96f71d7aaf5d0c865a8a133dbc457c3", "parents": [ "6bec91c7d4670905cd67440991ec76fd54d0f000" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jan 13 13:02:58 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 13:39:24 2017 +0000" }, "message": "Implement HLoadClass/kBssEntry for boot image.\n\nTest: m test-art-host\nTest: m test-art-host with CC\nTest: m test-art-target on Nexus 9\nTest: Nexus 9 boots.\nTest: Build aosp_mips64-eng\nBug: 30627598\nChange-Id: I168f24dedd5fb54a1e4215ecafb947ffb0dc3280\n" }, { "commit": "6bec91c7d4670905cd67440991ec76fd54d0f000", "tree": "05f4ba288e629270773c65b34b71be7bae5e92ff", "parents": [ "4155998a2f5c7a252a6611e3926943e931ea280a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 09 15:03:12 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 12:09:53 2017 +0000" }, "message": "Store resolved types for AOT code in .bss.\n\nTest: m test-art-host\nTest: m test-art-target on Nexus 9.\nTest: Nexus 9 boots.\nTest: Build aosp_mips64-eng.\nBug: 30627598\nBug: 34193123\nChange-Id: I8ec60a98eb488cb46ae3ea56341f5709dad4f623\n" }, { "commit": "ac141397dc29189ad2b2df41f8d4312246beec60", "tree": "a2f481463a14695bf9327fd2f549878ecf30c77b", "parents": [ "5c9f90c5ecf2ff6f93ada0f7b18b46d866c59ea1" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Jan 13 11:53:47 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Sun Jan 15 15:18:07 2017 +0000" }, "message": "Revert \"Revert \"ART: Compiler support for invoke-polymorphic.\"\"\n\nThis reverts commit 0fb5af1c8287b1ec85c55c306a1c43820c38a337.\n\nThis takes us back to the original change and attempts to fix the\nissues encountered:\n\n- Adds transition record push/pop around artInvokePolymorphic.\n- Changes X86/X64 relocations for MacSDK.\n- Implements MIPS entrypoint for art_quick_invoke_polymorphic.\n- Corrects size of returned reference in art_quick_invoke_polymorphic\n on ARM.\n\nBug: 30550796,33191393\nTest: art/test/run-test 953\nTest: m test-art-run-test\n\nChange-Id: Ib6b93e00b37b9d4ab743a3470ab3d77fe857cda8\n" }, { "commit": "0d3998b5ff619364acf47bec0b541e7a49bd6fe7", "tree": "a4763c0660372f6311b612c09267cbbc2fe71e89", "parents": [ "aa89a4c6fca095904521842c018399f1e3501a45" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 12 15:35:12 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 12 16:51:12 2017 +0000" }, "message": "Revert \"Revert \"Make object allocation entrypoints only take a class.\"\"\n\nThis reverts commit f7aaacd97881c6924b8212c7f8fe4a4c8721ef53.\n\nChange-Id: I6756cd1e6110bb45231f62f5e388f16c044cb145\n" }, { "commit": "c8144cdad955b77988a48777cfbdc6fd2e8c1916", "tree": "354e00610ec25279a8c4b77e8b685e380815813f", "parents": [ "d1a277954284c4dd4b5b14fd4e58f1854daed848", "f7aaacd97881c6924b8212c7f8fe4a4c8721ef53" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Jan 12 06:19:22 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jan 12 06:19:23 2017 +0000" }, "message": "Merge \"Revert \"Make object allocation entrypoints only take a class.\"\"" }, { "commit": "f7aaacd97881c6924b8212c7f8fe4a4c8721ef53", "tree": "780209ac8e992fa63307062977f672aa5bb55d9e", "parents": [ "2b615ba29c4dfcf54aaf44955f2eac60f5080b2e" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Jan 12 02:58:38 2017 +0000" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Jan 12 02:58:38 2017 +0000" }, "message": "Revert \"Make object allocation entrypoints only take a class.\"\n\n960-default-smali64 is failing.\n\nThis reverts commit 2b615ba29c4dfcf54aaf44955f2eac60f5080b2e.\n\nChange-Id: Iebb8ee5a917fa84c5f01660ce432798524d078ef\n" }, { "commit": "dcf52765ab5886abdd85a4436fa0358b2a31341d", "tree": "b68ee975792a5bf488ed93cfbe09a37f80008288", "parents": [ "a28ddf5140cd1f4a2ae93dbf8be2f200b1552003", "0fb5af1c8287b1ec85c55c306a1c43820c38a337" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 19:18:51 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 11 19:18:51 2017 +0000" }, "message": "Merge \"Revert \"ART: Compiler support for invoke-polymorphic.\"\"" }, { "commit": "0fb5af1c8287b1ec85c55c306a1c43820c38a337", "tree": "66239e7f745fae54e1630e91fb44a859bff615d6", "parents": [ "02e3092f8d98f339588e48691db77f227b48ac1e" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 18:58:15 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 19:17:33 2017 +0000" }, "message": "Revert \"ART: Compiler support for invoke-polymorphic.\"\n\nThis reverts commit 02e3092f8d98f339588e48691db77f227b48ac1e.\n\nReasons for revert:\n\n- Breaks MIPS/MIPS64 build.\n- Fails under GCStress test on x64.\n- Different x64 build configuration doesn\u0027t like relocation.\n\nChange-Id: I512555b38165d05f8a07e8aed528f00302061001\n" }, { "commit": "79f9928fc9e0a88430f3329069bfb2f9a0d37f0c", "tree": "e3142e4829c808c3df1059f3b05c0b3a37193ce9", "parents": [ "716eb25353390f699778a79d69006a5b8d8289c2", "02e3092f8d98f339588e48691db77f227b48ac1e" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Jan 11 18:08:03 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 11 18:08:04 2017 +0000" }, "message": "Merge \"ART: Compiler support for invoke-polymorphic.\"" }, { "commit": "716eb25353390f699778a79d69006a5b8d8289c2", "tree": "ba86b06bb9dc02deea0420d4432e00b219ea94f2", "parents": [ "07d2e7a760aafa54fb043917bf84dad0484dbd68", "f5769e15aa9f058a5af987f0d1354f16f4464a7d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jan 11 17:59:34 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 11 17:59:35 2017 +0000" }, "message": "Merge \"Move most of art::Thread to ObjPtr\"" }, { "commit": "02e3092f8d98f339588e48691db77f227b48ac1e", "tree": "127dd23346206b0547b7c6453a776253252b3c6e", "parents": [ "bc7d0deda4549f314e68ee3e0e6afd68c4a8fd06" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Dec 01 10:33:51 2016 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 16:26:43 2017 +0000" }, "message": "ART: Compiler support for invoke-polymorphic.\n\nAdds basic support to invoke method handles in compiled code.\n\nEnables method verification for methods containing invoke-polymorphic.\n\nAdds k45cc/k45rc output to Instruction::DumpString() which\nwas found to be missing when enabling verification.\n\nInclude stack traces in test 957-methodhandle-transforms for\nfailures so they can be easily identified.\n\nBug: 30550796,33191393\nTest: art/test/run-test 953\nTest: m test-art-run-test\nChange-Id: Ic9a96ea24906087597d96ad8159a5bc349d06950\n" }, { "commit": "2b615ba29c4dfcf54aaf44955f2eac60f5080b2e", "tree": "0a2fe5f9243645a054d4aa094bff5a69cc1abb88", "parents": [ "c9a060f2688599d4a402ee6234db46c2e9b7463f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 06 14:40:07 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 11 10:34:10 2017 +0000" }, "message": "Make object allocation entrypoints only take a class.\n\nChange motivated by:\n- Dex cache compression: having the allocation fast path do a\n dex cache lookup will be too expensive. So instead, rely on the\n compiler having direct access to the class (either through BSS for\n AOT, or JIT tables for JIT).\n- Inlining: the entrypoints relied on the caller of the allocation to\n have the same dex cache as the outer method (stored at the bottom of\n the stack). This meant we could not inline methods from a different\n dex file that do allocations. By avoiding the dex cache lookup in\n the entrypoint, we can now remove this restriction.\n\nCode expansion on average for Docs/Gms/FB/Framework (go/lem numbers):\n- Around 0.8% on arm64\n- Around 1% for x64, arm\n- Around 1.5% on x86\n\nTest: test-art-host, test-art-target, ART_USE_READ_BARRIER\u003dtrue/false\nTest: test-art-host, test-art-target, ART_DEFAULT_GC_TYPE\u003dSS ART_USE_TLAB\u003dtrue\n\nChange-Id: I41f3748bb4d251996aaf6a90fae4c50176f9295f\n" }, { "commit": "f5769e15aa9f058a5af987f0d1354f16f4464a7d", "tree": "8a55c514c70a701a06102d3cbda6e4fe8a84242c", "parents": [ "2e17788df086284845108c6b39954a5c74e93a69" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 10 15:54:41 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 10 15:57:11 2017 -0800" }, "message": "Move most of art::Thread to ObjPtr\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I73d0de3c875d3ec3d732034fdc961dae79ef4070\n" }, { "commit": "dba61481035b7944173181ec9ee02aea41dd0e29", "tree": "8629775e657cc8968e2f8fa2af6256c7a58c7dd4", "parents": [ "cb2ce333ee2d1053919fe0acc390369bea3ad77e" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 21 08:20:29 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 21 11:43:14 2016 -0800" }, "message": "Revert \"Revert \"Revert \"Revert \"Basic obsolete methods support\"\"\"\"\n\nA GetDeclaringClass()-\u003eGetDexCache() got inserted during the\nmerge/review process meaning that we would try to access incorrect\ndex-cache in obsolete methods in some situations.\n\nAlso when using tracing we would loop forever (or at least until an\nOOM error) in test 916 due to tracing forcing InterpretOnly mode\nmeaning methods would never be jitted.\n\nBug: 32369913\nBug: 33630159\n\nTest: ART_TEST_TRACE\u003dtrue \\\n ART_TEST_JIT\u003dtrue \\\n ART_TEST_INTERPRETER\u003dtrue mma -j40 test-art-host\n\nThis reverts commit f6abcda293b115a9d7d8a26376ea2dcf2d1dc510.\n\nChange-Id: I0773bfcba52e3cd51a83be815c6a50c189558f48\n" }, { "commit": "f6abcda293b115a9d7d8a26376ea2dcf2d1dc510", "tree": "a30ce596848d14c95a73449dbf2bfccf26a510b9", "parents": [ "ce77fc0e7f60a15354bb20c356537cbf8b53b722" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 21 09:26:18 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 21 09:26:18 2016 +0000" }, "message": "Revert \"Revert \"Revert \"Basic obsolete methods support\"\"\"\n\nFails in tracing mode\n\nBug: 32369913\nBug: 33630159\n\nThis reverts commit ce77fc0e7f60a15354bb20c356537cbf8b53b722.\n\nChange-Id: I1bdcf6ad467f2e31f9c5d0c3c987b90a4f5efc69\n" }, { "commit": "ce77fc0e7f60a15354bb20c356537cbf8b53b722", "tree": "b4301967c423ab190a4274409d99a3d6e23f0e8a", "parents": [ "d54f43ca39dfa92f08c2d760123f185f0f65fb86" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 15 01:05:52 2016 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Dec 19 14:26:18 2016 -0800" }, "message": "Revert \"Revert \"Basic obsolete methods support\"\"\n\nThis reverts commit b81a9840b44480bfeacd74b8d9f51e06f295411d.\n\nThere were 2 issues with the original CL\u0027s test 916-obsolete-jit that\ncaused it to sporadically fail.\n\nFirst, when checking if we had jitted the function under test in\n916-obsolete-jit we failed to check to see if the function \u0027doCall\u0027,\nwhich is used to work-around bugs in our deoptimization \u0026 compilation\nsystems, had also been jitted.\n\nIn the case where the \u0027sayHi\u0027 function had been jitted but the\n\u0027doCall\u0027 function had not we would (correctly) fail to redefine the\n\u0027Transform\u0027 class since we would not be able to deoptimize the \u0027sayHi\u0027\nfunction since it is under a quick_to_interpreter_bridge (runtime)\nframe.\n\nSecondly, the function Main.isInterpretedFunction was incorrect and\nwould always return false, regardless of the actual state of the\nfunction. This would cause the test to fail as the\nquick_to_interpreter_bridge frame prevented deoptimization of the\nobsoleted function. Usually the warm-up period was enough to make sure\nthe methods were jitted anyway but this was not guaranteed.\n\nBoth of these problems become more likely to occur on systems with\nmore cpu contention such as the buildbots.\n\nTest: stress --cpu 60 \u0026; while ./test/run-test --host --jit 916; do ; done\n\nReason for revert: Fixed bug in test that was causing failures.\n\nOriginal Tests:\n\nTest: ./test/run-test --host 914\nTest: ./test/run-test --host 915\nTest: ./test/run-test --host 916\nTest: mma -j40 test-art-host\nTest: ART_TEST_JIT\u003dtrue \\\n ART_TEST_INTERPRETER\u003dtrue mma -j40 test-art-host\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64 --no-jit\n\nBug: 32369913\nBug: 33630159\n\nChange-Id: If1a92e47b90965a7dc21c5826185debe62bd1554\n" }, { "commit": "b81a9840b44480bfeacd74b8d9f51e06f295411d", "tree": "d79a3567b272b6eb4fb94d66a537cf92bef455db", "parents": [ "d8936da27b792d1ca02e59c92456a1a53c7b9905" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 15 00:59:05 2016 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 15 00:59:05 2016 +0000" }, "message": "Revert \"Basic obsolete methods support\"\n\nThis reverts commit d8936da27b792d1ca02e59c92456a1a53c7b9905.\n\nReason for revert: Some sort of race in JIT\n\nChange-Id: Ibb4e520bb0721d6d7aa2c841a52eb5baff07740e\n" }, { "commit": "d8936da27b792d1ca02e59c92456a1a53c7b9905", "tree": "6a593dd426e7f4a15c651dd3b872f22fb64f09ce", "parents": [ "7c73bf3518eea7544406eadfed6946411c54a267" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Nov 28 16:24:32 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 14 13:01:27 2016 -0800" }, "message": "Basic obsolete methods support\n\nAdd support for executing obsolete methods following redefinitions.\nThis support includes methods that have been jitted. This does not add\nany additional validity checks to our redefinition functions.\n\nNote using work-arounds to ensure 916 works pending some fixes to the\ncompiler, deoptimizer, or both.\n\nTest: ./test/run-test --host 914\nTest: ./test/run-test --host 915\nTest: ./test/run-test --host 916\nTest: mma -j40 test-art-host\nTest: ART_TEST_JIT\u003dtrue \\\n ART_TEST_INTERPRETER\u003dtrue mma -j40 test-art-host\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64 --no-jit\n\nBug: 32369913\nBug: 33630159\n\nChange-Id: I78ef95f484146f1fb93c37fc50f56575bdab2432\n" }, { "commit": "b08265b2d61cd3923dd6fc01d6c82f73d5230e82", "tree": "cebb0dd2570ed90265dc376d89c17768700fd90d", "parents": [ "b4ee681c21564ee9afe0202e1006cfa21019e88b", "1b8464d17c2266763714ae18be7c4dc26e28bf61" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Dec 07 17:15:08 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Dec 07 17:15:08 2016 +0000" }, "message": "Merge \"MIPS32: Pass more arguments in registers.\"" }, { "commit": "1b8464d17c2266763714ae18be7c4dc26e28bf61", "tree": "19ab81b439fc216e289cb14be8a7618dbafb4f50", "parents": [ "d7a7c7f3e93de9fa915e66d54dfc799efcc12ffb" ], "author": { "name": "Alexey Frunze", "email": "Alexey.Frunze@imgtec.com", "time": "Sat Nov 12 17:22:05 2016 -0800" }, "committer": { "name": "Alexey Frunze", "email": "Alexey.Frunze@imgtec.com", "time": "Tue Dec 06 13:57:04 2016 -0800" }, "message": "MIPS32: Pass more arguments in registers.\n\nSpecifically, use A0-A3,T0-T1 for non-floats and F8-F19 for floats.\n\nTest: booted MIPS32R2 in QEMU\nTest: test-art-target-run-test-optimizing (MIPS32R2) on CI20\nTest: test-art-target-gtest (MIPS32R2) on CI20\nTest: booted MIPS64 (with 2nd arch MIPS32R6) in QEMU\nTest: test-art-target-run-test-optimizing (MIPS32R6) in QEMU\nTest: test-art-target-gtest (MIPS32R6) in QEMU\nTest: test-art-host-gtest\n\nChange-Id: Ib8b0310a109d9f3d70119c1e605e54b013e60728\n" }, { "commit": "07ab8831ff6025f91f927cc9e60c04801583666c", "tree": "908a6ea9f01c47f11cb9fb7d9cd78e107ea15493", "parents": [ "7874c0c9265b827393f49201d003fd6e6fcef9c0", "5ace201d84adb7753680bf4c7877b3b71558da82" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 30 21:02:25 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Nov 30 21:02:26 2016 +0000" }, "message": "Merge \"Revert \"Revert CC related changes.\"\"" }, { "commit": "5ace201d84adb7753680bf4c7877b3b71558da82", "tree": "4687d41f5a1c8358e8356066d1e1b43b231b6037", "parents": [ "304f2d3a1d090846a7a55a0016efc9fc8eb14078" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 30 10:15:41 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 30 11:01:52 2016 -0800" }, "message": "Revert \"Revert CC related changes.\"\n\nDisable entrypoint switching in ResetQuickAllocEntryPointsForThread\ninstead of callers. Fixes bug where instrumentation would switch\nto non CC entrypoints for non X86_64 architectures causing aborts.\n\nBug: 31018974\n\nTest: test-art-host\nTest: test/run-test 099\n\nThis reverts commit 96172e0172c5fca6e9a5ad4b857a24d8c7b064e5.\n\nChange-Id: If206694ae35ff4446c6a8a97bfbcbf2dac35e3f9\n" }, { "commit": "96172e0172c5fca6e9a5ad4b857a24d8c7b064e5", "tree": "04bb8fe644c9bc961c8ac33e609c1b162ec45222", "parents": [ "626b839f881f09a1481377a76712d08580c47a16" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 30 11:52:19 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 30 11:52:19 2016 +0000" }, "message": "Revert CC related changes.\n\nRevert: \"X86_64: Add allocation entrypoint switching for CC is_marking\"\nRevert: \"Fix mips build in InitEntryPoints\"\nRevert: \"Fix mac build in ResetQuickAllocEntryPoints\"\n\nTest: test-art-target-run-test\nChange-Id: If38d44edf8c5def5c4d8c9419e4af0cd8d3be724\n" }, { "commit": "8003cafbcade4cd2929b91de3e50d2660c424280", "tree": "8ee82e8c6778dec2e84fdde12231fd268088d3a9", "parents": [ "ae7f6b965e15794b0014a0c1db5e1fcf9e841e5c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 29 14:17:09 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 29 14:17:33 2016 -0800" }, "message": "Fix mac build in ResetQuickAllocEntryPoints\n\nUnused arg warning.\n\nTest: None\n\nChange-Id: I5b147b1a584b96ad5457c9fee36f3edb0498cecf\n" }, { "commit": "ab191538a1d9eee6ec96bc3fa86dde36a007a6f5", "tree": "64c4ffaf96a8cf0aeb27bf8cbbbd5cfa42d1ff40", "parents": [ "8fb28dcf0e83f7153e76e176671cd4ad1f20205b", "f5de23265360e15fcfceb7d07bdadca0e5bb5f0a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 29 21:32:11 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Nov 29 21:32:12 2016 +0000" }, "message": "Merge \"X86_64: Add allocation entrypoint switching for CC is_marking\"" }, { "commit": "8a0128a5ca0784f6d2b4ca27907e8967a74bc4c5", "tree": "0dec75200282ae5e49785395e97bd4e6459f1c09", "parents": [ "60438b46090d22bb9b978196f5aa53fab3b89759" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 28 07:38:35 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 29 11:11:46 2016 -0800" }, "message": "ART: Add dex::StringIndex\n\nAdd abstraction for uint32_t string index.\n\nTest: m test-art-host\nChange-Id: I917c2881702fe3df112c713f06980f2278ced7ed\n" }, { "commit": "a5b09a67034e57a6e10231dd4bd92f4cb50b824c", "tree": "304be738f4fa528b7ad2676103eecc84c79eaeeb", "parents": [ "dac7ad17c78387d15d7aefae0f852dddf5f37e34" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Nov 17 15:21:22 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 21 10:57:00 2016 -0800" }, "message": "ART: Add dex::TypeIndex\n\nAdd abstraction for uint16_t type index.\n\nTest: m test-art-host\nChange-Id: I47708741c7c579cbbe59ab723c1e31c5fe71f83a\n" }, { "commit": "f5de23265360e15fcfceb7d07bdadca0e5bb5f0a", "tree": "1bf893445a4c6fb35c430a68d79f66fa02fe4d2b", "parents": [ "e89667815b9d0a1eacb91678fed2a7518bb07cc2" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 15 17:56:15 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 16 23:26:16 2016 +0000" }, "message": "X86_64: Add allocation entrypoint switching for CC is_marking\n\nOnly X86_64 done so far. Use normal TLAB allocators if GC is not\nmarking.\n\nAllocation speed goes up by ~8% based on perf sampling.\n\nWithout change:\n1.19%: art_quick_alloc_object_region_tlab\n\nWith change:\n0.63%: art_quick_alloc_object_tlab\n0.47%: art_quick_alloc_object_region_tlab\n\nBug: 31018974\nBug: 12687968\n\nTest: test-art-host-run-test\n\nChange-Id: I4c4d9eb229d4ad2f41b856ba5c2958a5eb3b7ffa\n" }, { "commit": "9fd8c60cdff7b28a89bb97fd90ae9d0f37cf8f8b", "tree": "630b6f23441a26e4b5d9434900993a710635c9e9", "parents": [ "26c8f54be31ccf99540906746b17fc8bba1bab9a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 14 14:38:53 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 15 13:02:24 2016 -0800" }, "message": "Pass object instead of class to instanceof entrypoint\n\nReduces code size. Also avoid read barrier for kArrayCheck case.\n\nBug: 32577579\n\nTest: test-art-host, test-art-target CC\n\nChange-Id: Ia890f656fe166b2d39c522b63a8a6469404134ae\n" }, { "commit": "b99f4d6463e7cb5654af3893ed7b3113665df658", "tree": "1bacc6a2bd01426c921e2f83f39bb63f9d9d8048", "parents": [ "67381d6298edf18760b0c9b3d16572dc3894e551" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 07 16:17:26 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 08 12:46:59 2016 -0800" }, "message": "Change check cast entrypoint to check instance of\n\nReduces code size since we do not need to reload class before\ncalling slow path.\n\nTODO: Delete read barriers in the check cast code since the slow\npath will retry with the proper read barriers if the check fails.\n\nBug: 12687968\nBug: 29516974\n\nTest: test-art-host + test-art-target with CC\n\nChange-Id: Ia4eb9bbe3fe2d2016e44523cf0451210828d7b88\n" }, { "commit": "12b58b23de974232e991c650405f929f8b0dcc9f", "tree": "bc08011e3b9f4c132fc605dc54d1d5f50c932a29", "parents": [ "b0b2080b4cea8365052d1be973e927f8ee85f609" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Nov 01 11:55:29 2016 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Wed Nov 02 16:31:04 2016 -0700" }, "message": "Clean up the runtime read barrier and fix fake address dependency.\n\n- Rename GetReadBarrierPointer to GetReadBarrierState.\n- Change its return type to uint32_t.\n- Fix the runtime fake address dependency for arm/arm64 using inline\n asm.\n- Drop ReadBarrier::black_ptr_ and some brooks code.\n\nBug: 12687968\nTest: test-art with CC, Ritz EAAC, libartd boot on N9.\nChange-Id: I595970db825db5be2e98ee1fcbd7696d5501af55\n" }, { "commit": "ef41db7a3f322a1feb305fdb457410c4cea94d00", "tree": "c03152c091ef62ab70f5be0a2fe3a965b189132c", "parents": [ "1458e0c09fe0a3b9fa5fd7beb9b6077d1fc46b1d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 25 15:08:01 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 26 15:25:40 2016 -0700" }, "message": "Move interpreter to ObjPtr\n\nMoved most of interpreter, interpreter_common,\ninterpreter_switch_impl, and some of mterp to ObjPtr.\n\nBug: 31113334\n\nTest: test-art-host ART_TEST_INTERPRETER\u003dtrue\nTest: art/tools/run-libcore-tests.sh \u0027--mode\u003dhost\u0027 \u0027--variant\u003dX32\u0027 --debug\n\nChange-Id: I0935d18287e1332205c17c5a018aa167788ab897\n" }, { "commit": "5b6fbd02f03fd41829c7dd4bfce97a62346be854", "tree": "bc1c72e0ac46d4e1c630e11fdf6ee8ba668cdf77", "parents": [ "3e653ff5d455a79b5fc8e272c622dd472fafd422", "9d7ef62b854289632791a83223c1a5a5b3c8fc64" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 25 15:18:14 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Oct 25 15:18:14 2016 +0000" }, "message": "Merge changes Iae3a933e,I08ff5d6e\n\n* changes:\n ART: Make IndirectReferenceTable resizable\n ART: Change IndirectReferenceTable\n" }, { "commit": "1bc4b178590fa204265363a0207f1388e8bd6d15", "tree": "791fd12e3fac0c9c4ec260233d471d8a97cca5ca", "parents": [ "3667e26de4856cccf24bcbab54ad3349a05267c0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 24 16:53:39 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 25 10:01:08 2016 +0100" }, "message": "Revert \"Disable kBssEntry LoadString sharpening.\"\n\nFix .bss GC root walking by registering the oat file\nwith the class loader\u0027s class table.\n\nAlso fix potentially outdated ObjPtr\u003c\u003e use in debug build.\n\nThis reverts commit b55fdbb30b3bc4e334c241153b98c0a6ea4a4a2b.\n\nTest: m test-art-host\nBug: 32124939\nChange-Id: I0b7e3b93cb53c7b22408aa10a04eaf5582c69ee8\n" }, { "commit": "e03662b71bbb4d262af0840bf90ce4fc84750b43", "tree": "0d89f7d90a46a7a2f8d120e3b0aabf3e1d277004", "parents": [ "dc061d038e4e48fe2a967fd4a9c200d112df5698" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Oct 13 17:12:56 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Oct 24 19:16:54 2016 -0700" }, "message": "ART: Change IndirectReferenceTable\n\nChange cookie structure to allow for more entries. Use a local\nhole-count caching scheme. The design is driven by two\nconsiderations. For one, the change is small and mostly local.\nThe other point is to still allow inlining of functions involved\nwith JNI transitions.\n\nThis change is in preparation for a resizable backing table for\n\"unlimite\" local references.\n\nmicro_native tests show changes are in the noise.\n\nBug: 32125344\nTest: m test-art-host\nChange-Id: I08ff5d6eaed75d13ec88f469fb0d18328a0eeb70\n" }, { "commit": "024662a3102df92fabf4f40d9ec40976f8e20c5d", "tree": "248a072ebfa8e3a7897ad08f819456f28e47c0bc", "parents": [ "4a4ff643b629b268787fb97648116e1ac391f53b", "af1e2990cd1406a0fb7cba1d2e208208e950e413" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Oct 20 14:41:18 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Oct 20 14:41:18 2016 +0000" }, "message": "Merge changes I16f8b7ec,I075bbf55\n\n* changes:\n jni: Support @FastNative methods that return objects\n Revert \"jni: Disable FastNative path for methods returning objects\"\n" }, { "commit": "d985a5a81a45595f5cb4d12da81eec760801c92c", "tree": "3d49d34870ddb439596321e6e6577845e97c61ac", "parents": [ "b5706dd4e86c3687153a02fdda715e40f69c55d1", "94ce9c2f41ea198f5fdcfc09c48b9984c95a9c61" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Oct 20 09:24:03 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Oct 20 09:24:03 2016 +0000" }, "message": "Merge \"Change pResolveString entrypoint to kSaveEverything.\"" }, { "commit": "af1e2990cd1406a0fb7cba1d2e208208e950e413", "tree": "07e80e4dcc31931003a87be7884134ea42ceec07", "parents": [ "3f699ae23266b2c4adc98958dcbd80c71c2c3284" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Oct 12 17:44:50 2016 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Oct 19 11:38:54 2016 -0700" }, "message": "jni: Support @FastNative methods that return objects\n\nBug: 32088975\nChange-Id: I16f8b7ec6b251812af60ab25f2153d9b72f37044\n" }, { "commit": "94ce9c2f41ea198f5fdcfc09c48b9984c95a9c61", "tree": "8dcaf27e8b34f51f5849b1e8d4366968f9bae849", "parents": [ "2f61867045ffbd0c38a4ecec5f59632004f7efff" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 30 14:50:51 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Oct 19 18:02:48 2016 +0100" }, "message": "Change pResolveString entrypoint to kSaveEverything.\n\nTest: Run ART test suite including gcstress on host and Nexus 9.\nTest: Run ART test suite including gcstress with baker CC on host and Nexus 9.\nBug: 20323084\nChange-Id: I63c21a7d3be8ff7a5765b5003c85b5317635efe6\n" }, { "commit": "709b070044354d9f47641f273edacaeeb0240ab7", "tree": "3a8ac051d7c35076303984d0d892cdd396b60427", "parents": [ "1a4de6a2453a3ad0310aca1a44e7e2d3b6f53bc1" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Oct 13 09:12:37 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Oct 18 14:10:04 2016 -0700" }, "message": "Remove mirror:: and ArtMethod deps in utils.{h,cc}\n\nThe latest chapter in the ongoing saga of attempting to dump a DEX\nfile without having to start a whole runtime instance. This episode\nfinds us removing references to ArtMethod/ArtField/mirror.\n\nOne aspect of this change that I would like to call out specfically\nis that the utils versions of the \"Pretty*\" functions all were written\nto accept nullptr as an argument. I have split these functions up as\nfollows:\n1) an instance method, such as PrettyClass that obviously requires\nthis !\u003d nullptr.\n2) a static method, that behaves the same way as the util method, but\ncalls the instance method if p !\u003d nullptr.\nThis requires using a full class qualifier for the static methods,\nwhich isn\u0027t exactly beautiful. I have tried to remove as many cases\nas possible where it was clear p !\u003d nullptr.\n\nBug: 22322814\nTest: test-art-host\nChange-Id: I21adee3614aa697aa580cd1b86b72d9206e1cb24\n" }, { "commit": "174d17b2f70f009d5fa349bb94b1703297c36ec4", "tree": "a73a00338e48376a0d68da19b3075eadfc5c109d", "parents": [ "8c0fee9ca38ea1b4ee6a8ab72c5f9c0ba924761f", "82f3a20b1f51b17b432d5164d5109abeb1b5a947" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 17 15:08:21 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Oct 17 15:08:21 2016 +0000" }, "message": "Merge \"Add DCHECK() for class loader in artResolveStringFromCode().\"" }, { "commit": "043eb9a4496cacaffe60f381c5686c989cb5d701", "tree": "42f89fbff75828cc962992000fa6f018fb46caa4", "parents": [ "cfdff144f154df876987fefb340058a975c43381" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Oct 14 11:21:38 2016 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Oct 14 13:04:25 2016 -0700" }, "message": "Add a mark bit check in the read barrier jni entrypoint.\n\nBug: 30437917\nBug: 12687968\nTest: test-art host/N9, N9 libartd boot, Ritz EAAC.\nChange-Id: Ic5f8080959f243e80f00da73ba5f4b91a8d8b617\n" }, { "commit": "82f3a20b1f51b17b432d5164d5109abeb1b5a947", "tree": "9abd1c854225b419d4bf7c1f5e75edd74a6970be", "parents": [ "4aed412037e6e7bad2cf589645449df960b88200" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Oct 14 16:28:52 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Oct 14 16:28:52 2016 +0100" }, "message": "Add DCHECK() for class loader in artResolveStringFromCode().\n\nAddress late comment from\n https://android-review.googlesource.com/281642\n\nTest: m test-art-host\nBug: 20323084\nBug: 30627598\nChange-Id: I3ba2617d2bc52ac8ba11307d206e6f5e6c017fb7\n" }, { "commit": "1a5337fff2cc6cb9d563c8b32aca75f485d23373", "tree": "03616e90a91c5c4c1bd7ff03c3d193c3b24bf0db", "parents": [ "c8b722b3cebff245d614e95d1849de45ef7bdf06" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 13:48:23 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 17:37:56 2016 -0700" }, "message": "Move mirror::ObjectArray to ObjPtr\n\nAlso reduce calls to ObjPtr::Ptr.\n\nBug: 31113334\n\nTest: test-art-host -j32\n\nChange-Id: I73e1b356972bb079b66332f00f7f07451601044e\n" }, { "commit": "28bd2e4f151267b34b8e1eb19c489d8d547bbf5c", "tree": "d99ae3fe74ea63b83091898d830d3efe68cd479d", "parents": [ "6e5fa09510c7280168e040382d27dd8b55760d9a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 04 13:54:57 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 10:59:28 2016 -0700" }, "message": "Move mirror::Class to use ObjPtr\n\nLeave the return types as non ObjPtr for now. Fixed moving GC bugs\nin tests.\n\nTest: test-art-host\n\nBug: 31113334\n\nChange-Id: I5da1b5ac55dfbc5cc97a64be2c870ba9f512d9b0\n" }, { "commit": "9d156d500801accee919b6d51e22d6ddcdcd4a05", "tree": "92129562c0ba1673c660297f26444a4a2ad31459", "parents": [ "078483d4dfd049d5b3f192a99e2dfdc355e4754f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 06 17:44:26 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 12 09:49:49 2016 -0700" }, "message": "Move Heap parameters to ObjPtr\n\nDeleted some unused object dumping code.\n\nTest: test-art-host\n\nBug: 31113334\n\nChange-Id: I747220caafe6679591fd4b361d7f50383a046164\n" }, { "commit": "1506bf332232907016fe3fcb3956c9d4b064b482", "tree": "f8a5c83a5c0ae627f8469b18743aa18d47719f54", "parents": [ "5065b09332328477d489b8ce26c3a1381e6f8d49" ], "author": { "name": "Goran Jakovljevic", "email": "Goran.Jakovljevic@imgtec.com", "time": "Mon Oct 10 19:12:09 2016 +0200" }, "committer": { "name": "Goran Jakovljevic", "email": "Goran.Jakovljevic@imgtec.com", "time": "Mon Oct 10 19:15:56 2016 +0200" }, "message": "Add missing cases in quick_trampoline_entrypoints_test for MIPS64\n\nTest: mma test-art-target-gtest-quick_trampoline_entrypoints_test\nChange-Id: I4596d47f7219b375a60d6b9d80f797d1f01facd7\n" }, { "commit": "c4f3925490a73da8dc74884a1deb965d4ecaf14e", "tree": "7fe566827f8ab903af1acb5697c86a6efe3187b3", "parents": [ "18f7de841fee3ca6f0c04e7caa57c3ce76b36231" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 18:32:08 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 06 13:55:58 2016 -0700" }, "message": "Move remaining jobject related functions to use ObjPtr\n\nAlso added ObjPtr::DownCast.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I59c253211dc435579ffdfd49f856861ab13d262c\n" }, { "commit": "1432a5bb0608a920e7281b38ee5f6e8dfcfae5ef", "tree": "804b9e78fa9d2705a1e58571944af7381858a5b0", "parents": [ "4aa6a93c46a959df1ab71ee7a68ad345338046ef" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 04 15:41:42 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 09:43:15 2016 -0700" }, "message": "Don\u0027t push handle scope for critical native generic JNI\n\nLeaving a stale handle scope causes problems for the GC the next\ntime roots are visited. At this point the stack will have other\ncontents and the GC will attempt to mark many invalid roots.\n\nBug: 31933313\n\nTest: non preopt eng build booting.\nTest: test-art-host\n\n(cherry picked from commit 92879f9bbcfc034660ed1ff5ef741d7f2bcb116f)\n\nChange-Id: I7d57964ccd2b59a05bb06f67062f646362ce1204\n" }, { "commit": "556c9a2f182c4fcb9eba9dc4937095665f314a2b", "tree": "ffe6d3711b565330fb883953144ddd7603356804", "parents": [ "e793bbb4a8a832423e7d6150b6a8fad75d63d36e", "1cc62e4ea24828fdb3f3da0b8603f0b107d09a04" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 00:39:07 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Oct 05 00:39:08 2016 +0000" }, "message": "Merge \"Rename ObjPtr::Decode to ObjPtr::Ptr\"" }, { "commit": "1cc62e4ea24828fdb3f3da0b8603f0b107d09a04", "tree": "718e322f3d05ac095770d4dc2a68b824bf3974a7", "parents": [ "82d4838d6bb3480cd25327cedc5179fb2d86881c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 03 18:01:28 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 04 10:46:16 2016 -0700" }, "message": "Rename ObjPtr::Decode to ObjPtr::Ptr\n\nDone to prevent ambiguity with ScopedObjectAccess::Decode.\n\nBug: 31113334\n\nTest: test-art-host\nChange-Id: I07a2497cc9cf66386311798933547471987fc316\n" }, { "commit": "aad75c6d5bfab2dc8e30fc99fafe8cd2dc8b74d8", "tree": "c1b9e1eabcf35c5cbb5b4f46313a4e062f2d5d51", "parents": [ "82d4838d6bb3480cd25327cedc5179fb2d86881c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 03 08:46:48 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 04 17:51:46 2016 +0100" }, "message": "Revert \"Revert \"Store resolved Strings for AOT code in .bss.\"\"\n\nFixed oat_test to keep dex files alive. Fixed mips build.\nRewritten the .bss GC root visiting and added write barrier\nto the artResolveStringFromCode().\n\nTest: build aosp_mips-eng\nTest: m ART_DEFAULT_GC_TYPE\u003dSS test-art-target-host-gtest-oat_test\nTest: Run ART test suite on host and Nexus 9.\nBug: 20323084\nBug: 30627598\n\nThis reverts commit 5f926055cb88089d8ca27243f35a9dfd89d981f0.\n\nChange-Id: I07fa2278d82b8eb64964c9a4b66cb93726ccda6b\n" } ], "next": "3398c7874e002beaa6c2b2fadf183e7d1ddad23a" }