)]}' { "log": [ { "commit": "e027d727b833930f1bd0029bc1b2cf750d8af5da", "tree": "0bb05206b8c019798bd3ffede02fb9243d08e615", "parents": [ "ac52000e86077b3c45c192ec259d72413599ff11" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Feb 05 10:13:49 2019 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 06 11:56:16 2019 +0000" }, "message": "Replace StringPiece with std::string_view in class.h.\n\nTranslate the std::string_view to StringPiece where\nneeded for comparing with Signature until we change\nthe Signature::operator\u003d\u003d() to take std::string_view.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 123750182\nChange-Id: I9699f13882353d84dc69e6b8e30d12dac4bace24\n" }, { "commit": "4bcd65753c1e49ceba2c41983f26af81ae319670", "tree": "3fc7bd53d37448621726dc508f173490807af9c7", "parents": [ "2da3cbb4af20a64108e474c0bbbe0cc5d3af2aa2" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Sat Feb 02 20:08:44 2019 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Feb 04 15:00:20 2019 +0000" }, "message": "Take hidden API into account during getDeclaredMethod()\n\nGenerics make it possible for two methods to have the same name and list\nof parameters but differ in their return type. Class.getDeclaredMethod()\ndoes not allow callers to specify the type, so either of the matching\nmethods can be returned (ART will prefer the non-synthetic one).\n\nHowever, Class::GetDeclaredMethodInternal() did not use to take hidden API\ninto account and could return a hidden method, despite a non-hidden one\nbeing available. The reflective call would then reject the method and\nthrow NoSuchMethodException.\n\nThis patch modifies Class:GetDeclaredMethodInternal() to consider:\n(a) hidden/non-hidden\n(b) virtual/direct\n(c) synthetic/non-synthetic\nin that decreasing order of importance and pick the best matching\nmethod. The hiddenness checks are performed with AccessMethod::kNone\nso as to not trigger warnings. A hidden method may still be returned and\nthe caller should do the access check again with the appropriate\nAccessMethod.\n\nBug: 122291025\nTest: art/test.py -r -t 690-hiddenapi-same-name-methods\nChange-Id: Iaee780c1e87f5587f51e24b517b2b37101c729e3\n" }, { "commit": "8f1c8e56a683ff9d77ec1d1ac7f3d81458047714", "tree": "10c19d1403361bf499a576000135aaf2a692dd05", "parents": [ "181b831d1a215504a80b5c7508fc2c6e1acac40c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 08 10:34:16 2019 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jan 17 17:13:51 2019 +0000" }, "message": "ART: Some more iwyu\n\nBug: 119869270\nTest: mmma art\nChange-Id: Ie67b57c7173986009fdc7b4aa01563846d30f25f\n" }, { "commit": "49b74a8c685acfe43dd33e3f51a24c486388bee1", "tree": "2c88245108b612904d1b3e377725894574ec1eed", "parents": [ "a1ffdba7e285435cbb3538473f25eb7cb47cfd66" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jan 07 10:19:49 2019 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 09 12:38:30 2019 -0800" }, "message": "ART: Remove thread from class.h\n\nMove only dependent functions to the inl.\n\nBug: 119869270\nTest: mmma art\nChange-Id: Ic2ceba1563ac44048afec6df1bccda91ceb42782\n" }, { "commit": "62629598f845c2efec0dff652e440e8fbcce4b8a", "tree": "7af96934e7ca5313787a9b328e85e6606645c4e5", "parents": [ "d77abd9e128f60917f712b8da84bab46e3ed5870" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jan 03 16:08:31 2019 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jan 03 16:09:07 2019 -0800" }, "message": "ART: Some more IWYU in common headers\n\nForward-declare a few structs and classes. Remove unused includes.\n\nBug: 119869270\nTest: mmma art\nChange-Id: I21215cee39269b373784d6a82577039322a4a950\n" }, { "commit": "436896cd020eee6de97b619a670832b0607c3f16", "tree": "7cc5cbafcfbe6f590cb3e9d816112815a3f00668", "parents": [ "7458a7afdfe3046d962ea13dd0f6b176283505b2" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 02 15:06:53 2019 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jan 03 09:19:10 2019 -0800" }, "message": "ART: Some IWYU for IterationRange\n\nRemove unnecessary include. Forward-declare in common headers.\n\nBug: 119869270\nTest: mmma art\nChange-Id: I0a1403259ea3f1142548009deb6313dab0fed26f\n" }, { "commit": "3f1dcd39e134d994ac88dcc4f30ec8cabcd8decf", "tree": "365d20ad6b68ff1dbd4903764b63880324136e4d", "parents": [ "0f0a4e40667c87fbd4ae5480eddbfd701bfabfa2" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Dec 28 09:39:56 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 02 10:32:25 2019 -0800" }, "message": "ART: Move dex structs into own header\n\nSeparating out the structs from DexFile allows them to be forward-\ndeclared, which reduces the need to include the dex_file header.\n\nBug: 119869270\nTest: m\nChange-Id: I32dde5a632884bca7435cd584b4a81883de2e7b4\n" }, { "commit": "62f6e9055363df6b6372d86960e8d987d6c439a1", "tree": "189957252780e2d61aa0d5ef874db5794805d39b", "parents": [ "81c61bf0ef90c1fd79694aab49758e16cd0dd044" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Oct 11 18:58:50 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 13 10:52:08 2018 -0800" }, "message": "ART: Add integrity check\n\nAdd access-flags setter DCHECK code similar to the getter. For now,\ncheck that the verification-attempted flag is not removed.\n\nBug: 115834172\nTest: m test-art-host\nChange-Id: Ia1027126b2ff1a2a2e3c26f674a5f8dc8d2f0c0a\n" }, { "commit": "97b281129e556e6f3a91e9893f71ca45e7715e3f", "tree": "3e90f0148f3f6d073a29b0bfcd774e0689fb9fb4", "parents": [ "0bbac3089b5637103585b04774eea3d959c4f24d" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Dec 04 09:09:12 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 06 11:37:19 2018 -0800" }, "message": "ART: Clean up base/utils.h\n\nRemove dead code. Remove unnecessary includes.\n\nBug: 115837065\nBug: 119869270\nTest: m\nChange-Id: Ieb07546e9ebd476b7b1459fb8dd6cf5545d0af20\n" }, { "commit": "85865697ff9fabede3d64ff64cde72727c3fc4c1", "tree": "0c67639c4c286149fa3f06f5f412683e39014790", "parents": [ "b321ac28f726a7ed41f277382d85702ffdfbe00f" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Oct 30 17:26:20 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri Nov 16 11:11:20 2018 +0000" }, "message": "Runtime flags only for fast/slow hiddenapi path\n\nWith more flags being supported in the dex file, stop copying all of\nthem into ArtField/ArtMethod access flags. Instead, store the\ninformation needed to figure out whether to enter the slow path and\nretrieve full access flags from dex or not.\n\nAt the moment, the only runtime flag is kAccPublicApi assigned to all\nclass members on the whitelist.\n\nThe CL also moves hardcoded API membership of intrinsics out of\nArtMethod and into hidden_api.h, and moves ArtMethod::SetIntrinsic\ninto the .cc file.\n\nTest: m test-art\nChange-Id: Ia1cc05060dbc22341768161dfd8697c6158e803a\n" }, { "commit": "bc1e0fab934b7fb03545f08085b109d6508b17d0", "tree": "87048ec7cccd78f9819ba16d95539a8deae51659", "parents": [ "0bcbe9399dfc546db5c81b13979afc85da19d025" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 14 16:18:18 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 14 16:40:48 2018 -0800" }, "message": "Reduce class loader updating overhead.\n\nAvoid unnecessary read barrier for comparing class loader agianst\nnull. Avoid transaction check.\n\nDeleted the helper class to simplify the code.\n\nThe time goes from ~3.8ms to ~2.9ms on Maps.\n\nTest: test-art-host\nBug: 116052292\n\nChange-Id: I4a966a9bb39400ba58d7cf3e0b2c8ab4747d65ed\n" }, { "commit": "dbcb48fc0d0841f327d108246487a4ef3df556ed", "tree": "3abcfa87be3178d040739c465ce54f90dda96aa6", "parents": [ "104883b04617a850adf11f05e57c2fd29e09c83c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Nov 12 11:47:04 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Nov 12 16:35:06 2018 +0000" }, "message": "Avoid unnecessary read barriers in mirror::Class.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nChange-Id: Id98ebedf185bfc7f31eda506604c072f68b7a084\n" }, { "commit": "104883b04617a850adf11f05e57c2fd29e09c83c", "tree": "51d436588505a3ead506d633bdf97c9b4c934ae0", "parents": [ "00e96d054e2b656d4d0f99fc141d1701af4dba23" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Nov 09 17:12:23 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Nov 12 15:26:21 2018 +0000" }, "message": "Clean up primitive array helpers in Object.\n\nRefactor these helpers and avoid read barriers. Remove\nClass::Is{Int,Long}ArrayClass() and use the Object helpers\ninstead.\n\nRemove the AsByteSizedArray() and AsShortSizedArray()\nhelpers that essentially break the type system and rewrite\ntheir users, adding appropriate notes. {Float,Double}Array\nuses in Unsafe would have previously failed a DCHECK().\n\nTest: Additional test in 004-UnsafeTest.\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nChange-Id: I88b7e3df7de883f64cfc5eb437a40646f2884685\n" }, { "commit": "70f5fd0770adbef645950895309a2a63c3bde7c0", "tree": "d9794484e83858cf7e33551eb8a108e07f46b18c", "parents": [ "52ecb65ae9e1ad6fe7f58beecc88cdc08e08f0c4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 24 19:58:37 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 26 08:44:12 2018 -0700" }, "message": "ART: Add class-alloc-inl.h\n\nIn an effort to reduce the (transitive) proliferation of heap-inl\nadd a specific inline header for class instance allocation.\n\nBug: 118385392\nTest: mmma art\nTest: m test-art-host\nChange-Id: I32529f0221a836452c58687330a91ac0d5fde162\n" }, { "commit": "98104995d8dfc6fec6ec46f4cd50acd67e435687", "tree": "390e8a3f8361a894fba2d6136d906aa1c62c29f3", "parents": [ "17c5e022f48a36fd160906b9c99ae14a2f014f17" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 16 12:49:47 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 16 12:49:47 2018 -0700" }, "message": "ART: Use ObjPtr for GetSuperClass\n\nBug: 37224696\nTest: mmma art\nTest: m test-art-host\nChange-Id: If55db6e26bb1bb05a13c59b849bed17839fca6cf\n" }, { "commit": "e8fdf725e380ef31441cceb02978b4b6c4e39fe7", "tree": "1c233d405bc4a259a108faea6d11f83fe330e222", "parents": [ "9d2f7f87c77b3a3e74da21d9962b92ed63ddfbbd", "66c7459a0ea8d84c9f52738d9400831de4d2819d" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Oct 04 09:45:40 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Oct 04 09:45:40 2018 +0000" }, "message": "Merge \"Check the size of pid_t in art::mirror::Class.\"" }, { "commit": "66c7459a0ea8d84c9f52738d9400831de4d2819d", "tree": "982400de801d3d8e3fa0d3b4872aa42df81bc3d4", "parents": [ "2c3d9a443abdfd491b187e7c7660895f8bafb529" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Oct 03 19:12:45 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Oct 03 19:12:45 2018 +0100" }, "message": "Check the size of pid_t in art::mirror::Class.\n\nEnsure that `art::mirror::Class::clinit_thread_id_` (which is a C\n`pid_t` value) and `java.lang.Class.clinitThreadId` (which is a Java\n32-bit `int` value) have the same size.\n\nTest: mmma art\nChange-Id: Ief8e962a3d012be8f0aa966849496fba6a9fb3b8\n" }, { "commit": "56de89aaf2a224de8c436291e3c23a1a61315437", "tree": "397fafa3662d1abde6fecff49f24a559c2b0ab32", "parents": [ "ededf08e2f4a5df3401a5d4badb98ff3c8cb8fb9" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Oct 01 15:32:20 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Oct 03 14:38:59 2018 +0100" }, "message": "Mark most *Offset helper functions as constexpr.\n\nMaking the values compile-time constants will help to\nclean up the cpp-define-generator.\n\nTest: test.py -b -g\nChange-Id: I612a19a54062784b501bfe4f41c6642d48e0dd21\n" }, { "commit": "924ad50f0bcfc49657c5d4d23a197b2e6fdc03fe", "tree": "daf8c97184751014cfca2434fe1a9baac2c68eb8", "parents": [ "a93364cbd4ebf4b8c309c4c18f30e16c8f9a8997" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Sep 19 09:48:04 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Sep 19 13:54:22 2018 +0100" }, "message": "Refactor Class::FixupNativePointers().\n\nSimplify Class::FixupNativePointers() and fix verify flags\nusage. Also clean up verify flags for a few array functions.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 77856493\nChange-Id: I5370757827a2a5061b956034434bdc63164eda3f\n" }, { "commit": "98db89cef0be48e4b8cdd6738f1af26b0d908f45", "tree": "bea405e4f48784305c6709d3f5b0c7355ed645fe", "parents": [ "c3776b8e1220bf33a596bb69d613adf5db9d8bd1" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 07 11:45:46 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Sep 17 18:11:54 2018 +0100" }, "message": "ART: Clean up verify and read barrier flags.\n\nRemove read barrier option from Class::IsArrayClass() and\nrelated functions. Comparison with null does not need the\nread barrier and the primitive type should be the same when\nread from the from-space version of a class as when read\nfrom the to-space version.\n\nPropagate the verify flags to a few more functions and\nsimplify Class::IsClassClass().\n\nRemove pointer size check from Class::EmbeddedVTableOffset()\nin preparation for patching the boot image in place before\nthe Runtime and Thread are fully initialized.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --gcstress\nBug: 77856493\nChange-Id: I8b7a6165fa8e25efd37778e0e6ea049fc2147b24\n" }, { "commit": "d7a7f2f0bf62d6ff564dd34aa41a19f016583a78", "tree": "52bb3eb11651c275a232e9d8b970fe02a1b54dd1", "parents": [ "d176e93d66df409e2eee5343efd26b35c73fb71b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 07 11:57:18 2018 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 07 13:05:11 2018 -0700" }, "message": "Remove read barriers in Class::IsClassClass and Object::IsClass\n\nRemove unnecessary read barriers since Class.class is not movable.\n\nTest: test-art-host\nBug: 114413743\n\nChange-Id: Id68562606bab75bf5ba99a7e8a38e7db9d46df1d\n" }, { "commit": "bbc6e7edb5fca4a61ac53dd9bce79cb4f0bb3403", "tree": "0fbce767bc383358cf4cd65aafc74140e1850982", "parents": [ "19379b58bd433da91230e4fe6cd96e7416d16adc" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Aug 24 16:58:47 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 28 11:06:07 2018 +0100" }, "message": "Use \u0027final\u0027 and \u0027override\u0027 specifiers directly in ART.\n\nRemove all uses of macros \u0027FINAL\u0027 and \u0027OVERRIDE\u0027 and replace them with\n\u0027final\u0027 and \u0027override\u0027 specifiers. Remove all definitions of these\nmacros as well, which were located in these files:\n- libartbase/base/macros.h\n- test/913-heaps/heaps.cc\n- test/ti-agent/ti_macros.h\n\nART is now using C++14; the \u0027final\u0027 and \u0027override\u0027 specifiers have\nbeen introduced in C++11.\n\nTest: mmma art\nChange-Id: I256c7758155a71a2940ef2574925a44076feeebf\n" }, { "commit": "9ddef18ae95859a985e7a0de7e22999fcbc28e07", "tree": "ff38945fc61f1e32717a18b0a6901c5d11c33ccb", "parents": [ "0366f3251c3078a0161d178e3b0afd5efc4c84c0", "bcf175247272d0e321c8d988c3c01c123b56e36e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 04 09:14:42 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jun 04 09:14:42 2018 +0000" }, "message": "Merge \"ObjPtr\u003c\u003e-ify array allocations.\"" }, { "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": "a5359a3bf47c4b086fe717ae35dfa6be779824bd", "tree": "08e786e30b2d1fbe886c688b5545cff9906b057f", "parents": [ "22bd3c969533986640e17e20f38975b22f30af1c", "317892b756cd2a87c01928b09e99e020c86dea10" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Jun 01 09:55:34 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jun 01 09:55:34 2018 +0000" }, "message": "Merge \"Remove static GcRoot\u003c\u003es from Class and ClassExt.\"" }, { "commit": "317892b756cd2a87c01928b09e99e020c86dea10", "tree": "bb337ff8cd81c049e781afbd8c68b72ca30907fc", "parents": [ "1539a11150ca9c28fec353b38508267c41d7814b" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 31 11:11:32 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 31 13:56:49 2018 +0100" }, "message": "Remove static GcRoot\u003c\u003es from Class and ClassExt.\n\nAnd clean up gc_root includes in runtime/mirror/.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: Ib5c42a3a892ced4440720350a63a94bcf3a1ca75\n" }, { "commit": "d32ead213f01c16c3ddd7c7a33c0f0d259e4c4dc", "tree": "bb68a6355872dadaa1e7f56a224cda40d77f0738", "parents": [ "08231f6cb3095a7dbde29299a7da5413a5f992e4" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed May 30 17:38:21 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu May 31 12:46:40 2018 +0100" }, "message": "Aesthetic changes in art::mirror::Object and art::mirror::Class.\n\nTest: mmma art\nChange-Id: I1c7bf2e3b0420238ee2ae249aca9b7fce492a3e6\n" }, { "commit": "6834d3414308e3d536bc685dbb3d60fe70186f23", "tree": "7c579de52d3acc5cc4d02babf25c7fcb2b94793b", "parents": [ "c7aa87e1666ac48ddf9149cfdfd64b026b3969e5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 25 13:12:09 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 25 14:12:31 2018 +0100" }, "message": "Remove mirror::Reference::java_lang_ref_Reference.\n\nAnd some #include cleanup after previous CLs removing\nstatic GcRoot\u003c\u003es.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: I56b34637f9d793ed7680696a1eeedd07d061ad00\n" }, { "commit": "59a49a6fe102b26aacda906b6a2c466d84819c73", "tree": "f33b21de298b1943ae3483ba877cdbf50313e049", "parents": [ "48af3b39cd7c8abaa7f9671191ec8260630239b2" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed May 09 10:58:27 2018 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu May 10 12:38:11 2018 +0100" }, "message": "Implement VMDebug API to mark a class \"platform\"\n\nAdd a native implementation of VMDebug.allowHiddenApiReflectionFrom\nwhich takes a Class object and sets a new access flag\nkAccSkipAccessChecks on the corresponding mirror::Class instance.\n\nThe function will throw a SecurityException if the process has not\nbeen forked as Java debuggable.\n\nhiddenapi::IsCallerInPlatformDex is extended to take an optional\nargument of the mirror::Class of the caller. If it is set and\nthe class has kAccSkipAccessChecks, the caller is allowed to\nacccess non-SDK APIs.\n\nNote that the mirror::Class of the caller is only provided for\nreflection and JNI. The access flag is ignored for other means of\naccess.\n\nBug: 64382372\nTest: N/A\nChange-Id: I2bf0ca7dcb45c17fe91eb2d421c947b892bd6fec\n" }, { "commit": "be4c2bd892bd167a50b4dfa7133e70a809197698", "tree": "150898533382fb98c160e265a5f31f66ded754f0", "parents": [ "bfce631960ae2ab381180beb55cf34ab3b445aaa" ], "author": { "name": "Alexey Grebenkin", "email": "a.grebenkin@samsung.com", "time": "Thu Feb 01 19:09:59 2018 +0300" }, "committer": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Tue Mar 13 17:01:21 2018 +0300" }, "message": "Fix dangling SingleImplementations left after class unloading\n\nTest: make test-art-host, manual using sample code\n\nbug: 73143991\n\nChange-Id: I4d56b39c69d4ed60266a8b90b9e9d18fba7b8227\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": "58143d2c47734c46c1fa4855cb603c24f2d15454", "tree": "261bd69b379e5a39e39e3c15947960e53f389d98", "parents": [ "a1b035b963d091ca6824f285c33de69c6e16ac77" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Tue Feb 20 08:44:20 2018 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Mar 05 15:21:52 2018 +0000" }, "message": "ART: Fixes for constructor parameter annotations\n\nSynthesize empty parameter annotations for implicit parameters on\nconstructors. Reflective methods for recovering parameter annotations\nexpect them to be present though they may not be present in the DEX file.\n\nBug: b/68033708\nTest: art/test/run-test --host 715\n\nChange-Id: I0827c7e71ff7c7e044fc9dd6c5aac639a0e1a4c6\n" }, { "commit": "67bf42e89592c3a1c648f927f2ce3ccb189a1161", "tree": "054d5b7adf7cc62d4d2a2118a70c0fbdd1751610", "parents": [ "d961043ff1dd6fddb68aa90c1f939cfafec24219" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Feb 26 16:43:04 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Feb 28 15:08:21 2018 -0800" }, "message": "Header library to remove dependence on runtime/\n\nAdd a new header library to remove libdexfile and others\u0027 dependence on\nruntime (typically runtime/base) includes in libdexfile. Also a small step\nto tease dexlayout and profman away from relying on these as well.\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make -j 50 test-art-host-gtest\n\nChange-Id: I38e2fe399a75f4bc6318c77a71954c00ea73ec2b\n" }, { "commit": "7287c4d75e38e7a01c7ea3fdebb47d9cfe128a56", "tree": "2b43d1b334e05e1c5719fee3f2a16bebaff059d9", "parents": [ "1efa7cb8c415f4df8a248242cd773a1fc671878e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 15 10:41:07 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 15 15:36:21 2018 +0000" }, "message": "Add Class.getPrimitiveClass().\n\nImplement the native part in ART and remove the native parts\nof the previously needed workaround for java.lang.Void.TYPE.\n\nCompanion libcore/ change:\n https://android-review.googlesource.com/619326\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nBug: 73346078\nChange-Id: Icfb201a1f08d8ff2a65c58f7c0ec25aa2296db29\n" }, { "commit": "305c38b395eddff7ca001fa7bdb621397033184d", "tree": "809965b36e35d8b5a06229a4904c7943525ea54c", "parents": [ "3bffbe8a8288cf01b7248fffdd320bf688529bcb" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 14 11:50:07 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 14 16:16:05 2018 +0000" }, "message": "Disable bitstring type check runtime hooks.\n\nIntroduce a build flag for the bitstring type check, put\nruntime hooks behind the flag and set the flag to false.\n\nAlso add bitstring initialization for proxy classes, a test\nand a benchmark for the type checks.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --interpreter\nTest: Repeat with kBitstringSubtypeCheckEnabled \u003d true.\nBug: 73299705\nChange-Id: Ibcd88a828c7addc0473d8c428818734f80226b19\n" }, { "commit": "8c0961f9e061ee4b04c1c4ba8ad5cca13bcf884d", "tree": "f6e5a970a6bcc0935e150cb6c28a520bf31ba964", "parents": [ "2b1a21ed3581f77f34d582507177ebce33565534" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 23 16:11:38 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Jan 24 09:24:29 2018 -0800" }, "message": "Move missed files to libdexfile\n\nReduce the dependencies on utf and utils in preparation for separate\ndirectory.\n\nBug: 22322814\nTest: make -j 50 test-art-host\n make -j 50 dexdump2 dexlist\n\nChange-Id: Icdecf895dafec63ef903514eef79d459abc14925\n" }, { "commit": "dc682aa9d0eae1a851af059434adb6f6cf8f06f8", "tree": "f93f00493ee5887b05b42a6a5dd99eb6794daad4", "parents": [ "d6b7e8c63f8eca25460f56f66dcae15eaa897ff0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jan 04 18:42:57 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jan 10 14:30:26 2018 +0000" }, "message": "Use 28 bits for type check bit string.\n\nAnd reverse the order of fields in the Class::status_. This\navoids generated code size increase:\n - ClassStatus in high bits allows class initialization\n check using \"status_high_byte \u003c (kInitialized \u003c\u003c 4)\"\n which is unaffected by the low 4 bits of LHS instead of\n needing to extract the status bits,\n - the type check bit string in the bottom bits instead of\n somewehere in the middle allows the comparison on ARM\n to be done using the same code size as with the old\n layout in most cases (except when the compared value is\n 9-16 bits and not a modified immediate: 2 bytes less for\n 9-12 bits and sometimes 2 bytes more for 13-16 bits; the\n latter could be worked around using LDRH if the second\n character\u0027s boundary is at 16 bits).\n\nAdd one of the extra bits to the 2nd character to push its\nboundary to 16 bits so that we can test an implementation\nusing 16-bit loads in a subsequent CL, arbitrarily add the\nother three bits to the 3rd character. This CL is only\nabout making those bits available and allowing testing, the\ndetermination of how to use the additonal bits for the best\nimpact (whether to have a 4th character or distribute them\ndifferently among the three characters) shall be done later.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nTest: testrunner.py --target --optimizing\nBug: 64692057\nChange-Id: I38c59837e3df3accb813fb1e04dc42e9afcd2d73\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": "2c64a837e62c2839521c89060b5bb0dcb237ddda", "tree": "65475ed2e313ff17354e741bac7e9c85739b8b95", "parents": [ "6cd0005698181e4cef2247b632d396e605d58fa3" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jan 04 11:31:56 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jan 05 11:27:50 2018 +0000" }, "message": "Change ClassStatus to fit into 4 bits.\n\nIn preparation for extending the type check bit string from\n24 to 28 bits, rewrite ClassStatus to fit into 4 bits. Also\nperform a proper cleanup of the ClassStatus, i.e. change it\nto an enum class, remove the \"Status\" word from enumerator\nnames, replace \"Max\" with \"Last\" in line with other\nenumerations and remove aliases from mirror::Class.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nTest: testrunner.py --target --optimizing\nBug: 64692057\nBug: 65318848\nChange-Id: Iec1610ba5dac2c527b36c12819f132e1a77f2d45\n" }, { "commit": "b0a6aeee250945b1d156ebab94053380f2e5a3c5", "tree": "7ddb52bb20997b8f1c81be3df2250949a088e697", "parents": [ "0db16e00e3927445585a588499731c58c1ae1bef" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Oct 27 10:34:04 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Nov 10 08:30:36 2017 +0000" }, "message": "Record @{Fast,Critical}Native in method\u0027s access flags.\n\nRepurpose the old kAccFastNative flag (which wasn\u0027t actually\nused for some time) and define a new kAccCriticalNative flag\nto record the native method\u0027s annotation-based kind. This\navoids repeated determination of the kind from GenericJNI.\nAnd making two transitions to runnable and back (using the\nScopedObjectAccess) from GenericJniMethodEnd() for normal\nnative methods just to determine that we need to transition\nto runnable was really weird.\n\nSince the IsFastNative() function now records the presence\nof the @FastNative annotation, synchronized @FastNative\nmethod calls now avoid thread state transitions.\n\nWhen initializing the Runtime without a boot image, the\nWellKnowClasses may not yet be initialized, so relax the\nDCheckNativeAnnotation() to take that into account.\n\nAlso revert\n https://android-review.googlesource.com/509715\nas the annotation checks are now much faster.\n\nBug: 65574695\nBug: 35644369\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nChange-Id: I2fc5ba192b9ce710a0e9202977b4f9543e387efe\n" }, { "commit": "86083f7cd118f3d6c757191e83b4e4abaabdc5d7", "tree": "8e5b81ae0d09d41bfd90284a1b6b16b2332435e5", "parents": [ "495e783c9180c3fc033ce459ee0a783e633f7754" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Oct 27 10:59:04 2017 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Nov 07 14:47:51 2017 -0800" }, "message": "runtime: Bitstring implementation for subtype checking (4/4).\n\nIntegrate the previous CLs into ART Runtime. Subsequent CLs to add\noptimizing compiler support.\n\nUse spare 24-bits from \"Class#status_\" field to\nimplement faster subtype checking in the runtime. Does not incur any extra memory overhead,\nand (when in compiled code) this is always as fast or faster than the original check.\n\nThe new subtype checking is O(1) of the form:\n\n src \u003c: target :\u003d\n (*src).status \u003e\u003e #imm_target_mask \u003d\u003d #imm_target_shifted\n\nBased on the original prototype CL by Zhengkai Wu:\nhttps://android-review.googlesource.com/#/c/platform/art/+/440996/\n\nTest: art/test.py -b -j32 --host\nBug: 64692057\nChange-Id: Iec3c54af529055a7f6147eebe5611d9ecd46942b\n" }, { "commit": "2466d627fc877ed328226ab001e7dd061ecbc49b", "tree": "4e6149e458f0113a9d3567c789f94cdb9e708ab4", "parents": [ "2dc2bcc09aff444fad4d0c0ae12458bde5716699" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 26 13:52:26 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 26 15:36:22 2017 -0700" }, "message": "Pass methods array to GetMethodsSliceRangeUnchecked\n\nAim to prevent cases where a different method array is used in the\ncaller than is used to compare end_offset against the number of\nmethods.\n\nTest: test-art-host\nBug: 66940748\nChange-Id: I53e6ed8192babfa3bbb0f2cde08a1e8e3008426a\n" }, { "commit": "04c6ab9266876137c9b46b69391aa2820cd56600", "tree": "1580e08b7a8e73197c64b499540ec613b755a6e1", "parents": [ "e2abbc604ce003c776c00ecf1293796bb4c4ac5a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 08 21:49:14 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 18 11:36:24 2017 -0700" }, "message": "ART: Forward declarations for InvokeType\n\nUse forward declarations in other header files.\n\nTest: m\nChange-Id: I8ef4492b6c48ff20d4d77d0516eacd362ed210bc\n" }, { "commit": "cf83aaa468fa1b87746ac2e8ca95e80aa8c29a81", "tree": "2e74a2ac49fab21ce80f5b7e11a7c5e77097d735", "parents": [ "de07216fe9824f68ba76bf4c14158ec4f55af2cb" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Sep 08 03:21:54 2017 +0000" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Sep 08 03:21:54 2017 +0000" }, "message": "Revert \"mirror: Make Class::Status 64-bit field\"\n\nThis reverts commit de07216fe9824f68ba76bf4c14158ec4f55af2cb.\n\nReason for revert: \u003cINSERT REASONING HERE\u003e\n\nChange-Id: Id86866a14ad7caf8e23181be19fc750cb3f46667\n" }, { "commit": "de07216fe9824f68ba76bf4c14158ec4f55af2cb", "tree": "a26c44ab0a2c45b8bcb24976ef74d28ff867d919", "parents": [ "82629c9182dffb823f05dec30f7ac72cf8fb3ba5" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Sep 07 14:42:46 2017 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Sep 07 15:15:44 2017 -0700" }, "message": "mirror: Make Class::Status 64-bit field\n\nTemporarily merge this CL, then revert.\n\nWe need it to submit Zhengkai\u0027s CL to go/lem because of the libcore\nchange.\n\nBug: 64692057\nTest: art/test.py -j32 --host\nChange-Id: If104832c66973e6ffe8baf81ad1b32237ee78a80\n" }, { "commit": "ee5303f76ef167714a6a04d3abc502584ac5e103", "tree": "7f414df9d48dda94872421de4192daf6ec731681", "parents": [ "c101222c854a0c476f5b6ae64e20adbd38126a3c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Aug 31 15:34:42 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 05 10:46:07 2017 -0700" }, "message": "ART: Move Class::Status to ClassStatus\n\nMove the enum to its own file to lower include burden on others.\nThis is an intentionally small and localized change. Users will\nbe cleaned up in a follow-up.\n\nBug: 63467744\nBug: 65318848\nTest: m test-art-host\nChange-Id: If3bfe6953973e24623e78a2a8dcc69b50117aa3c\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": "ba118827465d12177f3996e50133960087b1c916", "tree": "f39728cdafc7810004d51c0bef2728b98993daa9", "parents": [ "64a102dde8c5daad83b991710decb418ce43aac5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 12 15:41:56 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 20 13:35:06 2017 +0100" }, "message": "ART: Change method lookup to be more consistent to JLS and the RI.\n\nThe method lookup for different invoke types was previously\nwidely different and didn\u0027t work well with the dex cache\nmethod array where we have only a single slot for each\nMethodId. The new behavior is to perform the same lookup for\nall cases, distinguishing only between interface and\nnon-interface referencing class, and to further align the\nbehavior with the JLS and the RI. Where the JLS conflicts\nwith the RI, we follow the JLS semantics.\n\nThe new lookup for class methods first searches the methods\ndeclared in the superclass chain (ignoring \"copied\" methods)\nand only then looks in the \"copied\" methods. If the search\nin the superclass chain finds a method that has not been\ninherited (i.e. either a private method or a package-access\nmethod where one of the classes in the chain does not belong\nto the same package, see JLS 8.4.8), we still search the\n\"copied\" methods as there may actually be a method inherited\nfrom an interface. This follows the JLS semantics where\ninherited methods are included in the search (JLS 15.12.2.1)\nbut conflicts with the RI where the private or\npackage-access method takes precedence over methods\ninherited from interfaces.\n\nNote that this search can find an accessible method that is\nnot inherited by the qualifying type, either for a package\naccess method when the referrer is in the same package but\nthe qualifying type is in another package, or for a private\nmethod where the referrer is in the same class but the\nqualifying type is actually a subclass. For the moment we\nallow such calls and we shall consider whether to throw\nan IncompatibleClassChangeError in this situation in future\nto comply with JLS 15.12.4.3.\n\nThe new lookup for interface methods searches the interface\nclass, then all the superinterfaces and then the\njava.lang.Object class, see implicitly declared methods in\ninterfaces, JLS 9.2. The search for the maximally-specific\nnon-abstract superinterface method is not yet implemented,\nbut the difference should be difficult to observe as the\nusual subsequent call to FindVirtualMethodForInterface()\nshould yield the same result for any matching method.\n\nThe new test 162-method-idx-clash exposes several cases\nwhere we previously completely messed up due to the effects\nof the DexCache, or where we were out of line with the RI.\nIt also tests a case where the JLS and the RI disagree and\nwe follow the JLS.\n\nTest: art/test/run-test --host --jvm 162-method-resolution\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: testrunner.py --host --interp-ac\nTest: Nexus 6P boots.\nTest: testrunner.py --target\nBug: 62855082\nBug: 30627598\nChange-Id: If450c8cff2751369011d649c25d28a482a2c61a3\n" }, { "commit": "0cb172874481f736c6b7c491dd621166cc25561b", "tree": "2408750b6e08c686b89cee5df0ddae736a775092", "parents": [ "890045e5a768257d8def42827a09a516ebe3e07e" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Jul 12 14:51:49 2017 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Jul 17 14:09:56 2017 -0700" }, "message": "Do superclass validation at compile time and log with new class status.\n\nTries to perform superclass validation for classes that are resolved,\nbut not initialized at runtime. If successful, saves the result in the\noat file with a new class status. At runtime, the superclass validation\ncan be skipped during class initialization, saving some time and\nreducing string accesses.\n\nResults show savings of 50kB PSS in maps on startup, with slight\ndecrease in startup time.\n\nMaps (average of 100 runs)\nBefore: dex 9941.3 odex 15159.8 total 25101.1 launch 908\nAfter: dex 9897.4 odex 15155.7 total 25053.1 launch 906.6\n\nBug: 63456114\nTest: mm test-art-host\nChange-Id: If67a4a49d61781b6d561c26118d7e0c6b9cc0d6f\n" }, { "commit": "6d3e76884710e639e79a46d505395bc83ad5ec7d", "tree": "3b51520701c740ccb69ce38483c47c25105dd40b", "parents": [ "00cca3a275562d110a8b35094b9b12fac37f67ab" ], "author": { "name": "Chang Xing", "email": "chxing@google.com", "time": "Tue Jul 11 10:31:29 2017 -0700" }, "committer": { "name": "Chang Xing", "email": "chxing@google.com", "time": "Tue Jul 11 17:05:12 2017 -0700" }, "message": "Abstract SetField functions with Transaction check\n\nAbstracted several functions which set objects\u0027 fields with a check to\ntest whether transaction is active to determine one template argument,\nthis make more object modification operations support transaction.\n\nReplaced some function calls with transaction checking version to\nsupport clinit for app image to run under transaction.\n\nTests shows not breaking anything, testcase cover this situation will be\nadded shortly.\n\nTest: make test-art-host -j64\nChange-Id: Ic15a4428a7c7cce6832651e1894eb760354de009\n" }, { "commit": "f79aa7fc107c661e5a6d6ffd2a33221864fa1fa3", "tree": "07bf40e8b5d75454ffe52e82f4d4e5300386fe37", "parents": [ "4a77b1e96733be419c0cb571448e8590c803bd91" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jul 04 16:58:55 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jul 05 16:32:24 2017 +0100" }, "message": "Small refactoring of field/method access checks.\n\nAnd some cleanup to reduce the size of the upcoming\nmethod resolution rewrite CL.\n\nTest: m test-art-host\nTest: testrunner.py --host\nTest: testrunner.py --host --interp-ac\nBug: 62855082\nChange-Id: I5bbf0a92d75fe9de6982b80280cd5877fcc37700\n" }, { "commit": "9ac7749591c4341f5bf7d17af59d3aa6366487f1", "tree": "509126cecfa61b338c6c13d554f9583b1e99cf36", "parents": [ "e1993c7d74fc60c468f55da78da490a5ce9c2f91" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jun 14 18:07:03 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jun 15 10:11:26 2017 +0100" }, "message": "Clean up ArraySlice\u003c\u003e.\n\nRemove constructors that take ranges (one of them created\nambiguity with the constructor taking just the length) and\nadd SubArray() to serve as a replacement API.\n\nRemove the AsRange() function as the translation to the\nIterationRange\u003c\u003e seems totally unnecessary, we can simply\nreturn ArraySlice\u003c\u003e from the functions that used it.\n\nRemove the At() function as the comparison with the\nstd::vector\u003c\u003e::at() is confusing given that we do not\nuse C++ exceptions.\n\nAdd some standard container API, i.e. nested types and\nmore functions.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nChange-Id: Ifcd39222cf8f36b7ebf8029398ed7fc67aafcaf0\n" }, { "commit": "3b7dc35f4e5c4d86c73b6784b7ee0df701c68ec2", "tree": "1e3a56846ec63148142ac6fb1fef214129f4a05e", "parents": [ "8228cdf4ad6322ec8133564aaa51f966d36c0f17" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jun 06 20:02:03 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 08 13:13:20 2017 -0700" }, "message": "ART: Clean up allocator.h\n\nMove the single-use typedefs to their users. Remove now-unused\nincludes. Fix up transitive includes.\n\nTest: m\nChange-Id: I953d774b28f1e4f3191f96943e3a69ce66aa398a\n" }, { "commit": "57cf00bde719ddc84a6015b107b90a20169e3099", "tree": "1522952470490e6b107d07649f511ef09f9abd32", "parents": [ "40c6710dfa0d10d34dcca8ed92c214faba36c833" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 05 17:15:32 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 05 18:29:04 2017 -0700" }, "message": "ART: Remove old object_callbacks.h includes\n\nRemove unused includes.\n\nTest: m\nChange-Id: I70c227d32c6900904e186b975d1f2131b718d93d\n" }, { "commit": "c6ea7d00ad069a2736f603daa3d8eaa9a1f8ea11", "tree": "a6d3332a8592fb806841314d55b206b573d37d86", "parents": [ "d68677c5fde1ace16ea58d65733776c954e7acb4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Feb 01 16:46:28 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Apr 21 08:40:33 2017 -0700" }, "message": "ART: Clean up art_method.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. Introduce new -inl files\nfor method handles and reference visiting.\n\nTest: source build/envsetup.sh \u0026\u0026 lunch aosp_angler-userdebug \u0026\u0026 mmma art\nTest: source build/envsetup.sh \u0026\u0026 lunch aosp_mips64-userdebug \u0026\u0026 mmma art\nChange-Id: I8f60f1160c2a702fdf3598149dae38f6fa6bc851\n" }, { "commit": "8f2b925473cfdc7650cef407102957befe0c6bb5", "tree": "4af3395adaf03cb4a60cd40e7b3ee4196645eb9e", "parents": [ "de9b8d63ef548d56de9c9d4657a7b916050d81ad", "6b2dc3156a2140a5bfd9cbbf5d7dad332ab5f5bd" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Fri Mar 17 09:05:55 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Mar 17 09:05:56 2017 +0000" }, "message": "Merge \"ART: Get rid of most of java.lang.DexCache.\"" }, { "commit": "4f2e957b07c8b6e48bbd7593bb64f3c5214fa489", "tree": "efcfcaaa1efdc567449c406e7107da2e94c127c2", "parents": [ "ed883a11b8cd7f3712bd30fca1e2b0fb641bff63" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Mar 16 13:13:31 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Mar 16 14:23:46 2017 -0700" }, "message": "Ensure the GC visits Obsolete Methods\n\nWe were previously not visiting obsolete methods during GCs. This\ncould lead to the use of stale pointers.\n\nBug: 36335999\nTest: ./test/testrunner/testrunner.py --host --interp-ac --gcstress -j40\n\nChange-Id: I2b5c7c75b29f9037204a860501fcdb78104b5e7a\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": "7ec0fde07816df30b5ab58d1a7a18aa4b68a374e", "tree": "e24f2d5ed8edb5600ca7fa02d52a12581a21b9fe", "parents": [ "335f644f617d9837bc44219c70a2943f36c3f496" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Wed Mar 08 12:43:30 2017 +0000" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Wed Mar 08 13:25:14 2017 +0000" }, "message": "Track libcore change 2c0c8889ff415431e3c2d6d5ac95748f8b284384.\n\nUpdate the size of Class\u0027 vtable.\n\nTest: make test-art-host\nChange-Id: I6613a3f1924a2347e0e4f9c4db31d126ab4deacc\n" }, { "commit": "7cc3ae5705416bd8fc4b7096904e2871aa761e73", "tree": "7828e0346d1961b1454a56fa7759b72c13e1b67a", "parents": [ "a86b359441f349d9057d05c32c88c375d794cf86" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 07 14:33:37 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 07 17:44:43 2017 +0000" }, "message": "Return the right value in VerifyClass.\n\nWe used to return kNoFailure when a class was already processed.\nBut it could have had soft failures.\n\nAlso remove IsCompileTimeVerified to avoid future confusions and\nintroduce ShouldVerifyAtRuntime.\n\nAdd some more checks to make sure we record the right things in\nthe vdex file.\n\nbug: 33845394\ntest: verifier_deps_test test-art-host\nChange-Id: Iff11a96e825c85db416083413761981515f405b7\n" }, { "commit": "f85fc6bd9d65ef4689d6af797adb889da8cf15e3", "tree": "8fe8d622e30a550eb3a336bc12f925eb40828e6b", "parents": [ "fe076a51b0498c2771341cc09a77db15b437328f", "8ca4e4e7a55f7a7079ec1f95a3188d9df434704b" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Feb 06 10:13:08 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Feb 06 10:13:09 2017 +0000" }, "message": "Merge \"Clean up art::mirror::Class::SetSuperClass.\"" }, { "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": "8ca4e4e7a55f7a7079ec1f95a3188d9df434704b", "tree": "e8bb240938fdadcbe363ac00598ea43621c2c7a8", "parents": [ "6058b682234060120c4407a1afad1182fa5a1baf" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Feb 03 15:12:38 2017 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Feb 03 17:35:54 2017 +0000" }, "message": "Clean up art::mirror::Class::SetSuperClass.\n\n- Move its definition from runtime/mirror/class.h to\n runtime/mirror/class-inl.h.\n- Prevent debug code from running in non-debug mode.\n\nTest: test-art-host\nChange-Id: I981cb9aa5c59c75c6a1c4d2145185cf8b147e2db\n" }, { "commit": "137cdfa1e8a3810fb86d3298ec10036c0491d13f", "tree": "db568fce8feec0ae6910ff1150cf46f08fe7e3e5", "parents": [ "fbf47ea64a9f797a82030e919fa4f085c9eb5b28" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 26 14:03:11 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 26 17:04:55 2017 -0800" }, "message": "Avoid read barriers for ArtMethod::GetDexFile\n\nShows up in pmd benchmark from Class::FindDeclaredDirectMethod and\nClass::FindDeclaredVirtualMethod. There are still calls to\nIsProxyMethod that could probably be eliminated.\n\nReadBarrier::Mark goes from 12.39% to 3.45% according to perf.\n\nTest: test-art-host\n\nChange-Id: I6a4f2fa2d68bf5f393f83b9b70e8d6fcc9dbdaa2\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": "f9bf250cc642c4e3e0c6904550fe922d3359a737", "tree": "953351c7e13b45e9246625ad81bce708788d96f9", "parents": [ "b73e659be3fb6474ff3a993cd25ecbfa1cc4715c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 14 14:59:04 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 15 14:52:22 2016 +0000" }, "message": "Remove dex cache string from mirror::Class.\n\nThe compiled code does not need it anymore.\n\nTest: test-art-host test-art-target\nChange-Id: I71c08ce12d450ab2b2cd018330d42a8b8c9e3da2\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": "e51ca8bfa8a193b64901ad214842f213adca92eb", "tree": "b07b2534edc592d68b5461a3bdb8a31241892d54", "parents": [ "eb9c58e88f8e0f4a0a413efce04b49b183ad23e3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 22 14:49:31 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 29 21:05:51 2016 +0000" }, "message": "Move inline caches GC handling in JitCodeCache.\n\nMake the classes they hold weak references and visit\nthem during SweepJitRoots.\n\nThis fixes the newly introduced deadlock:\n\nThread1:\n1) Lock JitCodeCache lock to create Profiling info for\n ArtMethod m.\n2) m is a copied method, we need to track the actual holder,\n needing to decode a weak reference.\n3) Weak references are not accessible due to GC.\n\nGC Thread:\n- Disallow weak reference access.\n- Wait for checkpoint.\n\nThread2:\n- Try to lock JitCodeCache lock\n- Deadlock, as Thread1 owns the JitCodeCache lock.\n\nTest: test-art-host\nbug: 31289185\nbug: 33198826\n\nChange-Id: I7ee17631015450ace8d2a0264415a81c5a902bb8\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": "0273ad1f702d7094b90ebb62c21c50b6a9568ab0", "tree": "759ae8b9f5bad5ee62e9e04d7dd5b35d307f627f", "parents": [ "21b880ce80f63ddeeab7afaffe0de6934e602551" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Nov 02 11:19:31 2016 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Nov 03 09:17:50 2016 -0700" }, "message": "Remove lock from ClassExt installation procedure.\n\nWe were using a lock on the class to ensure that we avoid races in\nsetting the ext_data_ field of a class object. We replace this with a\nCAS of the field in order to prevent deadlocks.\n\nTest: mma test-art-host\nChange-Id: Ie436ff9526f2c3b38a9af49c5606a7cee6d718f1\n" }, { "commit": "d62515891b15e73bc8eebb55cc09cb4dec4fcf21", "tree": "20c485231c93fa83384312e428202d8140dfa09d", "parents": [ "77a0e4541a29c26dba21ed8c1ed1c29ef8d0ce4c" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Oct 31 11:12:30 2016 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Nov 02 12:55:10 2016 -0700" }, "message": "Add ClassExt mirror.\n\nWe add the ClassExt type to hold values in Class that are not always\nneeded without making the Class type larger. For the initial change we\nmove the verifyError field into this type. It is expected to grow as\ntime goes on.\n\nTest: mma test-art-host\n\nChange-Id: I1d97df2e1267203841ad47b2effcb66dd76ac12a\n" }, { "commit": "9f54896e96a363a5b1ddb2f1b34b58ad4f32ac10", "tree": "5ea316acc6fd641f9b9a70a0e7f83deb3fcd3c9b", "parents": [ "dd7c25e5af5b88a1bc58ed7de076037e593b176d", "94bee022594187158e6fb2050d5198f973375336" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Wed Nov 02 09:32:17 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Nov 02 09:32:17 2016 +0000" }, "message": "Merge \"MethodHandles: Track libcore change cc4ded.\"" }, { "commit": "2ecfd270adb29861b493de51cfb8670e276e0c5e", "tree": "6aa5d0062a59f43b2c188fe0b1f11d56aedc588a", "parents": [ "19dfeb0e725bca7fd85b4a8a96ce9f3d30408aa3" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 01 10:10:05 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 01 15:49:51 2016 -0700" }, "message": "Add interface fast path to art_quick_check_cast for X86_64\n\nX86_64 CC ritzperf results from perf:\nart_quick_check_cast: 0.44% -\u003e 0.76%\nartIsAssignableFromCode: 1.78% -\u003e 0.11%\n\nAdded stub test.\n\nBug: 32577579\n\nTest: test-art-host\n\nChange-Id: I5ed5675c4674fac8eed8826eb50527f4876e5f07\n" }, { "commit": "94bee022594187158e6fb2050d5198f973375336", "tree": "d38d958f197e318ae512117ffd097746191d35ea", "parents": [ "bda454e9a09018cde1ea2ee10f0fdd5bf2ad756b" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Tue Nov 01 10:57:15 2016 +0000" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Tue Nov 01 12:51:04 2016 +0000" }, "message": "MethodHandles: Track libcore change cc4ded.\n\nTrack increase in vtable size of java.lang.Class, and add additional\ntests for the method that resulted in the increase as well as its\ncaller.\n\nBug: 30550796\nTest: make test-art-host\nChange-Id: I0ea6cbf8840d45b74f3a50e4c9df2af5397d41e3\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": "1a5337fff2cc6cb9d563c8b32aca75f485d23373", "tree": "03616e90a91c5c4c1bd7ff03c3d193c3b24bf0db", "parents": [ "c8b722b3cebff245d614e95d1849de45ef7bdf06" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 13:48:23 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 17:37:56 2016 -0700" }, "message": "Move mirror::ObjectArray to ObjPtr\n\nAlso reduce calls to ObjPtr::Ptr.\n\nBug: 31113334\n\nTest: test-art-host -j32\n\nChange-Id: I73e1b356972bb079b66332f00f7f07451601044e\n" }, { "commit": "28bd2e4f151267b34b8e1eb19c489d8d547bbf5c", "tree": "d99ae3fe74ea63b83091898d830d3efe68cd479d", "parents": [ "6e5fa09510c7280168e040382d27dd8b55760d9a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 04 13:54:57 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 10:59:28 2016 -0700" }, "message": "Move mirror::Class to use ObjPtr\n\nLeave the return types as non ObjPtr for now. Fixed moving GC bugs\nin tests.\n\nTest: test-art-host\n\nBug: 31113334\n\nChange-Id: I5da1b5ac55dfbc5cc97a64be2c870ba9f512d9b0\n" }, { "commit": "9d156d500801accee919b6d51e22d6ddcdcd4a05", "tree": "92129562c0ba1673c660297f26444a4a2ad31459", "parents": [ "078483d4dfd049d5b3f192a99e2dfdc355e4754f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 06 17:44:26 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 12 09:49:49 2016 -0700" }, "message": "Move Heap parameters to ObjPtr\n\nDeleted some unused object dumping code.\n\nTest: test-art-host\n\nBug: 31113334\n\nChange-Id: I747220caafe6679591fd4b361d7f50383a046164\n" }, { "commit": "b890130a66e167404a9a60cf0893a015538778ca", "tree": "e50899e754132f3281c932c31203f8c34c8ca85f", "parents": [ "3398c7874e002beaa6c2b2fadf183e7d1ddad23a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 30 10:27:43 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 03 16:00:55 2016 -0700" }, "message": "Use ObjPtr for ResolvedFieldAccessTest and ResolvedMethodAccessTest\n\nAlso added LookupResolvedType that is guaranteed to not do thread\nsuspension but deals with multidex since GetResolvedType will\nreturn null if the type was resolved in another dex file.\n\nAdded test.\n\nBug: 31113334\n\nTest: test-art-host CC baker\n\nChange-Id: I50493bca7d8ce9760546c3116b717484c62c47a4\n" }, { "commit": "3398c7874e002beaa6c2b2fadf183e7d1ddad23a", "tree": "7db754b5b0c3a6c68bc1f91d7cb2477f534d2ad3", "parents": [ "462e21cc8d28a4d48342cf0f634bd1ecc63647d7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 30 10:27:43 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 03 14:06:27 2016 -0700" }, "message": "Move ArtField to ObjPtr\n\nAdded EXPECT_OBJ_PTR_EQ and variants to gtests.\n\nFixed moving GC bugs in:\nClassLinker::CreatePathClassLoader\nClassLinkerTest: StaticFields\n\nObjPtr Decode call sites: 186 -\u003e 181.\n\nSome tests fail due to ResolvedFieldAccessTest, will fix in follow\nup CL.\n\nBug: 31113334\n\nTest: test-art-host CC baker\n\nChange-Id: I8b266ad00f3c20c8cbe7cfdf280d175083df0b88\n" }, { "commit": "c38a6f8e8d18b85113d1a585235e997779583c80", "tree": "ce04e813a10e1edfdd0ad1bad520fc5f3f8b06f2", "parents": [ "d1224dce59eb0019507e41da5e10f12dda66bee4" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Sep 29 17:07:20 2016 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Sep 29 18:31:22 2016 +0100" }, "message": "dex_cache: rewrite StringDexCachePair as DexCachePair\u003cT\u003e\n\n.. with [T \u003d mirror::String]. This is in preparation for introducing\na dex cache array for MethodTypes, which will be treated the same way.\n\nTest: make test-art-host\nbug: 30550796\nChange-Id: Ief4455b4c6e4c9dd897f2c40b14b843a57b1dc8e\n" }, { "commit": "279ac5c060a1a642c786730e3f2b05d556c00839", "tree": "2c7a4faea98f43828f465f46d77e9b12d6f2f09a", "parents": [ "30a6a8830f6bec346de3e9ad046c8576283e36d8" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 08 17:34:25 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 09 14:16:13 2016 -0700" }, "message": "Don\u0027t assume resolved classes are filled in for ResolvedFieldAccessTest\n\nThe assumption that the dex cache type for class of the field is\nresolved is not correct since FindClass does not fill in the\nresolved type array. This resulted in crashes if dex_access_to \u003d\u003d\nnullptr.\n\nAlso implemented for ResolvedMethodAccessTest\n\nBug: 31277064\n\nTest: test-art-host CC baker, launch ZeroTouch with PGO.\n\n(cherry picked from commit c90450a201707111e33d6b6f37d431b201849a8b)\n\nChange-Id: Id5e71ea0eced1b631a3bd7d297229a66cb190954\n" }, { "commit": "161db1dea4808d32db3623cc1a8e91b53df0fb02", "tree": "43b1481040788e63ced53518061714539e5e5406", "parents": [ "4fd93aebe856151f818cfca515a902f7a171c604" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 01 14:06:54 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 02 10:56:08 2016 -0700" }, "message": "Use the fast path object size for RosAlloc asm stubs\n\nAlso address comments. MemAllocTest perf on N5X speedup in the noise.\n\nBug: 9986565\n\nTest: test-art-host -j32, N5X booting\n\nChange-Id: Ic22ca92aab88b37fd66928949bf11264ee3476dc\n" }, { "commit": "93bbee0c1c97bd9a5336ed0c9e41cd17613df585", "tree": "7f2fe8d9ec240b34e7ee153b95e69b821554ab7e", "parents": [ "1c29876b303ab392e615b2294db009a971613bea" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 31 09:38:40 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 31 18:32:33 2016 -0700" }, "message": "Faster allocation fast path\n\nAdded a new object size field to class, this field contains the\naligned object size if the object is not finalizable and is\ninitialized. If the object is finalizable or uninitialized the field\nis set to some large value that forces the ASM allocators to go slow\npath.\n\nOnly implemented for region/normal TLAB for now, will add the to\nRosAlloc stubs soon.\n\nCC N6P MemAllocTest: 1067 -\u003e 1039 (25 samples)\nCC N6P EAAC: 1281 -\u003e 1260 (25 samples)\n\nRAM overhead technically 0 since mirror::Class was not 8 byte aligned\npreviously. Since the allocators require 8 byte allignment, there\nwould have been 1 word of padding at the end of the class. If there\nwas actually 4 extra bytes per class, the system overhead would be\n36000 * 4 \u003d 120KB based on old N6P numbers for the number of loaded\nclasses after boot.\n\nBug: 9986565\n\nTest: test-art-host CC baker, N6P phone boot and EAAC runs.\n\nChange-Id: I119a87b8cc6c980bff980a0c62f42610dab5e531\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": "6e03d19e638ea64260f2be4890944b40409d6afa", "tree": "e723b0b96c379d991e5cecb32686decf1353704f", "parents": [ "9495e0c56b25349220aed7adb997d81f9b0c86cc" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Tue Aug 23 14:32:21 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Tue Aug 30 10:21:46 2016 +0100" }, "message": "Track addition of Class.toGenericString() method\n\nIncreasing the vtable size for class by one.\n\nBug: 28666126\nChange-Id: I89019d5f1b65725180b409decbd5b2570e24f5fe\nTest: cts ran for libcore (libcore, harmony_java_lang, org) and host art tests.\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": "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": "3d0aa88808052436981f589b7af4f1706c365187", "tree": "6a6d232b20d5b275dac7ee849ba2fc81d9256afd", "parents": [ "95afcbcc25bb17c9c34e8b7708013b612a3bb581" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Tue Aug 16 19:53:07 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Wed Aug 17 10:27:36 2016 +0100" }, "message": "Adjustments for changes made to java.lang.Class\n\nFix up the runtime after removal of the annotationType field\nfrom java.lang.Class.\n\nTest: make test-art-host / boot a device\nChange-Id: I5f37567d2b5ec381b35e66fef29287abe307b840\n" }, { "commit": "8261d02f9523b95013108f271b82bb157ef6f71d", "tree": "026969c0ee61c0deae7a625fba81a711acd218aa", "parents": [ "9cff32df754c428ef69ddb61e7600abfd4c75266" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Aug 08 09:41:04 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Aug 09 09:57:01 2016 -0700" }, "message": "Revert \"Revert \"ARM64 asm for region space array allocation\"\"\n\nAlso added missing large object check. No regression from the check\nN6P CC EAAC time at 1313 for 10 samples vs 1314 before reverts.\n\nBug: 30162165\nBug: 12687968\n\nTest: test-art-target with CC + heap poisoning\n\nThis reverts commit 6ae7f3a4541e70f04243a6fe469aa3bd51e16d79.\n\nChange-Id: Ie28f652f619898d7d37eeebf3f31a88af8fac949\n" }, { "commit": "6ae7f3a4541e70f04243a6fe469aa3bd51e16d79", "tree": "db05e699a9929d11ff47a2b19845a921e360b5ff", "parents": [ "f686c3feabe3519bedd1f3001e5dd598f46946ef" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Aug 08 14:41:32 2016 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Aug 08 14:41:32 2016 +0000" }, "message": "Revert \"ARM64 asm for region space array allocation\"\n\nThis change breaks many tests on the ARM64 concurrent\ncollector configuration.\n\nBug: 30162165\nBug: 12687968\n\nThis reverts commit f686c3feabe3519bedd1f3001e5dd598f46946ef.\n\nChange-Id: I5d7ef5fa2ffb6a8d9a4d3adbcc14854efa257313\n" }, { "commit": "f686c3feabe3519bedd1f3001e5dd598f46946ef", "tree": "234494fbb4fcf640df4676659d6cf6f987e0d3ab", "parents": [ "20b975cc5c5a1f4e6f0a8e525b5d2e63abc413df" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Aug 04 11:08:26 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 05 10:42:03 2016 -0700" }, "message": "ARM64 asm for region space array allocation\n\nWrote region space tlab array and array resolved allocators in\nassembly code. The speedup is a combined increase from checking the\nmark bit and having an assembly fast path.\n\nAdded resolved, initialized entrypoints for object region TLAB\nallocator.\n\nN6P (960000 mhz) EEAC benchmark (average of 50 samples):\nCC 1442.309524 -\u003e 1314 (10% improvement)\nCMS: 1382.32\n\nRead barrier slow paths reaching C++ code go from 5M to 2.5M.\n\nBug: 30162165\nBug: 12687968\n\nTest: With CC: N6P boot, run EAAC, test-art-target\n\nChange-Id: I51515b11ef3f795f57eb72fe0f5759618fef5084\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": "5ffa0780a2738eed1f86efb967ea99badcbd5d9c", "tree": "ee3a4bde21aa03ad564c84877b3cbab3048e6cfd", "parents": [ "93afc5f20a23ece78d6d6e506ed75775b8ef2113" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 27 10:45:47 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 27 13:57:01 2016 -0700" }, "message": "Reduce unnecessary read barriers in GC\n\nRemoved read barrier from IsUnprocessed, DequeuePendingReference,\nEnqueueReference, and a few other places.\n\nHard to tell if GC time goes down.\n\nEAAC:\nBefore GC slow path count: 254857\nAfter GC slow path count: 1005\n\nBug: 30162165\nBug: 12687968\n\nTest: test-art-host, volantis boot with CC\n\nChange-Id: Ic2add3a9b1e1d7561b0b167f2218b10f8dbff76c\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().\"" }, { "commit": "a62cb9bb6cb2278cb41ab0664191623e178c6a4f", "tree": "62263dac644e2d80a34a04f2649e4741a1bed6f4", "parents": [ "bb8d501c9bb882a8927c6ceda07bf9577e06c3e1" ], "author": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Thu Jun 30 09:18:25 2016 +0000" }, "committer": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Thu Jul 07 10:01:04 2016 +0000" }, "message": "Revert \"Revert \"Optimize IMT\"\"\n\nThis reverts commit 88f288e3564d79d87c0cd8bb831ec5a791ba4861.\n\nChange-Id: I49605d53692cbec1e2622e23ff2893fc51ed4115\n" }, { "commit": "bb268b14da040c5df27a413426aa2baab602ed15", "tree": "04e2fbfa54bc278170d361f97781b066b4abe46a", "parents": [ "d80c5d2368b4455c452e8b4fe2d8511fe1c0430c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jun 30 15:52:56 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jul 04 17:44:17 2016 +0100" }, "message": "Clean up Class::FindStaticField().\n\nPreviously we have wrapped the class pointer in a Handle\nfor the GetDirectInterface() call but we didn\u0027t wrap the\nDexCache pointer. Since GetDirectInferface() call from\nthis function cannot cause thread suspension anyway, fix\nthe inconsistency by using raw pointers where possible\nand asserting that no thread suspension takes place.\n\nTest: Run standard ART test suite in gcstress mode.\nChange-Id: Ie4c6101aabd6fa68733e4920db277f835793d943\n" }, { "commit": "88f288e3564d79d87c0cd8bb831ec5a791ba4861", "tree": "dd051a7b2985d1af3fea91ad6000de4bdc701a19", "parents": [ "e36389f20c83083c0aaba2dcac0888951e55cae1" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:17:52 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:41:25 2016 +0000" }, "message": "Revert \"Optimize IMT\"\n\nBug: 29188168 (for initial CL)\nBug: 29778499 (reason for revert)\n\nThis reverts commit badee9820fcf5dca5f8c46c3215ae1779ee7736e.\n\nChange-Id: I32b8463122c3521e233c34ca95c96a5078e88848\n" }, { "commit": "fb7a0b4e898a88079ddd335b2d864fe04a32766a", "tree": "e183af7d3312f365a04997a43692c9f8bfcfe647", "parents": [ "efeb9f01308c5c4b723ff905cf5bea1636f7a549" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:19:04 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:19:04 2016 +0000" }, "message": "Revert \"Propagate flags to ShouldHaveEmbeddedVTable\"\n\nBug: 29188168 (for initial CL)\nBug: 29778499 (reason for revert)\n\nThis reverts commit efeb9f01308c5c4b723ff905cf5bea1636f7a549.\n\nChange-Id: I21bc32fa07014bbce2eedba3686f09d3d45aec65\n" } ], "next": "efeb9f01308c5c4b723ff905cf5bea1636f7a549" }