)]}' { "log": [ { "commit": "e9cd81c14609fa9dae2125e96ec4d86ca959117f", "tree": "e261a0445fc6a8bbd1c21cda9c012b3392e606fe", "parents": [ "a70b683cd796f7e5bb50d8d282f72c50c89af250", "9e57abacecc683f50dd6c9870a32ee075eaf86a8" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Mar 16 13:30:22 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Mar 16 13:30:24 2017 +0000" }, "message": "Merge \"Fix String::DoReplace() using obsolete `this`.\"" }, { "commit": "9e57abacecc683f50dd6c9870a32ee075eaf86a8", "tree": "27ef93c87f3ce9f09414620f77b35f6d0949fb22", "parents": [ "26ec3cad4968ae32ce9d04c3046f766df46d9bd7" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Mar 16 10:45:40 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Mar 16 10:55:24 2017 +0000" }, "message": "Fix String::DoReplace() using obsolete `this`.\n\nChange it to a static function taking a Handle\u003c\u003e.\n\nTest: testrunner.py --host --interp-ac --gcstress -t 021-string2\nBug: 36335996\nChange-Id: I5ab3e7adc59d6a9095290e57d5ce5d46b79f089b\n" }, { "commit": "f44d36c8423f81cbb5e9f55d8813e26ffa1a7f3b", "tree": "324b41485ce6c414c1a006c72cbcc5ed9f466138", "parents": [ "8d6768d47b66a688d35399d524ad5a5450e9d9d4" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 14 14:18:46 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 14 19:03:20 2017 +0000" }, "message": "Revert^2 \"Hash-based DexCache field array.\"\n\nTest: testrunner.py --host --interpreter\nBug: 30627598\n\nThis reverts commit 6374c58f2ea403b3a05fb27376110fe4d0fc8e3f.\n\nChange-Id: I275508e288a85d3aa08f7405a1a4f362af43b775\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": "6374c58f2ea403b3a05fb27376110fe4d0fc8e3f", "tree": "0aacaaf1f193e96b55e0b203b678ae724c0fb6f5", "parents": [ "8f323e09e692ff4f95f40300391fe41fb96a6c49" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Mar 13 14:51:19 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Mar 13 14:52:17 2017 +0000" }, "message": "Revert \"Hash-based DexCache field array.\"\n\nReverting to allow rebasing the revert\n https://android-review.googlesource.com/351689\nwithout too many conflicts.\n\nBug: 30627598\n\nThis reverts commit 1aea3510b8dd0c512cec61c91c5ef1f1e5d53d64.\n\nChange-Id: I4af65e9f41c8bad8106c028947eca7c5a9534c53\n" }, { "commit": "1aea3510b8dd0c512cec61c91c5ef1f1e5d53d64", "tree": "94cbab7c3097ce7d3a1feb1a69f28406644af085", "parents": [ "d1d4530ffa97729aa8944932a7ac2009ae51c7e3" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Dec 08 11:39:42 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Mar 10 12:59:37 2017 +0000" }, "message": "Hash-based DexCache field array.\n\nTest: m test-art-host, both AOT and interpreter\nTest: m test-art-target, both AOT and interpreter\nTest: m valgrind-test-art-host\nBug: 30627598\nChange-Id: If992f091aadd862d17b09928d21659573dd285a0\n" }, { "commit": "77651c4bbd56d502bcf05347e641061bbebca099", "tree": "7965e6e8cf29f1c78fc15e3eb3d645bb681a3f7d", "parents": [ "f9d80d72a686048200bcf6a7f37192f9eb75d5bd" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Mar 03 18:04:02 2017 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Tue Mar 07 11:38:22 2017 -0800" }, "message": "ART: Clean up code related to foreign dex use\n\nWe simplified the way we track whether or not a dex file is used by\nother apps. DexManager in the framework keeps track of the data and we\nno longer need file markers on disk.\n\nTest: device boots, foreign dex markers are not created anymore\n\nBug: 32871170\n\n(cherry picked from commit 868515e2606820ea96f8b9022b442f5bcc770867)\n\nChange-Id: If51b3301c845a8c3bbaf87d0e35a12c700e1d0aa\nMerged-In: I5a04d3bba29581dedf05e21b8b20d79afa9b405a\n" }, { "commit": "84740267390c28f7f88cdb66f8429badfa15456e", "tree": "9347bdc4cea0f1b7d0ec986a8b97441f00b6c321", "parents": [ "861af894a5d2d4a77eb66041bddfbaf54d8c8ee8" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Mar 01 09:00:28 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Mar 03 08:55:05 2017 -0800" }, "message": "Clean up ZygoteHooks stack walking slightly.\n\nTest: Build and run marlin with libartd.so\nChange-Id: I4a67935c5c7270636085c3ee12db0ebc1798a89e\n" }, { "commit": "861af894a5d2d4a77eb66041bddfbaf54d8c8ee8", "tree": "757339c632345f2e5f5005ba084d2020e7f91548", "parents": [ "edb792d53fd8b9f94244396a698fb1dc68cb6905" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Feb 28 12:45:36 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Feb 28 14:35:47 2017 -0800" }, "message": "Fix mutex issue\n\nWe were having some issues with mutex locking order related to\nupgrading local to global references when recording non-debuggable\nclasses. We refactored this code so there is no longer this issue.\n\nBug: 35838746\nTest: Build and boot a marlin with libartd.so\n\nChange-Id: I093b433d921478307130c49a07d0c7ec34dd070d\n" }, { "commit": "c0ca5fd8f54f5b8361548174cadb236d01789d6d", "tree": "7a4710d29947921c6e63fed8a1bf9ee27aa04818", "parents": [ "e44d32542794bbc32e0aa7e3b37fc5b914e1de0a", "e00e5590a9cfb72c89294dd7a5480e5fa69a90b9" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Feb 28 14:15:21 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Feb 28 14:15:21 2017 +0000" }, "message": "Merge \"Fix java.lang.Void.TYPE (void.class) initialization.\"" }, { "commit": "e00e5590a9cfb72c89294dd7a5480e5fa69a90b9", "tree": "7eb9254bd6cfe10b2c4005e9cdec3a7cfbb2f442", "parents": [ "06f658ab309a5f1930b8a6bc306ef79d0ef0616e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Feb 24 14:58:29 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Feb 27 16:31:42 2017 +0000" }, "message": "Fix java.lang.Void.TYPE (void.class) initialization.\n\nDo not rely on the DexCache resolved types now that entries\ncan be evicted.\n\nTest: 157-void-class\nTest: testrunner.py --host\nBug: 35652776\nBug: 30627598\nChange-Id: Ic384174ae1849072568dd6d2ff9cc60d7f7f1880\n" }, { "commit": "6e311acac755e0446660fdda955e9352895f2b01", "tree": "19d654b562d328c338f95c05687d3170d207ab7e", "parents": [ "69c4c27cc965620511e876248f679bbcf4fde646" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 27 11:46:03 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 27 12:48:57 2017 +0000" }, "message": "Re-arrange System.identityHashCode for better performance.\n\nBrings in ~8% improvement to Ritz::EvaluateComplexFormulas.\n\nTest: run-libcore-tests test-art-host\nbug: 30933338\nChange-Id: I9966da4143d013caf2e610eb176c0a8fffabca45\n" }, { "commit": "2baf0756b1fe0bda98b7fcceeaf0bde1346031d9", "tree": "914d88033e34c6b2281a8cca43dda5764111213f", "parents": [ "1883c6df11e93139f1b2923091c5771f5656d894", "e77b48b7d0df3cd7aa650ded8373f86974556373" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Feb 23 23:24:22 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Feb 23 23:24:23 2017 +0000" }, "message": "Merge \"Check stacks for unmodifiable frames\"" }, { "commit": "e77b48b7d0df3cd7aa650ded8373f86974556373", "tree": "397def95ccf39a0d2c777e4a656cf4f2a0fdf69c", "parents": [ "88837a783d8c700f597da55b8e0e091b4d44f87c" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Feb 22 11:08:06 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Feb 23 13:11:51 2017 -0800" }, "message": "Check stacks for unmodifiable frames\n\nWe keep a generate a list of classes that have unmodifiable frames\nduring the zygote fork and check for them in IsModifiableClass.\n\nTest: Start apps on aosp_marlin-userdebug phone with libartd.so\n\nChange-Id: I6bbaa20d307c3803a5808fb4108638365895e802\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": "92907f3f7af106a9eeb341caba17f3770203f7fb", "tree": "c44f6b487c9ad078c044f19a4a6d72eaf4979d11", "parents": [ "1b7ba9e15d9525eec7ea68e9ecf95e5777a46069" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Feb 20 14:08:30 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Feb 21 12:31:52 2017 +0000" }, "message": "Remove String.setCharAt().\n\nThe internal API String.setCharAt() breaks the assumption\nthat strings are really immutable. That in turn breaks\nstring compression invariants - compressible strings must\nbe compressed. This CL removes the String.setCharAt() API.\n\nThe method was used only in String.replace(char, char)\nwhen we found a match, copying the string on first match.\nInstead, introduce a new native method that does the whole\nreplacement with a single call when we find the first match.\n\nStringReplaceBenchmark results on Nexus 6P, lower is better:\n timeReplaceCharNonExistent/EMPTY 41.93 -\u003e 38.25 (-9%)\n timeReplaceCharNonExistent/L_16 114.90 -\u003e 95.09 (-17%)\n timeReplaceCharNonExistent/L_64 419.97 -\u003e 320.65 (-24%)\n timeReplaceCharNonExistent/L_256 1667.01 -\u003e 1091.25 (-35%)\n timeReplaceCharNonExistent/L_512 3253.50 -\u003e 2075.62 (-36%)\n timeReplaceCharRepeated/EMPTY 41.93 -\u003e 39.58 (-6%)\n timeReplaceCharRepeated/L_16 114.87 -\u003e 95.40 (-17%)\n timeReplaceCharRepeated/L_64 1267.29 -\u003e 704.32 (-44%)\n timeReplaceCharRepeated/L_256 5139.14 -\u003e 1361.80 (-74%)\n timeReplaceCharRepeated/L_512 10787.81 -\u003e 2338.41 (-78%)\n timeReplaceSingleChar/EMPTY 41.78 -\u003e 37.16 (-11%)\n timeReplaceSingleChar/L_16 449.54 -\u003e 497.51 (+11%)\n timeReplaceSingleChar/L_64 942.08 -\u003e 891.35 (-5%)\n timeReplaceSingleChar/L_256 2756.18 -\u003e 2174.64 (-21%)\n timeReplaceSingleChar/L_512 5489.91 -\u003e 3983.32 (-27%)\n\nTest: testrunner.py --host\nTest: run-libcore-tests.sh --mode\u003dhost\nTest: testrunner.py --host with string compression enabled.\nTest: run-libcore-tests.sh --mode\u003dhost with string compression enabled.\nBug: 31040547\nChange-Id: I9cf0d5457182f0a33ca8251c29931d3eb624ae07\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": "ea9465eaaa55646b0de242d2a21f9c1f0f0aa01f", "tree": "12590422d05d459a63a56937e77520e84cf4c288", "parents": [ "4c9c57054578022d9ab8442264fbc661769f97f5" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Feb 16 15:38:35 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Feb 17 08:33:06 2017 -0800" }, "message": "Move InMemoryDexClassLoader to be a subtype of BaseDexClassLoader\n\nAlso write a test that class transformation works with this type of\nclass loader.\n\nTest: mma -j40 test-art-host\n\nChange-Id: Ibc2214191e04876ff7bbea010be6ec03b6f41904\n" }, { "commit": "8dc12b1546c7409be19a7b5dc48932011db13067", "tree": "a157d2fb7bee08f7c0290e8a41f0c7c9c883794d", "parents": [ "e92ba687f9ab2a3cd9724f494b6899e8e6e472ec", "ffc8cad04135192a8d10e7f38bd97c48101294a8" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Feb 15 09:39:05 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Feb 15 09:39:06 2017 +0000" }, "message": "Merge \"Add a DCHECK in thread::GetPeer.\"" }, { "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": "ffc8cad04135192a8d10e7f38bd97c48101294a8", "tree": "fb90432ea5e6d218891c0b3fcccc0fa77d202f7d", "parents": [ "4e4b62e21672dbacf5d5abb688a126aabad29269" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 10 10:59:22 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 14 15:42:29 2017 +0000" }, "message": "Add a DCHECK in thread::GetPeer.\n\nMotivated by https://android-review.googlesource.com/#/c/333205/.\n\nTest: test-art-host run-jdwp-test.sh\nChange-Id: I173c060324aa0dc39144db55e3a97e672c012ba8\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": "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": "8d4b1189639f0d8982bde681ccbdd7c03fe6ddbf", "tree": "a16a1efaaadcb51001554a1ab829fbc456b104ae", "parents": [ "e6dddf0f4f31db42c67661adc3845165c64d4c3d", "cd556b003adbb53739d4b3f43135e6a0ae69509a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 09 12:30:41 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Feb 09 12:30:42 2017 +0000" }, "message": "Merge \"Fix dex cache resolved types and class table mismatch.\"" }, { "commit": "cd556b003adbb53739d4b3f43135e6a0ae69509a", "tree": "a30c9f03071d87e1f75a0d0b8c2961d113ea767d", "parents": [ "357dcb73934356239292c46d6fbedba734da5e00" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Feb 03 11:47:34 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 09 10:10:29 2017 +0000" }, "message": "Fix dex cache resolved types and class table mismatch.\n\nRecord class table in ClassLinker::DexCacheData and use\nit in DexCache.setResolvedType() to store the type also\nin the initiating loader\u0027s class table if the dex file\nhas been registered.\n\nAlso throw InternalError when trying to register the\nsame DexFile with multiple class loaders. (Different\nDexFile instances referencing the same file are OK.)\n\nTest: 155-java-set-resolved-type\nTest: m test-art-host\nBug: 30627598\nBug: 34193123\nBug: 34839984\nChange-Id: Ia48acb300337c45880ea1459d2d32789546d67f4\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": "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": "bd2fed5a813b68fa42fe941b07325ba4ff3190b0", "tree": "2e8ce7221f842133a1b7b8e6fc4793acbf4ed8f6", "parents": [ "ca21dc47adeed92a15a9d3fd090bdd0e6654679c" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Wed Jan 25 10:46:54 2017 +0000" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Fri Jan 27 13:47:50 2017 +0000" }, "message": "MethodHandles: Support and tests for revealDirect / reflectAs.\n\nWe need runtime support to materialize a Constructor, Field or\nMethod object from a MethodHandle and its associated ArtField\nor ArtMethod pointer.\n\nAn alternate approach might be to have all handles hold a reference\nto their associated Field or Method object (in addition to the\nraw ArtField or ArtMethod) but that seems unnecessary given that\nthese APIs are not expected to be called often.\n\nTest: test-art-host\n\nChange-Id: I9450706b9c30359edebf2e8a7afbc8d89bd68b26\n" }, { "commit": "92e393a5ef08b63ae68621c51764d69abce2c9a2", "tree": "ad3128469639995f98ba8d2c7c01e2aaa9cb8270", "parents": [ "727e9110243392bd8599ae8ae73b1a169668ded7", "ca6e14ebd19ab972e4962f18d99a882969d7bee1" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Jan 25 22:35:49 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 25 22:35:50 2017 +0000" }, "message": "Merge \"[ART] Prepare profile streaming for framework to use.\"" }, { "commit": "ca6e14ebd19ab972e4962f18d99a882969d7bee1", "tree": "9bd4bc18fcc56294e01db0ad4e551b13ae533fea", "parents": [ "67edcfdb2151af66cdacf61e44c618c41601d0c8" ], "author": { "name": "Shukang Zhou", "email": "shukang@google.com", "time": "Wed Jan 25 00:35:21 2017 -0800" }, "committer": { "name": "Shukang Zhou", "email": "shukang@google.com", "time": "Wed Jan 25 10:40:46 2017 -0800" }, "message": "[ART] Prepare profile streaming for framework to use.\n\nMake it possible for framework\u0027s Java code to optionally\nenable streaming output of profile.\n\nTest: m -j48 ART_TEST_TRACE\u003dtrue ART_TEST_TRACE_STREAM\u003dtrue test-art-host\nChange-Id: I70f6cf3b72c6cec2ac1f1f1e9c2b2c4727d72e8a\n" }, { "commit": "72ab684871f870aead76b23cb67deb046107b380", "tree": "d4bb40cab65df449246b35ae582234238676b7bb", "parents": [ "e36c51aee58e61e9fc89851b767379c587f050e3" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jan 20 19:32:50 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jan 24 12:05:02 2017 +0000" }, "message": "Add class status for resolved erroneous classes.\n\nSplit the old ambiguous status mirror::Class::kStatusError\ninto kStatusErrorUnresolved and kStatusErrorResolved. Once\na class has been resolved, IsResolved() shall return true\neven if the class later becomes erroneous. Allow returning\nerroneous class from ClassLinker::EnsureResolved() if it has\nbeen previously resolved. This allows consistent behavior\nfor retrieving classes, immune to multi-threaded races and\nmulti-dex weirdness. It also allows JVMTI to properly report\n\"prepared\" (i.e. resolved) classes that are also erroneous.\n\nThe new behavior is consistent with the RI.\n\nAdd regression tests to 008-exceptions for inconsistent\nbehavior for multi-dex retrieval of erroneous resolved class\n(wrapping or not wrapping the old exception based on which\ndex file is used for lookup) and for a CHECK(IsResolved())\ncrash in ClassLinker::LoadSuperAndInterfaces() (without any\ntests for similar checks that could have previously failed\nonly due to extremely unlikely race conditions; these should\nnow also be fixed).\n\nInconsistency still remains for class verification as shown\nby the new exceptionsForSuperClassInitFailure() test in\n008-exceptions, where interpreter and Optimizing still\ncause different exceptions to be thrown.\n\nNote: This is partially changing behavior implemented for\nbug 28787733. Since we allow the class loader to retrieve an\nerroneous resolved class, the ExceptionInInitializerError is\nnot thrown at all from VMClassLoader_findLoadedClass(), so\nthere is nothing to wrap in ClassNotFoundException.\n\nTest: m test-art-host\nBug: 30627598\nBug: 28787733\nChange-Id: I86cdca00f35a0d6221d2559e3026ac0428a3613c\n" }, { "commit": "a35510de5aa1ec5d39c6870da556e2d60666cdc3", "tree": "65e2c9f63cbe409b2fe7a48194a69ac4ecca6dde", "parents": [ "0d478f289f0e33f19693d135f1d562b57427ed32" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 22:42:09 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 23:03:24 2017 +0000" }, "message": "Fix missing OOME check in reflection.\n\nTest: 080-oom-throw\nChange-Id: I5f18ea756e815978bf8682555470505959a8fad5\n" }, { "commit": "4201cf014cfe00c145edc0b32bf30b1ceaf1495f", "tree": "086c5df83832b6d80864a294503b6dc6025f270c", "parents": [ "9e68f5043323eaf4543ebf3b6f496145d8ae77a2" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 12 14:51:44 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 12 15:15:51 2017 -0800" }, "message": "Avoid suspending heap task thread for getting stack traces\n\nInstead of suspending the heap task thread, GetThreadStack (called by\nVMStack_fillStackTraceElements and VMStack_getThreadStackTrace) will\nreturn an empty thread stack. This fixes possible deadlocks caused by\nsuspending the GC thread and doing allocations for the stack trace.\n\nBug: 28261069\n\nTest: test-art-host\n\nChange-Id: I45a0b8ac94a99d6bbcfcdc2b41afadf941ec0138\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": "6f43e3a8fd839a97073516be0607656dd771362d", "tree": "9b7aac9a4077fd6bece019179c6cac0448f7c0a8", "parents": [ "7a8e16c827efc866d01b5cbca835b18fe29a83ab" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Dec 14 14:12:17 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Dec 14 14:41:27 2016 -0800" }, "message": "Rename Reference.clear to Reference.clearReferent\n\nRenamed on java side.\n\nTest: test-art-host ART_TEST_INTERPRETER\u003dtrue ART_TEST_READ_BARRIER\u003dtrue\n\nChange-Id: I57fc1f46c8c3c2b6d35741ab8549020894de7026\n" }, { "commit": "c9a7028430d95090ad3d7690203fd887d1e201a4", "tree": "83e690d43ce87913c274b788c61d3425e6570e51", "parents": [ "36994ba006c18c1933815cc0c4c036df086e6814" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Dec 13 14:44:33 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Dec 13 16:40:14 2016 -0800" }, "message": "Add exclusion for Reference.clear and reference processing\n\nPrevents race conditions like unclearing cleared references or\ncalling IsMarkedHeapReference on null references.\n\nBug: 33389022\n\nTest: test-art-host\n\nChange-Id: Iee83b76d84453e929172f1a83f284aa4910e126c\n" }, { "commit": "19a4d374738da4dc668a078f92dbe887ff9f00d9", "tree": "a210638e93d80cf0a297de07f4c9cdc60784ce50", "parents": [ "aea9ffece7eb32f3884a4ad0553e1df4d90fd9e4" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Dec 08 14:41:46 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Dec 09 13:30:00 2016 +0000" }, "message": "Clean up Class::GetDirectInterface().\n\nOnce the `klass` has been resolved, this function should not\nneed to resolve interface types anymore. Drop the type\nresolution from this function and pass the Class as ObjPtr\u003c\u003e\ninstead of Handle\u003c\u003e. Make callers that expect non-null\nresult DCHECK() that assumption. For the callers that may\nactually need to resolve these interface classes, introduce\na new function ResolveDirectInterface().\n\nAlso improve ObjPtr\u003c\u003e constructors and assignment operator\nto improve overload resolution. For example, if we have\nfoo(ObjPtr\u003cT1\u003e) and foo(ObjPtr\u003cT2\u003e), calling foo(.) with\na T1* would have previously been ambiguous even if T1 is\nnot a base of T2.\n\nTest: m test-art-host\nChange-Id: Iff44a7285f68f0af4507fc9ba37343865422fe84\n" }, { "commit": "d12f04cb563432c1c072c7446255c57ec2f8db53", "tree": "020f1dae51c3427d40e18f940353cc21e44f4546", "parents": [ "72ef70de77fb77afcc34ee4f523a1f7522daf8a8", "a8888c1b6513a6f5335e72073efc301c86467e92" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Dec 06 20:04:53 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Dec 06 20:04:54 2016 +0000" }, "message": "Merge \"Use getReferent() in enqueueSentinelReference to avoid race with GC.\"" }, { "commit": "a8888c1b6513a6f5335e72073efc301c86467e92", "tree": "3228f8c9146b30cecf1cf902bfef44d4ac5f0e89", "parents": [ "5d0c3009fc56afdb4aeae5ee6dd98c3d1d2e7711" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Dec 02 13:58:58 2016 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Dec 02 14:32:40 2016 -0800" }, "message": "Use getReferent() in enqueueSentinelReference to avoid race with GC.\n\nThis is the art part.\n\nFix a check failure due to the direct referent access that races with\nthe GC reference processing for CC.\n\nBug: 33207594\nBug: 12687968\nTest: test-art-host with and without CC.\nChange-Id: I3c98066d09dd98d8fddc59b6784c75843b45338b\n" }, { "commit": "cc1b5357f83f0b787d51fbfde3fe870c8a2fa050", "tree": "253d23dba63e1003dc8e953002b994d01c97f181", "parents": [ "5d0c3009fc56afdb4aeae5ee6dd98c3d1d2e7711" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 01 16:58:38 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Dec 02 14:00:22 2016 -0800" }, "message": "ART: Clean up ClassLinker\n\nTry to clean up and simplify ClassLinker.\n\nMove dex_lock to Locks. Remove dead code. Move single-use\ncode to user. Hide implementation details from header.\n\nTest: m test-art-host\nChange-Id: I58150fa9c2a9524f8304370270c2197d655cb3a8\n" }, { "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": "46cc64f11b1b1f0ac5e4aebd665b325a68d4d42f", "tree": "182f28f92fb576b2ad586a368ac2d0761896e832", "parents": [ "3e580bc9b248586fa5d87db2098fa5c5abb7afc1" ], "author": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Mon Nov 14 14:53:55 2016 +0000" }, "committer": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Thu Nov 24 07:54:03 2016 +0000" }, "message": "Move dumping of oat file status inside of the OatFileAssistant.\n\nRather than exposing what could otherwise be internal to the\nOatFileAssistant, move the logic for dumping a human readable\nrepresentation of the status of the oat files inside the\nOatFileAssistant.\n\nBug: 30937355\nTest: oat_file_assistant_test\n\nChange-Id: I79c6cc1286a822f1dbe0035be934a2be4792563c\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": "3773f2106864cf71fa17be33058717ecc402a5ad", "tree": "1bf43b280f3a8f484758d308383c67af328e1a9f", "parents": [ "8acdebc1a1b5821d3bb8e9461f7877fc234a37ff", "ca620d7bc03b23a0bcf0ef58df58603ee000dca0" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Nov 09 01:48:19 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Nov 09 01:48:19 2016 +0000" }, "message": "Merge \"ART: Fix tidy warnings\"" }, { "commit": "ca620d7bc03b23a0bcf0ef58df58603ee000dca0", "tree": "07cb026075b70a958d14ae84b4e213178a6ba0b4", "parents": [ "b02b8d7df48ea3314cfcb3c08d84ac9556363833" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 08 08:09:33 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 08 15:04:48 2016 -0800" }, "message": "ART: Fix tidy warnings\n\nSwitch to char versions of find variants.\n\nAdd \"explicit\" constructor variants or refactor and\nremove defaults.\n\nUse const references.\n\nBug: 32619234\nTest: m test-art-host\nChange-Id: I970cc2f47d6cf8f0c74104b994b075b2fafb3d45\n" }, { "commit": "13b27842e88ccf1a42807c92daeb108e867dc4cd", "tree": "d95968f678070058ec084376d561c7ed4f1bcff3", "parents": [ "689a331aded82340d2ab144d0adb9bbbbd4b53ff" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 07 16:48:23 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 07 19:45:48 2016 -0800" }, "message": "ART: Refactor jmethodID handling\n\nArtMethod objects have been native since Marshmallow.\nRemove the dependency on being runnable. Refactor the\ncode into the jni_internal header.\n\nTest: m test-art-host\nChange-Id: I1385fcd4c08981491701da55a87036b447aa2fc2\n" }, { "commit": "7d8d8ff0727a7aa9d11c738f13a7e06d3c4c3d68", "tree": "0f3f131d2d3438d06ecdfc05137ebfcc1ff6f1f3", "parents": [ "af3572bc21cf66f970090c852b976095bb028083" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 02 12:38:05 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 02 12:38:05 2016 +0000" }, "message": "Revert \"Revert \"ART: Generalize FindClassInPathClassLoader\"\"\n\nBug: 32510706\n\nChange is not responsible for the flakiness after all.\n\nThis reverts commit af3572bc21cf66f970090c852b976095bb028083.\n\nChange-Id: I399049c217deeac6a5c3f9d678a46bdb84386e3f\n" }, { "commit": "af3572bc21cf66f970090c852b976095bb028083", "tree": "597b436db28186864524d5208628082e52a5ff52", "parents": [ "fd1ae75a03b0b17fdbbb0097e856f494edf12312" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 02 09:49:25 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 02 09:49:25 2016 +0000" }, "message": "Revert \"ART: Generalize FindClassInPathClassLoader\"\n\nBug: 32510706\n\nThis reverts commit fd1ae75a03b0b17fdbbb0097e856f494edf12312.\n\nChange-Id: Ie247477eea44a6a241d90864bf5dca29897ff4b8\n" }, { "commit": "fd1ae75a03b0b17fdbbb0097e856f494edf12312", "tree": "0f3f131d2d3438d06ecdfc05137ebfcc1ff6f1f3", "parents": [ "46aba36a1c982567d7e80a62b0cff9c0935e7455" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 28 15:34:26 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 01 08:23:27 2016 -0700" }, "message": "ART: Generalize FindClassInPathClassLoader\n\nAlso accept DexClassLoader. Do a fast check for subclasses of\nBaseDexClassLoader that are in the boot classpath.\n\nBug: 32510706\nTest: m test-art-host\nChange-Id: I7d79aa90e0241e7717c5d146d1123f41746e4ffa\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": "c369f6b0cd132ab5bba3f127b162f128dd7d2599", "tree": "691b4ae4ca4da162d403d047095a82c18b5607bd", "parents": [ "d6435da102851e155acd12adc420e7df3db47903", "a1aa3b1f40e496d6f8b3b305a4f956ddf2e425fc" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Oct 28 10:09:33 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Oct 28 10:09:34 2016 +0000" }, "message": "Merge \"Add support for Baker read barriers in UnsafeCASObject intrinsics.\"" }, { "commit": "a1aa3b1f40e496d6f8b3b305a4f956ddf2e425fc", "tree": "efe3dd581dbe9fb33e615d1608f0571ec3984208", "parents": [ "188edb3a3ec36ad5fc42373b1e1bed3a85b4f112" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Oct 26 13:03:38 2016 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Oct 26 13:03:38 2016 +0100" }, "message": "Add support for Baker read barriers in UnsafeCASObject intrinsics.\n\nPrior to doing the compare-and-swap operation, ensure the\nexpected reference stored in the holding object\u0027s field is\nin the to-space by loading it, emitting a read barrier and\nupdating that field with a strong compare-and-set operation\nwith relaxed memory synchronization ordering (if needed).\n\nTest: ART host and target tests and Nexus 5X boot test with Baker read barriers.\nBug: 29516905\nBug: 12687968\nChange-Id: I480f6a9b59547f11d0a04777406b9bfeb905bfd2\n" }, { "commit": "75dccb7238714358fca9e5e993e4daabe24af085", "tree": "f46591a920fcd8fda21ab93eb5f25e6dbae1fa2b", "parents": [ "c33daa3739bbb7288f231a034f3ff80c5df2dd5a", "34ee6842a283afe107f0c1df941393f955323b52" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Oct 26 00:21:33 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Oct 26 00:21:35 2016 +0000" }, "message": "Merge \"ART: Refactor class-linker methods\"" }, { "commit": "eb84221ffc00357be6d69e2e461c7a45ee96334a", "tree": "00e6a0884a62dfb448e4d617699fe511307db74f", "parents": [ "19c5e5f3e92db92e4a51b354743cbb0edd8e0f48" ], "author": { "name": "Leonard Mosescu", "email": "mosescu@google.com", "time": "Thu Oct 06 17:26:36 2016 -0700" }, "committer": { "name": "Leonard Mosescu", "email": "mosescu@google.com", "time": "Tue Oct 25 19:46:58 2016 +0000" }, "message": "Enable agent attaching during live phase\n\nThis is the ART part, the plumbing from VMDebug_attachAgent() to\nactually loading the agent into the runtime.\n\nTest: m test-art-host\n\nBug: 31682382\n\nChange-Id: I3ccc67aa050c1f78278882128983686ed44ddec2\n" }, { "commit": "34ee6842a283afe107f0c1df941393f955323b52", "tree": "63614db22d589a42f843f1b96e2332e8c89d4860", "parents": [ "5b6fbd02f03fd41829c7dd4bfce97a62346be854" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Dec 02 15:43:52 2014 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 25 11:18:05 2016 -0700" }, "message": "ART: Refactor class-linker methods\n\nHide the LookupClass with hash version. Clients should not have to\nknow about that performance detail.\n\nHide FindClassInPathClassLoader. This is an implementation detail.\n\nTest: m test-art-host\nChange-Id: I2378c6fed8d7d1fb1ead8e042b4cf07228adf25c\n" }, { "commit": "28357fab628bd9b91749988b554977398caf9963", "tree": "957fac3c1003713c349e0e216762836138c596a7", "parents": [ "c89f9776a107ca20d0146c16fa881db91c4f8266" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 18 16:27:40 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 19 10:04:37 2016 -0700" }, "message": "Move most of class linker to ObjPtr\n\nReturn values are still mirror pointer. Fix some failing asserts in\ncompiler driver and tests.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I4450bf9dfb2541749496b8388616e8aae8488919\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": "bc5a795c0d486c84913d987cad5846ded840cea6", "tree": "90db29a97a21fa15f4cbc5ffbc1f6dd191add49c", "parents": [ "38a4223fcc0493553d9ad324a1dc145869eb663a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 17 15:46:31 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 17 16:16:50 2016 -0700" }, "message": "Move art/native to ObjPtr\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I67eb89cf042c762c6dcd5eb8b008b9a28e9b3319\n" }, { "commit": "5d3f73aef1b778d81ba101ece41935adee259cf2", "tree": "fec0c2fdbb88df0a366e29b6470d1274aa6e2ec1", "parents": [ "cfdff144f154df876987fefb340058a975c43381" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 14 14:28:47 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 14 14:38:43 2016 -0700" }, "message": "Move reference processor and mirror::Reference to ObjPtr\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I2c7c3dfd88ebf12a0de271436f8a7781f997e061\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": "6b3d12b3d233734bb6669faccddcba7abeb5ea99", "tree": "ce2f3a3188b94e70c880590bf4f99a899ada9119", "parents": [ "b18bfac8e95ffdc6ef20d2602d2e7003021815b6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 13:59:58 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 14:56:51 2016 -0700" }, "message": "Move common throws to ObjPtr\n\nAlso a few other throws. Removed some calls to ObjPtr::Ptr.\n\nTest: test-art-host\n\nBug: 31113334\n\nChange-Id: I4082d64134a5a8961489f4309522c28a30968a18\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": "e8a3c576301fd531d5f73a65fc8b84a63619d580", "tree": "a00e488f153505ee890c28873308d373e6ab231b", "parents": [ "e8317d90c61dde07c12e404a2bc1fabf584905c1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 11 16:52:17 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 12 13:30:54 2016 -0700" }, "message": "Replace StackHandleScopeCollection with VariableSizedHandleScope\n\nVariableSizedHandleScope\u0027s internal handle scopes are not pushed\ndirectly on the thread. This means that it is safe to intermix with\nother types of handle scopes.\n\nAdded test.\n\nTest: clean-oat-host \u0026\u0026 test-art-host\n\nChange-Id: Id2fd1155788428f394d49615d337d9134824c8f0\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": "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": "9d07414a255cd53f9582497cbc918b7df698c1e4", "tree": "757ebcdd5420f564ab51b0369d0f17292efbb84c", "parents": [ "74a69eda053486e2d563f888a689586c5cfa7217" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 30 09:51:32 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 30 10:09:08 2016 +0100" }, "message": "Move two dangling mirror objects into handles.\n\ntest: libcore.java.lang.reflect.MissingClassesTest#testGetFieldFails\ntest: ./art/tools/run-libcore-tests.sh --mode\u003dhost --variant\u003dX64\nChange-Id: Ib708e914cac6abe4d4fc7cb88e258da83a8219ab\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": "4d46970b8a2719fefc2e8ca5ee52eeb6c3ce49f0", "tree": "6ba332eccdcd0adc5975a4ea8f24b685da520a4b", "parents": [ "1c3ca75e63c67c1ab080ede937f21eb1f1184076", "a59d9b228b1eda3bf71a81b6201ec64e26086c23" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 28 22:04:13 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Sep 28 22:04:14 2016 +0000" }, "message": "Merge \"Use ObjPtr for reflection.cc/h/inl\"" }, { "commit": "79a21e798abde263afdb5c8d1c191c84748fe06e", "tree": "f8a0b129dfdafba1f3520930b49f16fbde2eea8a", "parents": [ "38e1882858d90ba147511af2df144c606fbf7c7c" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Fri Sep 09 14:24:51 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Wed Sep 28 11:19:29 2016 +0100" }, "message": "Add support for Executable.getParameters() metadata\n\njava.lang.reflect.Executable native code to create Parameter\nobjects and supporting code for obtaining the system annotations\nthat hold the parameter metadata.\n\nBug: 30391692\nTest: test-art-host and CtsLibcoreTestCases\nChange-Id: I23d7e36014716967ce189fba5955cc5e064fe8d0\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": "3fec9ac0d5af1358d216eb2fdc2000ec0205f3f0", "tree": "f38d8d8aae51f53e7ee6b474f47597b784fc2316", "parents": [ "0cfe19af3b7395658210ea6044a65c9811962a7a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 13 10:47:28 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 26 10:59:22 2016 -0700" }, "message": "ART: Use libbase logging\n\nMove most of our logging infrastructure over to system/core/base.\nRetain VLOG.\n\nUsing unified Android infrastructure has two main advantages. First,\nit reduces the complexity/maintenance burden in ART. Second, it\nallows to detach logging for the cases where we do not want or need\na runtime, e.g., dexdump, the disassembler, etc. As a part of the\nlatter, libbase is also supported for all hosts (including Windows).\n\nFrom a developer viewpoint, there are minor behavior changes for the\nLOG statements (see above), but otherwise usage is the same. Explicit\nseverity enum items are in the android::base namespace now.\n\nBug: 31338270\nTest: m test-art-host\nChange-Id: I5abcb2f45f5b03d49951874c48544f72a283a91b\n" }, { "commit": "0e8443995af602bcc2e70dcef1d20ee48acec934", "tree": "2d81b8db23bea139e40e33e4e9b8c69027fb1e56", "parents": [ "c4cb3d532db8a500c1a15e8a0052895960259209" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Thu Sep 08 13:43:31 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Wed Sep 21 11:42:21 2016 +0100" }, "message": "Collapse ART\u0027s AbstractMethod class into Executable\n\nAfter the OpenJDK 8 pull there is now a common base class\nfor Method and Constructor that can be used instead.\n\nMethods that are abstract on Executable but on Android\nwere implemented in AbstractMethod are now consistently\nrenamed to \"{method}Internal\", declared final and\ngiven package-protected access.\n\nFor consistency with all other private, natively\nimplemented methods, the method getDeclaredAnnotations()\nhas been renamed getDeclaredAnnotationsNative().\n\nBug: 28666126\nBug: 31052885\nTest: make test-art-host and cts run of CtsLibcoreTestCases\nChange-Id: Ibfe6af6a47fe0cfffd0859ffbe58980aeb84e053\n" }, { "commit": "9aa352e92b6ca0f2250cb7f54dfbf4b1be714c19", "tree": "647b9274daa64b3497c202da4327be26fe4e494b", "parents": [ "47c22ff721503c0eb325508450663adacf0bd377" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Sep 15 18:13:52 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Sep 16 16:31:19 2016 -0700" }, "message": "Remove TypeLookupTable from DexFile.\n\nOne more step towards removing runtime dependencies from the DexFile\nAPI. This severs the ties to OatFile. Work remains to move MemMap out\nof DexFile.\n\nBug: 22322814\nChange-Id: I29e7ad8fd292c7919ed2689dc754b958b88d6819\nTest: test-art-host\n" }, { "commit": "26ead4975e1752e8ae2f5ed6fda73876c4f9ff59", "tree": "2ce1b047f54bfb4287483de8eaf79cd12f6058b6", "parents": [ "79f6d706185714dccf80aca20d9f3261337473ae", "268764da8022cafa5661c5b514eaa343c5257e57" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 14 16:55:56 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Sep 14 16:55:56 2016 +0000" }, "message": "Merge \"Make ScopedAssertNoThreadSuspension no overhead for non-debug\"" }, { "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": "268764da8022cafa5661c5b514eaa343c5257e57", "tree": "7c4b46847a6dcfac7a1492edd79b434fb1a0d6ea", "parents": [ "f9decf915f6fa5b33fceba7c10c119e4cad481e5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 13 12:09:38 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 13 15:18:29 2016 -0700" }, "message": "Make ScopedAssertNoThreadSuspension no overhead for non-debug\n\nPreviously it required Thread::Current() which may not be free.\nThe plan is to add a lot more ScopedAssertNoThreadSuspension in\nthe codebase.\n\nAlso cleaned up callers.\n\nBug: 31458474\nChange-Id: I5a1621a5435476504d22266cc01a9bf26aab7568\n" }, { "commit": "60458a072c4f0b24038420b0f1bed0fe94c72965", "tree": "e1e10a29c99f99538695c12da1af914d84ddd4c3", "parents": [ "9d185da3bef8caf015d3dbf4ad79c520af7ce3b1" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Thu Sep 01 15:32:44 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Wed Sep 07 11:12:02 2016 +0100" }, "message": "Track libcore Executable.getParameter() changes\n\nExecutable.getParameter() has been added which added\ntwo new fields to java.lang.reflect.Executable,\nimpacting class arrangements and padding.\n\nThe java.lang.reflect.Parameter class has been added\nwhich has some associated native code and support\nmethods in DexFile.\n\nTest: make test-art-host\nBug: 28666126\nChange-Id: I4ff6469388879651a6c9e568eca0b1cd716f0c0d\n" }, { "commit": "cffc52978fb0ad30f7f33b275f9b0ec15faeb89c", "tree": "80a5c0a5e5535e67afa210d94be8eb556e3d858d", "parents": [ "ebb7b22f4e227176eca40a554990a8e89e283fcf" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Sep 05 13:43:11 2016 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Sep 05 13:45:55 2016 +0100" }, "message": "Address a couple of TODOs and remove an unused header.\n\nFixing overlooked comments in:\n https://android-review.googlesource.com/#/c/258402/\n\nTest: test-art-target-gtest-dex_file_test\n\nChange-Id: I5dc4ddb0cfaf1445a92620d8e2fa35cdb277df6a\n" }, { "commit": "ebb7b22f4e227176eca40a554990a8e89e283fcf", "tree": "574152c57f5dc44f5f877d994d34b541afeb67b9", "parents": [ "38c8fcd78dcb6707e30b489b57d61c1232ee5757", "3a21e386fa55a8b86dffa5250985b263fcfd2155" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Mon Sep 05 09:18:53 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Sep 05 09:18:54 2016 +0000" }, "message": "Merge \"Clean up some includes.\"" }, { "commit": "3a21e386fa55a8b86dffa5250985b263fcfd2155", "tree": "ce6fecaae067f86e094ae7844ee5fa4ae8ee63e8", "parents": [ "f94a4cf18946bda4a4f19378436d2bf131a492ca" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 02 12:38:38 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 02 15:15:28 2016 +0100" }, "message": "Clean up some includes.\n\nRemove some unnecessary includes from header files, replace\nothers with forward references and add includes to source\nfiles as needed. Reduce dependency on stack.h by pulling\nStackReference\u003c\u003e out to its own file.\n\nTest: m test-art-host\nChange-Id: I0fb182145e328870cbd918b0ef6ae2950223c1b2\n" }, { "commit": "a4c2a054ca946eb14496fe8e9fd6057a59ccf692", "tree": "e6ba8afbff39b30775c24d81a05ee255fcc9457b", "parents": [ "79c3e4e558215ffae1601fc6d5f1626e063a4b22" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Aug 17 10:51:42 2016 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Sep 02 11:40:21 2016 +0100" }, "message": "Support for loading classes from DEX files stored in a ByteBuffer.\n\nBug: http://b/26878838\nTest: test-art-target-gtest-dex_file_test\n\nChange-Id: I68845c35784386bde6270102f0079d4f07bc4d03\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": "16b21cd4874be8550f43987b0bd2e4ba8ba3ca42", "tree": "475aeec7476e039442009ca36c1ddde1c9cdbd47", "parents": [ "ca11dc008457e1596554eb9b1b77c823ae9dcf54" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Fri Aug 12 09:37:02 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Fri Aug 26 14:31:26 2016 +0100" }, "message": "Tracking java.lang.reflect.Executable changes\n\nAdded a method to Class: total vtable entries increases\nby one.\n\nHandling knock-ons of introducing Executable into the\nclass hierarchy above AbstractMethod.\n\nRearranging java_lang_reflect_*.cc code to track libcore\nchanges that reduce duplication between Constructor / Method\n/ Executable and AbstractMethod and track the OpenJDK 8\nmore closely.\n\nBug: 28666126\nChange-Id: I6b5c476a14b8ea25556c35448431de744519b6c2\n" }, { "commit": "3aaa37bba53d6df0265793de48b4b0b57327e57a", "tree": "ce795803f0f10003eb3d96d9348da620937675b3", "parents": [ "792c98bb773c8c2390f9cbf774f85be9d9a75332" ], "author": { "name": "jessicahandojo", "email": "jessicahandojo@google.com", "time": "Fri Jul 29 14:46:37 2016 -0700" }, "committer": { "name": "jessicahandojo", "email": "jessicahandojo@google.com", "time": "Tue Aug 23 15:12:26 2016 -0700" }, "message": "creating workflow for mirror::String compression\n\nAll-ASCII String characters are stored in 8-bit blocks\ninstead of 16-bit. The compression has not taken place, but all\nworkflow are in the code already (changing kUseStringCompression in\nstring.h file to TRUE will enable the feature)\n\nNotes: Feature works on interpreter only without optimizing\n\nTest art: m ART_TEST_INTERPRETER\u003dtrue ART_TEST_OPTIMIZING\u003dfalse\n test-art-host\n\nAlso tested with String tests from libcore/:\n1. libcore.java.lang.StringTest\n2. libcore.java.lang.StringBufferTest\n3. libcore.java.lang.StringBuilderTest\n4. libcore.java.lang.OldStringTest\n5. libcore.java.lang.OldStringBufferTest\n\nMemory improvement is 33% (from 6.03% to 4.03%, total String memory\nfrom all apps per total memory of all apps) measured on Angler\nwith Hprof tools\n\nBug: 31040547\nChange-Id: I9cc92c265ebf1305fc06b5fc33efd83797660cce\n" }, { "commit": "9c07ab332b8ebbcb1586c311bfcb75e19b8a35b4", "tree": "26a300b803f77e64c17e3d239a1880a4c5960666", "parents": [ "6670bd2098264d4c4e19750ab4741121da7ee54b", "bf44e0e5281de91f2e38a9378b94ef8c50ad9b23" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 19 17:33:03 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Aug 19 17:33:03 2016 +0000" }, "message": "Merge \"ART: Implement a fixed size string dex cache\"" }, { "commit": "bf44e0e5281de91f2e38a9378b94ef8c50ad9b23", "tree": "bb6e65a3434806dc58f286ee75ad3b78ba9d6c36", "parents": [ "d99565069c64fefc069005286de04599dc2619b8" ], "author": { "name": "Christina Wadsworth", "email": "cwadsworth@google.com", "time": "Thu Aug 18 10:37:42 2016 -0700" }, "committer": { "name": "Christina Wadsworth", "email": "cwadsworth@google.com", "time": "Thu Aug 18 16:18:36 2016 -0700" }, "message": "ART: Implement a fixed size string dex cache\n\nPreviously, the string dex cache was dex_file-\u003eNumStringIds() size, and\n@ruhler found that only ~1% of that cache was ever getting filled. Since\nmany of these string dex caches were previously 100,000+ indices in\nlength, we\u0027re wasting a few hundred KB per app by storing null pointers.\nThe intent of this project was to reduce the space the string dex cache\nis using, while not regressing on time that much. This is the first of a\nfew CLs, which implements the new fixed size array and disables the\ncompiled code so it always goes slow path. In four other CLs, I\nimplemented a \"medium path\" that regresses from the previous \"fast path\"\nonly a bit in assembly in the entrypoints. @vmarko will introduce new\ncompiled code in the future so that we ultimately won\u0027t be regressing on\ntime at all. Overall, space savings have been confirmed as on the order\nof 100 KB per application.\n\nA 4-5% slow down in art-opt on Golem, and no noticeable slow down in the\ninterpreter. The opt slow down should be diminished once the new\ncompiled code is introduced.\n\nTest: m test-art-host\n\nBug: 20323084\n\nChange-Id: Ic654a1fb9c1ae127dde59290bf36a23edb55ca8e\n" }, { "commit": "2ca5828b08b1c4184d802c90585b48dbfd3efb3d", "tree": "a7cd7ac14eecd6b3b37aad9c5da6a4f8f0c3a6eb", "parents": [ "c6efcaa17319e9f81def246c277fec523f5b85e0" ], "author": { "name": "Dmitriy Filchenko", "email": "dmitriyf@google.com", "time": "Wed Aug 17 17:58:08 2016 -0700" }, "committer": { "name": "Dmitriy Filchenko", "email": "dmitriyf@google.com", "time": "Thu Aug 18 01:41:27 2016 +0000" }, "message": "Remove unused include.\n\nBUG: 29824203\n\nChange-Id: Iea74f7b166c3e81c0d00c92442f27e162631e5ae\n" }, { "commit": "542451cc546779f5c67840e105c51205a1b0a8fd", "tree": "11e09bb5abaee12dddffefbe7e425291076dfa7a", "parents": [ "85c4a4b8c9eabfe16e4e49f9b4aa78c1bf4be023" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 26 09:02:02 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Aug 01 18:54:48 2016 -0700" }, "message": "ART: Convert pointer size to enum\n\nMove away from size_t to dedicated enum (class).\n\nBug: 30373134\nBug: 30419309\nTest: m test-art-host\nChange-Id: Id453c330f1065012e7d4f9fc24ac477cc9bb9269\n" }, { "commit": "e01e3644205c3644546889237c20185391a0092e", "tree": "413fcf279b415a305605a601297e3856f13a2948", "parents": [ "de4cf16f467b531373560ca6eb785f0f36606aae" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 25 13:06:04 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 25 13:09:41 2016 -0700" }, "message": "ART: Add pointer-size template to some reflection functions\n\nThe unstarted runtime may run code for a different pointer size,\neven when no transaction is active (e.g., during startup). To\nretain performance when the runtime is up and executing under\nnormal conditions, add a template parameter and use sizeof(void*)\nin places where it is adequate.\n\nFor maintainability, it is necessary to drop the default for\nthe transaction template parameter. Implicit conversions from\nbool to size_t may lead to incorrect code and hard to diagnose\nproblems. So instead ensure that all callers must give all\ntemplate parameter values.\n\nTest: m test-art-host\nChange-Id: I3076883422c8553ede4de5642409c5684a5a9aa8\n" }, { "commit": "65ad9b3516c4f4bc4e7abf5c6e065a675cf024d8", "tree": "7650ae342ce201d498526609d506df7f52783d3a", "parents": [ "a92938a17b75eed3213030faaff9cfb321d47f37", "bb268b14da040c5df27a413426aa2baab602ed15" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 21 10:33:13 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jul 21 10:33:13 2016 +0000" }, "message": "Merge \"Clean up Class::FindStaticField().\"" } ], "next": "09b710c6e3e496edb2ff0312eb59d10ba53f3922" }