)]}' { "log": [ { "commit": "fbfc3945c1877cfb97ebd217ccceb98abc4bc441", "tree": "19259839bdeec7d62f8cb9724ddfad493545854c", "parents": [ "ae616fa0e828cd6cbbae53edf3cf14338ad49dd9" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 27 16:51:35 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 27 16:56:42 2017 +0100" }, "message": "Remove ClassLinker::SetEntryPointsToCompiledCode().\n\nThis function was used only for tests and it was essentially\njust calling ArtMethod::SetEntryPointFromQuickCompiledCode()\nin a very obscure way and required the memory before the\ncode pointer to contain a non-zero code size.\n\nThis is a follow-up to\n https://android-review.googlesource.com/445648\nthat removed a problematic use of the function, calling it\nwith a pointer to an entrypoint instead of a pointer to\ncompiled managed code, i.e. without the pre-header that\nshould contain the non-zero code size.\n\nTest: m test-art-host-gtest\nChange-Id: I483450832443ea0589eb41c74491384bcd5d6ab8\n" }, { "commit": "be7af227c41c524fd38fc695c78f575d14db0e91", "tree": "aa9864ad8ead6f057a9f714c8b14541cfbb0fc91", "parents": [ "c1ac9ee48ddf483ec770b424d63de4982262ea33" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 25 09:57:28 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 25 11:16:05 2017 -0700" }, "message": "ART: Move app image verification to its own method\n\nMove the app image verification to its own method to allow easier\nextension without bloating app image loading code. Refactor the\nvisitors into local scope.\n\nBug: 34193647\nTest: m test-art-host\nChange-Id: Id1f471a5020f4f70017eaac4ac20e4c40607e349\n" }, { "commit": "c1ac9ee48ddf483ec770b424d63de4982262ea33", "tree": "f0fe2958793a2eccef85c40ac94d056b046ca7de", "parents": [ "aed968d130d62d23cfe7f6051ca72c6a84fe1510" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 24 22:35:49 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 25 11:15:09 2017 -0700" }, "message": "ART: Move CHA to ClassLinker\n\nMove the ClassHierarchyAnalysis singleton to ClassLinker. Hierarchy\nanalysis is a link-time effort and belongs to class-loading instead\nof the global Runtime object.\n\nAlso clean up to use a unique_ptr, and add null checks to allow\nbypassing the analysis.\n\nBug: 34193647\nTest: m test-art-host\nChange-Id: I14ef536e37eb19c0dede372ffd68a826c482cf71\n" }, { "commit": "4f342b01a9d6661261a233d2221ae4a3c4e6ac4b", "tree": "62d76d0784c86a48f2140c708bff0502514a2491", "parents": [ "2727d7ca49eea03fc1cd4cf81447e1c2bb7027e7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jul 21 17:12:39 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jul 24 16:31:40 2017 +0000" }, "message": "Move logic to get profile class descriptors to profile info\n\nMove and simplify logic used to get a descriptor set for app image\ncreation. Code is covered by TestLayoutAppImage in dex2oat_test.\n\nMotivation:\nIf we are class unloading in dex2oat, we won\u0027t have all the dex files\nregistered at once.\n\nBug: 63467744\nTest: test-art-host\nChange-Id: I1e3616ee0a731061c347677713dba07e603eaa08\n" }, { "commit": "07bfbace6f835e6c748fd68ec7624992478b16c1", "tree": "5d094a00fbc90455bd9b53e042cf8b4fe8433462", "parents": [ "ba118827465d12177f3996e50133960087b1c916" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 06 14:55:02 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 20 16:33:00 2017 +0100" }, "message": "Hash-based DexCache methods array.\n\nTotal boot*.art size for aosp_angler-userdebug:\n - arm64:\n - before: 11603968\n - after: 10129408 (-1.4MiB, -12.7%)\n - arm:\n - before: 8626176\n - after: 7888896 (-0.7MiB, -8.5%)\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: Nexus 6P boots.\nTest: testrunner.py --target\nTest: Build aosp_mips64-eng\nBug: 30627598\nChange-Id: I7f858605de5f074cbd7f0d9c4c072fbd44aee28f\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": "f8db2c3945e746fcbf307832ac2f7826d5ec178a", "tree": "fc579f67f17b2ccbb218889f2484ef6f5013da1c", "parents": [ "c5515fd96887f70101640fddf9a3189ff936c08b" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Jul 07 20:07:12 2017 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Jul 14 13:04:30 2017 +0100" }, "message": "ART: Add support for constructor method handles\n\nAdd well known methods for MethodHandles.lookup() and\nMethodHandles.Lookup.findConstructor(). These are used to call the\nJava code that create a constructor transform.\n\nSeparate method handle resolution paths for fields and methods in the\nclass linker.\n\nUpdate test 952-invoke-custom-kinds to include a constructor method\nhandle. The test classes now match dx/tests/135-invoke-custom.\n\nBug: 62774190\nTest: Update 952-invoke-custom-kinds\nChange-Id: I9a007254a856422c24397c4df3ef3dfbf6bdd840\n" }, { "commit": "06bed3009b94b392ce6be43d72f4b10803565c72", "tree": "63acb06a7ad44782c588cafb21c5c2eb1665409c", "parents": [ "99d50e80f3d186c9ae19736b9b224ee319d936f9" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 13 13:23:18 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 13 16:59:18 2017 -0700" }, "message": "Revert \"Revert \"Record post startup methods in profile\"\"\n\nTest: test/run-test 595-profile-saving\n\nThis reverts commit a867f7ab437dd6c1458088964986d0fe7275625c.\n\nChange-Id: I2826bdddda446925856991de4b0b786c8c05831e\n" }, { "commit": "a867f7ab437dd6c1458088964986d0fe7275625c", "tree": "e575d817d4cdc79626d4c70b87d1e09b02a78d81", "parents": [ "9f3a7c5e59a0a0caec950970fe5c636973d0421c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jul 13 07:57:57 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jul 13 07:59:23 2017 +0000" }, "message": "Revert \"Record post startup methods in profile\"\n\nBreaks target tests.\n\nThis reverts commit 5594f11fa8a8a1efc24d1b258592f68cf462eec8.\n\nChange-Id: I5ff4aab14bb241e287f042a9a90458e61306985e\n" }, { "commit": "5594f11fa8a8a1efc24d1b258592f68cf462eec8", "tree": "8b077a4ee54f0c49aaced410f57ce03a7467a7a0", "parents": [ "65f1c2edd02f3936aef94c7313932069e66a1167" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jun 07 15:18:44 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jul 11 13:51:01 2017 -0700" }, "message": "Record post startup methods in profile\n\nRecord post startup methods in the profile by visiting the methods\nfor all the loaded classes. This data will be used for layout\noptimizations and generating the boot image profile.\n\nThe maps profile size doesn\u0027t seem to get larger after doing a basic\npost launch use case. The change in profile size is hard to reliably\nmeasure.\n\nVerified by dumping a profile with profman and confirming that\nthere are post startup methods in the profile.\n\nAdded test to 595-profile-saving.\n\nDid a test for maps to see how long we hold the class table lock. The\nresults are 4ms in the longest case. This should be short enough to\nnot cause jank if run every 20s or so.\n\nTest: test-art-host\nBug: 36457259\nChange-Id: I82e36957fcdc8d9e072c0193478bd1ef0ce09a67\n" }, { "commit": "7865ac7e233b8a8dcfd6ca8466d30b39a8089c3e", "tree": "c845ef2149d819dd512b87f1898aff6d583f91c6", "parents": [ "415dc3d63fc05f89b77c2f6249a5a89dade4df27" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Jun 28 11:03:12 2017 -0700" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Jul 06 12:05:54 2017 -0700" }, "message": "Add DelegateLastClassLoader to the list of WellKnownClassLoaders\n\nGeneralize CreatePathClassLoader from the class linker to be able to\ncreate a well known class loader with a given parent.\n\nFor now, it only supports PathClassLoader and DelegateLastClassLoader.\n\nTest: m test-art-host\nBug: 38138251\nChange-Id: Iced0de7c21010c6d61dbf29a631e670011833a7d\n" }, { "commit": "415dc3d63fc05f89b77c2f6249a5a89dade4df27", "tree": "c80669ca9def11e5c32addfdc3f4c88c4eb25686", "parents": [ "16d59b2b0ff202be99cbe24830e8a5080b774357" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Jun 28 11:03:12 2017 -0700" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Jul 06 12:02:52 2017 -0700" }, "message": "Refactor FindClass code in class linker\n\nExtract the code searching the current class loader classpath in a new\nmethod (FindClassInClassLoaderClassPath). This will make things easier\nwhen adding support for DelegateLastClassLoader.\n\nTest: m test-art-host\nBug: 38138251\nChange-Id: I742cd614f12e8a7b6ccd0a23a99d1f6653afaf73\n" }, { "commit": "2ff3b97500b717f69415ae474d5cfb1613cd15ee", "tree": "a6bd76eeda26ad85173e23a14f69ec8ed3440570", "parents": [ "5d08fcc21a14190c4dba3028a3e35d281afdd016" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 05 18:14:53 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 05 20:28:20 2017 -0700" }, "message": "ART: More header cleanups\n\nClean up some more includes. Move some -inls from object-inl to\nobject-refvisitor-inl. Fix up transitives.\n\nTest: m test-art-host\nChange-Id: I0e3e268d95a1c71b0a80fc963d13d191a97b2283\n" }, { "commit": "07f0621463e7b480c86ddba3e72d3fb9f0ae820f", "tree": "130691a38d30d788306ca546f3e7b4d17c808a53", "parents": [ "596c58b3dc73a4017d49af6c5037bbd7109fd31e" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Jun 01 14:01:43 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Jun 02 11:30:01 2017 -0700" }, "message": "Fix redefinition related use-after-free bug\n\nA redefinition that failed with JVMTI_ERROR_FAILS_VERIFICATION could\ncause a use-after-free of java DexCache objects if a art::DexFile* that\naliases the one created for the failed redefinition is created.\n\nBug: 62237378\nTest: ./test.py --host -j40\n\nChange-Id: Ia080dfa6c702b2e3a735c4c3cd80ca3974386934\n" }, { "commit": "2af99028f508071d5cd056e4ea42e988e0b83e79", "tree": "8043da86d7a9e72c69d75d0ac7e8eaf5da64cf99", "parents": [ "d482e73fe26cb9161511a80e3db39e08b9808ab6" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Apr 25 08:32:59 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue May 30 16:26:39 2017 -0700" }, "message": "ART: More header cleanup - class_linker.h\n\nHide the ClassTable. Users of ClassLinker don\u0027t care about it.\n\nTest: mmma art\nChange-Id: I282b25a3d1844cc6c7f716159fc58fca9b65135e\n" }, { "commit": "d482e73fe26cb9161511a80e3db39e08b9808ab6", "tree": "0b8c874f24c211e4e3d87083f8017580aa23ff63", "parents": [ "a14100ccf51cc63a5c472188d1e2d337627e49eb" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 24 17:59:09 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue May 30 16:26:35 2017 -0700" }, "message": "ART: More header cleanup - runtime.h\n\nRemove unnecessary runtime.h includes. Refactor other headers\n(move to forward declarations) to make this possible.\n\nTest: mmma art\nChange-Id: I297f0ad444ba92169edd16d8c7cbe95bb6dbb800\n" }, { "commit": "6d7abbd2324f544c6b6da42bb6b9b531df0ce3cd", "tree": "669705a042bbc77c7963c117a9bf6f7a2308f9c8", "parents": [ "0dfc315a096bf5678cb5182771e3fdcca8ea4433" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 24 13:19:09 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 24 14:18:45 2017 -0700" }, "message": "ART: More header cleanup - method_verifier.h\n\nMove enumerations to own header. Move the compiler interface (of what\nthe compiler can tolerate) into its own header. Replace or remove\nmethod_verifier.h where possible.\n\nTest: mmma art\nChange-Id: I075fcb10b02b6c1c760daad31cb18eaa42067b6d\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": "db01a091aefbd78b56777f4c2e8c5e3f2d8c2712", "tree": "62dbbd082c5a4cfc122830a4df0a2ced219d8d6d", "parents": [ "432fccc4c001fcd822f401aea1a4214b713bd896" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Apr 03 15:39:55 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Apr 04 11:44:35 2017 -0700" }, "message": "Ensure invoking obsolete methods throws errors.\n\nThis sets the entrypoint for obsolete methods to a special stub that\nwill ensure that calling them results in an Error being thrown.\nPreviously we were allowing obsolete methods to be run if they could\nreach the appropriate places in the runtime.\n\nGetting into the state where this is possible is extremely difficult\nsince one can only get an jmethodID to an obsolete method by snatching\nit off the stack (or by inspecting internal runtime data). From there\nnormally invoking it will do lookup on the receiver which will get you\nthe original version.\n\nBug: 36867251\nBug: 31455788\n\nTest: ./test.py --host -j40\nTest: (with aosp_marlin-userdebug device) ./test.py --target -j4\n\nChange-Id: I2ca0503966a4e3de18dd89cb7ff224eba1459b49\n" }, { "commit": "f25cc730ff70f085daf527f69a02fdee69aee16b", "tree": "2f36f61990add16e7519d8b93506d6049dd7764c", "parents": [ "7591bdadf72a2fe155f937b8dc66f6249cedd014" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Mar 16 16:18:15 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Mar 23 15:25:27 2017 +0000" }, "message": "Deterministic DexCache in heap images.\n\nPreload each slot in dex cache arrays with the candidate\nentry with the lowest index.\n\nAlso fix a discrepancy where we used to keep an ArtField*\nin the DexCache if we kept the declaring class rather than\nbased on the class referenced in the corresponding FieldId.\nThis can result in accessing a field through a FieldId with\nan unresolved class as demonstrated by the regression test.\n\nTest: 159-app-image-fields\nTest: testrunner.py --host\nBug: 30627598\nChange-Id: I34735047cb064ff462cd9859e86a61ddf8615ecd\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": "046854b83e54eef0919d27b41d34061c2274a4dd", "tree": "b1a49f6657c83e6850276fe6ab976a5314b30fb8", "parents": [ "425b5d23e2c60d295471817a75b1b554481c5334" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Mar 01 17:16:22 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 03 12:50:26 2017 -0800" }, "message": "Fix dex location filtering in dex2oat\n\nPreviously we were filtering dex location against profile keys,\nthis meant qualified ones like /system/.../app.apk would not match\nthe profile key app.apk in the profile. This CL fixes changes the\nbehavior to filter based on the profile key of the dex file location.\n\nFixed OatWriter checksum for raw data case (also found by regression\ntest).\n\nAdded missing FlushCloseOutputFiles to CompileImage causing DCHECK\nfailures for File destructor.\n\nAll the fixes are regression tested by dex2oat_test.\n\nTest: test-art-host-gtest-dex2oat_test\n\nBug: 34929159\nBug: 35761072\n\nChange-Id: I1bdc949bd644bfab1c8fea0b737a132b487a653b\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": "c069a30d42aefd902c20e8bc09dfad1683f07ded", "tree": "8bbf72bea7ea5d243b57f8e0ab64b687a9f60e4b", "parents": [ "3f38398380b80d1ded078ebed1211b7e4f51460f" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 18 09:23:12 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Tue Feb 14 14:04:33 2017 +0000" }, "message": "ART: invoke-custom support\n\nAdds invoke-custom instruction to the interpreter.\n\nBug: 33191717,30550796\nTest: art/test/run-test --host 952\nChange-Id: I3b754128649a8b3a00ade79ba2518d0e377f3a1e\n" }, { "commit": "5812e20ff7cbc8efa0b8d7486ada2f58840a6ad5", "tree": "be0e9fdeb90bcbad76b17b3aae4e746d977d80ee", "parents": [ "518f373324f355f8e58440dfbc3f5a1f2244bde7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 13 18:32:04 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 13 19:34:54 2017 -0800" }, "message": "Revert^3 \"Hash-based dex cache type array.\"\n\nAssert failing for \"earchbox:search\":\nF zygote64: class_linker.cc:4612] Check failed: handle_scope_iface.Get() !\u003d nullptr\n\nTest: m test-art-host\nBug: 34839984\nBug: 30627598\nBug: 34659969\n\nThis reverts commit 85c0f2ac03417f5125bc2ff1dab8109859c67d5c.\n\nChange-Id: I39846c20295af5875b0f945be7035c73ded23135\n" }, { "commit": "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": "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": "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": "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": "4b0cedf04e8d136256f3afd35730da3d22bbee43", "tree": "9216f71cfa3b3ac136e624fa2299e1e2e2c5990b", "parents": [ "cc92b213c1931a68b1317471b0e8fd12a98e96b0" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Jan 24 11:15:20 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Jan 24 11:16:17 2017 -0800" }, "message": "Fix typo in argument name.\n\nTest: mma -j40 test-art-host\nChange-Id: Ifff0e7b7ae64ce322562244a705fbe4e8f879689\n" }, { "commit": "b0f1192bfac159008e1f2367751083740ba05698", "tree": "4b5d4385c27b600f5488962d0f7f6f00d9836843", "parents": [ "a7e38d8aaacfca85b40b5df654f85c0979968672" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Jan 23 14:25:17 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Jan 23 14:44:23 2017 -0800" }, "message": "Add ClassPreDefine hook.\n\nThis hook is called prior to defining a class for the first time. It\nallows callbacks to modify the dex-file used to load the class if they\nwish. The event handler is responsible for ensuring that the returned\ndex-file is set-up correctly.\n\nBug: 31684920\nTest: mma -j40 test-art-host\nChange-Id: Iaed79c1597913148ead795e033a0a10a4ebe6b2b\n" }, { "commit": "0f01b583c3952d0219696480654a0db8fac4b661", "tree": "33184022e0c9bc0a6af4d89b21864fd460f32df4", "parents": [ "04bbb5be5b9c0f0b3a72116353d23ea63c8bc5e9" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 18 15:22:37 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jan 19 15:12:12 2017 -0800" }, "message": "ART: Add ClassLoadCallback\n\nAdd callback for class-load and class-prepare events. Move Dbg\nover. Add tests.\n\nBug: 31684920\nTest: m test-art-host-gtest-runtime_callbacks_test\nTest: art/tools/run-jdwp-tests.sh --mode\u003dhost\nChange-Id: I871f6b3c54448fd6ece8d9a7571b2042be50d525\n" }, { "commit": "d4a282789669b27ce0352c99c26469b07a86d871", "tree": "037d234b5410becd373ae42c37b8b6c265b57741", "parents": [ "24e2ded0f9b7715e3dccfe41d2467cc5a7eea84d", "ce7732bcb504fc91ed201d8e2128e30d51ee9bdb" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Jan 18 17:16:06 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 18 17:16:07 2017 +0000" }, "message": "Merge \"ART: Add Search API\"" }, { "commit": "a3974581751cd73a896f7c4fcab71beb17c4f9dc", "tree": "66bcf219ae6cf6ab50793c7ea9e233e4061ab28d", "parents": [ "c66a76f94ec823d2021879ab6f4a1934a2274ace", "942fd3130254d8276cbfe8e6201825e9c49e9b2c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jan 18 10:52:20 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 18 10:52:21 2017 +0000" }, "message": "Merge \"Reduce using ArtMethod\u0027s dex_cache_resolved_types_.\"" }, { "commit": "ce7732bcb504fc91ed201d8e2128e30d51ee9bdb", "tree": "8875e446d90a3436576da3abe5c45ed9b81aaada", "parents": [ "3a5e34b65727d2c9e7e6ebe45c337dae0d0398a4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 17 15:50:26 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 17 20:40:31 2017 -0800" }, "message": "ART: Add Search API\n\nAdd initial support for AddToBootstrapClassLoaderSearch and\nAddToSystemClassLoaderSearch. The current implementation does\nnot support the OnLoad phase.\n\nAdd tests.\n\nBug: 34359699\nTest: m test-art-host-run-test-929-search\nChange-Id: I42955ff381cd3828bfd772bee2f9b0745195ee51\n" }, { "commit": "942fd3130254d8276cbfe8e6201825e9c49e9b2c", "tree": "62dbcd85ab150604e2d72a3c2d0c3639a28c9b7a", "parents": [ "8bd59a0fd46db83616785168231e09fb95ed2ead" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 20:52:19 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jan 17 16:12:15 2017 +0000" }, "message": "Reduce using ArtMethod\u0027s dex_cache_resolved_types_.\n\nAvoid using the ArtMethod\u0027s dex cache type array shortcut\nin runtime, preparing for its removal. We do not completely\nremove the shortcut yet because it is still used by array\nallocation entrypoints.\n\nFix ArgArray::BuildArgArrayFromObjectArray in reflection.cc\nto not ask for the parameter type to be resolved. It should\nhave been previously resolved when retrieving the Method.\n\nAlso partially revert\n https://android-review.googlesource.com/310717\nbecause it relied on the removed AIOOBE check in the removed\nArtMethod::GetDexCacheResolvedType(). The removed check was\nsimply defensive but it could not be triggered without some\nmemory corruption.\n\nTest: m test-art-host\nBug: 30627598\nChange-Id: Ic45a5ff8c66b79429e440cbc08d67bf22a083682\n" }, { "commit": "70f16393e7859f489e8403d2cc287db6a442455a", "tree": "0c68f881c893fbaf1623d8a2c790f035467dedda", "parents": [ "0d478f289f0e33f19693d135f1d562b57427ed32" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jan 16 14:20:10 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jan 16 18:03:56 2017 -0800" }, "message": "ART: Add GetClassLoaderClasses\n\nAdd support for GetClassLoaderClasses, extracting all classes a\ngiven classloader initiated. Add test.\n\nBug: 31684578\nTest: m test-art-host-run-test-912-classes\nChange-Id: I0aba27fb1674baf1263c0a19dc2dcce7af013760\n" }, { "commit": "1998cd02603197f2acdc0734397a6d48b2f59b80", "tree": "aa639c7ec96f71d7aaf5d0c865a8a133dbc457c3", "parents": [ "6bec91c7d4670905cd67440991ec76fd54d0f000" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jan 13 13:02:58 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 13:39:24 2017 +0000" }, "message": "Implement HLoadClass/kBssEntry for boot image.\n\nTest: m test-art-host\nTest: m test-art-host with CC\nTest: m test-art-target on Nexus 9\nTest: Nexus 9 boots.\nTest: Build aosp_mips64-eng\nBug: 30627598\nChange-Id: I168f24dedd5fb54a1e4215ecafb947ffb0dc3280\n" }, { "commit": "921094ad3af4484537e19d62235f22792b7ce595", "tree": "c2c3a66b9c55c301ad52bb6347110b63386a7394", "parents": [ "7620a3b58a9286fae376f4705b602310e5fd464a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jan 12 18:37:06 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jan 12 19:09:03 2017 +0000" }, "message": "Revert \"Revert \"Refactor ClassLinker::LinkInterfaceMethods().\"\"\n\nDrop the fix for using the correct linear allocator.\nThat fix breaks some debug checks that need non-trivial\nchanges and extra testing, so leave that for another CL.\n\nThis reverts commit dce413d333c88c86f28b9d0fe3a8052942ed9080.\n\nTest: m test-art-host\nChange-Id: I69935236a843a08d79b77c5fce74b4cc55f8ea99\n" }, { "commit": "7620a3b58a9286fae376f4705b602310e5fd464a", "tree": "3ded5576a330585c8edc503b32d6abba93f14860", "parents": [ "8fdba845ffeb516aa0ede900def3a833861eff15", "dce413d333c88c86f28b9d0fe3a8052942ed9080" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 12 18:00:02 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jan 12 18:00:03 2017 +0000" }, "message": "Merge \"Revert \"Refactor ClassLinker::LinkInterfaceMethods().\"\"" }, { "commit": "dce413d333c88c86f28b9d0fe3a8052942ed9080", "tree": "428c8889adc2f56614e90a586165a7c007908dc8", "parents": [ "afbd71ffbcc7b37067d3a4703648e62fc0e55e6f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 12 17:59:39 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 12 17:59:39 2017 +0000" }, "message": "Revert \"Refactor ClassLinker::LinkInterfaceMethods().\"\n\nFails libcore tests.\n\nThis reverts commit afbd71ffbcc7b37067d3a4703648e62fc0e55e6f.\n\nChange-Id: I611620f915025a0c077a78a480b2b730b3be3de8\n" }, { "commit": "aa89a4c6fca095904521842c018399f1e3501a45", "tree": "c2df327bbd61e6c231effce854c0bf816fd6fe04", "parents": [ "97e1cb551c7faff934ca69ce9185cb9e8e6e74aa", "afbd71ffbcc7b37067d3a4703648e62fc0e55e6f" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Jan 12 12:39:01 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jan 12 12:39:01 2017 +0000" }, "message": "Merge \"Refactor ClassLinker::LinkInterfaceMethods().\"" }, { "commit": "afbd71ffbcc7b37067d3a4703648e62fc0e55e6f", "tree": "d2c194d57b794ee7281763cb41c8f52b9b2c7322", "parents": [ "263cc99a5f7a2cc81094e18b08eaa66ceed3dd1b" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Dec 19 17:32:17 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jan 12 11:05:16 2017 +0000" }, "message": "Refactor ClassLinker::LinkInterfaceMethods().\n\nAdd a helper class and split LinkInterfaceMethods() into\nseveral functions to improve readability. Even with this\nchange, it is still almost 250 lines long.\n\nFix the method array reallocation to use the appropriate\nlinear allocator for the class\u0027s defining loader. Given that\nthis is a native reallocation, failure to reallocate would\nresult in an abort, so CHECK() that we get a non-null array\ninstead of expecting a null and asserting a pending OOME.\n\nTest: m test-art-host\nChange-Id: I8ff8257e4271ae8c975ea4b33fd8af36e1a1f77e\n" }, { "commit": "dba61481035b7944173181ec9ee02aea41dd0e29", "tree": "8629775e657cc8968e2f8fa2af6256c7a58c7dd4", "parents": [ "cb2ce333ee2d1053919fe0acc390369bea3ad77e" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 21 08:20:29 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 21 11:43:14 2016 -0800" }, "message": "Revert \"Revert \"Revert \"Revert \"Basic obsolete methods support\"\"\"\"\n\nA GetDeclaringClass()-\u003eGetDexCache() got inserted during the\nmerge/review process meaning that we would try to access incorrect\ndex-cache in obsolete methods in some situations.\n\nAlso when using tracing we would loop forever (or at least until an\nOOM error) in test 916 due to tracing forcing InterpretOnly mode\nmeaning methods would never be jitted.\n\nBug: 32369913\nBug: 33630159\n\nTest: ART_TEST_TRACE\u003dtrue \\\n ART_TEST_JIT\u003dtrue \\\n ART_TEST_INTERPRETER\u003dtrue mma -j40 test-art-host\n\nThis reverts commit f6abcda293b115a9d7d8a26376ea2dcf2d1dc510.\n\nChange-Id: I0773bfcba52e3cd51a83be815c6a50c189558f48\n" }, { "commit": "c5798bf82fc0ccd0bb90e0813d8e63df4d0576cc", "tree": "89c94659ad977b66f4d2a90981ed7c63c0af636a", "parents": [ "aea9ffece7eb32f3884a4ad0553e1df4d90fd9e4" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Dec 09 10:20:54 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Dec 09 13:02:40 2016 +0000" }, "message": "Revert^8 \"Make sure that const-class linkage is preserved.\"\n\nReplaced two ReaderMutexLocks with WriterMutexLocks.\nRemoved some unnecessary debugging output.\n\nTest: m test-art-host\nBug: 30627598\nOriginal-Change-Id: Ie9b721464b4e9a5dcce8df8095548e983bba1fe8\n\nThis reverts commit 2c8c6b63da6ecb2ac701cc30f9b4fa4a8eea5cc8.\n\nChange-Id: I3a1aeecf64e4b202cef61cceb248d48106a2f4a6\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": "2c8c6b63da6ecb2ac701cc30f9b4fa4a8eea5cc8", "tree": "7b5e29f66a840e5e83c02df0b23d05501b0d63c5", "parents": [ "6afaa42f50157095a3cdc742afdbc3d58b833eea" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Dec 01 17:42:00 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Dec 01 17:43:09 2016 +0000" }, "message": "Revert \"Make sure that const-class linkage is preserved, try again.\"\n\nReverting due to test failures as expected.\n\nBug: 30627598\nBug: 33231647\n\nThis reverts commit cb5ab35980a86b05586c402924d2e7ca9df25758.\nSquashed revert \"Additional debug logging for bug 33231647.\"\n This reverts commit 00a441033db28d243fc33692d30eb2755fa81728.\n\nChange-Id: I0c0ee1f70d47540fec99f8a797ce13571c16147c\n" }, { "commit": "903691455319f72219853a46340b59c9b89b7553", "tree": "42f33b1e3b949fcd8246f34877b46a304bb36dfe", "parents": [ "982e0bf9ed2b00ef85b328de674e28342eb0b2dd", "cb5ab35980a86b05586c402924d2e7ca9df25758" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Dec 01 12:11:07 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Dec 01 12:11:07 2016 +0000" }, "message": "Merge \"Make sure that const-class linkage is preserved, try again.\"" }, { "commit": "cb5ab35980a86b05586c402924d2e7ca9df25758", "tree": "acf33ce7ac7627682aaf822489247ca1804b60d0", "parents": [ "07e6a7947f3279f66f14f7af045065b5297294c2" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 30 15:31:13 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Dec 01 10:13:19 2016 +0000" }, "message": "Make sure that const-class linkage is preserved, try again.\n\nThis CL causes occasional test failures on the build servers\nwhich we were not able to reproduce locally. So we add some\nsome additional debug output to help pinpoint the cause.\n\nBug: 30627598\nBug: 33231647\nTest: m test-art-host\n\nThis reverts commit 171cf811a1cdf8b1cbc5151505d8630741ce4cf3.\n\nChange-Id: Id56a3f0e86e8212fd547e09c61794401bff47fb0\n" }, { "commit": "211fd802903c130ed1c47c18d1b87809d290aa69", "tree": "e049fbad2dd7fbf244d5163e084be339f02fc8d7", "parents": [ "304f2d3a1d090846a7a55a0016efc9fc8eb14078", "8a0128a5ca0784f6d2b4ca27907e8967a74bc4c5" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Nov 30 18:21:22 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Nov 30 18:21:22 2016 +0000" }, "message": "Merge \"ART: Add dex::StringIndex\"" }, { "commit": "171cf811a1cdf8b1cbc5151505d8630741ce4cf3", "tree": "2e5e16040024115d75baebdbf064987961e36e5f", "parents": [ "e47172b75b8b04a450d235cf72bdc4e223a29cb5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 29 21:10:22 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 29 21:10:22 2016 +0000" }, "message": "Revert \"Revert \"Revert \"Revert \"Revert \"Make sure that const-class linkage is preserved.\"\"\"\"\"\n\nBug: 30627598\n\nThis reverts commit e47172b75b8b04a450d235cf72bdc4e223a29cb5.\n\nChange-Id: I038029c71195a80f041982e82ae556d3157db7b8\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": "e47172b75b8b04a450d235cf72bdc4e223a29cb5", "tree": "9301e9bc08944f3b4590d9ccf6aa7656cc238adf", "parents": [ "d0111420a9f924fe560a97132d09ae531852fd69" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Nov 25 11:47:20 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 29 13:49:20 2016 +0000" }, "message": "Revert \"Revert \"Revert \"Revert \"Make sure that const-class linkage is preserved.\"\"\"\"\n\nAvoid class loading triggered by verification in the\n626-const-class-linking test. Expect spurious wakeups.\n\nAdd extra debugging output for a very strange failure\n image_writer.cc:1144] Check failed:\n !IsBootClassLoaderClass(as_klass)\n java.lang.Class\u003cjava.lang.NoSuchMethodError\u003e\nThat class is supposed to be in the boot image (core.art for\ntests), so the failing check should not be reached at all.\n\nTest: m test-art-host\nTest: m ART_TEST_OPTIMIZING\u003dtrue \\\n ART_TEST_JIT\u003dtrue \\\n ART_TEST_INTERPRETER\u003dtrue \\\n ART_TEST_INTERPRETER_ACCESS_CHECKS\u003dtrue \\\n test-art-host-run-test-626-const-class-linking\nBug: 30627598\n\nThis reverts commit 09ded9201fbeec605b12741ae255663c14bda225.\n\nChange-Id: I65a68cb53f0707b8abb3555ac3dc44c82907aef5\n" }, { "commit": "09ded9201fbeec605b12741ae255663c14bda225", "tree": "c0276a06f8b6dba35e843098a05be326c8f5e2ac", "parents": [ "25dcbad4462ea7279ee2bbe0884abc25bdfac77a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 24 13:03:57 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 24 13:03:57 2016 +0000" }, "message": "Revert \"Revert \"Revert \"Make sure that const-class linkage is preserved.\"\"\"\n\n626-const-class-linking is failing (spurious wakeups?)\n970-iface-super-resolution-gen is failing:\n dex2oatd F 11-24 10:57:16 6410 6410 image_writer.cc:1144]\n Check failed: !IsBootClassLoaderClass(as_klass)\n java.lang.Class\u003cjava.lang.NoSuchMethodError\u003e\n\nBug: 30627598\n\nThis reverts commit 25dcbad4462ea7279ee2bbe0884abc25bdfac77a.\n\nChange-Id: Ie010169bdde45e6ccf2e04a521da4682bd817114\n" }, { "commit": "25dcbad4462ea7279ee2bbe0884abc25bdfac77a", "tree": "f8f8db090bf6735ead2c6999b7b574f4b7278056", "parents": [ "dd77c2b4ff36db082edfbcf2631c78a9282689e1" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 23 13:35:26 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 23 17:01:16 2016 +0000" }, "message": "Revert \"Revert \"Make sure that const-class linkage is preserved.\"\"\n\nFix jdwp getting multiple instances of the same class.\nFix counting \"zygote\"/\"non-zygote\" classes in class table.\nFix FindClass() to EnsureResolved() classes added by\na racing thread to the class table.\n\nTest: m test-art-host\nTest: art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64 --debug\nBug: 30627598\n\nThis reverts commit ecffc67068a28d55d131553bf915fdb9fafbbc03.\n\nChange-Id: I15807949da4f163c2693fac2fbfc274f17685f8a\n" }, { "commit": "ecffc67068a28d55d131553bf915fdb9fafbbc03", "tree": "3f63982d609fa01a5b271b5f393ee9c6d83e1107", "parents": [ "b46782abf6752ccf688b0d9f5b4bb83e5d33793b" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 22 17:37:21 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 22 17:41:49 2016 +0000" }, "message": "Revert \"Make sure that const-class linkage is preserved.\"\n\nBug: 30627598\n\nThis reverts commit 19886dbeced9430579a27979beb92b09c95197b4.\nSquashed reverts of subsequent partial fixes.\n This reverts commit bad475a63b7c938b7ff7903ea1dae27369c3f629.\n This reverts commit 0ba0c7d9b0328af39a385014652eb9af4e42d8b5.\n This reverts commit ddb8afec79492750f16f98e5bd07d44f3dc485d4.\n\nChange-Id: Id94af5ae56a0335f9641a79c885cc75bba971713\n" }, { "commit": "9ee5fdf92f498d716c740cc1770943097f968092", "tree": "a1d7a4cb0a86500850d7eb96f5f2eb2bef90c487", "parents": [ "da2d78ba97d177b80e0e9528362bcebc1fd31f0f", "19886dbeced9430579a27979beb92b09c95197b4" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 22 11:23:16 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Nov 22 11:23:16 2016 +0000" }, "message": "Merge \"Make sure that const-class linkage is preserved.\"" }, { "commit": "19886dbeced9430579a27979beb92b09c95197b4", "tree": "33c85aada415308d8207d63bce903262ea7504fe", "parents": [ "bb4366d2b0b56f2cc66041243bd93de4b8e4d15a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Nov 11 11:40:40 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Nov 21 19:11:31 2016 +0000" }, "message": "Make sure that const-class linkage is preserved.\n\nAfter calling loadClass() on the initiating loader, insert\nthe Class to the ClassTable. Check that the returned class\nhas the correct name and that the ClassTable does not yet\ncontain a different Class for the same name. If either check\nfails, follow the RI behavior for parallel capable class\nloaders, i.e. throw NoClassDefFoundError for name mismatch\n(with more descriptive message than RI) or return the Class\nalready present in the ClassTable. This ensures that the\nconst-class instruction for a given name always gives the\nsame result as the first time it succeeded for the same\ninitiating loader, even for non-well-behaved loader.\n\nInserting the Class to the ClassTable is also a necessary\nstep for changing to hash-based dex cache type array when\nthe DexCache will not necessarily keep the Class and\ntherefore the defining loader alive while the initiating\nloader stays alive, so the ClassTable entry is necessary\nto keep the dependecy.\n\nAdd tests that a const-class instruction always returns the\nsame Class\u003c?\u003e as it returned the first time, i.e. when it\nwas \"linked\", even for class loaders that are not\nwell-behaved, even if the dex cache type array is cleared.\nAlso test the edge cases, i.e. NoClassDefFoundError and\nreturning old entry from ClassTable when already inserted\nby another thread.\n\nBug: 30627598\nTest: 626-const-class-linking\nTest: m test-art-host\nChange-Id: Ie9b721464b4e9a5dcce8df8095548e983bba1fe8\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": "d98b4ed7a73af40f5292337495333616d9974da0", "tree": "6b7d64d1f85fa4df9f99602b85ca264bded8f950", "parents": [ "53e6c51bf1e2bd7f09f42b509017f4fac2fe9ee1" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Nov 04 20:27:24 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 07 10:00:59 2016 -0800" }, "message": "ART: Make excludes explicit\n\nFix some missing lock excludes.\n\nTest: m\nChange-Id: If74708c5d16469490e3374b7d5ffc23dd2ea5ec1\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": "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": "000e1885701c8fdad07a784e3771881fa9be1d8a", "tree": "77d11180fb1d93903fa6c6cd58383d91a82affc5", "parents": [ "b5cc1d1321e36d38cd48ae079438a8758b82edc1" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Oct 24 17:14:25 2016 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Fri Oct 28 14:11:45 2016 +0100" }, "message": "method_handles: Complete support for emulated stack frames.\n\nMost of this change is a refactor that templatizes the code\nthat performs argument conversions. This allows us to copy arguments\nbetween two shadow frames, or an emulated stack frame and a shadow\nframe.\n\nTest: make test-art-host\nBug: 30550796\n\nChange-Id: I23e65735a2dbd28f3c7b7d1ccf9762e77e0cf1f1\n" }, { "commit": "eb242cd311e5e93dd85042321a63d55e690873d4", "tree": "1c27a9759441e87a950d04c7d46b7d6f4039f958", "parents": [ "168c58ce18b139d313cbc6131e3bb43539952c55", "0802518a6a5af8182131eb3fe66bf58dd77f9fe2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 26 14:24:43 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Oct 26 14:24:43 2016 +0000" }, "message": "Merge \"Save the non-verified classes in the VerifierDeps.\"" }, { "commit": "0802518a6a5af8182131eb3fe66bf58dd77f9fe2", "tree": "94b84da33a0ef542f6c4c3888e786dc8762ff1b2", "parents": [ "f30b1582c20079a1266f1fdcac621d4df2d7ce79" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 25 17:20:18 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 26 12:41:10 2016 +0100" }, "message": "Save the non-verified classes in the VerifierDeps.\n\nWe will need that information when taking an OTA to make sure\nthe same set of classes needs to be verified at runtime.\n\nCurrently, the vdex file will contain a list of unverified\nclasses. We could alternatively encode a bit vector of the size\nof the type_id array, but the few experiments I did show that\nthe bit vector is actually larger. We can refine this later.\n\nbug: 30937355\ntest: m test-art-host\ntest: verifier_deps_test.cc\n\nChange-Id: I2670e4fd2e54ee7a148246baa705fda3a56617ff\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": "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": "1a1de67c4923436eb7de7da334c6d77703375dbd", "tree": "bf82b30cddfeaa822e14a63c0ad9fedd3886958d", "parents": [ "6e5fa09510c7280168e040382d27dd8b55760d9a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Oct 13 12:53:15 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Oct 14 11:25:28 2016 +0100" }, "message": "Do not write empty class table and intern table to image.\n\nAnd remove dead ClassLinker and InternTable code that would\nbe erroneously triggered by not writing the tables.\n\nTest: m test-art-host\nChange-Id: I2520daa8d1231e16112d44e364a72d782fa32527\n" }, { "commit": "7638651be29f117ef651841fbb9ffda76921ae1b", "tree": "5354cc999cc64dd2bfe78d60bef994059fcca2ba", "parents": [ "cf42ad62df2dda8b5a39cab50669a9446a880b41", "6c60d8420e51fda65ad247ae04b5a823c88c26b6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 19:59:08 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Oct 05 19:59:09 2016 +0000" }, "message": "Merge \"Initialize dex cache while holding dex_lock\"" }, { "commit": "6c60d8420e51fda65ad247ae04b5a823c88c26b6", "tree": "8c4222eadad97235b5f6d860fb63ab8a749d111a", "parents": [ "c13fbeb2201dc0bf783670d4548a49f03b1cb33e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 15 10:24:43 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 09:39:07 2016 -0700" }, "message": "Initialize dex cache while holding dex_lock\n\nFixes multiple threads calling RegisterDexFile occasionally getting\nDCHECK failures due to the arrays not being null since the BSS ones\nis per dex file.\n\nBug: 31369621\n\nTest: test-art-host, no DCHECK failure during debug booting\n\nChange-Id: I7b6e4cd03460dd1213eb4e044bdcf5f6103fd5f9\n" }, { "commit": "97d7e1cd7f733cb33a0e238bec6d7ed525638cd1", "tree": "d2f02420f82d76d10dc0d80f431c9f4aba03ac82", "parents": [ "eebe142e935e9444c7c947d02246e7fbe88b2cc4" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 04 14:44:28 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Oct 05 13:21:44 2016 +0100" }, "message": "Remove #include \"oat_file.h\" from class_linker.h .\n\nRefactor the OatClass and OatMethod related functions from\nClassLinker to OatFile and ArtMethod, respectively. Refactor\nthe remaining ClassLinker dependencies on OatFile to break\nthe #include dependency and reduce incremental build times.\n\nTest: m test-art-host\nChange-Id: Iebc5b9f81b48fbcf79821cc827a5d7c4a0261bf6\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": "25352fc06c84cdab8a2ab6d173b0514066ade2b9", "tree": "bb8a6be0c9a144ccf33b07b3158409769a32daa2", "parents": [ "51cbe84ca33800c57fef9c96f1155f60bcc70bba" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Wed Aug 03 12:46:58 2016 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Oct 03 09:48:50 2016 +0100" }, "message": "class_linker: Add support for resolving method types.\n\n- Add a new fixed size dex cache array for resolved method types.\n The size of this array is set to 1024.\n- Also introduces a new runtime flag that controls this feature.\n\nTest: make test-art-host\nBug: 30550796\n\nChange-Id: I147b33398d71ee21f2e91b418d3700d4630801ff\n" }, { "commit": "afa48270228832ecd7df4506915354d1a41f374a", "tree": "7d63c5bde346eb129f3159b07b2a66dfa86aeb41", "parents": [ "72795d0a3368a7f921ce2f470917131cd946b717" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Wed Aug 03 12:46:58 2016 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Sep 29 13:15:32 2016 +0100" }, "message": "java.lang.invoke: Add mirror types for MethodHandle / MethodType.\n\nBug: 30550796\nTest: make test-art-host\nChange-Id: I096160464bc6e84f7e5ad021306a7e462cf3b0c5\n" }, { "commit": "496577fb4d6d7baea9c093d35d17e208c831bbca", "tree": "1da9976883a8fea0dc05a58ec9bbd347b010fc19", "parents": [ "304533ed8475e9438e6ca7ef5fa644fd5dab5e97" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 20 15:33:31 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 23 15:27:26 2016 -0700" }, "message": "Smarter image layout\n\nPut strings in the dex file that resolves them.\n\nDepth first traversal with overrides for class and dex cache. The\nwork list keeps track of what oat_index with each pushed item. This\nmeans the static fields of a class will usually be in the same image.\n\nAdded layout test to image_test to make sure things are somewhat\nreasonably attributed.\n\nBug: 28640955\n\nTest: test-art-host\n\n(cherry picked from commit 4e9c4e746617bad6a012d799d2f5cf9e01d24ea2)\n\nChange-Id: I67a536c33aeed603b252d8e0f75622c9efbf2559\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": "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": "5fd66d0546805f0d86ec3365762c00b8dd3415f5", "tree": "82fb3cbb657f970a5c1d7c65e5567026816e284f", "parents": [ "6a8c69f9d9d7840cac8cd17057d58c4cf91da4c6" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 12 20:22:19 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 12 20:25:01 2016 -0700" }, "message": "ART: Remove LogSeverity NONE\n\nRemove non-standard severity, and instead use a new enum to\nconfigure verifier logging.\n\nPartially reverts commit 7fe30233d32a872f5349e921dba65bd767ad994b.\n\nBug: 31338270\nTest: m test-art-host\nChange-Id: If52ee23c4902fbf2dabe7b232f6742448764ff8c\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": "65975776f807d55c83af6cca1e447f8daa794413", "tree": "3b30b9a6ba00e72df89f59387c383ef747772025", "parents": [ "3f3201a89ec19257b3bc93c25b20abdcfe61f3e4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 05 10:46:36 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Aug 09 10:45:18 2016 -0700" }, "message": "Card mark holding class instead of declaring class\n\nFor profiling info, we need to mark the card of the holding class\ninstead of declaring class. This is required for GC correctness since\nthe GC relies on the card table to track cross space references.\n\nTest: test-art-host ART_TEST_JIT\u003dtrue\n\nBug: 30655270\n\nChange-Id: Ia4690219ded0df38032b644440273e06bc303956\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": "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": "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": "abdda2fb52eb7e9a6916899a913f78251d8ed277", "tree": "165a31831ae077827b1e7b973bd5602125b5b142", "parents": [ "2b87a47c09823031c4fe336dcef0867605debfd7", "badee9820fcf5dca5f8c46c3215ae1779ee7736e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jun 15 17:44:14 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jun 15 17:44:14 2016 +0000" }, "message": "Merge \"Optimize IMT\"" }, { "commit": "badee9820fcf5dca5f8c46c3215ae1779ee7736e", "tree": "982948a67a88a1f9a734c935f919f8d307969f48", "parents": [ "614968198625a6693666bdc1e5609e2f663f5638" ], "author": { "name": "Nelli Kim", "email": "nelli.kim@samsung.com", "time": "Fri May 13 13:08:53 2016 +0300" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Tue Jun 14 10:05:47 2016 -0700" }, "message": "Optimize IMT\n\n* Remove IMT for classes which do not implement interfaces\n* Remove IMT for array classes\n* Share same IMT\n\nSaved memory (measured on hammerhead):\nboot.art:\nTotal number of classes: 3854\nNumber of affected classes: 1637\nSaved memory: 409kB\n\nChrome (excluding classes in boot.art):\nTotal number of classes: 2409\nNumber of affected classes: 1259\nSaved memory: 314kB\n\nGoogle Maps (excluding classes in boot.art):\nTotal number of classes: 6988\nNumber of affected classes: 2574\nSaved memory: 643kB\n\nPerformance regression on benchmarks/InvokeInterface.java benchmark\n(measured timeCall10Interface)\n1st launch: 9.6%\n2nd launch: 6.8%\n\nChange-Id: If07e45390014a6ee8f3c1c4ca095b43046f0871f\n" }, { "commit": "ece5386eeb30aa6be663af2416b68a7ebf9bd42e", "tree": "6a951fc8c1a2358591327d107828d7d48ee47ea4", "parents": [ "614968198625a6693666bdc1e5609e2f663f5638", "7a62e6728463237684d3d9834d81bd7bba7ab197" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Jun 13 23:11:33 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jun 13 23:11:33 2016 +0000" }, "message": "Merge \"Propagate the read barrier option to Class::VisitNativeRoots.\"" }, { "commit": "7a62e6728463237684d3d9834d81bd7bba7ab197", "tree": "7f53e92a3303475cb8c164d98274d465bd32ef00", "parents": [ "1e5dda549c11d02aff51f132760dd693e351ce85" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Jun 10 17:22:48 2016 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Jun 13 13:57:34 2016 -0700" }, "message": "Propagate the read barrier option to Class::VisitNativeRoots.\n\nPropagate the read barrier option from Object::VisitReferences to\nClass::VisitNativeRoots.\n\nThis is a step toward the GC thread avoiding graying objects (and\nreducing dirty pages) in the immune spaces.\n\nBug: 12687968\n\nChange-Id: I29c4126a4ad4c40e63a934e62451fb3fb36aad43\n" }, { "commit": "c92a7a14ce44c4bb7e63e4c447a008b558bc0bca", "tree": "c48ebef140dd292cc2cc29726045c03f5b2b02ea", "parents": [ "6c54dfb86a6cd6f63a9518893562595460b480ad" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Jun 06 11:09:20 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Jun 07 20:04:02 2016 -0700" }, "message": "Wrap certain exception types when loading an erroneous class.\n\nBug: 28787733\n\n(cherry-picked from commit 7c8aa8357196781c811a73d2eb66aaaa1681ce36)\n\nChange-Id: Iea55486c4b95ee16e1f19c8ba2d24c18b9100c97\n" }, { "commit": "696632e6113a379f24dc6fe4c38e4dee57a249f8", "tree": "2bc5da7e66573bd2a50bda6db03bc2be1aa1b036", "parents": [ "6d335635fc8b575261be4a7efea0ea5f4adb9fac" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jun 03 17:47:32 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 06 13:09:09 2016 -0700" }, "message": "Revert \"Revert \"Hold dex caches live in class table\"\"\n\nBug: 29083330\n\nThis reverts commit f102faf1bcbdb2149e3e7bf27b1819f621b7894b.\n\n(cherry picked from commit c9dbb1df3b5c06ba122cacaf35b17cb53c6be3c6)\n\nChange-Id: Ib5d82bde2e4a031c1cab0ea6116925ef99d9f07f\n" }, { "commit": "6d335635fc8b575261be4a7efea0ea5f4adb9fac", "tree": "9b3d2c06470319fbb91d683efac163dfd263888d", "parents": [ "55554a9c0a5ea80b0a6e361be75944d560e75197" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sat Jun 04 00:22:32 2016 +0000" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 06 13:08:51 2016 -0700" }, "message": "Revert \"Hold dex caches live in class table\"\n\nThis reverts commit d6d49e56c2b7b11f474acb80cb02bb1fe9b7861e.\n\nBug: 29083330\n\n(cherry picked from commit f102faf1bcbdb2149e3e7bf27b1819f621b7894b)\n\nChange-Id: Id0c921bd6fe422159f8daa598dc8e7c9ed6eca47\n" }, { "commit": "f284d448e3edd428b6ade473d0993028638b2064", "tree": "e24f57f952f4e6c850ec910de3361aef6340727a", "parents": [ "3f432d5a7c184b7580bd5aba27158c1455c328ff" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 02 11:48:30 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 02 18:03:41 2016 -0700" }, "message": "Hold dex caches live in class table\n\nPrevents temporary dex caches being unloaded for the same dex file.\nUsually this is OK, but if someone resolved a string in that dex\ncache, it could leave stale pointers in BSS. Also it can use extra\nmemory in linear alloc if we allocate dex cache arrays multiple\ntimes.\n\nBug: 29083330\nChange-Id: Ia44668f013ceef1f5eb80f653a48d0f8004548c9\n" }, { "commit": "f0192c86a58b2f43378c9a2113007538dd38ddbf", "tree": "df3fef8e9ccd30152819e893e6212ce45b5f51a5", "parents": [ "783d02e527107e91880f12bddf41afd313919132" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Mar 28 20:39:50 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue May 17 11:08:41 2016 -0700" }, "message": "Support to pass \u003cuses-library\u003e option through to dex2oat.\n\nThis change takes an app\u0027s shared libraries specified by \u003cuses-library\u003e\nand passes it through to dex2oat to be used during compilation.\n\nPart of a multi-project change.\n\nBug: 26880306\n\n(cherry-picked from commit 26e8a2f150cd7f7195a10650ab8a5b6fa5014bc8)\n\nChange-Id: I72a352abdfc37eacd8bedfa6c218e3809ca8e39c\n" } ], "next": "ab880f4371e1ae5239c1b887911bd88f0c216a3d" }