)]}' { "log": [ { "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": "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": "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": "35e42f0ab3b70203038fe037ee50d39e2d37af9a", "tree": "2c8e6fd5a88ce40dff7daf078954832bebec46a0", "parents": [ "46e67be2da4623025c306b3bd21b96ce10ae1b6e" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Jun 26 18:14:39 2017 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Jun 27 14:13:14 2017 +0100" }, "message": "Only look for method optimization annotations in the bootstrap class loader.\n\nWhen checking for an optimization annotation (FastNative or\nCriticalNative) on a method, do not resolve the method\u0027s annotations\u0027\nclasses as a side effect -- instead, look them up in the bootstrap\nclass loader\u0027s resolved types. This is to prevent exceptions from being\nthrown (during class resolution) in JNI transitions.\n\nThis change does not affect annotation lookup rules in the context of\nreflection.\n\nTest: art/test/testrunner/testrunner.py -t 656-annotation-lookup-generic-jni\nBug: 38454151\nBug: 34659969\nChange-Id: Ie6b8b30b96a08baa629c449e3803a031515508d1\n" }, { "commit": "0d2323e9fc700388ac0134390819c653c7893204", "tree": "d8430611d1866a7126a0436aa9dbe87fe1f180ea", "parents": [ "80b99c23b09bf1c4ca49385cff15879d821aecb9" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Jun 26 18:14:39 2017 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Jun 26 23:27:41 2017 +0100" }, "message": "Fix the stack at the beginning of the Generic JNI trampoline.\n\nFix up a callee-save frame at the bottom of the stack while we\ncheck for optimization annotations, thus allowing stack walking\nuntil the completion of the JNI frame creation.\n\nTest: art/test/testrunner/testrunner.py -t 656-annotation-lookup-generic-jni\nBug: 38454151\nBug: 34659969\nChange-Id: I70dc3d40139198f29457dba5282b45a9d0d09e49\n" }, { "commit": "b7edcda968bb0cbaa69a3ad387fcd3194f5612be", "tree": "50748aac195d20a7e8f6d59a6e499f3fe69418de", "parents": [ "049f2a58ea9276dfd162760271ad443570f2e660" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Apr 27 13:20:31 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Jun 09 10:54:02 2017 -0700" }, "message": "Add method tracing JVMTI callbacks\n\nAdd MethodEntryHook and MethodExitHook callbacks and associated\ncapabilities.\n\nSplit --jvmti-stress option in run-test into --jvmti-trace-stress and\n--jvmti-redefine-stress to test each different component.\n\nNB 3 differences from RI found:\n 1) RI will call methodExitHook again if the method exit hook throws\n an exception. This can easily cause an infinite loop and the test\n is specifically tweaked to prevent this from happening on the RI.\n 2) RI always includes the method being exited in the stack trace of\n errors thrown in the hooks. In ART we will not include the method\n if it is native. This is due to the way we call native methods\n and would be extremely difficult to change.\n 3) The RI will allow exceptions thrown in the MethodEnterHook to be\n caught by the entered method in some situations. This occurs with\n the tryCatchExit test in 989. In ART this does not happen.\n\nBug: 34414073\nTest: ./test.py --host -j40\nTest: ART_TEST_FULL\u003dtrue DEXTER_BINARY\u003d\"/path/to/dexter\" \\\n ./test/testrunner/testrunner.py --host -j40 -t 988\nTest: ART_TEST_FULL\u003dtrue DEXTER_BINARY\u003d\"/path/to/dexter\" \\\n ./test/testrunner/testrunner.py --host -j40 -t 989\nTest: lunch aosp_angler-userdebug; \\\n m -j40 droid build-art \u0026\u0026 \\\n fastboot -w flashall \u0026\u0026 \\\n ./test.py --target -j4\n\nChange-Id: Iab229353fae23c2ea27c2b698c831627a9f861b1\n" }, { "commit": "8228cdf4ad6322ec8133564aaa51f966d36c0f17", "tree": "6d2f5cf4e742ad644ad30eb96a9f943c9ebcb34b", "parents": [ "83b140474aa1759739c8ee4464bf226c4fa0f6d7" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue May 30 15:03:54 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 08 13:13:15 2017 -0700" }, "message": "ART: Move CalleeSaveType to callee_save_type.h\n\nMove the type out of runtime to decrease dependencies. Make it\na proper enum class. Fix up usage.\n\nTest: m test-art-host\nChange-Id: Id84c44bf3c59f37c8a43548e720c5fb65707b198\n" }, { "commit": "0eb882bfc5d260e8014c26adfda11602065aa5d8", "tree": "e66dbebfb1e9a254c20954a2f2f98541aebfd5af", "parents": [ "b5f5d746ac3f2c3088292395603cb1470e7749d2" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon May 15 13:39:18 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jun 07 15:39:24 2017 +0100" }, "message": "Use ArtMethod* .bss entries for HInvokeStaticOrDirect.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: testrunner.py --target\nTest: Nexus 6P boots.\nTest: Build aosp_mips64-userdebug.\nBug: 30627598\nChange-Id: I0e54fdd2e91e983d475b7a04d40815ba89ae3d4f\n" }, { "commit": "513061a792b22c417c938d31c19581390709561c", "tree": "80b4fdce03711170626aa5640d07b07de4a326a1", "parents": [ "38c4ae5f4c5a033b7a7441032f39ea58f5772d4c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 01 09:17:34 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 01 12:59:13 2017 -0700" }, "message": "ART: Clean up thread.h and thread_list.h\n\nRemove dependency on stack.h and gc_root.h. Remove unused object\ncallbacks include. Factor out ManagedStack into its own set of files.\nFix up users of transitive includes.\n\nTest: m test-art-host\nChange-Id: I01286c43d8c7710948c161b1348faabb05922e59\n" }, { "commit": "d78ddec5f8eaf1f27e9043f6f42be90149ccb966", "tree": "e3c21f93e419ba35486cf8f641d816523a06b9a3", "parents": [ "66f43b988ad84568a46c1760b314723e9229e6d0" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Apr 18 15:20:38 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Apr 19 09:44:25 2017 -0700" }, "message": "Implement can_generate_native_method_bind capability\n\nThis capability lets one observe and even replace the implementations\nof native methods when they are bound.\n\nTest: ./test.py --host -j40\n\nBug: 37432636\nChange-Id: I2432a8e4da1a677e8011ce495296f4ab9f42eb3e\n" }, { "commit": "9e6dee2569ba3ec393b91ee4afe0f63a4daecfa9", "tree": "a4349d6188ed7ad2a4c35bb1cc3be7a50c74df68", "parents": [ "bec07a05dd1aa564895f0d9c58c8277c576dc9bb" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Apr 11 13:50:23 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Apr 11 13:50:23 2017 -0700" }, "message": "ART: Move trampoline logging to -verbose:dex\n\nMove invoke logging to VLOG(dex).\n\nBug: 37158297\nTest: m test-art-host\nChange-Id: Ib29b3920e8f227395b99a9ed65de5ca10c737cc0\n" }, { "commit": "cbcedbf9382bc773713cd3552ed96f417bf1daeb", "tree": "58eed766a69ddddbc41d21624244e14547fa1dff", "parents": [ "ef62210a1466c133056c4bfdae9f8f5417065eff" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Mar 12 22:24:50 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Mar 16 10:20:34 2017 -0700" }, "message": "Add method info to oat files\n\nThe method info data is stored separately from the code info to\nreduce oat size by improving deduplication of stack maps.\n\nTo reduce code size, this moves the invoke info and inline info\nmethod indices to this table.\n\nOat size for a large app (arm64): 77746816 -\u003e 74023552 (-4.8%)\nAverage oat size reduction for golem (arm64): 2%\n\nRepurposed unused SrcMapElem deduping to be for MethodInfo.\nTODO: Delete SrcMapElem in a follow up CL.\n\nBug: 36124906\n\nTest: clean-oat-host \u0026\u0026 test-art-host-run-test\n\nChange-Id: I2241362e728389030b959f42161ce817cf6e2009\n" }, { "commit": "5667f56867383fc4113aa4a6551efdf9f48ee5e7", "tree": "fed4de9fbacff02d86afdeb93cee54a7174d52b0", "parents": [ "68948e01f56ad1996af77f4c0aab721940b0e18d" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Feb 27 19:32:01 2017 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Mar 08 16:46:37 2017 -0800" }, "message": "Modify invoke interface trampoline to pass interface method.\n\nThis avoids touching the code item in the case that the interface method\nis resolved and in the dex cache. If it\u0027s not, the trampoline will still\nhave to go to the code item to try to resolve it.\n\nBug: 35800981\nTest: mm test-art-host\nChange-Id: Ia0e1d23429b3b928bcec727f4f2f16b7834408e5\n" }, { "commit": "d776ff08e07494327716f0d2ea1a774b2ebfbca9", "tree": "cedf874dd494d881adc572a10a9d14bca852add6", "parents": [ "3fb852a88d2a8ffaa87089752f4b1d5f9d6ce3c1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 17 09:32:18 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 15 14:46:15 2017 -0800" }, "message": "Add invoke infos to stack maps\n\nInvoke info records the invoke type and dex method index for invokes\nthat may reach artQuickResolutionTrampoline. Having this information\nrecorded allows the runtime to avoid reading the dex code and pulling\nin extra pages.\n\nCode size increase for a large app:\n93886360 -\u003e 95811480 (2.05% increase)\n\n1/2 of the code size increase is from making less stack maps deduped.\nI suspect there is less deduping because of the invoke info method\nindex.\n\nMerged disabled until we measure the RAM savings.\n\nTest: test-art-host, N6P boots\n\nBug: 34109702\n\nChange-Id: I6c5e4a60675a1d7c76dee0561a12909e4ab6d5d9\n" }, { "commit": "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": "575d3e60c68b5cf481b615dde4a16283507b19ed", "tree": "4718f2747233e74268de21d804df9bfe0b7e4362", "parents": [ "857acf51fbc5a931939c20e9e299c69676baf654" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 06 11:00:40 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 08 09:51:27 2017 -0800" }, "message": "Clean up code info table layout\n\nPreviously:\nTable layout was computed multiple places like stack_map_stream,\nand getters. This made it difficult to add new stack map tables and\nmade the code hard to understand.\n\nThis change makes the table layout specified all inside of the code\ninfo. Updating the layout only requires changing ComputeTableOffsets.\n\nChanged the stack map inline info offset to be an index, so that it is\nnot require the inline infos are directly after the dex register table.\n\nOat file size for a large app: 94459576 -\u003e 93882040 (-0.61%)\n\nUpdated oatdump and fixed a bug that was incorrectly computing the\nregister mask bytes.\n\nBug: 34621054\n\nTest: test-art-host\n\nChange-Id: I3a7f141e09d5a18bce2bc6c9439835244a22016e\n" }, { "commit": "433b79a9130df5f2375b9d3d7518c1f3fb84a467", "tree": "dddc053886464b99be29b9ca69039d74d49605ca", "parents": [ "81356645157af44152c7b7db383596b5cf3479b5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 20:54:45 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 20:54:45 2017 +0000" }, "message": "Revert \"Revert \"Make --debuggable rely on JIT code.\"\"\n\nFixed by https://android-review.googlesource.com/#/c/330165/\n\nThis reverts commit 81356645157af44152c7b7db383596b5cf3479b5.\n\nChange-Id: Ifb74e1cc90ab6dea621f7f54a00b540d6ccd0cf6\n" }, { "commit": "81356645157af44152c7b7db383596b5cf3479b5", "tree": "59aca3792abfa47c7424e4dac13248651e4662b2", "parents": [ "a0619e25aacf8b8074132f4951f75fdbcfd42925" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 20:18:02 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 20:19:28 2017 +0000" }, "message": "Revert \"Make --debuggable rely on JIT code.\"\n\nBreaks 137-cfi with ART_TEST_RUN_TEST_DEBUGGABLE\u003dtrue\n\nThis reverts commit a0619e25aacf8b8074132f4951f75fdbcfd42925.\n\nbug: 28769520\nChange-Id: Ifd7b8fc7c9b72c0a523fd57c4b1b80edd3547caa\n" }, { "commit": "a0619e25aacf8b8074132f4951f75fdbcfd42925", "tree": "dddc053886464b99be29b9ca69039d74d49605ca", "parents": [ "ca21dc47adeed92a15a9d3fd090bdd0e6654679c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 20 13:57:43 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 12:41:52 2017 +0000" }, "message": "Make --debuggable rely on JIT code.\n\nRemoves -Xfully-deoptable in the process, which was added as a\ntemporary workaround until this CL.\nPartial revert of https://android-review.googlesource.com/#/c/302232/\n\nMakes things consistent with existing infrastructure:\n\n- Parse the --debuggable from the compiler options, just like\n --compiler-filter.\n\n- Add DEBUG_JAVA_DEBUGGABLE, passed by the zygote, for debuggable apps.\n\n- Java debuggable now solely relies on JIT for simplicity.\n\n- Debugging under userdebug for non-java-debuggable apps is still\n best effort.\n\nTest: test-art-host, jdwp\n\nbug: 28769520\nChange-Id: Id0593aacd85b9780da97f20914a50943957c858f\n" }, { "commit": "fea84ddbccae1c1242d6a1815f6580d758bae8c7", "tree": "68ac300b7c09e3aaadc7e103b1bd03f5f1985304", "parents": [ "a48d74a59af68d45e5c9b9273c2017ae2470d58a" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Jan 16 13:52:20 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Jan 16 13:52:20 2017 +0000" }, "message": "ART: Fix artInvokePolymorphic.\n\nartInvokePolymorphic called the wrong visit method on\nRememberForGcArgumentVisitor. It should have called VisitArguments()\nrather than Visit().\n\nBug: 30550796,33191393\nTest: art/test/run-test 953\nTest: m test-art-run-test\nChange-Id: I46442203e8dc7000efd2cd64b2dcd5ed83163be6\n" }, { "commit": "ac141397dc29189ad2b2df41f8d4312246beec60", "tree": "a2f481463a14695bf9327fd2f549878ecf30c77b", "parents": [ "5c9f90c5ecf2ff6f93ada0f7b18b46d866c59ea1" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Jan 13 11:53:47 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Sun Jan 15 15:18:07 2017 +0000" }, "message": "Revert \"Revert \"ART: Compiler support for invoke-polymorphic.\"\"\n\nThis reverts commit 0fb5af1c8287b1ec85c55c306a1c43820c38a337.\n\nThis takes us back to the original change and attempts to fix the\nissues encountered:\n\n- Adds transition record push/pop around artInvokePolymorphic.\n- Changes X86/X64 relocations for MacSDK.\n- Implements MIPS entrypoint for art_quick_invoke_polymorphic.\n- Corrects size of returned reference in art_quick_invoke_polymorphic\n on ARM.\n\nBug: 30550796,33191393\nTest: art/test/run-test 953\nTest: m test-art-run-test\n\nChange-Id: Ib6b93e00b37b9d4ab743a3470ab3d77fe857cda8\n" }, { "commit": "dcf52765ab5886abdd85a4436fa0358b2a31341d", "tree": "b68ee975792a5bf488ed93cfbe09a37f80008288", "parents": [ "a28ddf5140cd1f4a2ae93dbf8be2f200b1552003", "0fb5af1c8287b1ec85c55c306a1c43820c38a337" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 19:18:51 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 11 19:18:51 2017 +0000" }, "message": "Merge \"Revert \"ART: Compiler support for invoke-polymorphic.\"\"" }, { "commit": "0fb5af1c8287b1ec85c55c306a1c43820c38a337", "tree": "66239e7f745fae54e1630e91fb44a859bff615d6", "parents": [ "02e3092f8d98f339588e48691db77f227b48ac1e" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 18:58:15 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 19:17:33 2017 +0000" }, "message": "Revert \"ART: Compiler support for invoke-polymorphic.\"\n\nThis reverts commit 02e3092f8d98f339588e48691db77f227b48ac1e.\n\nReasons for revert:\n\n- Breaks MIPS/MIPS64 build.\n- Fails under GCStress test on x64.\n- Different x64 build configuration doesn\u0027t like relocation.\n\nChange-Id: I512555b38165d05f8a07e8aed528f00302061001\n" }, { "commit": "79f9928fc9e0a88430f3329069bfb2f9a0d37f0c", "tree": "e3142e4829c808c3df1059f3b05c0b3a37193ce9", "parents": [ "716eb25353390f699778a79d69006a5b8d8289c2", "02e3092f8d98f339588e48691db77f227b48ac1e" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Jan 11 18:08:03 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jan 11 18:08:04 2017 +0000" }, "message": "Merge \"ART: Compiler support for invoke-polymorphic.\"" }, { "commit": "02e3092f8d98f339588e48691db77f227b48ac1e", "tree": "127dd23346206b0547b7c6453a776253252b3c6e", "parents": [ "bc7d0deda4549f314e68ee3e0e6afd68c4a8fd06" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Dec 01 10:33:51 2016 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Jan 11 16:26:43 2017 +0000" }, "message": "ART: Compiler support for invoke-polymorphic.\n\nAdds basic support to invoke method handles in compiled code.\n\nEnables method verification for methods containing invoke-polymorphic.\n\nAdds k45cc/k45rc output to Instruction::DumpString() which\nwas found to be missing when enabling verification.\n\nInclude stack traces in test 957-methodhandle-transforms for\nfailures so they can be easily identified.\n\nBug: 30550796,33191393\nTest: art/test/run-test 953\nTest: m test-art-run-test\nChange-Id: Ic9a96ea24906087597d96ad8159a5bc349d06950\n" }, { "commit": "f5769e15aa9f058a5af987f0d1354f16f4464a7d", "tree": "8a55c514c70a701a06102d3cbda6e4fe8a84242c", "parents": [ "2e17788df086284845108c6b39954a5c74e93a69" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 10 15:54:41 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 10 15:57:11 2017 -0800" }, "message": "Move most of art::Thread to ObjPtr\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I73d0de3c875d3ec3d732034fdc961dae79ef4070\n" }, { "commit": "1b8464d17c2266763714ae18be7c4dc26e28bf61", "tree": "19ab81b439fc216e289cb14be8a7618dbafb4f50", "parents": [ "d7a7c7f3e93de9fa915e66d54dfc799efcc12ffb" ], "author": { "name": "Alexey Frunze", "email": "Alexey.Frunze@imgtec.com", "time": "Sat Nov 12 17:22:05 2016 -0800" }, "committer": { "name": "Alexey Frunze", "email": "Alexey.Frunze@imgtec.com", "time": "Tue Dec 06 13:57:04 2016 -0800" }, "message": "MIPS32: Pass more arguments in registers.\n\nSpecifically, use A0-A3,T0-T1 for non-floats and F8-F19 for floats.\n\nTest: booted MIPS32R2 in QEMU\nTest: test-art-target-run-test-optimizing (MIPS32R2) on CI20\nTest: test-art-target-gtest (MIPS32R2) on CI20\nTest: booted MIPS64 (with 2nd arch MIPS32R6) in QEMU\nTest: test-art-target-run-test-optimizing (MIPS32R6) in QEMU\nTest: test-art-target-gtest (MIPS32R6) in QEMU\nTest: test-art-host-gtest\n\nChange-Id: Ib8b0310a109d9f3d70119c1e605e54b013e60728\n" }, { "commit": "ef41db7a3f322a1feb305fdb457410c4cea94d00", "tree": "c03152c091ef62ab70f5be0a2fe3a965b189132c", "parents": [ "1458e0c09fe0a3b9fa5fd7beb9b6077d1fc46b1d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 25 15:08:01 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 26 15:25:40 2016 -0700" }, "message": "Move interpreter to ObjPtr\n\nMoved most of interpreter, interpreter_common,\ninterpreter_switch_impl, and some of mterp to ObjPtr.\n\nBug: 31113334\n\nTest: test-art-host ART_TEST_INTERPRETER\u003dtrue\nTest: art/tools/run-libcore-tests.sh \u0027--mode\u003dhost\u0027 \u0027--variant\u003dX32\u0027 --debug\n\nChange-Id: I0935d18287e1332205c17c5a018aa167788ab897\n" }, { "commit": "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": "1432a5bb0608a920e7281b38ee5f6e8dfcfae5ef", "tree": "804b9e78fa9d2705a1e58571944af7381858a5b0", "parents": [ "4aa6a93c46a959df1ab71ee7a68ad345338046ef" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 04 15:41:42 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 09:43:15 2016 -0700" }, "message": "Don\u0027t push handle scope for critical native generic JNI\n\nLeaving a stale handle scope causes problems for the GC the next\ntime roots are visited. At this point the stack will have other\ncontents and the GC will attempt to mark many invalid roots.\n\nBug: 31933313\n\nTest: non preopt eng build booting.\nTest: test-art-host\n\n(cherry picked from commit 92879f9bbcfc034660ed1ff5ef741d7f2bcb116f)\n\nChange-Id: I7d57964ccd2b59a05bb06f67062f646362ce1204\n" }, { "commit": "1cc62e4ea24828fdb3f3da0b8603f0b107d09a04", "tree": "718e322f3d05ac095770d4dc2a68b824bf3974a7", "parents": [ "82d4838d6bb3480cd25327cedc5179fb2d86881c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 03 18:01:28 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 04 10:46:16 2016 -0700" }, "message": "Rename ObjPtr::Decode to ObjPtr::Ptr\n\nDone to prevent ambiguity with ScopedObjectAccess::Decode.\n\nBug: 31113334\n\nTest: test-art-host\nChange-Id: I07a2497cc9cf66386311798933547471987fc316\n" }, { "commit": "0795f23920ee9aabf28e45c63cd592dcccf00216", "tree": "ff3f880c5e84f3316532b47d0e9a7729ade848ac", "parents": [ "d1224dce59eb0019507e41da5e10f12dda66bee4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 27 18:43:30 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 29 17:31:09 2016 -0700" }, "message": "Clean up ScopedThreadStateChange to use ObjPtr\n\nAlso fixed inclusion of -inl.h files in .h files by adding\nscoped_object_access-inl.h and scoped_fast_natvie_object_access-inl.h\n\nChanged AddLocalReference / Decode to use ObjPtr.\n\nChanged libartbenchmark to be debug to avoid linkage errors.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I4d2e160483a29d21e1e0e440585ed328b9811483\n" }, { "commit": "75a7db67f0e56bc0ccc63df4a6a1bb04ab3e86b1", "tree": "2d426df6f2807606811c45ae22597e2098dd8a2f", "parents": [ "fd8339e58e3c09097165feeed7a2d70482d40130" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 26 12:04:26 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Sep 28 09:00:08 2016 -0700" }, "message": "ART: Factor out IMT from ArtMethod\n\nMove IMT Conflict Table to its own file. Move IMT index computation to\nImTable.\n\nIn preparation for code using this independent of specific ArtMethods.\n\nBug: 31594153\nTest: m test-art-host\nChange-Id: Ifd6e304bb86724c5284984c4655c43d3af357f84\n" }, { "commit": "06a04e0e776875303577c2d871b53a53c78da1b5", "tree": "6f8b38fa918f49cc502b573c96d9a6338ad343df", "parents": [ "48d10ee126838f060aa6dcd304fc161b57bc14af" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Sep 13 15:57:37 2016 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Sep 21 13:32:00 2016 -0700" }, "message": "jni: Implement generic JNI support for @CriticalNative/@FastNative\n\nAlso:\n* Extend existing test cases to check generic JNI for the above.\n* Re-enable previously blacklisted @CriticalNative JNI run-tests.\n\nBug: 31400248\nChange-Id: I436ed00c8b8880e936a0c3483bc0dc251f0c0ce2\n" }, { "commit": "be08cf5782c04d876d38be8b8d1caf1b4c3fbe55", "tree": "3efb73eb12ccb967003157d30670c5a283ddc8b4", "parents": [ "32772cbdbcb35f5475b01f31314a3c7289bdb589" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 13 13:41:24 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 14 16:53:11 2016 -0700" }, "message": "Fix moving GC bugs in entrypoints\n\nIn FindMethodFromCode, there was missing handles for objects used\nafter a suspend point (ResolveType).\n\nFor check_jni, JniMethodEndWithReferenceHandleResult did not\nhandlarize the return value when calling CheckReferenceResult.\n\nFor CheckReferenceResult, the argument was not in a handle.\n\nBug: 31458474\nChange-Id: I06617fc63f1fb63d7c13ad4245223a8c54fa4b98\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": "fd36f1f927c138575184a1f4c7ea4e7abb3e2dbf", "tree": "2debb35c9193ac019071e0625986698ccbcaabb8", "parents": [ "c218427ab96e521e0c4e9a3ffeb87e6c57eea0a5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Aug 03 18:49:58 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Aug 10 14:19:49 2016 +0100" }, "message": "Rename callee save enumerators.\n\nAnd related image method enumerators, macros, etc.\nClean up some entrypoint assembly comments.\n\nThis is a follow-up to\n https://android-review.googlesource.com/252348\n\nTest: Run ART test suite on host and Nexus 9.\nBug: 30212852\nChange-Id: I2707342d4255c88c547655be83ed97a67e12ae9e\n" }, { "commit": "542451cc546779f5c67840e105c51205a1b0a8fd", "tree": "11e09bb5abaee12dddffefbe7e425291076dfa7a", "parents": [ "85c4a4b8c9eabfe16e4e49f9b4aa78c1bf4be023" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 26 09:02:02 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Aug 01 18:54:48 2016 -0700" }, "message": "ART: Convert pointer size to enum\n\nMove away from size_t to dedicated enum (class).\n\nBug: 30373134\nBug: 30419309\nTest: m test-art-host\nChange-Id: Id453c330f1065012e7d4f9fc24ac477cc9bb9269\n" }, { "commit": "e01e3644205c3644546889237c20185391a0092e", "tree": "413fcf279b415a305605a601297e3856f13a2948", "parents": [ "de4cf16f467b531373560ca6eb785f0f36606aae" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 25 13:06:04 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 25 13:09:41 2016 -0700" }, "message": "ART: Add pointer-size template to some reflection functions\n\nThe unstarted runtime may run code for a different pointer size,\neven when no transaction is active (e.g., during startup). To\nretain performance when the runtime is up and executing under\nnormal conditions, add a template parameter and use sizeof(void*)\nin places where it is adequate.\n\nFor maintainability, it is necessary to drop the default for\nthe transaction template parameter. Implicit conversions from\nbool to size_t may lead to incorrect code and hard to diagnose\nproblems. So instead ensure that all callers must give all\ntemplate parameter values.\n\nTest: m test-art-host\nChange-Id: I3076883422c8553ede4de5642409c5684a5a9aa8\n" }, { "commit": "465ecc86ff65ca546629630c9469deb6d2d8137e", "tree": "3a0ed3f2d3ab9fd2cffec44a659036efab35fb4e", "parents": [ "bae13af2fc2fa89985d0466cedf155cb96767910" ], "author": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Tue Jul 19 21:32:52 2016 +0000" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Tue Jul 19 21:48:06 2016 +0000" }, "message": "Revert \"Revert \"Refactor GetIMTIndex\"\"\n\nOriginally reverted in order to revert\nhttps://android-review.googlesource.com/#/c/244190/\nbut can now be merged again.\n\nThis reverts commit d4ceecc85a5aab2ec23ea1bd010692ba8c8aaa0c.\n\nTest: m test-art-host\n\nChange-Id: Id9205f2b77a378fc0f06088e78c66e81a49f712d\n" }, { "commit": "83e4903f9f966b52d758b52ac1d522e13fd43e60", "tree": "966df75b9d69f43bb18ece89b1ef16db6c546e45", "parents": [ "dadc075149eb322bf28ba35a75a70157147c1c30", "a62cb9bb6cb2278cb41ab0664191623e178c6a4f" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Jul 08 20:06:58 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jul 08 20:06:58 2016 +0000" }, "message": "Merge \"Revert \"Revert \"Optimize IMT\"\"\"" }, { "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": "5bf7bacc0993a2efd9b0765813928ff72ea4bb9f", "tree": "c8d6df8d62ea03664199420d2866cb66c3fdf945", "parents": [ "a5f0a1ab3f632f1e9c0bc048604aa0c0e98a4a93" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 14:18:23 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 15:50:18 2016 +0100" }, "message": "Revert \"Revert \"Inline and optimize interface calls.\"\"\n\nThis reverts commit 19dc255bf94a4229de8627a2079ee6f0e9005e2d.\n\nChange-Id: Ifc3a92280878d4db1b460d486137497b3456beae\n" }, { "commit": "19dc255bf94a4229de8627a2079ee6f0e9005e2d", "tree": "38ce4ec6ab0ae5ed99f16c695441135fd36d26dd", "parents": [ "d9faceaa8da92f4a56c2907de949081bd42faf79" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 14:01:50 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 14:01:50 2016 +0000" }, "message": "Revert \"Inline and optimize interface calls.\"\n\nWent ahead too quickly.\n\nThis reverts commit d9faceaa8da92f4a56c2907de949081bd42faf79.\n\nChange-Id: I1610deaf89b38037cf2786d135a59c48b10ced0c\n" }, { "commit": "d9faceaa8da92f4a56c2907de949081bd42faf79", "tree": "2efc00cfe8d5211e420174db81e732f8e92cbb33", "parents": [ "74c0d1bb67f9c6ee8306f0318ab7251d56dc99d6" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 10:19:23 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 13:30:13 2016 +0100" }, "message": "Inline and optimize interface calls.\n\n- Support for inlining methods with invoke-interface (the\n previous performance limitation is now fixed with the new\n ImtConflictTable).\n- Turn non inlineable invoke-interface into invoke-virtual to\n avoid any potential execution of the conflict trampoline.\n\ntest:609-checker-inline-interface\n\nChange-Id: Ibe33bc945deaada8fb78541e6e0bf034ebb52420\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": "d4ceecc85a5aab2ec23ea1bd010692ba8c8aaa0c", "tree": "982948a67a88a1f9a734c935f919f8d307969f48", "parents": [ "50706437d8216e41f0fea1e413cda7891324d397" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:39:47 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:39:47 2016 +0000" }, "message": "Revert \"Refactor GetIMTIndex\"\n\nI need to revert this to get https://android-review.googlesource.com/#/c/244190/ to cleanly revert. Matthew, do you mind rewriting it?\n\nThis reverts commit 50706437d8216e41f0fea1e413cda7891324d397.\n\nChange-Id: I5c1435f5dffb46dbb5b613b22adb88c7770304f2\n" }, { "commit": "50706437d8216e41f0fea1e413cda7891324d397", "tree": "23c7c5d1198b8cdde6261198626cfe443eab59ba", "parents": [ "badee9820fcf5dca5f8c46c3215ae1779ee7736e" ], "author": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Tue Jun 14 11:31:04 2016 -0700" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Wed Jun 15 09:08:35 2016 -0700" }, "message": "Refactor GetIMTIndex\n\nThis allows us to more easily maintain and experiment with\ninterface method table indexing and hashing.\n\nChange-Id: I719920fae7490dcedcda7c1c36db225c2b8b16df\n" }, { "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": "a3549d2b93f6ab4fe42f7c137cecae51b3b5fa14", "tree": "71de49768cb78980015dac65a4cbc49d23e82e12", "parents": [ "1b66fdf3f33c72dfdda4d31f6f17b6a0d8607402" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Jun 02 17:01:02 2016 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Jun 02 22:58:13 2016 -0700" }, "message": "Fix an issue of partial fragment deoptimization\n\nIf the interpreter bridge returns to the instrumentation exit stub,\nwe need to let the stub makes the decision whether the calling code\nis deoptimizeable or not since the stub knows the real return pc.\n\nBug: 28769520\nChange-Id: I262d1222e50a1ccbcb3675d05dcab414dc242a28\n" }, { "commit": "f711f2cf3c28dfe865e36f17419a16f06a0ebb5a", "tree": "3657d68b2a2fb67f4919cb97ca279cf9310224e9", "parents": [ "bbfa42aa4c4b5cbc2ed2d8bdea6ff76da615a8aa" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon May 23 12:29:39 2016 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu May 26 13:24:54 2016 -0700" }, "message": "Partial fragment deoptimization\n\nWe used to do either single frame deoptimization, or full fragment\ndeoptimization which deoptimizes all the frames in a fragment.\nThis change allows some methods to be not deoptimizeable, likely due\nto some kind of optimization. So we need another deoptimization mode\nthat unwinds partial fragment. Deoptimizations are now generalized into\neither full or partial fragment. A full fragment deoptimization will\ndeopt all frames in the fragment, and then returns from the invoke stub\nto enter interpreter. A partial fragment deoptimization will deopt a\nsingle frame, or all frames up to the method that\u0027s not deoptimizeable,\nand then jumps to the interpreter bridge.\n\nCurrently code not deoptimizeable is the code in boot image since the\ncode may not be compiled with debuggable flag.\n\nBug: 28769520\nChange-Id: I875c694791cc8ebd5121abcd92ce7b0db95aca38\n" }, { "commit": "e42888f9df4163303244070c65d5229d3e201742", "tree": "c3c3e380ed16676186040f1da6042fc2cd58e041", "parents": [ "7f98c9a6babe3a21d84ce1f1e1273c99975a47f5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 14 10:49:19 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 27 12:45:30 2016 -0700" }, "message": "Write conflict tables in image\n\nAdd image sections for runtime methods and conflict tables. This\nmakes it that we do not need to fake up a length prefixed array\nfor runtime methods.\n\nReduces .art private dirty and PSS.\n\nSystem wide .art PSS goes from 32.5MB to 30.5MB after system boot.\n\nBusiness card .art private dirty goes from 588K to 504K.\n\nIncreases image size by ~100K.\n\nBug: 27906566\n\n(cherry picked from commit cda9386add68d94697449c6cb08b356747e55c21)\n\nChange-Id: I38cbe3150c3eeb385b8cad7cf93614e3980f4162\n" }, { "commit": "7f98c9a6babe3a21d84ce1f1e1273c99975a47f5", "tree": "e4e19523d64b36ace1314d7d2493b64b31eb7442", "parents": [ "3ce07dd1a1d000a5439e32d64fcee19268846c08" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 14 10:49:19 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 27 12:44:47 2016 -0700" }, "message": "Add support for eagerly calculating conflict tables\n\nWill be used to put them in the image by having the compiler eagerly\ncalculate them.\n\nEnabled for debug builds (non compiler). Support for having conflict\ntables written in the image will come in the next CL.\n\nBug: 27906566\n\n(cherry picked from commit 49b5cede15d69930a8c156a3aea240164ca7af80)\n\nChange-Id: Ia05cb31f85eacfeabe64a8caf9a0b3029114a749\n" }, { "commit": "393fdb8b4822d80bbbd6347b088e28c03a72289e", "tree": "134955e86b08514fb88ecd0e5a037adfbf738fd6", "parents": [ "f7cda8088ec57ab1422f85f08df78e217a9f7094" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Apr 25 14:58:06 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Apr 25 16:56:56 2016 +0000" }, "message": "Make sure the referring class is in the dex cache.\n\nThe method CanAccessResolvedMethod expects the referring class\nin the dex file is already in the dex cache, which is true during AOT,\nbut not necessarilly during JIT.\n\nbug:28295348\nChange-Id: I58739903f0dff3867b920a7444f53b99ecf86e85\n" }, { "commit": "2f92ec8a52faa4bc566af8fa34212f3e749e82c8", "tree": "78b9ee38ae016f1198bc21ae7d913bc961f7624a", "parents": [ "75ee5c28ab583c532066b64d6337a5f78d6aa2ad", "61b28a17d9b6e8e998103646e98e4a9772e11927" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Mon Apr 04 13:35:53 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Apr 04 13:35:53 2016 +0000" }, "message": "Merge \"Use bit-level packing for InlineInfo in stackmaps as well.\"" }, { "commit": "cbfa10557ab6e2669747a1e21a87adf212ec44c2", "tree": "5c041e9dd51300a5a84159adf390d68b1ab075bc", "parents": [ "01afcbba19e40db984d41974086d63bbdd9cb955", "9fc547ac3936fe88e9592f4a47afd7b134cb607c" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Apr 01 17:30:41 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Apr 01 17:30:41 2016 +0000" }, "message": "Merge \"Make InvokeInterfaceTrampoline check methods\"" }, { "commit": "9fc547ac3936fe88e9592f4a47afd7b134cb607c", "tree": "1f604cb34a00418694515e7790657be8306650e1", "parents": [ "87ad82eb1e085ccc6ed3ec54945937582334dbbc" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Mar 31 14:34:33 2016 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Apr 01 09:46:37 2016 -0700" }, "message": "Make InvokeInterfaceTrampoline check methods\n\nInvokeInterfaceTrampoline was causing problems by looking into the\nImtConflictTable of non-imt-conflict-methods. This makes it check for\nthat before doing so.\n\nBug: 27931085\n\nChange-Id: I993178a371f8f46535a752e5c4d46d74777cefaf\n" }, { "commit": "61b28a17d9b6e8e998103646e98e4a9772e11927", "tree": "438c4a73f72423ddc71e6dca9f082d9e1d533996", "parents": [ "cf0f8560f41467331427418584687830932e66ed" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Feb 25 21:55:03 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Apr 01 17:01:55 2016 +0100" }, "message": "Use bit-level packing for InlineInfo in stackmaps as well.\n\nUse the same approach as we do for stackmaps to reduce the size.\n\nIt saves 4.0 MB from non-debuggable boot.oat (AOSP).\nIt does not affect debuggable boot.oat.\n\nIt saves 3.6 MB (of 96.6 MB) from /system/framework/arm/ (GOOG).\nIt saves 0.6 MB (of 26.7 MB) from /system/framework/oat/arm/ (GOOG).\n\nField loads from inline-info get around 5% slower.\n(based on the time it takes to load all inline-infos from boot.oat)\n\nChange-Id: I67b0fa5eef74c1fdb013680d0231fd44ea696176\n" }, { "commit": "09ed09866da6d8c7448ef297c148bfa577a247c2", "tree": "dad6a5dae6ca6131f1eba201eaa371edc6d9929d", "parents": [ "e28ad4b91591c226ed404a2b01104bb99bfeb28f" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Feb 12 21:58:43 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Mar 31 22:12:07 2016 +0100" }, "message": "Pack stack map entries on bit level to save space.\n\nUse only the minimum number of bits required to store stack map data.\nFor example, if native_pc needs 5 bits and dex_pc needs 3 bits, they\nwill share the first byte of the stack map entry.\n\nThe header is changed to store bit offsets of the fields rather than\nbyte sizes. Offsets also make it easier to access later fields without\ncalculating sum of all previous sizes.\n\nAll of the header fields are byte sized or encoded as ULEB128 instead\nof the previous fixed size encoding. This shrinks it by about half.\n\nIt saves 3.6 MB from non-debuggable boot.oat (AOSP).\nIt saves 3.1 MB from debuggable boot.oat (AOSP).\n\nIt saves 2.8 MB (of 99.4 MB) from /system/framework/arm/ (GOOG).\nIt saves 1.0 MB (of 27.8 MB) from /system/framework/oat/arm/ (GOOG).\n\nField loads from stackmaps seem to get around 10% faster.\n(based on the time it takes to load all stackmap entries from boot.oat)\n\nBug: 27640410\nChange-Id: I8bf0996b4eb24300c1b0dfc6e9d99fe85d04a1b7\n" }, { "commit": "796d63050a18f263b93ea34951a61deaecab3422", "tree": "813865c31b25ac06006e4ee3932b4e918c708c9b", "parents": [ "eecf60d51b481647c8508f22b3d6ce437773ea0c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sun Mar 13 22:22:31 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 22 15:17:58 2016 +0000" }, "message": "Add an ImtConflictTable to better resolve IMT conflicts.\n\n- Attach a ImtConflictTable to conflict runtime ArtMethod.\n- Initially 0, a new one will be created at the first hit of\n the conflict method.\n- If the assembly code does not find a target method in the table,\n we will create a new one again, copying the data from the previous\n table and adding the new mapping.\n\nImplemented for arm/arm64/x86/x64.\n\nbug:27556801\nbug:24769046\n\nChange-Id: Ie74d1c77cf73d451a1142bdc5e3683f9f84bb4e7\n" }, { "commit": "e9dd04f633fcffc485f8d74861f39f66fc8bc522", "tree": "894c9061f726899e0988c5c0c987eb18bab2d527", "parents": [ "e848c29bb35d91f23bb6f52712aa7f2dff342ea0" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Mar 16 16:09:45 2016 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Mar 17 16:29:55 2016 +0000" }, "message": "Revert \"Revert \"Fix issue with proxy invocation on default methods\"\"\n\nThis reverts commit daf58c80d42f024aae0cb94ebd2c0bd61ebbf240.\n\nBug: 27621360\nChange-Id: I56fccf19ae80451be4c316ba14911c667cbe9e47\n" }, { "commit": "fedd91d50930e160c021d65b3740264f6ffec260", "tree": "5f356423e51f3bd71020ba0a5f024c54adc959e7", "parents": [ "97f4bc04b61d5cf78b0820dbf18e999b20d7a108" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Jan 07 14:49:16 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Jan 28 15:46:55 2016 -0800" }, "message": "Optimizing compiler support for directly calling interface methods\n\nThis teaches the optimizing compiler how to perform invoke-super on\ninterfaces. This should make the invokes generally faster.\n\nBug: 24618811\n\nChange-Id: I7f9b0fb1209775c1c8837ab5d21f8acba3cc72a5\n" }, { "commit": "f39745e663f8f2634fc8858e427b77da98f8f2b4", "tree": "b8cd5abd22e0252511166930ac6326a2f38e6980", "parents": [ "36867b43efadc637ab74b1bacb1caecea2c5413d" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jan 26 12:16:55 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jan 26 12:16:55 2016 +0000" }, "message": "ART: Remove some unnecessary mutator lock annotations.\n\nThe StackReference\u003c\u003e pointer held by a Handle\u003c\u003e can be used\nwithout holding the mutator lock. We already do that when\nwe copy Handle\u003c\u003es around. Only accessing the actual content\nof the pointed-to StackReference\u003c\u003e needs to be done while\nholding the mutator lock.\n\nChange-Id: I5f93bd7e277383192f1f16dff6883ecb26387414\n" }, { "commit": "42ef8ab151a3d0cbb42cb43f6841c3708d65fca3", "tree": "ba8eb3fdecd226a8c516a86ac418f73f6c0ff254", "parents": [ "170e01a6b59b3242a5afc76c3a03c00ce288150f" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 03 17:27:32 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 10 13:52:33 2015 -0800" }, "message": "ART: Stash a resolved method late in the verifier\n\nInvoke-interface should only be called on an interface method.\nWe cannot move the check earlier, as there are other checks\nthat must be done that can fail a class hard. So postpone\na push to the dex cache.\n\nClean up the test a bit.\n\nAlso templatize ResolveMethod with a version always checking\nthe invoke type, and on a cache miss check whether type target\ntype is an interface when an interface invoke type was given.\n\nBug: 21869691\nChange-Id: I94cbb23339cbbb3cb6be9995775e4dcefacce7fd\n" }, { "commit": "c96205e03bd4f991ad1540df38f9f692ef211b22", "tree": "4ae775d15f51efb704ca5b00a2ee4b6fb6615812", "parents": [ "34760f9a97e6a1733edfca1b13199f749106a30f" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 10 13:25:27 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 10 13:27:10 2015 -0800" }, "message": "Revert \"ART: Check invoke-interface earlier in verifier\"\n\nThis reverts commit dae24142127c64551142a50423085aabdb0a6060.\n\nIt is important to check the name of the method being called.\n\nBug: 21869691\n" }, { "commit": "eb5ddd7b02ce2d25d4f28f85a13aac628526e1c1", "tree": "8f655b382e73222e70e30daaeedc3999014aa7ae", "parents": [ "2433d4e17c3006b8262a0d9421e201fc84777208", "dae24142127c64551142a50423085aabdb0a6060" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Dec 08 22:33:23 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Dec 08 22:33:23 2015 +0000" }, "message": "Merge \"ART: Check invoke-interface earlier in verifier\"" }, { "commit": "dae24142127c64551142a50423085aabdb0a6060", "tree": "35c06bc37e2dd2caad4fef1fba4de16a12aad6f9", "parents": [ "ef45113d01c16102538a8848d6979e772dad6225" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 03 17:27:32 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Dec 08 08:50:12 2015 -0800" }, "message": "ART: Check invoke-interface earlier in verifier\n\nInvoke-interface should only be called on an interface method.\nMove the check earlier, as otherwise we\u0027ll try to resolve and\npotentially inject a method into the dex cache.\n\nAlso templatize ResolveMethod with a version always checking\nthe invoke type, and on a cache miss check whether type target\ntype is an interface when an interface invoke type was given.\n\nBug: 21869691\nChange-Id: Ica27158f675b5aa223d9229248189612f4706832\n" }, { "commit": "73be1e8f8609708f6624bb297c9628de44fd8b6f", "tree": "47a22745797a345dea8dd846aad00334da0d5bf1", "parents": [ "47229aa5848df7d45578dbdd9285f57dfa9399a8" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Sep 17 15:22:56 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Dec 04 12:08:02 2015 +0000" }, "message": "Inline monomorphic calls.\n\nChange-Id: If38171c2dc7d4a4378df5d050afc4fff4499c98f\n" }, { "commit": "e523423a053af5cb55837f07ceae9ff2fd581712", "tree": "6c2d9c570bf0d9a0e2cd056e052c0be618b03fc5", "parents": [ "08a84acc7adb1bb076595eb961bd4667896e5075" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 02 09:06:11 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 02 13:01:25 2015 +0000" }, "message": "Revert \"Revert \"Don\u0027t use the compiler driver for method resolution.\"\"\n\nThis reverts commit c88ef3a10c474045a3476a02ae75d07ddd3230b7.\n\nChange-Id: I0ed88a48b313a8d28bc39fae40631123aadb13ef\n" }, { "commit": "3a0909248e04b22c3981cbf617bc2502ed5b6380", "tree": "b0b7013ea78b10f23fae75ef145e53a696ff797c", "parents": [ "457e874459ae638145cab6d572e34d48480e39d2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 24 09:17:30 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 24 09:17:30 2015 +0000" }, "message": "Revert \"lambda: Add support for invoke-interface for boxed innate lambdas\"\n\n955-lambda is flaky\n\nBug: 24618608\nBug: 25107649\n\nThis reverts commit 457e874459ae638145cab6d572e34d48480e39d2.\n\nChange-Id: I24884344d21d7a4262e53e3f5dba57032687ddb7\n" }, { "commit": "457e874459ae638145cab6d572e34d48480e39d2", "tree": "b192c4698c4889ae90db6abe2652199802213fe6", "parents": [ "3944f7175dcf60316ba58a42698ccf23c65ac57c" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Oct 22 17:37:50 2015 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Nov 20 16:33:06 2015 -0800" }, "message": "lambda: Add support for invoke-interface for boxed innate lambdas\n\nLambda closures created with the \u0027create-lambda\u0027 instruction\n(termed \"innate lambdas\") can be turned into an object with \u0027box-lambda\u0027.\n\nThis CL enables support for those kinds of lambdas to work with\n\u0027invoke-interface\u0027 by generating a proxy class for the lambda.\n\nNote: MIPS32/64 support not included.\n\nBug: 24618608\nBug: 25107649\nChange-Id: Ic8f1bb66ebeaed4097e758a50becf1cff6ccaefb\n" }, { "commit": "9139e008abe30b7beaf4afd6533228a1dd9b202c", "tree": "506be89287507bd777bd132627ad963729434969", "parents": [ "d7c76bd7e867b151ac81ef41810614c7b4cc898e" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Oct 09 15:59:48 2015 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Nov 10 17:20:41 2015 -0800" }, "message": "Correct exception behavior for default methods\n\nDefault methods are defined to throw an IncompatibleClassChangeError\n(ICCE) when they are called and there is no \"best\" implementation.\nPreviously we would simply throw an ICCE during class loading as soon\nas we noticed that this would happen if called. This makes us wait\nuntil we actually attempt to execute the method. Furthermore, this\nallows us to use other, non-conflicting, methods on the object as\nnormal.\n\nFurthermore, this makes us correctly throw AbstractMethodErrors in\ncases where all default implementations of a method are overridden by\nabstract declarations.\n\nAdds 3 tests for this new behavior.\n\nBug: 24618811\n\nChange-Id: Id891958a81f9b3862b2ce5919636aabef7d3422e\n" }, { "commit": "524e7ea8cd17bad17bd9f3e0ccbb19ad0d4d9c02", "tree": "ad07cc96f633bdae839ff2f1553d84b9c864a930", "parents": [ "b697c447eb61c2e14315166ec3b0d16375ae403c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Oct 16 17:13:34 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 20 11:52:11 2015 +0100" }, "message": "Remove ArtCode.\n\n- Instead use OatQuickMethodHeader.\n- Various cleanups now that we don\u0027t have all those\n ArtMethod -\u003e ArtCode -\u003e OatQuickMethodHeader indirections.\n\nAs a consequence of this cleanup, exception handling got a bit\nfaster.\n\nParserCombinators benchmark (exception intensive) on x64: (lower is better)\nBefore:\nParserCombinators(RunTime): 1062500.0 us.\nAfter:\nParserCombinators(RunTime): 833000.0 us.\n\nChange-Id: Idac917b6f1b0dc254ad68fb3781cd61bccadb0f3\n" }, { "commit": "4b8f1ecd3aa5a29ec1463ff88fee9db365f257dc", "tree": "d113f8a5c6b61c078256cf15c7cbb9f7c8de0390", "parents": [ "114873103db3d4d6e0da42ca02bad1ea8826443b" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Aug 26 18:34:03 2015 +0100" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Oct 15 12:22:39 2015 -0700" }, "message": "Use ATTRIBUTE_UNUSED more.\n\nUse it in lieu of UNUSED(), which had some incorrect uses.\n\nChange-Id: If247dce58b72056f6eea84968e7196f0b5bef4da\n" }, { "commit": "6bc4374e3fa00e3ee5e832e1761c43e0b8a71558", "tree": "38118523aa6b9a92fbdcd7bf1d32ca6185a3d3f0", "parents": [ "16065ce56394c73c87dcb78ead4164ddc80fddb3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Oct 12 18:11:10 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 13 12:05:27 2015 +0100" }, "message": "Add an abstraction over a compiled code.\n\nThat\u0027s just step 1, moving code-related functions of ArtMethod to\nanother class. That class is only a wrapper on an ArtMethod, but will\nbe changed to be a wrapper around compiled code.\n\nChange-Id: I6f35fc06d37220558dff61691e51ae20066b0dd6\n" }, { "commit": "03ec930faded5bbfa096533ce60b6893847922db", "tree": "085ec76235f72647cd2e083801f15d82394e038b", "parents": [ "14412079e6eba4d16f40c0bdc5e94812e1c7f5bc" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Aug 27 17:41:47 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 09 16:32:13 2015 -0700" }, "message": "ART: Lock counting in the interpreter\n\nTo support structured locking when balanced-locking verification\nfailed, count lock and unlock operations in the verifier.\n\nBug: 23502994\nChange-Id: I2bb915da6e3d43c49723a943b42d4d5a7c939aa1\n" }, { "commit": "e024162070e3ce7e92a65e3b0f62dce7b8a8505a", "tree": "707a432372a66f099c23e96a4601cb3f03458d0e", "parents": [ "0766e3ff5638b14d6b3ea6157c800eed11be5b98", "a23b4680a8565f28e8aa958febf8949db74a3064" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Sep 29 19:07:50 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Sep 29 19:07:50 2015 +0000" }, "message": "Merge \"Fix a mutator lock violation in the generic JNI end sequence.\"" }, { "commit": "a23b4680a8565f28e8aa958febf8949db74a3064", "tree": "bc86ebbdf93e4318ab89835626996f91bbf94c18", "parents": [ "8a0014dd3ad1562d93695fa0d3033856ec7da133" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Sep 28 17:47:32 2015 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Tue Sep 29 11:33:13 2015 -0700" }, "message": "Fix a mutator lock violation in the generic JNI end sequence.\n\nartQuickGenericJniEndTrampoline() was accessing objects without a shared\nmutator lock right after returning from a JNI call but before switching\nto the runnable state.\n\nThis fixes crashes with table lookup read barriers enabled.\n\nBug: 12687968\nChange-Id: I94ad9ca276750f58cb68b2fa9eb8cdeb371f021b\n" }, { "commit": "931e26843bbb688eacfa67b40414c6b8f221a56a", "tree": "63511ae48d4d485823e82f9f0c0af2bda4b78102", "parents": [ "ac87001cbbd44b436cc7866f7a41037ca83b17f1", "639bdd13993644a267f177f8f5936496bda65e2b" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 28 22:40:18 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Sep 28 22:40:18 2015 +0000" }, "message": "Merge \"ART: Single-frame deopt\"" }, { "commit": "639bdd13993644a267f177f8f5936496bda65e2b", "tree": "fc68e344355ab1c0cfef4d47dd8f2cbc9fc7e022", "parents": [ "b175a45be78152606a1db1eac0b79ef8777644f5" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 03 11:22:45 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 28 15:37:28 2015 -0700" }, "message": "ART: Single-frame deopt\n\nAdd deoptimization of a single frame. Works by removing the managed\ncode frame and jumping into the quick-to-interpreter bridge, and\nthe bridge understanding a stored ShadowFrame.\n\nWe need a separate fixup pass. For x86, we leave the return address\non the stack so we don\u0027t need to push it there.\n\nBug: 21611912\nChange-Id: I06625685ced8b054244f8685ab50b238a705b9d2\n" }, { "commit": "b30259251b22430fad12f1adeab671e4bf8f88f5", "tree": "80bff979beae84702b739962656a16d245a40ef4", "parents": [ "eae0927cb81437389dc96437e9f04903783282b4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 01 14:45:00 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 24 15:42:14 2015 -0700" }, "message": "ART: Use unique_ptr for alloca-ed ShadowFrames\n\nTo enable automated calls of the destructor (for extensions of\nShadowFrame), return a unique_ptr with a custom deleter. Use a\nmacro so that the alloca happens in the caller.\n\nChange-Id: Id28a6ea131f108e4a94ff1699fc22e7b44aec018\n" }, { "commit": "05792b98980741111b4d0a24d68cff2a8e070a3a", "tree": "bad79a387bcbdaefc87c07b388099960ca9caff3", "parents": [ "c26b4512a01d46756683a4f5e186a0b7f397f251" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Aug 03 11:56:49 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 03 17:30:57 2015 +0100" }, "message": "ART: Move DexCache arrays to native.\n\nThis CL has a companion CL in libcore/\n https://android-review.googlesource.com/162985\n\nChange-Id: Icbc9e20ad1b565e603195b12714762bb446515fa\n" }, { "commit": "0747466fca310eedea5fc49e37d54f240a0b3c0f", "tree": "6d27922fa35330dd8d863ea56c82c5f04b711822", "parents": [ "9ee5d6cdc14ac94b64ea1961bf221bad48746929" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:12:33 2015 +0000" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Aug 26 16:54:32 2015 +0200" }, "message": "Revert \"Revert \"Fix deoptimization with pending exception\"\"\n\nThis reverts commit 6e2d5747d00697a25251d25dd33b953e54709507.\n\nFixes the deoptimization path from compiled code (generated by the\nOptimizing compiler) by adding wrapper artDeoptimizeFromCompiledCode.\nThis wrapper, called through the matching assembler stub\nart_quick_deoptimize_from_compiled_code, pushes the deoptimization\ncontext just before deoptimizing the stack.\n\nBug: 23371176\nBug: 19944235\nChange-Id: Ia7082656998aebdd0157438f7e6504c120e10d3e\n" }, { "commit": "ff73498a5539d87424a964265e43765e788aec44", "tree": "3f8a525ef91d2f12e4b2a71c04fc8147a12c12bd", "parents": [ "9dc601eb65da0cd5f53172699dacd6e5dd38ab44" ], "author": { "name": "Goran Jakovljevic", "email": "Goran.Jakovljevic@imgtec.com", "time": "Mon Aug 24 12:58:55 2015 +0000" }, "committer": { "name": "Goran Jakovljevic", "email": "Goran.Jakovljevic@imgtec.com", "time": "Tue Aug 25 17:17:22 2015 +0200" }, "message": "Revert \"Revert \"[MIPS] Use hard float calling convention for managed code\"\"\n\nThis reverts commit 7fee84c087e0f903e7d43bef180df047db1c8051.\n\nFixed issue with temporary registers on Mips32r6.\n\nChange-Id: I93018927e6a6036cff2d55e6cda66d3212a4316b\n" }, { "commit": "6e2d5747d00697a25251d25dd33b953e54709507", "tree": "b68a418e93780ed55abd79dc8e76f3459e884254", "parents": [ "54b62480636ae846d705fc180c7bd6cd08ec1e42" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:05:17 2015 +0000" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:05:17 2015 +0000" }, "message": "Revert \"Fix deoptimization with pending exception\"\n\nThis reverts commit 54b62480636ae846d705fc180c7bd6cd08ec1e42.\n\nThis is causing test failures with Optimizing compiler.\n\nBug: 23371176\nBug: 19944235\nChange-Id: Ie3ffbcf2b6d2ca8bc93cb008a4e29a7567d04a7c\n" }, { "commit": "54b62480636ae846d705fc180c7bd6cd08ec1e42", "tree": "d46a6c7bebb45ddbbd5b0a6818b3bb2ce49f48ed", "parents": [ "f0170ded84fbcd3d9152cdbcf9ffbf74c71b03ba" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Aug 20 12:07:57 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Aug 25 15:13:49 2015 +0200" }, "message": "Fix deoptimization with pending exception\n\nWhen deoptimizing the stack, we set a fake exception in the current\nThread* (see method Thread::GetDeoptimizationException). On the next\nexception check, the QuickExceptionHandler will deoptimize the stack.\n\nThe issue is when we deoptimize while an exception is already pending\nin the current Thread*: setting the fake exception will clobber the\npending exception which is not correct. This happens in the\nartQuickToInterpreterBridge when returning from the interpreter and\nwe want to deoptimize the stack for debugging (like single-stepping).\n\nThis CL saves the pending exception before asking for deoptimization.\nThen the exception is restored just before executing the deoptimized\nframes with the interpreter.\n\nAlso cleans up the way we save deoptimization context (return value\nand pending exception).\n\nBug: 23371176\nBug: 19944235\nChange-Id: I7f4c8347b328817c452beda3399e210eba3a88a4\n" }, { "commit": "7fee84c087e0f903e7d43bef180df047db1c8051", "tree": "c35065aeef23f857563eacd82e55ae47d8ceb67c", "parents": [ "a29449dcf57c57fe0876f51367985477317cc557" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Aug 21 18:39:26 2015 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Aug 21 18:39:26 2015 +0000" }, "message": "Revert \"[MIPS] Use hard float calling convention for managed code\"\n\nMipsMir2Lir::LockCallTemps() is broken for secondary architecture on aosp_mips64-eng.\n\nThis reverts commit a29449dcf57c57fe0876f51367985477317cc557.\n\nChange-Id: I480ea7569d73aea7894fc0a6dd804b1135286a37\n" }, { "commit": "a29449dcf57c57fe0876f51367985477317cc557", "tree": "3b077e6f286bfdb48f211e0d0672ff301c5b7006", "parents": [ "28de0f652e32e112edab0e5e7dc779943ae488d5" ], "author": { "name": "Goran Jakovljevic", "email": "Goran.Jakovljevic@imgtec.com", "time": "Wed Jul 22 11:08:57 2015 +0200" }, "committer": { "name": "Goran Jakovljevic", "email": "Goran.Jakovljevic@imgtec.com", "time": "Fri Aug 21 14:40:30 2015 +0200" }, "message": "[MIPS] Use hard float calling convention for managed code\n\nNote that this isn\u0027t o32 ABI. Same set of registers is used for\narguments ($a0-$a3 and $f12-$f15), but we don\u0027t skip registers\nand fp arguments are never passed via core registers.\n\nChange-Id: Ifb883ff6e15758b539137898b49ac2f8ee075f49\n" }, { "commit": "3a35714ebca10c989aa383c0861e2e84fe8dadf8", "tree": "3218ed8f5962af0351e1d285ccaff5207050672d", "parents": [ "275bc5e5f318ba6ea1bad6c89c2924d1248b7523" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Aug 07 17:20:11 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Aug 07 17:22:26 2015 -0700" }, "message": "ART: Clean up unnecessary ArtMethod**\n\nArtMethods are no longer Java objects, so the additional indirection\nis no longer necessary here.\n\nChange-Id: If76756d875b418b3f6e83f51b3225a158e9ce29b\n" }, { "commit": "90443477f9a0061581c420775ce3b7eeae7468bc", "tree": "8c74b81dfae162e0fd0ccf8d5ac50827ba815174", "parents": [ "6078aec213dfaf111c29969706e8e5967cfc9bea" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 16 20:32:27 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 22 15:13:56 2015 -0700" }, "message": "Move to newer clang annotations\n\nAlso enable -Wthread-safety-negative.\n\nChanges:\nSwitch to capabilities and negative capabilities.\n\nFuture work:\nUse capabilities to implement uninterruptible annotations to work\nwith AssertNoThreadSuspension.\n\nBug: 20072211\n\nChange-Id: I42fcbe0300d98a831c89d1eff3ecd5a7e99ebf33\n" }, { "commit": "1f2d3ba6af52cf6f566deb38b7e07735c9a08fb6", "tree": "af6ae612d43c50703292d39a8a0b37dda3f919a6", "parents": [ "ccd6337f31d13706c602f3d9436e9b4025075b63" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon May 18 12:12:50 2015 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Jun 08 00:53:55 2015 -0700" }, "message": "Fix nested deoptimization.\n\nHandle nested deoptimization cases. Create a stacked shadow frame\nrecords to keep track of deoptimization shadow frames. Shadow frames\nunder construction can be tracked in the same stack.\n\nBug: 20845490\nChange-Id: I768285792c29e7c3cfcd21e7a2600802506024d8\n" }, { "commit": "f677ebfd832c9c614fea5e6735725fec2f7a3f2a", "tree": "051b3ba6343bdbca4929d728dd5c0341e4383a08", "parents": [ "c47908e8c32fd58bc4dc75998a80f706954db1dc" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri May 29 16:29:43 2015 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jun 04 17:25:02 2015 +0100" }, "message": "Cache stack map encoding\n\nOperations on CodeInfo and StackMap objects repeatedly read encoding\ninformation from the MemoryRegion. Since these are 3-bit-loads of\nvalues that never change, caching them can measurably reduce compile\ntimes.\n\nAccording to benchmarks, this patch saves 1-3% on armv7, 2-4% on x86,\nand 0-1% on x64.\n\nChange-Id: I46b197513601325d8bab562cc80100c00ec28a3b\n" }, { "commit": "4edd8476339fd93ba8ff384ad107f1fc662e64a3", "tree": "6937496e8d32791ca450ef923d7e4b27eb07bf08", "parents": [ "7b9f214463b9a5a49b497421713efead3fb32ec4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 01 10:47:36 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 01 10:51:42 2015 -0700" }, "message": "Fix accidental IMT and root marking regression\n\nWas always using the conflict trampoline. Also included fix for\nregression in GC time caused by extra roots. Most of the regression\nwas IMT.\n\nFixed bug in DumpGcPerformanceInfo where we would get SIGABRT due to\ndetached thread.\n\nEvaluateAndApplyChanges:\nFrom ~2500 -\u003e ~1980\nGC time: 8.2s -\u003e 7.2s due to 1s less of MarkConcurrentRoots\n\nBug: 19264997\nChange-Id: I4333e80a8268c2ed1284f87f25b9f113d4f2c7e0\n" }, { "commit": "e401d146407d61eeb99f8d6176b2ac13c4df1e33", "tree": "17927f9bfe7d2041b5942c89832d55f9dedb24c5", "parents": [ "2006b7b9b8e32722bd0d640c62549d8a0ac624b6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 13:56:20 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 29 18:45:49 2015 -0700" }, "message": "Move mirror::ArtMethod to native\n\nOptimizing + quick tests are passing, devices boot.\n\nTODO: Test and fix bugs in mips64.\n\nSaves 16 bytes per most ArtMethod, 7.5MB reduction in system PSS.\nSome of the savings are from removal of virtual methods and direct\nmethods object arrays.\n\nBug: 19264997\nChange-Id: I622469a0cfa0e7082a2119f3d6a9491eb61e3f3d\n" }, { "commit": "d23eeef3492b53102eb8093524cf37e2b4c296db", "tree": "57d3e9ab2853d5b8092568bb3d29bc850c113315", "parents": [ "a15c78d3cc28f514a482ffd792a767e97fe53c95" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 18 22:31:29 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri May 29 14:15:04 2015 +0100" }, "message": "Support for inlining methods that call/throw.\n\nMostly fixes here and there to make it working.\n\nChange-Id: I1b535e895105d78b65634636d675b818551f783e\n" }, { "commit": "33d75bc75c72db7e5b1ea4ad0be287b1dfd72e69", "tree": "f2bdf02aba34e7d59c760df1b54bb3c789ee6648", "parents": [ "65efe9192390187674108b9c895ba225628bb07b", "3b45ef277e4a5f7e0314d5df7ef82e480156ba75" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 27 22:41:30 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed May 27 22:41:31 2015 +0000" }, "message": "Merge \"ART: Fix VerifyObject runtime verification\"" }, { "commit": "3b45ef277e4a5f7e0314d5df7ef82e480156ba75", "tree": "1df28568b9e7e4e50102b7f834fab6e943c444a5", "parents": [ "c06841dac1443e99e92596f15caa82c99bfee010" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue May 26 21:34:09 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 27 08:24:03 2015 -0700" }, "message": "ART: Fix VerifyObject runtime verification\n\nUpdate some bit-rotted code to work again. Most tests now work, for\nsome the verification overhead results in a timeout.\n\nChange-Id: Ieab4f2de474a05e915e24abc93da3c2eeed996eb\n" }, { "commit": "8ea18d0f066f63fa4e5d154f14327468bf288e2b", "tree": "a575e8d0f826ab8b3aaae949c22cc6c73ca040fb", "parents": [ "092a5656937a319449346e8c356f3f4c2870d81c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue May 26 16:29:08 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed May 27 12:07:34 2015 +0100" }, "message": "Pass the dex method index directly to interface trampoline.\n\nThis avoids computing the dex pc and re-finding the method\nindex again. I have kept the code for kDebugBuild.\n\nChange-Id: Icd60e0deade755e32b54021c0875b1af592b8c3e\n" } ], "next": "9bd88b0933a372e6a7b64b850868e6a7998567e2" }