)]}' { "log": [ { "commit": "55a45f3edfe4b17424de3c6c29c55fff1a2c10fc", "tree": "ae0083333c25c51e7365706d88e9da8eb0858dba", "parents": [ "de1449e2ed9003897c743145261240c8e38f04d2" ], "author": { "name": "Lokesh Gidra", "email": "lokeshgidra@google.com", "time": "Thu Feb 25 10:45:14 2021 -0800" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Sun Feb 28 03:46:38 2021 +0000" }, "message": "Fix indentation in Field.get()\n\nBug: 174433134\nTest: art/test/testrunner/testrunner.py --host\nChange-Id: Ic3b865e1a2819214349493323d91bf6aae29d54f\n" }, { "commit": "3a8d7c2de4392ff4c40858803447efe14c3c9fa8", "tree": "a6a13854431cc59646f3b94e5e3df420a83e6547", "parents": [ "ae7c8da2dcb775b58ee9285cb7c3433a8be66ebe" ], "author": { "name": "Lokesh Gidra", "email": "lokeshgidra@google.com", "time": "Wed Feb 24 11:29:26 2021 -0800" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Feb 25 02:19:19 2021 +0000" }, "message": "Return null for PhantomReference.referent in Field_get()\n\nTo be consistent with the behavior of PhantomReference\u0027s get(), we\nshould always return null. Otherwise it\u0027s possible that an object\nthat\u0027s only reachable via a PhantomReference becomes strongly reachable,\nviolating the defining property of PhantomReferences.\n\nBug: 174433134\nTest: art/test/testrunner/testrunner.py --host\nChange-Id: Ic0c5ba1826c69cd7e8c0d2b35dc3fc6ac5b93c0e\n" }, { "commit": "8dcdd4735bff02673e5a3ec1238b8aff1d217957", "tree": "1107ea6e73bc6fce6d23eec0b07b1325cf7b5233", "parents": [ "5de5efef38f52f248d1e1c88fdcf3d211aabe712" ], "author": { "name": "Lokesh Gidra", "email": "lokeshgidra@google.com", "time": "Wed Feb 17 19:30:02 2021 -0800" }, "committer": { "name": "Lokesh Gidra", "email": "lokeshgidra@google.com", "time": "Thu Feb 18 17:51:18 2021 +0000" }, "message": "Handle Reference-type object in Field_get()\n\nBy fetching referent from Reference-type object directly (without\ngetReferent()) we run the risk of invoking the read-barrier (and hence\npushing an unmarked object to the mark-stack) when the GC doesn\u0027t expect\nthat to happen.\n\nBug: 174433134\nTest: art/test/testrunner/testrunner.py --host\nChange-Id: I5ff7598f9f945518db28b86a5f6ed25c39b73d05\n" }, { "commit": "de05550bf33d3441655cfddd66e788791f2412e7", "tree": "6815d4af1a0a825df90318f753dc3afa71704ece", "parents": [ "988c3911671598d7c840c65bf1cdfafa1e05c582" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Aug 02 13:41:35 2019 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 01 13:05:17 2019 +0000" }, "message": "Use ClassStatus::kVisiblyInitialized in reflection.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: aosp_taimen-userdebug boots.\nTest: run-gtests.sh\nTest: testrunner.py --target --optimizing\nBug: 36692143\nChange-Id: I53342f5bb6285c9ca95445791dd10e04c6d8f962\n" }, { "commit": "38a062eb7390f07ee0a318517f71d9b56491a5f7", "tree": "67204f1fbc785b888f9d5718e31cf22de6b2d727", "parents": [ "e80ecf31e0d58a90d6b2a1d65ec090c03abc86dc" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Aug 02 13:59:27 2019 +0100" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Mon Aug 05 19:27:58 2019 +0000" }, "message": "Weaken 2 DCHECK()s in reflection.\n\nReflection can be used on classes that are being initialized\nrather than just classes that are already initialized as we\nwere previously asserting.\n\nTest: Additional tests in 100-reflect2.\nChange-Id: I072c28533e9248856b49fddb4bc46109448ee8a9\n" }, { "commit": "9186b182c0e0e5d57dc15a03e4f8d0f2951f3a96", "tree": "e230bdce1f86d411c50ded9efc588590e98df261", "parents": [ "a5175541c197e7bf9b03651ea5da4e64a2ac2d27" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 06 14:55:54 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 07 10:06:43 2018 +0000" }, "message": "Refactor ClassLinker::FindPrimitiveClass().\n\nIntroduce ClassLinker::LookupPrimitiveClass() that never\nthrows an exception and use it to clean up callers that\nare not supposed to cause thread suspension.\n\nTest: m test-art-host-gtest\nBug: 118909457\nChange-Id: Ibfee8b7991c4a0d1fa636c85f14437c9865d1658\n" }, { "commit": "52ecb65ae9e1ad6fe7f58beecc88cdc08e08f0c4", "tree": "7a8ee84cad0df34ab33bdcb0115fc0c79d6c83be", "parents": [ "c2099b0b0c052dd61137b9cf5f9f8b30bab1a7a8" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 24 15:18:21 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 26 08:44:12 2018 -0700" }, "message": "ART: Add object-array-alloc-inl.h\n\nIn an effort to reduce the (transitive) proliferation of heap-inl\nadd a specific inline header for object array allocation.\n\nBug: 118385392\nTest: mmma art\nTest: m test-art-host\nChange-Id: I0d7c40ed53708d4c759190961b40f0cac3fe696d\n" }, { "commit": "18090d118bfb04620aeef719e2d7780c26298bf8", "tree": "040e5195f0278c9bc846d5543c0e36771ffc5b48", "parents": [ "9ddef18ae95859a985e7a0de7e22999fcbc28e07" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 01 16:53:12 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 04 17:52:35 2018 +0100" }, "message": "Refactor String resolution.\n\nUse the same pattern as type resolution and avoid some\nunnecessary read barriers in the fast path. Consolidate\nnaming between ArtField and ArtMethod.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nChange-Id: Iea69129085f61f04a4add09edd0eadbb7ac9ecb2\n" }, { "commit": "bcf175247272d0e321c8d988c3c01c123b56e36e", "tree": "9f65ece5ce9474aa4fcf16fbfca6278109dc9a67", "parents": [ "09c5ca40635faee00f40f6ca0581dd475efd545e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 01 13:14:32 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 01 16:31:30 2018 +0100" }, "message": "ObjPtr\u003c\u003e-ify array allocations.\n\nAnd remove some unnecessary calls to ObjPtr\u003c\u003e::Ptr().\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: Ie313980f7f23b33b0ccea4fa8d5131d643c59080\n" }, { "commit": "c5b7564fdc187853d1f8266a54316ef402f83e02", "tree": "b0aeab247ceeef6ffa262c70251559643b4c2f0d", "parents": [ "a7d37cf735f05cfc3a3585a5c44eb65d8a279874" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 16 15:12:11 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 16 15:14:22 2018 -0700" }, "message": "ART: Fix jvalue includes\n\nJValue::SetL is defined in the -inl header. Fix up some clients.\n\nTest: mmma art\nChange-Id: I903532c767873847f7746680cf35083cb3408ecf\n" }, { "commit": "a3ad0cdd711857f04f477e2cdc5b56a2c74a3018", "tree": "7e0e4c7888c2497d86ce96c9574d7c9886721ccf", "parents": [ "abd9e1515bc6be88372c61071971c5432a51553d" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 04 10:06:38 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 09 11:11:09 2018 +0100" }, "message": "ART: Move JNI files to runtime/jni/ .\n\nTest: Rely on TreeHugger.\nChange-Id: I9cae11191ef1567ae9453be498882a7767285140\n" }, { "commit": "c431b9dc4b23cc950eb313695258df5d89f53b22", "tree": "422273559c3ae52caff0c6b1cf1a62a8312f0e26", "parents": [ "f46f46cf5bd32788d5252b7107628a66594a5e98" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Mar 02 12:01:51 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Mar 05 13:58:20 2018 -0800" }, "message": "Move most of runtime/base to libartbase/base\n\nEnforce the layering that code in runtime/base should not depend on\nruntime by separating it into libartbase. Some of the code in\nruntime/base depends on the Runtime class, so it cannot be moved yet.\nAlso, some of the tests depend on CommonRuntimeTest, which itself needs\nto be factored (in a subsequent CL).\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make -j 50 test-art-host\n\nChange-Id: I8b096c1e2542f829eb456b4b057c71421b77d7e2\n" }, { "commit": "06537f7a6f0772cbef08bdded933828378b2d32a", "tree": "e1505a3dea19d6585973d2db0d9dd6988bf0f03d", "parents": [ "a502c7202f56b8f5f2c5de567359fa7f6bab406f" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Feb 22 15:03:05 2018 -0800" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Feb 22 15:28:50 2018 -0800" }, "message": "native: Cleanup jni usage code\n\nMake the C++ signature consistent with the JNI descriptor by using the\nsame type as the descriptor whenever possible.\n\ne.g. \"()Ljava/lang/String;\" should actually be \"jstring fn(JNIEnv*,jobject)\"\ninstead of \"jobject fn(JNIEnv*,jobject)\".\n\nBug: 35325126\nChange-Id: I72318525fc3b18b013b8d6fa604d8dd6b5dd4400\nTest: make -j32 test-art-host # and also manually that it boots\n" }, { "commit": "a02cb113cd0dd023eec61716d3d1bd0dc1c1453a", "tree": "8553be51d5f6c2f0ebce20d7e364085ccf0a3206", "parents": [ "c7e546ff3963a1d51b1f100d308db735bd19f736" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Jan 31 11:36:39 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Feb 01 13:47:33 2018 +0000" }, "message": "Move hidden API warnings into resolvers\n\nChecks inserted to maybe warn about accessing a hidden API member\nare proving too expensive. Move them into the resolvers.\n\nBug: 64382372\nBug: 72649186\nTest: art/test.py --host -b -r -t 674-hiddenapi -v\nChange-Id: I11bceccb8fe647e829c491be38ba0af72f1b3996\n" }, { "commit": "ee7d2fd16d47b54d7fb5b9d5ec772fbc315faf4b", "tree": "844ed2998f78723aa7909ce0a7b19995c34f0bd9", "parents": [ "5a61bb7969347ffe8e0bf4f4dff841cc6c21ed85" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Sat Jan 20 17:25:23 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jan 25 10:40:56 2018 +0000" }, "message": "Start warning on hidden API greylist\n\nInsert checks into reflection, JNI and the verifier to print\na warning when greylisted methods are invoked and fields accessed.\nWe do this on actual access, because reflection allows to list\nall methods/fields and simply listing a greylisted member would\nprint too many false positives.\n\nIssuing a warning also sets a boolean flag in Runtime. This will\nbe made accessible through VMRuntime to the framework which will\nissue a Toast on Activity start.\n\nThe change was tested with internal microbenchmarks of reflection\nand those flagged one issue. Microbenchmark invoking a field getter\nhas regressed by 35%. We will profile this benchmark in detail and\nconsider options for improvement. Bug b/72482474 was created to track\nprogress.\n\nTest: art/test.py -b -r -t 674-hiddenapi\nBug: 64382372\nBug: 72482474\nChange-Id: I323244935e9091a2f8d012385cefaac6b1fe3777\n" }, { "commit": "9e734c7ab4599d7747a05db0dc73c7b668cb6683", "tree": "dce1d1993734a947fb2e6f626eb1b425cb72143b", "parents": [ "b496af808eaf3af5ebac50aef4fbec33323b5016" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Jan 04 17:56:19 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Jan 05 11:07:19 2018 -0800" }, "message": "Create dex subdirectory\n\nMove all the DexFile related source to a common subdirectory dex/ of\nruntime.\n\nBug: 71361973\nTest: make -j 50 test-art-host\nChange-Id: I59e984ed660b93e0776556308be3d653722f5223\n" }, { "commit": "895f92218f705ff8ad9c47b8be0c093130d9fbbc", "tree": "de590436a5feb2db183e3f3e35e29817c5ea3fb6", "parents": [ "c116154a4aae8ed1608a15ac602f8bd69d8ca0ff" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 05 09:53:32 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Aug 04 07:31:52 2017 -0700" }, "message": "ART: Fix up small header includes\n\nTest: m\nChange-Id: I6978d6eb4b95a6ee810e5a48ca6f5d6c590d4ce1\n" }, { "commit": "8cf9cb386cd9286d67e879f1ee501ec00d72a4e1", "tree": "88e86e214b425e444760fe4e0ffeee677e1558a2", "parents": [ "914b7b6a6c9f399b26b41e9160e9871ef749e0db" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 19 09:28:38 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 24 16:07:10 2017 -0700" }, "message": "ART: Include cleanup\n\nLet clang-format reorder the header includes.\n\nDerived with:\n\n* .clang-format:\n BasedOnStyle: Google\n IncludeIsMainRegex: \u0027(_test|-inl)?$\u0027\n\n* Steps:\n find . -name \u0027*.cc\u0027 -o -name \u0027*.h\u0027 | xargs sed -i.bak -e \u0027s/^#include/ #include/\u0027 ; git commit -a -m \u0027ART: Include cleanup\u0027\n git-clang-format -style\u003dfile HEAD^\n manual inspection\n git commit -a --amend\n\nTest: mmma art\nChange-Id: Ia963a8ce3ce5f96b5e78acd587e26908c7a70d02\n" }, { "commit": "87583b3de6e811e141f8c97c1f796eb516e8a751", "tree": "5b4688f306a2fda2f29bf319a8e1ff21309f93a3", "parents": [ "854461a4cd5e4a38debe3616e12b52fe7f160782" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu May 25 11:22:18 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 01 08:24:12 2017 -0700" }, "message": "ART: Move RegisterNatives helper\n\nMove the helper closer to the users.\n\nTest: m test-art-host\nChange-Id: Id05ce5f60af54d0c5aef0c7d8932706c4602fc94\n" }, { "commit": "a14100ccf51cc63a5c472188d1e2d337627e49eb", "tree": "d659a038e35fdd790b642f51e3eddbc9b51d003a", "parents": [ "1171deea4cb2f2db67a310ea8797d06206b2bdea" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 24 15:09:56 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue May 30 16:10:27 2017 -0700" }, "message": "ART: More header cleanup - jni_internal.h\n\nMove jni_macros.h to the clients (users of RegisterNativeMethods).\n\nTest: mmma art\nChange-Id: I052ac9f703ec69f0a2df1efcaf9f4a450eb7be1c\n" }, { "commit": "a1d2f957a21319d1110bebb9a52f46fd1c67ffaf", "tree": "db9b6eda7cd4764e13c07256df14a7c14888ee75", "parents": [ "c6ea7d00ad069a2736f603daa3d8eaa9a1f8ea11" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Apr 20 22:53:58 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Apr 21 08:41:28 2017 -0700" }, "message": "ART: Clean up art_field.h\n\nClean up the header. Fix up other headers including the -inl file,\nin an effort to prune the include graph. Fix broken transitive\nincludes by making includes explicit.\n\nTest: m\nChange-Id: I6ff2d6f89b741d8188e19426eccf4743fec688fc\n" }, { "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": "3b6f440dbd066f03a737da6d292074f47b3fbc29", "tree": "af0da4bdea7ff689929c8e23e1eaf6361dfc3ee8", "parents": [ "5c19b373ae45ec6d67b41917af63cac8c3ebbe89" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Feb 16 16:13:17 2017 -0800" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Feb 21 11:40:51 2017 -0800" }, "message": "jni: Switch to @FastNative for all JNI functions. Deprecate !bang JNI.\n\nSwitches all (248) methods that previously used !bang JNI in art/libcore\nto all use @FastNative.\n\nAlso deprecate !bang JNI since nothing in Android seems to (or should\nbe) using it anymore.\n\nThis measures to be a 3% startup time improvement in system_server.\n\nTest: make test-art-host\nBug: 34955272\nChange-Id: I0881f401c7660c79f275235362777bfa58241deb\n" }, { "commit": "46ee31b67d7ee1bd085fbc240502053caa3cf8fa", "tree": "83e870f293cd60d533207e389d7b094acd87b31f", "parents": [ "9538f9e2a5d03f1b1bc07ebfbd93b61dcf8ad604" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Dec 14 10:11:49 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 15 09:50:39 2016 -0800" }, "message": "ART: Move to libbase StringPrintf\n\nRemove ART\u0027s StringPrintf implementation. Fix up clients. Add\nmissing includes where necessary.\n\nTest: m test-art-host\nChange-Id: I564038d5868595ac3bb88d641af1000cea940e5a\n" }, { "commit": "3d617ac3d87f7fa98793406818e54b057bd701a1", "tree": "5230ceedfaf2928da0404fd71b64fa7f08439f21", "parents": [ "b5cc1d1321e36d38cd48ae079438a8758b82edc1" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Oct 19 14:00:46 2016 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Oct 31 09:16:19 2016 +0000" }, "message": "Getter/Setter support for invoke-polymorphic of invokeExact().\n\nTest: make test-art-host\nBug: 30550796\n\nChange-Id: I427a6e0afba88b223655ad1ba30843aaf255182b\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": "f8ac97f94aad9ce2d324c2c7c21dde3abf13315b", "tree": "9359680f07a65531898113dd335020db33b9b6e6", "parents": [ "299958fc681f591fee7ff9ae0d81bd3adfebdd2b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 15:56:52 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 15:59:57 2016 -0700" }, "message": "ObjPtr misc cleanup\n\nCheck for subtypes in constructor.\n\nRemove various calls to MakeObjPtr and ObjPtr::Ptr.\n\nBug: 31113334\n\nTest: test-art-host\nChange-Id: I8d680d514ea52cd64fc4f7c2e988926bc726174e\n" }, { "commit": "299958fc681f591fee7ff9ae0d81bd3adfebdd2b", "tree": "cb09ffa4dec7cd106e82fe630ed2941f49e8b566", "parents": [ "463a67dd60ab0341e56713426a34c3c0a0749a35" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 13:48:41 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 15:59:57 2016 -0700" }, "message": "Change java_lang_reflect_Field.cc to use ObjPtr\n\nAlso deleted some calls to ObjPtr::Ptr and fixed formatting.\n\nBug: 31113334\n\nTest: test-art-host\nChange-Id: I951b80886b723e1beff4783f28ee7c928cb10650\n" }, { "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": "0795f23920ee9aabf28e45c63cd592dcccf00216", "tree": "ff3f880c5e84f3316532b47d0e9a7729ade848ac", "parents": [ "d1224dce59eb0019507e41da5e10f12dda66bee4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 27 18:43:30 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 29 17:31:09 2016 -0700" }, "message": "Clean up ScopedThreadStateChange to use ObjPtr\n\nAlso fixed inclusion of -inl.h files in .h files by adding\nscoped_object_access-inl.h and scoped_fast_natvie_object_access-inl.h\n\nChanged AddLocalReference / Decode to use ObjPtr.\n\nChanged libartbenchmark to be debug to avoid linkage errors.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I4d2e160483a29d21e1e0e440585ed328b9811483\n" }, { "commit": "a59d9b228b1eda3bf71a81b6201ec64e26086c23", "tree": "91adf9f0e282eec50ccb28aa2dbc432a5e28d13f", "parents": [ "05ce52b406dcf6b9a24287d14f0e611211e0db3d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Sep 26 18:13:17 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 27 16:10:35 2016 -0700" }, "message": "Use ObjPtr for reflection.cc/h/inl\n\nChanged Pretty helpers to use this to reduce usage of Decode. The\neventual goal is not have almost any calls to ObjPtr::Decode.\n\nMoved ObjPtr out of mirror namespace for convenience. Added more\nPoisonObjectPointers calls in class linker, thread suspension.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I44d08db5143d95ed1b65e2f00f9749ef5cf379f7\n" }, { "commit": "9323e6e44060545d6c4f925139572868f65b7fe3", "tree": "65f3f380605d75f34d49aec49fca494bec26ef5b", "parents": [ "58735fbfbb2da0c0a19b3239da07c2353c40767d" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Sep 13 08:58:35 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Sep 14 08:01:39 2016 -0700" }, "message": "Separate annotations from dexfile reading.\n\nBug: 22322814\nChange-Id: I867d66da407dd80394a10d19903ebbc1ec3986ff\nTest: test-art\n" }, { "commit": "bdf7f1c3ab65ccb70f62db5ab31dba060632d458", "tree": "25cc77adfeb05232d0ab00aa561a693f1d71745c", "parents": [ "d7eabc2cc1a88c1f7f927da61246ae65aab0626c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 30 16:38:47 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 30 17:02:53 2016 -0700" }, "message": "ART: SHARED_REQUIRES to REQUIRES_SHARED\n\nThis coincides with the actual attribute name and upstream usage.\nPreparation for deferring to libbase.\n\nTest: m\nTest: m test-art-host\nChange-Id: Ia8986b5dfd926ba772bf00b0a35eaf83596d8518\n" }, { "commit": "3a0909248e04b22c3981cbf617bc2502ed5b6380", "tree": "b0b7013ea78b10f23fae75ef145e53a696ff797c", "parents": [ "457e874459ae638145cab6d572e34d48480e39d2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 24 09:17:30 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 24 09:17:30 2015 +0000" }, "message": "Revert \"lambda: Add support for invoke-interface for boxed innate lambdas\"\n\n955-lambda is flaky\n\nBug: 24618608\nBug: 25107649\n\nThis reverts commit 457e874459ae638145cab6d572e34d48480e39d2.\n\nChange-Id: I24884344d21d7a4262e53e3f5dba57032687ddb7\n" }, { "commit": "457e874459ae638145cab6d572e34d48480e39d2", "tree": "b192c4698c4889ae90db6abe2652199802213fe6", "parents": [ "3944f7175dcf60316ba58a42698ccf23c65ac57c" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Oct 22 17:37:50 2015 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Nov 20 16:33:06 2015 -0800" }, "message": "lambda: Add support for invoke-interface for boxed innate lambdas\n\nLambda closures created with the \u0027create-lambda\u0027 instruction\n(termed \"innate lambdas\") can be turned into an object with \u0027box-lambda\u0027.\n\nThis CL enables support for those kinds of lambdas to work with\n\u0027invoke-interface\u0027 by generating a proxy class for the lambda.\n\nNote: MIPS32/64 support not included.\n\nBug: 24618608\nBug: 25107649\nChange-Id: Ic8f1bb66ebeaed4097e758a50becf1cff6ccaefb\n" }, { "commit": "13e748b28c5f2bd1e83674d2ca899ff61ae5c0a1", "tree": "387724eebfe7cd0f7be368e64b22af3894eea4c7", "parents": [ "dda43e7e9680e6eead02876e67718d01837d89e0" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Aug 25 20:44:19 2015 +0000" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Aug 25 14:20:09 2015 -0700" }, "message": "Revert \"Revert \"Move annotations to native.\"\"\n\nThis reverts commit 7db6dd79a24570448ae737ee1946b00396696cac.\n\nAdds check if field\u0027s declaring class is proxy.\nBug: 23508574\n\nChange-Id: Ie829f1526e74427711e818b56d1588d92946cbf6\n" }, { "commit": "7db6dd79a24570448ae737ee1946b00396696cac", "tree": "490c05f1488141adf5668b34cbec28830fe8295c", "parents": [ "0042c6d49b8488c78f0b937063e316e8d6244439" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 25 10:53:19 2015 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 25 10:53:19 2015 +0000" }, "message": "Revert \"Move annotations to native.\"\n\nThis reverts commit 0042c6d49b8488c78f0b937063e316e8d6244439.\n\nReverting this change (as well as the companion CL\nhttps://android-review.googlesource.com/#/c/167510/ in\nplatform/libcore) as they make libcore test\nlibcore.java.lang.reflect.ProxyTest#test24846 fail.\n\nChange-Id: Ie0676cabb128277c7df5dab7bde17aefd3b2c09c\n" }, { "commit": "0042c6d49b8488c78f0b937063e316e8d6244439", "tree": "170286a01d08badebacaebed1db325bc889e259e", "parents": [ "300432d27ba9dd6961458fb1e1c845766f25c50a" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Jul 29 20:14:10 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Aug 21 17:05:27 2015 -0700" }, "message": "Move annotations to native.\n\nArt side of this change. There is also a corresponding Libcore change.\n\nSeeing ~2-3x speedup over dalvik KK MR1 in AnnotatedElementBenchmark.\nBenchmark\t\t\t Speedup of Art AOSP to Dalvik KK MR1\nGetAllReturnsLargeAnnotation\t 2.99\nGetAllReturnsMarkerAnnotation\t 2.20\nGetAllReturnsNoAnnotation\t 2.43\nGetAllReturnsSmallAnnotation\t 2.52\nGetAllReturnsThreeAnnotations\t 2.87\nGetAnnotationsOnSubclass\t 2.42\nGetDeclaredAnnotationsOnSubclass 2.49\nGetFieldAnnotation\t\t 2.68\nGetFieldAnnotations\t\t 2.60\nGetMethodAnnotation\t\t 2.66\nGetMethodAnnotations\t\t 2.61\nGetParameterAnnotations\t\t 2.52\nGetTypeAnnotation\t\t 2.56\nGetTypeAnnotations\t\t 2.17\nIsFieldAnnotationPresent\t 3.26\nIsMethodAnnotationPresent\t 4.99\nIsTypeAnnotationPresent\t\t 1.34\n\nChange-Id: Ibdbb6d23b17eaab6e83c8774b1bb9401e8227941\n" }, { "commit": "3152c82b0d33e5fb0a4aa964ea58451c72734444", "tree": "b5de74aa2d878aa9867d6e6589ec47917dd59f8a", "parents": [ "0fae2f5851dd79c4100134a92f997b2e7a520083" ], "author": { "name": "Fredrik Roubert", "email": "roubert@google.com", "time": "Tue Jul 28 00:14:08 2015 +0200" }, "committer": { "name": "Fredrik Roubert", "email": "roubert@google.com", "time": "Tue Jul 28 00:55:06 2015 +0200" }, "message": "Bugfix: Java char is 16 bits, can not be treated as boolean.\n\nUsing SetFieldBooleanVolatile() and SetFieldBoolean() happens to work\nfor char values that only use the lower 8 bits, but is a mistake that\nwas introduced by the \"Add AccessibleObject and Field to mirror\" commit:\n\nhttps://android.googlesource.com/platform/art/+/daaf326\n\nChange-Id: Iec02ba3084c992ea239ecef688d7d29c7e21ae16\n" }, { "commit": "90443477f9a0061581c420775ce3b7eeae7468bc", "tree": "8c74b81dfae162e0fd0ccf8d5ac50827ba815174", "parents": [ "6078aec213dfaf111c29969706e8e5967cfc9bea" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 16 20:32:27 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 22 15:13:56 2015 -0700" }, "message": "Move to newer clang annotations\n\nAlso enable -Wthread-safety-negative.\n\nChanges:\nSwitch to capabilities and negative capabilities.\n\nFuture work:\nUse capabilities to implement uninterruptible annotations to work\nwith AssertNoThreadSuspension.\n\nBug: 20072211\n\nChange-Id: I42fcbe0300d98a831c89d1eff3ecd5a7e99ebf33\n" }, { "commit": "e401d146407d61eeb99f8d6176b2ac13c4df1e33", "tree": "17927f9bfe7d2041b5942c89832d55f9dedb24c5", "parents": [ "2006b7b9b8e32722bd0d640c62549d8a0ac624b6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 13:56:20 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 29 18:45:49 2015 -0700" }, "message": "Move mirror::ArtMethod to native\n\nOptimizing + quick tests are passing, devices boot.\n\nTODO: Test and fix bugs in mips64.\n\nSaves 16 bytes per most ArtMethod, 7.5MB reduction in system PSS.\nSome of the savings are from removal of virtual methods and direct\nmethods object arrays.\n\nBug: 19264997\nChange-Id: I622469a0cfa0e7082a2119f3d6a9491eb61e3f3d\n" }, { "commit": "80afd02024d20e60b197d3adfbb43cc303cf29e0", "tree": "ef054c7b4f2a739f7cf063e0bc4c501c2c7e41b5", "parents": [ "559b178e34c5d92e7932f92e5d8a981ac334606f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 19 18:08:00 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 26 15:59:02 2015 +0100" }, "message": "ART: Clean up arm64 kNumberOfXRegisters usage.\n\nAvoid undefined behavior for arm64 stemming from 1u \u003c\u003c 32 in\nloops with upper bound kNumberOfXRegisters.\n\nCreate iterators for enumerating bits in an integer either\nfrom high to low or from low to high and use them for\n\u003carch\u003eContext::FillCalleeSaves() on all architectures.\n\nRefactor runtime/utils.{h,cc} by moving all bit-fiddling\nfunctions to runtime/base/bit_utils.{h,cc} (together with\nthe new bit iterators) and all time-related functions to\nruntime/base/time_utils.{h,cc}. Improve test coverage and\nfix some corner cases for the bit-fiddling functions.\n\nBug: 13925192\nChange-Id: I704884dab15b41ecf7a1c47d397ab1c3fc7ee0f7\n" }, { "commit": "ca239af73e512df5eeb80fe6c09c2ca614649e06", "tree": "cee8641c42385f40b567c686cd5053fde152a088", "parents": [ "f381645a336f7092ab6f5900c0a2cf183a9dbdf7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Mar 29 18:27:50 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Mar 29 18:54:20 2015 -0700" }, "message": "Fix some reflection errors\n\nFixed incorrectly using 2nd frame instead of 1st in VerifyAccess.\nAdded regression test to ART. Fixed broken setShort,\ngetDeclaredFieldInternal.\n\nChange-Id: I4b21d52d998cb768fe9503b8bccec506b7b972e5\n" }, { "commit": "daaf3265806eb2eadb2e03302bd68022fab5ca28", "tree": "aff5d6d53d6d2b65995aa204839f88ee66400989", "parents": [ "68e22f3b982ff9ccbdfb3b65b7cfc16fcae907ba" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 24 13:30:28 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Mar 29 14:13:08 2015 -0700" }, "message": "Add AccessibleObject and Field to mirror\n\nMain motivation is to remove all the functionality / field access on\njava side to ArtField. Also comes with some reflection speedups /\nslowdowns.\n\nSummary results:\ngetDeclaredField/getField are slower mostly due to JNI overhead.\nHowever, there is a large speedup in getInt, setInt,\nGetInstanceField, and GetStaticField.\n\nBefore timings (N5 --compiler-filter\u003deverything):\n\n benchmark ns linear runtime\n Class_getDeclaredField 782.86 \u003d\u003d\u003d\n Class_getField 832.77 \u003d\u003d\u003d\n Field_getInt 160.17 \u003d\n Field_setInt 195.88 \u003d\n GetInstanceField 3214.38 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n GetStaticField 6809.49 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nAfter:\n Class_getDeclaredField 1068.15 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Class_getField 1180.00 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Field_getInt 121.85 \u003d\n Field_setInt 139.98 \u003d\n GetInstanceField 1986.15 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n GetStaticField 2523.63 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nBug: 19264997\n\nChange-Id: Ic0d0fc1b56b95cd6d60f8e76f19caeaa23045c77\n" }, { "commit": "0aa50ce2fb75bfc2e815a0c33adf9b049561923b", "tree": "9a3f9603ab30d5cbc7fc21aee0ceb48bbb0dd25a", "parents": [ "e8e42f3548fd894f860912bb1b71ce6fa2d7daf3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 10 11:03:29 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 10 14:51:11 2015 +0000" }, "message": "Remove ThrowLocation.\n\nNote that this is a cleanup change, and has no functionality change.\nThe ThrowLocation had no use anymore.\n\nChange-Id: I3d2126af1dc673cec3a0453ff3d56a172663a5f6\n" }, { "commit": "08f1f50d6c2e8f247b8f5f19711d75a792851c7a", "tree": "dca490545d56168f7f9ae4a4616199b1b1c8ba0c", "parents": [ "8443637f71a777a13317fe7635028d758a0adf97" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 15:04:37 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 15:12:27 2014 -0800" }, "message": "Remove FieldHelper.\n\nChange-Id: I2d74e2d5b3c35a691c95339de0db9361847fca11\n" }, { "commit": "277ccbd200ea43590dfc06a93ae184a765327ad0", "tree": "d89712e93da5fb2748989353c9ee071102cf3f33", "parents": [ "ad17d41841ba1fb177fb0bf175ec0e9f5e1412b3" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 03 21:36:10 2014 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 04 18:40:08 2014 -0800" }, "message": "ART: More warnings\n\nEnable -Wno-conversion-null, -Wredundant-decls and -Wshadow in general,\nand -Wunused-but-set-parameter for GCC builds.\n\nChange-Id: I81bbdd762213444673c65d85edae594a523836e5\n" }, { "commit": "6a3c1fcb4ba42ad4d5d142c17a3712a6ddd3866f", "tree": "9df58b57af13240a93a6da4eefcf03f70cce9ad9", "parents": [ "c6e0955737e15f7c0c3575d4e13789b3411f4993" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 31 00:33:20 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Nov 03 20:01:04 2014 -0800" }, "message": "Remove -Wno-unused-parameter and -Wno-sign-promo from base cflags.\n\nFix associated errors about unused paramenters and implict sign conversions.\nFor sign conversion this was largely in the area of enums, so add ostream\noperators for the effected enums and fix tools/generate-operator-out.py.\nTidy arena allocation code and arena allocated data types, rather than fixing\nnew and delete operators.\nRemove dead code.\n\nChange-Id: I5b433e722d2f75baacfacae4d32aef4a828bfe1b\n" }, { "commit": "fc787ecd91127b2c8458afd94e5148e2ae51a1f5", "tree": "ef48c0f511ee9bf4ed85607cc4d530bace7e6cae", "parents": [ "8fa8c904f7c783204a1dc9438429391d256658da" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 21:56:44 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 22:22:46 2014 -0700" }, "message": "Enable -Wimplicit-fallthrough.\n\nFalling through switch cases on a clang build must now annotate the fallthrough\nwith the FALLTHROUGH_INTENDED macro.\nBug: 17731372\n\nChange-Id: I836451cd5f96b01d1ababdbf9eef677fe8fa8324\n" }, { "commit": "c0d8229898c44c0f604f08a5df1de83ff56c18fd", "tree": "13b150e68e806204c547e04bc071246c06199c37", "parents": [ "c70535b4f9f1ff3e3da451734bb7d9601012ccc1" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 23 10:38:30 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 29 11:40:16 2014 -0700" }, "message": "ART: Better IllegalAccessException message\n\nBug: 17618578\nBug: 17614623\nChange-Id: I0e3f15e676acd6ed5844fc86e136f75cc335372d\n" }, { "commit": "76433275dbe39e5ced1c223b006d1b900b1937f6", "tree": "a72c101defa86ceff8b633cf2eaa606c87f783eb", "parents": [ "88adf36315237947d3a5d10b28fdf8b9da7794a4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 26 14:32:37 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 26 19:54:39 2014 -0700" }, "message": "Optimize get/set reflection performance\n\nSpeedups mostly from reducing how often access checks are needed,\nand adding more inlining, and adding templates.\n\nField_getInt from ~850ns -\u003e 350ns.\nField_setInt from ~900ns -\u003e 370ns.\n\nBug: 14063288\n\n(cherry picked from commit ffc788cb7b5b9f53734d7bb8af2d5e45d885546b)\n\nChange-Id: I2441581ff3478c6ae43b6aa49939ff3f07555ec8\n" }, { "commit": "7b078e8c04f3e1451dbdd18543c8b9692b5b067e", "tree": "414229c6b87eb20ea24c40780752da5a3999a49a", "parents": [ "f79ba17defbd9342e44ab9f3de0807054673d3c9" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 10 14:44:24 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Sep 12 14:57:53 2014 -0700" }, "message": "Compile time performance improvements focusing on interpret-only.\n\nReduce virtual method dispatch in the method verifier and make more code\ninline-able.\nAdd a StringPiece with const char* equality operator to avoid redundant\nStringPieces and strlens.\nRemove back link from register line to verifier and pass as argument to reduce\nsize of RegisterLine.\nRemove instruction length from instruction flags and compute from the\ninstruction, again to reduce size.\nAdd suspend checks to resolve and verify to allow for more easy monitor\ninflation and reduce contention on Locks::thread_list_suspend_thread_lock_.\nChange ThrowEarlierClassFailure to throw pre-allocated exception.\nAvoid calls to Thread::Current() by passing self.\nTemplate specialize IsValidClassName.\nMake ANR reporting with SIGQUIT run using checkpoints rather than suspending\nall threads. This makes the stack/lock analysis less lock error prone.\nExtra Barrier assertions and condition variable time out is now returned as a\nboolean both from Barrier and ConditionVariable::Wait.\n\n2 threaded host x86-64 interpret-only numbers from 341 samples:\nBefore change: Avg 176.137ms 99% CI 3.468ms to 1060.770ms\nAfter change: Avg 139.163% 99% CI 3.027ms to 838.257ms\nReduction in average compile time after change is 20.9%.\nSlow-down without change is 26.5%.\n\nBug: 17471626 - Fix bug where RegTypeCache::JavaLangObject/String/Class/Throwable\ncould return unresolved type when class loading is disabled.\nBug: 17398101\n\nChange-Id: Id59ce3cc520701c6ecf612f7152498107bc40684\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": "61c5ebc6aee2cac1c363de6fbdac25ada1697fdb", "tree": "444936c3e3718b692ba7f1981b863190f26b4ed7", "parents": [ "25c4f6a25b3de9b9d7ca5162f1629753a0b7f003" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 05 17:42:53 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jun 06 13:12:17 2014 -0700" }, "message": "Change FieldHelper to use a handle.\n\nFixed compaction bugs related to FieldHelper::GetType in:\nartSet32InstanceFromCode\nSetFieldValueImpl\nCheckReceiver\nField_set\ninterpreter::DoFieldPut\nMethodVerifier::VerifyISGet\nMethodVerifier::VerifyISPut\nMethodVerifier::VerifyIGetQuick\n\nBug: 13077697\n\nChange-Id: I7de9ded2893b5568d43e4daa86fd135bf5508b72\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": "cb4581aa13d6f43f705535818a4d0893d551be3a", "tree": "903cbcb3852650c7e3105c0c1819e6e480091cd4", "parents": [ "e759c9f01b558093466fc78f01900042120cd832" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 28 15:43:37 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 28 15:43:37 2014 -0700" }, "message": "Some fixes for comments and implied conversions.\n\nAddresses comments in:\nhttps://android-review.googlesource.com/#/c/89148/3\n\nChange-Id: If21cfaa541210c8702371efd1e6d4f071a7b9ec3\n" }, { "commit": "11d5d8fffe41cc7daadbfa2ca98ecb978f3029af", "tree": "fae2cf974b502420a942043d135d301aa75967e7", "parents": [ "a708e32a9f764a48175e705ec4bcd2201c84f492" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Mar 26 15:08:20 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 28 14:27:55 2014 -0700" }, "message": "Add access checks to Method and Field reflection.\n\nArt side of this change. Has a corresponding libcore change.\n\nBug: 13620925\nChange-Id: Ie67f802a2a400e8212b489b9a261b7028422d8ba\n" }, { "commit": "549cc8b76582c7fb8256d79639c227ade69928d1", "tree": "1db1eeb82bcd4644ef3b15787161e8ba2112ffee", "parents": [ "41ec12c09573d6a18ce7bc095996567dcbf88d5a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 27 14:27:09 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 27 14:27:09 2014 -0700" }, "message": "Remove bogus DCHECK.\n\nBug: 13637540\nChange-Id: I39f19443156cf5bdc26e3eeb0bb501a850d40dca\n" }, { "commit": "84956ff6c0ff150d86a08157216ded79217d860a", "tree": "04177625872b89f27f00a53b574085416617ee0d", "parents": [ "636583c586b78e188168e510452cef0645578f8d" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Mar 26 23:52:41 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Mar 26 23:56:58 2014 -0700" }, "message": "Reflection tidying.\n\nMove out arguments to the right and make pointer not reference types.\nRemove unused unbox for argument routine.\nSimplify convert primitive routine for the case of identical types.\n\nChange-Id: I6456331b0f3f3e5f0b2c361a9f50b4ed1c9462a3\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": "53b8b09fc80329539585dcf43657bc5f4ecefdff", "tree": "cac0f82fbb89bd907104e3fed6c36203e11a3de0", "parents": [ "0dea9872082bc3e576ed6cefed86b0d6c0c45ffd" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 13 23:45:53 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 14 11:28:10 2014 -0700" }, "message": "Refactor reflective method invocation.\n\nMove invocation code out of JNI internal into reflection, including ArgArray\ncode. Make reflective invocation use the ArgArray to build arguments rather\nthan allocating a jvalue[] and unboxing arguments into that.\nMove reflection part of jni_internal_test into reflection_test.\nMake greater use of fast JNI.\n\nChange-Id: Ib381372df5f9a83679e30e7275de24fa0e6b1057\n" }, { "commit": "0f6784737882199197796b67b99e5f1ded383bee", "tree": "139888bdfad743c50e800ad46b9aec97ae3415d3", "parents": [ "73a471336a9515fa7fdc5859e3c481ec451237a5" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Mar 10 16:18:37 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Mar 10 18:02:07 2014 -0700" }, "message": "Unify 64bit int constant definitions.\n\nLL and ULL prefixes are word size dependent, use the INT64_C and UINT64_C\nmacros instead.\n\nChange-Id: I5b70027651898814fc0b3e9e22a18a1047e76cb9\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": "b122a4bbed34ab22b4c1541ee25e5cf22f12a926", "tree": "624f16271f4481a8fd5aa2f607385f490dc7b3ae", "parents": [ "e40687d053b89c495b6fbeb7a766b01c9c7e039c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Nov 19 18:00:50 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Dec 20 08:01:57 2013 -0800" }, "message": "Tidy up memory barriers.\n\nChange-Id: I937ea93e6df1835ecfe2d4bb7d84c24fe7fc097b\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": "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": "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": "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": "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" }, { "commit": "af6e67a4816d2593586115b89faa659225363246", "tree": "e3b330d6ed994f82972da4e5b8c3d06be2c66eae", "parents": [ "a6f3aaf8ca5385d870c35d1555b7787d56a70c95" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 16 08:38:37 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 16 15:37:56 2013 -0800" }, "message": "Further proxy refactorings.\n\nFactor the crawling of the quick stack arguments into a common visitor.\nFactor the proxy invocation dispatch into common runtime support code,\nfix numerous bugs relating to GC in the LLVM runtime support with this.\nClean up BoxPrimitive to not use an in argument as an out.\n\nChange-Id: I7b12c8d88d5083614e480b8fb1d2f2ef7c0a51b7\n" }, { "commit": "2fa6b2e2fc3d2a2fc27808ce518dc76b80ce369a", "tree": "daba5f5e64409452e31ae42e3b1b6b6491ccb007", "parents": [ "6ed960d260d0ec3cbbef455c646ccd47f4e78119" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 17 00:10:17 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 26 16:12:59 2012 -0700" }, "message": "Interpreter.\n\nThe opcodes filled-new-array and packed-switch aren\u0027t implemented but\nare trivial given that they are variants of implemented opcodes.\nRefactor Field::Get routines to take the declaring class in the case of\nstatic field accesses. This avoids a check on every use of a field.\nRefactor arg array builder to be shared by JNI invokes and invocations\ninto the interpreter.\nFix benign bug in const decoding in the verifier.\n\nChange-Id: I8dee6c1f4b7f033e6c003422c56e9471cfaccda8\n" }, { "commit": "b726dcb581bf72da46527378ccb6889020f0e6e9", "tree": "c0383ed788652926e93147e50d659ed226519548", "parents": [ "fd678beb171a4686a4f2d53ca4188a4ade8fa54e" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 05 08:57:23 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 05 09:06:19 2012 -0700" }, "message": "Rename GlobalSynchronization to Locks\n\nAlso address some review comments in common_throws relating to\nToStr\u003cInvokeType\u003e and exception detail messages.\n\nChange-Id: Ibf2c0f147689fa236d349bd7f01eed3c2522552b\n" }, { "commit": "00f7d0eaa6bd93d33bf0c1429bf4ba0b3f28abac", "tree": "6a2172ece15c4699e6c2a67ce76f019db0a9a21d", "parents": [ "634ea28f78c4a138e6a1de54eae8696095422415" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 19 15:28:27 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 14 10:45:52 2012 -0700" }, "message": "Global lock levels.\n\nIntroduce the notion of the mutators/GC being a shared-exclusive (aka\nreader-writer) lock. Introduce globally ordered locks, analysable by\nannotalysis, statically at compile time. Add locking attributes to\nmethods.\n\nMore subtly, remove the heap_lock_ and split between various locks that\nare held for smaller periods (where work doesn\u0027t get blocked). Remove\nbuggy Dalvik style thread transitions. Make GC use CMS in all cases when\nconcurrent is enabled. Fix bug where suspend counts rather than debug\nsuspend counts were sent to JDWP. Move the PathClassLoader to\nWellKnownClasses. In debugger refactor calls to send request and\npossibly suspend. Break apart different VmWait thread states. Move\nidentity hash code to a shared method.\n\nChange-Id: Icdbfc3ce3fcccd14341860ac7305d8e97b51f5c6\n" }, { "commit": "365c10235438607541fa2259a5fec48061b90bd8", "tree": "26516a4985a8bbef4c8da36b99bf1e54af3ad24d", "parents": [ "147440427984816b6ed5235a9b1fb2cbf99496d0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jun 22 15:05:28 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jun 26 15:21:01 2012 -0700" }, "message": "Increase use of ScopedJniThreadState.\n\nMove the routines for changing Object* to jobject and vice-versa\n(AddLocalReference and Decode) to ScopedJniThreadState to enforce use of\nObject*s in the Runnable thread state. In the Runnable thread state\nsuspension is necessary before GC can take place.\n\nReduce use of const ClassLoader* as the code bottoms out in FindClass\nand with a field assignment where the const is cast away (ie if we\u0027re\nnot going to enforce the const-ness we shouldn\u0027t pretend it is).\n\nRefactor the Thread::Attach API so that we\u0027re not handling raw Objects on\nunattached threads.\n\nRemove some unreachable code.\n\nChange-Id: I0fa969f49ee6a8f10752af74a6b0e04d46b4cd97\n" }, { "commit": "eac766769e3114a078c188ea26776a81f0edb3cf", "tree": "4b9b8dfc89725173431817baba4455c013e40577", "parents": [ "3f9ace8d90bfc48c8d7bf35af66de8ce8238de7f" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu May 24 21:56:51 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri May 25 12:04:43 2012 -0700" }, "message": "DO NOT MERGE Fix all unsafe caching to be like libcore.\n\nThis way, if a runtime is restarted within a process, we re-initialize all\nthe cached data.\n\nConflicts:\n\n\tsrc/native/java_lang_Runtime.cc -- nativeExit lost an argument in dalvik-dev\n\n(cherry picked from commit 7756d5473fa27ce7e6ac7c31770eef7030431da4)\n\nChange-Id: I6184fc20c2a9ec16c4b053584a4d1c3b64452d0f\n" }, { "commit": "aaa5edcf2deb1bddcbf5fb27820ad2240ac5b4f2", "tree": "c2d0f408237ad5a30bfd67819e5c9a72cc3ac45e", "parents": [ "983f2e411aee6b1d09e6da30e059b782b2699909" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed May 16 15:54:30 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed May 16 15:54:30 2012 -0700" }, "message": "Improve reflection IllegalArgumentException detail messages.\n\nAlso add a missing InstanceOf check that was causing CheckJNI to kill us\nif someone tried to pass an inappropriate reference type through\nMethod.invoke. (Amusingly, CheckJNI produced pretty much the exact detail\nmessage that Method.invoke should have.)\n\nPlus a new test for this stuff.\n\nBug: 6504175\nChange-Id: Ice95eecbdba5a0927c6eaf68e56d6500dc52ad2e\n" }, { "commit": "0dae08ead9112adb81ad507b81187f00f77bc168", "tree": "5c09e5ab16fa544360e6354796f999cfeaf991e7", "parents": [ "776ac1fa61237db645adb4370a4aab888530caf4" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Apr 14 13:55:11 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Apr 14 13:55:11 2012 -0700" }, "message": "Move JNI methods out of top-level.\n\nChange-Id: Ia9c883ba9a13b205cda7b72024e2939f02583acb\n" }, { "commit": "f24d3cedd395690f6904aaac80f84a100420f7a3", "tree": "1867ff0b6f3bebb4f611eaf72b05b14843c55d93", "parents": [ "14fe6aea00598ba5d40d3e5bfa62f26a1c8cfdff" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Apr 11 17:43:37 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Apr 11 17:43:37 2012 -0700" }, "message": "Enforce the use of accessors on JValue so we can ensure sign extension.\n\nChange-Id: I34810af7661d8674c84707afe5cfdf9e2c233f8b\n" }, { "commit": "1d878f3e1c39162496d7c061f7f6710e414d182b", "tree": "203a26165bff88dd352580c088c0775b953d363a", "parents": [ "4b952e74b4b2efa15cecedfc643c212eaeac2cc7" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Apr 11 15:17:54 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Apr 11 15:17:54 2012 -0700" }, "message": "Add a default constructor to JValue.\n\nChange-Id: I7abe801b26f5003c480215c1f1ca64dfc07667ed\n" }, { "commit": "d8a68a6356c1a4d0f488631a488981010313bed7", "tree": "8fd2629d687c0f8346b0aa8357cc126358f5b825", "parents": [ "c71f652a50e562d88f6aa1d691285101b8912959" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Apr 11 14:39:38 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Apr 11 14:39:38 2012 -0700" }, "message": "Ensure byte and short values are sign extended before boxing.\n\nChange-Id: Ib97b933eaceffe71f19affb22bec030f13516c2f\n" }, { "commit": "34e069606d6f1698cd3c33b39e72b79ae27e1c7b", "tree": "0feb7e9bde6cfe01c0df3ef2d8de3210570f7ce2", "parents": [ "e62934d85fbc2d935afdad57eeade39ecbd7440a" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Apr 09 13:55:55 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Apr 09 13:55:55 2012 -0700" }, "message": "Remove the useless \"suspend count already zero\" message for new threads.\n\nWe can actually detect the expected case of this warning ourselves, and\nnot emit it. Then we can upgrade the WARNING to a FATAL.\n\nI also tripped over the fact that the operator\u003c\u003c for Thread::State was out\nof date, so I\u0027ve moved the Thread enums up to namespace scope so the script\ncan automatically generate correct operator\u003c\u003c implementations for us. (All\nthe high-numbered thread states have been off by one for a couple of weeks.)\n\nChange-Id: I5de573d33d641e5a3cba87b370e9620c8c66e633\n" }, { "commit": "0045a290e6b79a274250e3112880c04bde437d4a", "tree": "a125d556b6c817218b14f6f3f72079b83d0032c4", "parents": [ "d23f5206cf72ca3fb53ac3049649bb7d018232bf" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Mar 31 21:08:41 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Mar 31 22:38:22 2012 -0700" }, "message": "Avoid a case of eager initialization.\n\nDon\u0027t mark as initialized classes that contain static field\ninitialization.\n\nChange-Id: Iedcabbdf355e8861eb7731650eee1467f68ae0cd\n" }, { "commit": "923e8b8e936ead33408e111682aa9372e3a7ed43", "tree": "189443deaaff7a72dcd602db59ef597273cabd9b", "parents": [ "634eb2eb14f87753519d0ef2c5f256e55888f378" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 23 11:44:07 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 23 14:42:39 2012 -0700" }, "message": "Expand test 046 and move reflective class initialization later.\n\nWe have no reason to do this later; we\u0027re just being conservative. We\nmight want to revisit this in a future release so you only pay once,\nwhen you cache your Field or Method instance.\n\nChange-Id: Ib18fa75c0648561d3c5e1554cb3955ed57ab9314\n" }, { "commit": "15216930d0d360e995480786a0a5cf9ee48a4ef7", "tree": "68eef3df89d935f86dbd261832854f78266a6af6", "parents": [ "e5b7894351ecc5ef99442eea0700c913178c95ce" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Mar 21 21:53:06 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Mar 21 21:53:06 2012 -0700" }, "message": "Ensure that classes we\u0027re using for reflection are initialized.\n\nDalvik did the equivalent of EnsureInitialized both when handing out\nfields and when receiving them back. We\u0027d prefer to just check for\ninitialization once, up front, so let\u0027s do that and just assert in\ndebug builds that there isn\u0027t some bizarre way to get here.\n\nChange-Id: I225ec30df5a78a60259121a50f6a7c4154956bce\n" }, { "commit": "dbac3090ace708eb27c707d4b72efc48c069129c", "tree": "2b811b40af24ce849d327643f05b4244ca2449fb", "parents": [ "5efa670d6a8cf6ad3771b9fb7d4c10598ad78f9d" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 16 18:00:30 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 16 18:00:30 2012 -0700" }, "message": "Ensure that BoxPrimitive only sees clean data...\n\n...rather than forcing it to zero irrelevant bits itself.\n\nChange-Id: Ibbd28db08b68fac84704a911a6f927f7d9343055\n" }, { "commit": "0512f02dd6623c0870c11fbf3274d7462f732136", "tree": "c90b32c42b285d3cea45272601b5cf007b4ea264", "parents": [ "88c5c355fc3d881f905564911d746b2313d5fc89" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Mar 15 22:10:52 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Mar 15 22:10:52 2012 -0700" }, "message": "Remove anonymous namespaces.\n\nNo one likes them and we\u0027re pretty inconsistent at using them. We\u0027re\nmuch better with \u0027static\u0027.\n\nChange-Id: I65eeffd0fb60633edca30923af0caedb06a0105d\n" }, { "commit": "b600b3f8360f535913fedc91aef7c8e5abaca48a", "tree": "b8f4fdca44a70caec314b210372d678ad9501dbf", "parents": [ "002b915276064dbe2ce9caf7474d3032301d8d7c" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Mar 14 13:57:24 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Mar 14 13:57:24 2012 -0700" }, "message": "Fix/improve exception detail messages for unboxing.\n\nThis fixes libcore/luni/src/test/java/libcore/java/lang/reflect/MethodTest.java\nfailures.\n\nChange-Id: Idf6aae169bd72f0534d4094170536e6b2515ca07\n" }, { "commit": "6d4d9fcb4f01e287ee29e81cd1c941ee5d11d379", "tree": "4c715bef86a3c5a47f9cea43c4a092cf4766ed4b", "parents": [ "86b0010c79ef95b5333cd540b7d3af34a9f1a643" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Nov 30 16:24:48 2011 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 06 14:02:09 2011 -0800" }, "message": "Reduce meta-data object sizes, introduce meta-data helper classes.\n\nChange-Id: Id14ad218f1c74c659701352fdf1a45bf6444daa3\n" }, { "commit": "c129a6b44193ac3a872f852a69d336e0e7ed8640", "tree": "ff5b5503aff4e5173c570b2e471d2df0895b7b2b", "parents": [ "434424a3f60f2129aaa1780ac6a5c2618bc789e0" ], "author": { "name": "Jesse Wilson", "email": "jessewilson@google.com", "time": "Thu Nov 24 14:47:46 2011 -0500" }, "committer": { "name": "Jesse Wilson", "email": "jessewilson@google.com", "time": "Thu Nov 24 14:47:46 2011 -0500" }, "message": "Make methods and fields accessible by default in art.\n\nThis also fixes a corner-case bug where calling Field.setBoolean()\non a non-boolean field would crash the runtime.\n\nChange-Id: I2f6693e3c3fe3bf23b4761335300506b3fbd3008\n" }, { "commit": "9a6bae896a2f003d7216603bf29771d105c10ca4", "tree": "ccf8c190b8b4daf799e17ec8b04b77edfc5c72ff", "parents": [ "169c9a7f46776b235d0a37d5e0ff27682deffe06" ], "author": { "name": "Jesse Wilson", "email": "jessewilson@google.com", "time": "Mon Nov 14 14:57:30 2011 -0500" }, "committer": { "name": "Jesse Wilson", "email": "jessewilson@google.com", "time": "Mon Nov 14 17:46:40 2011 -0500" }, "message": "Don\u0027t run managed code until the runtime has started.\n\nPreviously we ended up running managed code because JNI was\ntriggering class initializers to run. This was triggered by\nboth Thread::CreatePeer() and InitBoxingMethods().\n\nWhen these initializers were prevented from running, other\ncode broke:\n - Creating the peer for the main thread was relying on\n ThreadGroup.\u003cclinit\u003e to assign the built-in thread groups.\n - Creating the boxed methods caused class initialization of\n the primitive wrapper classes; these need to be initialized\n before Thread.\u003cclinit\u003e is run to avoid a crash in its own\n initializer.\n\nThis change works around those breaks by splitting thread peer\ncreation into two parts (allocation and running \u003cinit\u003e) and\nby calling InitBoxingMethods() during runtime start.\n\nChange-Id: I44be7170ce08451adf876ee73cba0f1f66d5a59e\n" }, { "commit": "b82b6878fb000d4731063b1bf15c39ff7c50b61f", "tree": "8764bcc9307fb2f57820b25b1d098e15166a24bc", "parents": [ "bfe487be25652c5456236661b9d9c3579d2296c1" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Oct 26 17:18:07 2011 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Oct 26 21:43:27 2011 -0700" }, "message": "Make sure threads allocating are in the kRunnable state\n\nChange-Id: I55a08fcd38b3388698ef571d2d4fa9553210e197\n" }, { "commit": "6b4ef025af12b158d117fc80fc79acf620f411a0", "tree": "223c580477cbb7bde240b599da4bbee793b8dcaf", "parents": [ "21b9f1355f0311b5c67bfdfa9dd63cacb1fba502" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Oct 23 14:59:04 2011 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Oct 25 00:03:03 2011 -0700" }, "message": "Make ResolveField not rely on Field::GetType resolution\n\nChange-Id: I10f4a874809ac9db2cd54e200cf10eb7c8979fce\n" }, { "commit": "582a7d16bb0db323d8bd730beb61578aa3765f43", "tree": "ec5ce05edac63b7849b39463d02470d4cc4fde3f", "parents": [ "df143242f4beaad4cc9fbabebfc033b68c40964e" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Oct 10 18:38:42 2011 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Oct 10 18:38:42 2011 -0700" }, "message": "Fix a handful of reflection bugs.\n\nWe now pass test 046.\n\nChange-Id: Ie605ed99e64038435608fe388c59526674af5489\n" }, { "commit": "ed1c1e335acc9f61c7e25a78204f159ee3d13350", "tree": "91be0a05aaf7e18f655cad675e411ed56a79d48a", "parents": [ "67786519786755d287bcda07dceb6b6110b4f3f0" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Sun Oct 02 14:31:05 2011 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Sun Oct 02 14:31:05 2011 -0700" }, "message": "Fix field access.\n\nThere was a bug in the implementation where the native methods weren\u0027t\nnecessarily getting a null receiver but art::Object insists that you only\nget/set static fields on a null receiver.\n\nAlso, these tests assume that fields are in the setAccessible(false) state\nby default. In our system, they\u0027re not.\n\nChange-Id: I935614fec013be465fc0bc30d3c717f12476e09b\n" }, { "commit": "5cb5ad27944efb08d4556b3c0d362302e37e832b", "tree": "63334f19072ca2ba7179973f0ce62101ee89605d", "parents": [ "d6fe38d96b6116bd53cf2cb14734af8d69e08661" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Sun Oct 02 12:13:39 2011 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Sun Oct 02 12:13:39 2011 -0700" }, "message": "Fix exception throwing to support no detail message.\n\n(The empty string as a detail message is distinct from a NULL detail message,\nand is treated differently by Throwable.printStackTrace.)\n\nChange-Id: I8c65deac9f18c5782dcf6e72e4c37e6dd4174fe9\n" }, { "commit": "fe6207f7d3a38e021f81f7c3d25f085883e4cf43", "tree": "c57ff9ab11f8ddcf89b2e33b8dafa4eca3965277", "parents": [ "20d8d673ed402e9d0292f1770313476ce94a9ef2" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Sep 26 17:24:06 2011 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Sep 26 17:24:06 2011 -0700" }, "message": "Add one last memory barrier.\n\n(At the time I added all the others, this code had yet to be written.)\n\nChange-Id: Id41e5730eec19d9968608cdcdfc487e48ddfb487\n" }, { "commit": "418d20fc407052d4152157f61e7453359f902383", "tree": "250a190e03cb4c8d509dc3a13fd4607f888965ca", "parents": [ "c41e5b5ae1befe2c602d55de1dbc04d1914f4a6c" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Sep 22 14:00:39 2011 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Sep 23 13:58:05 2011 -0700" }, "message": "Implement reflective method invocation.\n\nChange-Id: Ib3af9d7e00bf226398610b5ac6efbfe3eb2d15e8\n" }, { "commit": "0c64f5ebef8ad6470be87cd2d50934180c91b70a", "tree": "8ea66d89ba6c92d6edfb8f7253cb29ec0583d207", "parents": [ "c3d530c371d5c915f4d4545810d77d391dbbfec9" ], "author": { "name": "Jesse Wilson", "email": "jessewilson@google.com", "time": "Thu Sep 22 16:52:20 2011 -0400" }, "committer": { "name": "Jesse Wilson", "email": "jessewilson@google.com", "time": "Thu Sep 22 16:55:25 2011 -0400" }, "message": "Move annotations to managed code.\n\nChange-Id: I8491f5d8b489b6fb4f80d4cbca9a05c0cf47816d\n" } ], "next": "fea966e51e65f2040e1510a2e39d407baad29b2c" }