)]}' { "log": [ { "commit": "97e78033e6cf81d72c9d4d016e349282b911303c", "tree": "6d24f271c7c707646ae2eb87161c887004fcd469", "parents": [ "ef43805b0e80015645a5fc52e53b93ad178f60e3" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Jun 27 17:51:55 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Jun 28 09:55:48 2017 -0700" }, "message": "Move debugger.cc GetCanonicalMethod to ArtMethod\n\nThis functionality will be useful to other parts of the runtime so put\nit in a more central place.\n\nTest: ./test.py --host -j40\nChange-Id: I4d19cc1016884f5921245ac284c7323cbf3ea8cd\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": "a792067fcf354b02cbab0a51b51842b73274d4cd", "tree": "6139b0928d66ef4f17b5386f7a9cc1dc81f1c59e", "parents": [ "2cf565901845e4bafc663c24071c91d13c692858" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jun 23 09:41:20 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jun 23 11:11:03 2017 +0100" }, "message": "Don\u0027t do a read barrier in GetProfilingInfo.\n\nProfilingInfos are handled by the JIT, which treats declaring classes\nof methods having these ProfilingInfo as weak roots. So it might be\npossible the JIT requests a ProfilingInfo for a class that is going\nto be unloaded (and that\u0027s ok, as the memory for the ProfilingInfo\nis reclaimed by the JIT and is done before unloading the class).\n\nbug: 62914384\nTest: test.py\nChange-Id: I84571786c1569782fb02d68257c4f70b195f27b6\n" }, { "commit": "aea05c17f8c12ac7032e252f997a4c61c071e2d6", "tree": "4b6a33b180103e1b8ee36099ed189224f7b2fb6c", "parents": [ "1c5b42f00933b8f28b447f039fb93a7d9a9db06f" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri May 19 08:45:02 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 21 12:55:47 2017 -0700" }, "message": "ART: Move debug-checks to runtime-debug-flags\n\nCurrent slow checks:\n* Class status checking in ArtMethod\n* Read barrier invariant checking\n\nBug: 35644369\nTest: m test-art-host\nChange-Id: I66138a9accc601fa0fa675cf600412ad3629d28f\n" }, { "commit": "d76615851af31791c7b1549e8c4609152237b9ce", "tree": "e438a8cb0892e02768c55c2bc99f8be1d00abacb", "parents": [ "6ecff4d2127e70738aa2493d6deceb946c204eff" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon May 01 13:48:16 2017 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Jun 07 11:10:50 2017 -0700" }, "message": "Handler-ize InstrumentationListener and related code\n\nWe are going to be making instrumentation listeners that can cause\nsuspensions during their running. We explicitly handler-ize all the\ninstrumentation listener functions in order to ensure this is safe.\n\nBug: 34414073\nTest: ./test.py --host --ntrace --trace -j40\nChange-Id: Ic719080d0991b104d41b7757df8d1f332c72cd04\n" }, { "commit": "5d08fcc21a14190c4dba3028a3e35d281afdd016", "tree": "c42de65ced190778342d9b022e400f4a7f9d346b", "parents": [ "57cf00bde719ddc84a6015b107b90a20169e3099" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 05 17:56:46 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 05 20:28:16 2017 -0700" }, "message": "ART: More header cleanups\n\nPull out more dependencies through forward declarations.\n\nTest: m test-art-host\nChange-Id: I7d86726928937f788b956ec9eac91532d66d57ae\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": "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": "8c19d2431f45861ca11cf5ebc4fdaf10200f2458", "tree": "2d376ce322be680898d55fde929e67032d4c711c", "parents": [ "4e03442619834c56dbf79a3493eb8a2d91b93e7f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Mar 06 12:35:10 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Mar 29 16:51:26 2017 -0700" }, "message": "Refactor image writer\n\nAim to have common functions for copying references and pointers.\nRequired for adding support for faster image fixups.\n\nTest: test-art-host -j32\n\nBug: 34927277\nBug: 34928633\nChange-Id: Ia654efc483b332eea3535570496bfeccd7c635ee\n" }, { "commit": "18ea1c9e9281b5c143b8c376d76c3ff9cae885fb", "tree": "1b2a4a2d8c15fc9a01e539f29275a05833cb44f9", "parents": [ "624dc59e7d0ab8b916a986b502cb358d16182234" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 27 08:00:18 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 27 09:02:22 2017 +0100" }, "message": "\"Revert^6 \"CHA for interface method.\"\"\"\n\nUpdate test expectations for CHA tests flaking on no-dex2oat.\n\nbug:36344221\n\nThis reverts commit 27ef25f084017421ca05508208f436b5fc11df73.\n\nChange-Id: Ie92adc7a2ec3b3081a1c57d71f8c89247e58cd46\n" }, { "commit": "27ef25f084017421ca05508208f436b5fc11df73", "tree": "2059bf861c7992cfa195e7301451b98908c8abc6", "parents": [ "fb185130f35bbfe2ba1875974ed39ef0b3c7a77f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 24 08:59:22 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 24 08:59:22 2017 +0000" }, "message": "Revert \"Revert^4 \"CHA for interface method.\"\"\n\nStill failing on the bots.\n\nThis reverts commit fb185130f35bbfe2ba1875974ed39ef0b3c7a77f.\n\nChange-Id: I1629e237dd41ce4784ccadbfc346352dfcbdb137\n" }, { "commit": "fb185130f35bbfe2ba1875974ed39ef0b3c7a77f", "tree": "47cab7e151d7de2d86d9499e264273f7262699b9", "parents": [ "a861b0daacd508dacfdac6c337c1128904e5b386" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Mar 16 16:19:15 2017 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Mar 22 14:26:20 2017 -0700" }, "message": "Revert^4 \"CHA for interface method.\"\n\nThis reverts commit 43e99be9db10111a2d6e094882cd06c248c69e11.\n\nAlso fix the issue that tries to change invoke-interface to\nan invoke-virtual of a cross-dexfile proxy method. Added a\ntestcase.\n\nTest: run-libcore-tests.sh --mode\u003dhost --variant\u003dX64 --debug\nTest: ART_TEST_JIT\u003dtrue m -j20 test-art-host-run-test\nChange-Id: Iacdf9d7ad93590e8163ed59d5838d70700a69018\n" }, { "commit": "f2f1c9dab95892ec2983f57185c2bb0fbce80f80", "tree": "163613230c2afb656f9ee2b232253d76624e3bf8", "parents": [ "8a31bb939f2ab1fe3f745ade40408ac07267208a" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Mar 15 15:35:46 2017 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Mar 15 09:19:22 2017 -0700" }, "message": "Revert \"Revert \"Refactor annotation code for obsolete methods.\"\"\n\nThe number of InterruptedExceptions created during runtime shutdown is\nsometimes inconsistent so we will filter those out of the test results.\n\nThis reverts commit 65e0775010121498a37fa26c64fcc5bb17a6c1b5.\n\nReason for revert: Fixed issue with test flakes\n\nBug: 31455788\nTest: stress --cpu 60; while ./test/run-test --host 980; do; done\n\nChange-Id: Idf53ea11ceec1f48c433da468b31092bf230de57\n" }, { "commit": "65e0775010121498a37fa26c64fcc5bb17a6c1b5", "tree": "5849c59b4e2de57fa8156149597f4a921a2f04d9", "parents": [ "5378b64af97f9ae06082c697a6b13d2b98e02abb" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 15 06:56:35 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 15 06:58:57 2017 +0000" }, "message": "Revert \"Refactor annotation code for obsolete methods.\"\n\nTest has flakes.\nBug: 31455788\n\nThis reverts commit 672a0b75c91af44749b1e946684e6a65a884d778.\n\nChange-Id: If00ecbd31abe204bcaf8341eceb674e07c0ac8f2\n" }, { "commit": "bfe66b01844328bab508f305df3b5d46ca6798a7", "tree": "de590d9346e3b7ffe3725b36792264dc28d5f4f4", "parents": [ "9f2498c21a7d02605e61c5b2145ba55abf1c3703", "43e99be9db10111a2d6e094882cd06c248c69e11" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 15 06:29:09 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Mar 15 06:29:10 2017 +0000" }, "message": "Merge \"Revert \"Revert \"Revert \"CHA for interface method.\"\"\"\"" }, { "commit": "43e99be9db10111a2d6e094882cd06c248c69e11", "tree": "fb4d9ed1e6a5230c3a54e07157f017cf60167e51", "parents": [ "8f301e26943c53485abc2da5ff1907f7c2e0ff0c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 15 06:28:52 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 15 06:28:52 2017 +0000" }, "message": "Revert \"Revert \"Revert \"CHA for interface method.\"\"\"\n\nBreaks libcore tests.\n\nThis reverts commit 8f301e26943c53485abc2da5ff1907f7c2e0ff0c.\n\nChange-Id: Iea46176118be9e05aceb06f2d290961bb1f38265\n" }, { "commit": "875e3225c52570a806a8d3719ea89ef9d72bf7ae", "tree": "10597eb6e63f7da237cca23e75889c5583561d4e", "parents": [ "bff6bac661d05ad184c831a2b0b224260f2e2956", "672a0b75c91af44749b1e946684e6a65a884d778" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Mar 14 21:10:32 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Mar 14 21:10:33 2017 +0000" }, "message": "Merge \"Refactor annotation code for obsolete methods.\"" }, { "commit": "672a0b75c91af44749b1e946684e6a65a884d778", "tree": "7ae1c2fd5b9e2e7d155d9427848000d381e138d0", "parents": [ "01ea2aa05d6889b4ec08679606d3cdf36b302a8f" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Mar 10 13:09:57 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Mar 14 10:18:12 2017 -0700" }, "message": "Refactor annotation code for obsolete methods.\n\nReading annotations on obsolete ArtMethod* objects would cause\nproblems due to reading from the wrong dex file. We refactored that\ncode so now it always uses the correct dex file when reading\nannotations. This was rarely a problem since obsolete ArtMethods are\nonly ever exposed to the runtime itself which rarely reads\nannotations.\n\nWe also add a test for redefining Object.class. This is one of the\nsmall number of classes where the runtime will read it\u0027s annotations\noff of obsolete methods.\n\nBug: 31455788\nTest: ./test/testrunner/testrunner.py --host -j40\nChange-Id: I5f1c58464b89a7a9198a7b26b015b102a7dc9c2f\n" }, { "commit": "8f301e26943c53485abc2da5ff1907f7c2e0ff0c", "tree": "e255d1ce3041dff95d2cd2b6bf1b1474962d811e", "parents": [ "dcab11d06860ae1e23d03926adb6c31f75404032" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Feb 27 16:23:51 2017 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Tue Mar 07 16:47:13 2017 -0800" }, "message": "Revert \"Revert \"CHA for interface method.\"\"\n\nThis reverts commit 85a1ec1347c8117365632dcaaa648ebb0a8691f5.\n\nIt also fixed the issue that\u0027s caused by changing an invokeinterface\nto an invokevirtual, when the method is the original default interface\nmethod. Since the default interface method is not in any vtable, unless\nit\u0027s copied and becomes a virtual method.\n\nTest: ART_TEST_JIT\u003dtrue m -j20 test-art-host-run-test\nChange-Id: I627616978690485c66cfca47c234f1504066ea1d\n" }, { "commit": "85a1ec1347c8117365632dcaaa648ebb0a8691f5", "tree": "d6a9dcfa89747b39ee334a149e6bcba9a3e28447", "parents": [ "747aeb423267aed1d22a8d66013c351026d68f55" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Feb 27 16:00:48 2017 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Feb 27 16:00:48 2017 -0800" }, "message": "Revert \"CHA for interface method.\"\n\nThis reverts commit 7130fc769896a96573f55496444d87161e4b41e8.\n\nThere might be some race condition going on. 960-default-smali\nfails sometimes.\n" }, { "commit": "747aeb423267aed1d22a8d66013c351026d68f55", "tree": "7015e8ca15d4abec04813a3a1fce6fa2756463b0", "parents": [ "86a185601ae4067742a2236ff2bb494ae69915c2", "7130fc769896a96573f55496444d87161e4b41e8" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Feb 27 19:45:10 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Feb 27 19:45:11 2017 +0000" }, "message": "Merge \"CHA for interface method.\"" }, { "commit": "b1106e2aa98105dec533cdac9074a8f0216a4106", "tree": "881beb77d6550fbd62fd82241d2bb9107b72e794", "parents": [ "02c488068f4793204b00b40d76eb3a891a332def" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Feb 23 11:34:48 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Feb 23 11:34:48 2017 -0800" }, "message": "ART: Add flag for ArtMethod class state checks\n\nThe checks, especially in GetAccessFlags, is expensive. To help\nwith running a debug build on devices, add a flag to be able to\nturn the checks off.\n\nBug: 35644369\nTest: m\nChange-Id: I2a3db1a56986df8f4a8b2dc5bcb26e1bcaea0a24\n" }, { "commit": "7130fc769896a96573f55496444d87161e4b41e8", "tree": "667ddb35e759618e065d902a29e8c70061a2a5fc", "parents": [ "c0514866d973d7906258b552bf65b5c0382e3509" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Feb 15 15:59:24 2017 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Feb 22 14:37:43 2017 -0800" }, "message": "CHA for interface method.\n\nTest: m test-art-host, boot device.\nChange-Id: Ia2da74c2af1974fe344f215ae667eaf03c59a26e\n" }, { "commit": "7532d58afabda43b03bb30a06d1a448428aaebbf", "tree": "7fdc83741b78670bef07357a6125786ece73f532", "parents": [ "6a669aac63ffc17b5e903aa4a1f285fe338eadcf" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Feb 13 16:36:06 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Feb 21 09:49:44 2017 -0800" }, "message": "Allow redefined intrinsics\n\nWe make a change to allow intrinsics to be redefined. Note that\nredefined intrinisics will no-longer be optimized as much and will not\nbe inlined or moved out of loops.\n\nTest: ART_TEST_JIT\u003dtrue mma -j40 test-art-host\n\nChange-Id: Id6df89bb247d21f7859b48356ceba310eef9d105\n" }, { "commit": "eee0bd448ec057d3f224895ddb868786758eeb5b", "tree": "32283e1b1ccb97bced2999a364cf21361c8d07ac", "parents": [ "02b2349b1cd2a78e86b1a7542f8330e6c3aaeb35" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Feb 14 15:31:45 2017 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Feb 14 09:28:19 2017 -0800" }, "message": "Revert^6 \"Make class redefinition work with native methods on stack.\"\n\nWe were incorrectly trying to obtain the profiling information of a\nnative method.\n\nThis reverts commit 02b2349b1cd2a78e86b1a7542f8330e6c3aaeb35.\n\nReason for revert: Fixed test failure with jit configurations\n\nTest: ART_TEST_JIT\u003dtrue mma -j40 test-art-host\n\nChange-Id: Ic7112104aec64e597d2df80db5dc2a505d2cc2dd\n" }, { "commit": "fcbafb36eff1facdd4c2b60e5f56dd986c558901", "tree": "443a55d0ac3698e7eaf82395d6f3567b35d8474d", "parents": [ "406402baa4900c36b3fe27b03bf9e04e978e50be" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Feb 02 15:09:54 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Feb 02 15:32:25 2017 -0800" }, "message": "Ensure we don\u0027t attempt to compile obsolete methods.\n\nWe would run into issues if we tried to compile an obsolete method\nsince some of the information needed to do so is lost as we redefine\nthe declaring class. Since compiling these methods is unlikely to have\na major effect on the performance of the program we simply tell the\nJIT not to bother with them.\n\nAlso update all places to use the new SetDontCompile helper.\n\nTest: mma -j40 test-art-host\nTest: stress --cpu 60 \u0026; \\\n while ./test/run-test --host --jit --gcstress --64 942; do ; done\n\nChange-Id: Ie70fee6f503818e7589571facf28ccc5e7ca79f8\n" }, { "commit": "90c5a9b889af84cbb840c60e461d5bdbf8dc0df6", "tree": "ef0cc70d9ad20546f950be3364cbbbdeb05b5db6", "parents": [ "3cb871ab1af47576959fd24a99d370381b8f193e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 01 13:10:06 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Feb 02 17:49:39 2017 +0000" }, "message": "Various improvements to stack walking speed\n\nMake BitMemoryRegion constructor inlined, remove read barrier\nfor IsProxyMethod.\n\nAround 15% speedup for pmd benchmark, maybe more for CC.\n\nTest: test-art-host\n\nChange-Id: Ib4392649e041406e538cc944c26c69f68d388fb4\n" }, { "commit": "4ba388a39333b13f0f3bcde826444c77fd7166ed", "tree": "bbb2b337ee5798f50c5720d2e8467c53477d7826", "parents": [ "8f28bd479af445293474867191a0bf1cf0f08a4f" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Jan 27 10:26:49 2017 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Jan 30 12:13:30 2017 -0800" }, "message": "Remove Deoptimization code from class transformation.\n\nSince we removed the current_method from the compiled code we don\u0027t\nneed to deoptimize all frames anymore.\n\nThis is a partial revert of commit dba614810.\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\nChange-Id: I44a6dd89e1d96bd8c82c2c24a2f42fef023a80be\n" }, { "commit": "66866e2b5710cf8135674d11c6912a6f023de186", "tree": "cfbfd895ac3c71aa3eae50e649866369b95ed987", "parents": [ "dee81ab18f6be61add94256fbec90a913d09cc1a", "137cdfa1e8a3810fb86d3298ec10036c0491d13f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jan 27 19:11:31 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jan 27 19:11:32 2017 +0000" }, "message": "Merge \"Avoid read barriers for ArtMethod::GetDexFile\"" }, { "commit": "e8fcd013493b800227bd7ea5f38f6cc27e9b90d1", "tree": "16bf195a81dac7f66d513be7437a0a5c98dc2365", "parents": [ "cb7b5dfa0522cb234b6f2e4893ce21fd9c1a6cda" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Fri Jan 20 10:43:30 2017 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Jan 26 22:31:03 2017 -0800" }, "message": "Revert \"Revert \"CHA for abstract methods.\"\"\n\nThis reverts commit 8ebc8bf055e8bd8f6f167e65a69cf4dae136db55.\n\nWhen we set the single-implementation method for an abstract method, it\nneed to be protected by the cha-lock.\n\nTest: new testcase and ART_TEST_JIT\u003dtrue m -j20 test-art-host-run-test\nChange-Id: I66acb20ffa7e49dd9c391f001e3bb52f961872e4\n" }, { "commit": "137cdfa1e8a3810fb86d3298ec10036c0491d13f", "tree": "db568fce8feec0ae6910ff1150cf46f08fe7e3e5", "parents": [ "fbf47ea64a9f797a82030e919fa4f085c9eb5b28" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 26 14:03:11 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 26 17:04:55 2017 -0800" }, "message": "Avoid read barriers for ArtMethod::GetDexFile\n\nShows up in pmd benchmark from Class::FindDeclaredDirectMethod and\nClass::FindDeclaredVirtualMethod. There are still calls to\nIsProxyMethod that could probably be eliminated.\n\nReadBarrier::Mark goes from 12.39% to 3.45% according to perf.\n\nTest: test-art-host\n\nChange-Id: I6a4f2fa2d68bf5f393f83b9b70e8d6fcc9dbdaa2\n" }, { "commit": "8ebc8bf055e8bd8f6f167e65a69cf4dae136db55", "tree": "4cc89efe98ddc6ef0421405affafce95c5aabae2", "parents": [ "ae6c189b9d63ca4c2ae0e952187819c5e442e3c9" ], "author": { "name": "Colin Cross", "email": "ccross@android.com", "time": "Fri Jan 20 17:37:02 2017 +0000" }, "committer": { "name": "Colin Cross", "email": "ccross@android.com", "time": "Fri Jan 20 17:37:50 2017 +0000" }, "message": "Revert \"CHA for abstract methods.\"\n\nThis reverts commit ae6c189b9d63ca4c2ae0e952187819c5e442e3c9.\n\nThis is causing sporadic build failures with:\ndex2oatd F 01-20 15:05:33 8343 10164 cha.cc:292] Check failed: method_in_super-\u003eHasSingleImplementation() \n\nChange-Id: I4435ab028d3f7893e18b44347f294326c573a255\n" }, { "commit": "ae6c189b9d63ca4c2ae0e952187819c5e442e3c9", "tree": "beb13f852a0e1b3a7867dfd9d64e962d3cb403fb", "parents": [ "a8b4390a3b817a455b0abdf575922fea6182170d" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Jan 05 13:46:36 2017 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Jan 19 10:49:08 2017 -0800" }, "message": "CHA for abstract methods.\n\nAdd the ability to devirtualize abstract method invocation if\nan abstract method has only one implementation. Only support in JIT\nmode currently.\n\nTest: new testcase and ART_TEST_JIT\u003dtrue m -j20 test-art-host-run-test\nChange-Id: I43da03a9fa9a73840f5eaf207c6611e0913a7f66\n" }, { "commit": "308fa4d9044f2f67c2a7569660fcdb3f01ec8c37", "tree": "ccd6bdd48cf702474f4d0708181e71a3ec439230", "parents": [ "6141e156f7a62646f0350c6f0b368b591fb1211d", "9cd522b50a5f0209e97a2e6bcf8c9c350d3f3f83" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 19 13:20:58 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jan 19 13:20:59 2017 +0000" }, "message": "Merge \"Remove now unused dex_cache_resolved_types_ from ArtMethod.\"" }, { "commit": "9e8c8ca2d4c73d16d58d2375421d30a47df2f81c", "tree": "5a8700a97ebc348050822fd2a67c3966af39600c", "parents": [ "ee64b4d83aca1874f78ee80f58ae61333d327436", "6c4921b3a13ab645045dd20e5386daf552a31f41" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Jan 19 10:35:47 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jan 19 10:35:48 2017 +0000" }, "message": "Merge \"ART: Improve class initializer and constructor verification.\"" }, { "commit": "6c4921b3a13ab645045dd20e5386daf552a31f41", "tree": "46ff2d2d320a250a1ce6e5094a67632293093a4b", "parents": [ "a6040d4cca09c1030bbfcfdc131e38c7d6da8888" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Sep 21 15:41:06 2016 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Jan 19 09:01:02 2017 +0000" }, "message": "ART: Improve class initializer and constructor verification.\n\nDEX file verifier checks additional properties of class initializers\nand constructors:\n\n(i) Names match expected \u003cclinit\u003e / \u003cinit\u003e.\n(ii) The method descriptor for \u003cclinit\u003e is ()V.\n(iii) The return type of \u003cinit\u003e is V.\n(iV) No other names start with \u0027\u003c\u0027.\n\nBug: 31313719\nChange-Id: I60bffa6561e1bae353f97c42377ea556bfa790af\nTest: m test-art-host-gtest-dex_file_verifier_test\n" }, { "commit": "9cd522b50a5f0209e97a2e6bcf8c9c350d3f3f83", "tree": "503af07a0a6122d161a81dfed2d2e64f8e501d5a", "parents": [ "c3b7bf3bcc55ec3c684210176ff6ddeb9d33ac19" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 18 17:05:59 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 18 17:08:57 2017 +0000" }, "message": "Remove now unused dex_cache_resolved_types_ from ArtMethod.\n\nTest: test-art-host test-art-target\nChange-Id: I004c040da165934f74eb8b8e85e99e7c6621c2df\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": "5d37c152f21a0807459c6f53bc25e2d84f56d259", "tree": "7d8cbce0a55f258150a047def70244f79afc866d", "parents": [ "aa89a4c6fca095904521842c018399f1e3501a45" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 12 13:25:19 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 16 22:56:56 2017 +0000" }, "message": "Put inlined ArtMethod pointer in stack maps.\n\nCurrently done for JIT. Can be extended for AOT and inlined boot\nimage methods.\n\nAlso refactor the lookup of a inlined method at runtime to not\nrely on the dex cache, but look at the class loader tables.\n\nbug: 30933338\ntest: test-art-host, test-art-target\nChange-Id: I58bd4d763b82ab8ca3023742835ac388671d1794\n" }, { "commit": "b1a7c9453e9abeaa09ab6c167138fa7d28b625d5", "tree": "a94b5a2ba78e76129f04a92adf85ed7589f6a4ab", "parents": [ "2e17788df086284845108c6b39954a5c74e93a69" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 10 16:48:11 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 10 16:59:36 2017 -0800" }, "message": "Fix possible deadlock in AddAccessFlags and ClearAccessFlags\n\nReload the value each loop iteration or else it will spin forever\nif the CAS fails due to another thread adding a different flag.\n\nTest: mm\n\nChange-Id: Idc320921597acc20d56301899b398756563029f2\n" }, { "commit": "fe150d9821a162bae987bfd2bdcb57d31d1970fe", "tree": "e46969125ce2c9bfbd6412b16b84956795cc2255", "parents": [ "a79003958e6844541bae7db4eafdb442f9ed31a5", "b466d807007b522c17c56f66376b7f439b8f1352" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Jan 06 09:04:18 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jan 06 09:04:19 2017 +0000" }, "message": "Merge \"Remove bogus DCHECK.\"" }, { "commit": "b466d807007b522c17c56f66376b7f439b8f1352", "tree": "91d0c0233997ac2f6fbc5b6b654933cd72bfb8e1", "parents": [ "059802455e80a92a0fac780246968588bb0cf88b" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 04 11:49:00 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 04 16:29:23 2017 +0000" }, "message": "Remove bogus DCHECK.\n\nThe code may be executed concurrently by multiple threads,\nso the flag may already be set.\n\nTest: test-art-host\nbug:34074968\nChange-Id: I162b90a853ce53e6b7063754bbfce5d68cffc82a\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": "f6abcda293b115a9d7d8a26376ea2dcf2d1dc510", "tree": "a30ce596848d14c95a73449dbf2bfccf26a510b9", "parents": [ "ce77fc0e7f60a15354bb20c356537cbf8b53b722" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 21 09:26:18 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 21 09:26:18 2016 +0000" }, "message": "Revert \"Revert \"Revert \"Basic obsolete methods support\"\"\"\n\nFails in tracing mode\n\nBug: 32369913\nBug: 33630159\n\nThis reverts commit ce77fc0e7f60a15354bb20c356537cbf8b53b722.\n\nChange-Id: I1bdcf6ad467f2e31f9c5d0c3c987b90a4f5efc69\n" }, { "commit": "ce77fc0e7f60a15354bb20c356537cbf8b53b722", "tree": "b4301967c423ab190a4274409d99a3d6e23f0e8a", "parents": [ "d54f43ca39dfa92f08c2d760123f185f0f65fb86" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 15 01:05:52 2016 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Dec 19 14:26:18 2016 -0800" }, "message": "Revert \"Revert \"Basic obsolete methods support\"\"\n\nThis reverts commit b81a9840b44480bfeacd74b8d9f51e06f295411d.\n\nThere were 2 issues with the original CL\u0027s test 916-obsolete-jit that\ncaused it to sporadically fail.\n\nFirst, when checking if we had jitted the function under test in\n916-obsolete-jit we failed to check to see if the function \u0027doCall\u0027,\nwhich is used to work-around bugs in our deoptimization \u0026 compilation\nsystems, had also been jitted.\n\nIn the case where the \u0027sayHi\u0027 function had been jitted but the\n\u0027doCall\u0027 function had not we would (correctly) fail to redefine the\n\u0027Transform\u0027 class since we would not be able to deoptimize the \u0027sayHi\u0027\nfunction since it is under a quick_to_interpreter_bridge (runtime)\nframe.\n\nSecondly, the function Main.isInterpretedFunction was incorrect and\nwould always return false, regardless of the actual state of the\nfunction. This would cause the test to fail as the\nquick_to_interpreter_bridge frame prevented deoptimization of the\nobsoleted function. Usually the warm-up period was enough to make sure\nthe methods were jitted anyway but this was not guaranteed.\n\nBoth of these problems become more likely to occur on systems with\nmore cpu contention such as the buildbots.\n\nTest: stress --cpu 60 \u0026; while ./test/run-test --host --jit 916; do ; done\n\nReason for revert: Fixed bug in test that was causing failures.\n\nOriginal Tests:\n\nTest: ./test/run-test --host 914\nTest: ./test/run-test --host 915\nTest: ./test/run-test --host 916\nTest: mma -j40 test-art-host\nTest: ART_TEST_JIT\u003dtrue \\\n ART_TEST_INTERPRETER\u003dtrue mma -j40 test-art-host\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64 --no-jit\n\nBug: 32369913\nBug: 33630159\n\nChange-Id: If1a92e47b90965a7dc21c5826185debe62bd1554\n" }, { "commit": "b81a9840b44480bfeacd74b8d9f51e06f295411d", "tree": "d79a3567b272b6eb4fb94d66a537cf92bef455db", "parents": [ "d8936da27b792d1ca02e59c92456a1a53c7b9905" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 15 00:59:05 2016 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 15 00:59:05 2016 +0000" }, "message": "Revert \"Basic obsolete methods support\"\n\nThis reverts commit d8936da27b792d1ca02e59c92456a1a53c7b9905.\n\nReason for revert: Some sort of race in JIT\n\nChange-Id: Ibb4e520bb0721d6d7aa2c841a52eb5baff07740e\n" }, { "commit": "d8936da27b792d1ca02e59c92456a1a53c7b9905", "tree": "6a593dd426e7f4a15c651dd3b872f22fb64f09ce", "parents": [ "7c73bf3518eea7544406eadfed6946411c54a267" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Nov 28 16:24:32 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 14 13:01:27 2016 -0800" }, "message": "Basic obsolete methods support\n\nAdd support for executing obsolete methods following redefinitions.\nThis support includes methods that have been jitted. This does not add\nany additional validity checks to our redefinition functions.\n\nNote using work-arounds to ensure 916 works pending some fixes to the\ncompiler, deoptimizer, or both.\n\nTest: ./test/run-test --host 914\nTest: ./test/run-test --host 915\nTest: ./test/run-test --host 916\nTest: mma -j40 test-art-host\nTest: ART_TEST_JIT\u003dtrue \\\n ART_TEST_INTERPRETER\u003dtrue mma -j40 test-art-host\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64\nTest: ./art/tools/run-jdwp-tests.sh --mode\u003dhost --variant\u003dX64 --no-jit\n\nBug: 32369913\nBug: 33630159\n\nChange-Id: I78ef95f484146f1fb93c37fc50f56575bdab2432\n" }, { "commit": "c571e73e21202cff0a8ec3bcfecb7d326e8648ef", "tree": "905cb5252b8bc8c335f569c869dfb8529301f40c", "parents": [ "edcd7929e2bf521316fa5305109c330d563d87db", "cfa325e4ca65603fdb03a836a6cb394d23ed511f" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Dec 02 11:52:14 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Dec 02 11:52:15 2016 +0000" }, "message": "Merge \"ART: Add verifier support for invoke-polymorphic.\"" }, { "commit": "cfa325e4ca65603fdb03a836a6cb394d23ed511f", "tree": "f14e628cc90f7b03f8f227a30361993f3f594f11", "parents": [ "35b6546b0a4fe423ed6f27d8c5b709d5191e99ef" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Oct 13 10:25:54 2016 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Dec 02 10:09:40 2016 +0000" }, "message": "ART: Add verifier support for invoke-polymorphic.\n\nChange-Id: I1e1860cad80db46320c3ef5a9eaceb7529ea68d7\nBug: 30550796,33099829,33191712\nTest: make test-art-host\n" }, { "commit": "063fc772b5b8aed7d769cd7cccb6ddc7619326ee", "tree": "bc165781989087a998721991504e589a7d5b0926", "parents": [ "48d08a4233ee4450b0d5073d41445f9dd1f17191" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Tue Aug 02 11:02:54 2016 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Dec 01 11:15:47 2016 -0800" }, "message": "Class Hierarchy Analysis (CHA)\n\nThe class linker now tracks whether a method has a single implementation\nand if so, the JIT compiler will try to devirtualize a virtual call for\nthe method into a direct call. If the single-implementation assumption\nis violated due to additional class linking, compiled code that makes the\nassumption is invalidated. Deoptimization is triggered for compiled code\nlive on stack. Instead of patching return pc\u0027s on stack, a CHA guard is\nadded which checks a hidden should_deoptimize flag for deoptimization.\nThis approach limits the number of deoptimization points.\n\nThis CL does not devirtualize abstract/interface method invocation.\n\nSlides on CHA:\nhttps://docs.google.com/a/google.com/presentation/d/1Ax6cabP1vM44aLOaJU3B26n5fTE9w5YU-1CRevIDsBc/edit?usp\u003dsharing\n\nChange-Id: I18bf716a601b6413b46312e925a6ad9e4008efa4\nTest: ART_TEST_JIT\u003dtrue m test-art-host/target-run-test test-art-host-gtest\n" }, { "commit": "a01de59eef990c07df9815376cd3c675e3acd481", "tree": "86f6ef41c13de572dc134956c836dd94c35d9d64", "parents": [ "424c03aac25835febbf5c695b943206bc40ef335" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Nov 15 10:43:06 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Nov 30 10:23:47 2016 -0800" }, "message": "Infrastructure for obsolete methods\n\nThis adds some of the structures and functions needed to eventually\nsupport obsolete methods. Specifically this adds the code to create\nand iterate through the obsolete dex-cache maps of classes and\nre-organizes the redefinition code to be more sensible and extensible.\n\nBug: 32369913\nTest: ./test/run-test --host 902\nTest: mma -j40 test-art-host\n\nChange-Id: I93d60fa66e7512e8b059cdf42af8a56e93ef4bd7\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": "28357fab628bd9b91749988b554977398caf9963", "tree": "957fac3c1003713c349e0e216762836138c596a7", "parents": [ "c89f9776a107ca20d0146c16fa881db91c4f8266" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 18 16:27:40 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 19 10:04:37 2016 -0700" }, "message": "Move most of class linker to ObjPtr\n\nReturn values are still mirror pointer. Fix some failing asserts in\ncompiler driver and tests.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I4450bf9dfb2541749496b8388616e8aae8488919\n" }, { "commit": "709b070044354d9f47641f273edacaeeb0240ab7", "tree": "3a8ac051d7c35076303984d0d892cdd396b60427", "parents": [ "1a4de6a2453a3ad0310aca1a44e7e2d3b6f53bc1" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Oct 13 09:12:37 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Oct 18 14:10:04 2016 -0700" }, "message": "Remove mirror:: and ArtMethod deps in utils.{h,cc}\n\nThe latest chapter in the ongoing saga of attempting to dump a DEX\nfile without having to start a whole runtime instance. This episode\nfinds us removing references to ArtMethod/ArtField/mirror.\n\nOne aspect of this change that I would like to call out specfically\nis that the utils versions of the \"Pretty*\" functions all were written\nto accept nullptr as an argument. I have split these functions up as\nfollows:\n1) an instance method, such as PrettyClass that obviously requires\nthis !\u003d nullptr.\n2) a static method, that behaves the same way as the util method, but\ncalls the instance method if p !\u003d nullptr.\nThis requires using a full class qualifier for the static methods,\nwhich isn\u0027t exactly beautiful. I have tried to remove as many cases\nas possible where it was clear p !\u003d nullptr.\n\nBug: 22322814\nTest: test-art-host\nChange-Id: I21adee3614aa697aa580cd1b86b72d9206e1cb24\n" }, { "commit": "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": "762869dee6e0eadab5be1c606792d6693bbabf4e", "tree": "8c986c621e8a5f3cf4e4e3b2cc13b400401ad89b", "parents": [ "b4cf427734c6839b46d0d6037e3189a5e8aa1bdb" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jul 15 15:28:35 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 30 10:44:31 2016 +0100" }, "message": "Simplify our intrinsic recognizer.\n\n- Use the modifiers for storing the intrinsic kind.\n- Delete dex_file_method_inliner and its associated map.\n\nThis work was also motivated by the fact that the inline\nmethod analyzer leaks intrinsic tables, and even worse, might re-use\na table from one dex file to another unrelated dex file in the presence\nof class unloading and the unlikely event of the dex files getting\nthe same address.\n\ntest: m test-art-host m test-art-target\n\nChange-Id: Ia653d2c72df13889dc85dd8c84997582c034ea4b\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": "9323e6e44060545d6c4f925139572868f65b7fe3", "tree": "65f3f380605d75f34d49aec49fca494bec26ef5b", "parents": [ "58735fbfbb2da0c0a19b3239da07c2353c40767d" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Sep 13 08:58:35 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Sep 14 08:01:39 2016 -0700" }, "message": "Separate annotations from dexfile reading.\n\nBug: 22322814\nChange-Id: I867d66da407dd80394a10d19903ebbc1ec3986ff\nTest: test-art\n" }, { "commit": "367f3dd32454858b8b25d87feb8f6599d3b4c9dd", "tree": "9fe45c9f6785c31918f70ed2d12683f9c8d702af", "parents": [ "fa7b5c97db681e3d64d145807927cfafae78729b" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Sep 01 17:00:24 2016 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Sep 09 10:48:56 2016 -0700" }, "message": "jni: Add @CriticalNative optimization to speed up JNI transitions\n\nChange-Id: I963059ac3a72dd8e6a867596c356d7062deb6da7\n" }, { "commit": "3a21e386fa55a8b86dffa5250985b263fcfd2155", "tree": "ce6fecaae067f86e094ae7844ee5fa4ae8ee63e8", "parents": [ "f94a4cf18946bda4a4f19378436d2bf131a492ca" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 02 12:38:38 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 02 15:15:28 2016 +0100" }, "message": "Clean up some includes.\n\nRemove some unnecessary includes from header files, replace\nothers with forward references and add includes to source\nfiles as needed. Reduce dependency on stack.h by pulling\nStackReference\u003c\u003e out to its own file.\n\nTest: m test-art-host\nChange-Id: I0fb182145e328870cbd918b0ef6ae2950223c1b2\n" }, { "commit": "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": "9d4b6da934934c322536ee3309b63ce402740f49", "tree": "9e7ee5023d6036b98e0560411bb0527efdedca01", "parents": [ "2af1aa066e3d20edd8fea5d5b6dbbbad73102d52" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Jul 29 09:51:58 2016 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Aug 16 20:19:36 2016 +0000" }, "message": "jni: Fast path for @FastNative annotated java methods\n\nAdds a faster path for java methods annotated with\ndalvik.annotation.optimization.FastNative .\n\nIntended to replace usage of fast JNI (registering with \"!(FOO)BAR\" descriptors).\n\nPerformance Microbenchmark Results (Angler):\n* Regular JNI cost in nanoseconds: 115\n* Fast JNI cost in nanoseconds: 60\n* @FastNative cost in nanoseconds: 36\n\nSummary: Up to 67% faster (vs fast jni) JNI transition cost\n\nChange-Id: Ic23823ae0f232270c068ec999fd89aa993894b0e\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": "84413a7617602c466e39de8e8897914a6d06779d", "tree": "0beb127170772d2a6075ea4820bdf0e29de15526", "parents": [ "d4b7ad71359b571d0056a9bcad3811015128cc8f", "479b1de64cd7a8c9c8ca182c699cb2048850c35d" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Jul 21 23:58:00 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jul 21 23:58:01 2016 +0000" }, "message": "Merge changes I295c7876,Ib4b84b7b\n\n* changes:\n ART: Remove PACKED from ArtMethod\u0027s ptr_sized_fields_\n ART: Rename ArtMethod JNI field\n" }, { "commit": "479b1de64cd7a8c9c8ca182c699cb2048850c35d", "tree": "bca0b26c4e210e56d25baca49f7a25ffe9178813", "parents": [ "75f0885d7c3d72a0351912b6d47682e1c860efdf" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 19 18:27:17 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 21 15:47:35 2016 -0700" }, "message": "ART: Remove PACKED from ArtMethod\u0027s ptr_sized_fields_\n\nRemove the PACKED(4) hack, as it\u0027s highly annoying when debugging\na 64-bit process. Instead, fix the actual offset and size computation\nfor cross-size accesses.\n\nTest: m test-art-host\nChange-Id: I295c78760b74b6a62946e76856f218b4eb159cdc\n" }, { "commit": "75f0885d7c3d72a0351912b6d47682e1c860efdf", "tree": "d8a586b205ad3a502d8a007565a0140591d340ce", "parents": [ "27d99ed24328509cd3df7f895ce2696a1884bb21" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 19 08:06:07 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 21 11:26:47 2016 -0700" }, "message": "ART: Rename ArtMethod JNI field\n\nThe field is multi-purpose, rename it to data and clean up\naccessors in preparation of more checks.\n\nTest: m test-art-host\nChange-Id: Ib4b84b7b1a51ca201544bc488ce8770aa858c7fd\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": "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": "e36389f20c83083c0aaba2dcac0888951e55cae1", "tree": "3fe74b637f8dc90e3cbc8ad3949f3f340236537f", "parents": [ "abf64415cf99a9a7ee048ae26c76f7bbe972a6b9", "d4ceecc85a5aab2ec23ea1bd010692ba8c8aaa0c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:40:14 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jun 29 08:40:14 2016 +0000" }, "message": "Merge \"Revert \"Refactor GetIMTIndex\"\"" }, { "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": "8a0b9bf88aa055b2669825048dff6353dd04aafb", "tree": "51ed6b7a5436f191acea09f42def0c44c2ee6bd3", "parents": [ "edbcf54199cab8557f93ef3d136e4c2f335b915e", "50706437d8216e41f0fea1e413cda7891324d397" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Jun 16 00:28:15 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jun 16 00:28:15 2016 +0000" }, "message": "Merge \"Refactor GetIMTIndex\"" }, { "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": "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": "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": "56fdd0e63812764bbeb8cc52e376d3fd1f270052", "tree": "6988e995bf02f788066fb6e748cd3049fe87d31a", "parents": [ "9d05b59a8595cf7eeece1795eb65b6b8706aad45" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Apr 28 14:56:54 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Apr 29 17:33:22 2016 -0700" }, "message": "ART: Disambiguate access-checks mode from lock-counting\n\nLock-counting (when structural locking verification failed) is a\nspecial sub-mode of access-checks and must be disambiguated, because\nwe currently use access-checks mode class-wide when at least one\nmethod soft-fails, but do not stop the compiler/JIT to compile\nthe \"working\" methods. So we may end up in the access-checks\ninterpreter for a working method through deopt without knowing\nwhich locks are already held.\n\nBug: 28351535\n\n(cherry picked from commit f517e283d477dd2ae229ee3f054120c6953895db)\n\nChange-Id: I083032f064d88df8f8f0611ad8b57d1b39cd09fb\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": "250a378d5a2152662e0fa820f2b38f794ddd3596", "tree": "51896bf755c3f7afd1e0616a9a54071bda6dccc0", "parents": [ "a584db5460a31198bf621cdacf5bc304a984efa4" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Apr 20 16:27:53 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Apr 25 09:29:21 2016 +0100" }, "message": "Add flag to ArtMethod to skip compilation.\n\nTo ensure even the JIT will not try to compile methods with\nsoft failures a runtime_throw.\n\nbug:28293819\nbug:28313047\n\nChange-Id: Ie3fd71ded0b77de8dab1c3c825b867cb321b8873\n" }, { "commit": "1d011d9306fd4ff57d72411775d415a86f5ed398", "tree": "fadba83b9fb32840ff777948ace3bd8eb477e38f", "parents": [ "82a0229e949eb3ea209c2bd33fe2d60908f5dbc7" ], "author": { "name": "Bill Buzbee", "email": "buzbee@google.com", "time": "Mon Apr 04 16:59:29 2016 +0000" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Fri Apr 08 09:04:10 2016 -0700" }, "message": "Revert \"Revert \"Revert \"Revert \"ART: Improve JitProfile perf in arm/arm64 mterp\"\"\"\"\n\nBug: 28081559\n\nThis reverts commit 961ea9fe42edcc2c57469bf451d1ca421da5cd59.\n\nChange-Id: I98a5bb8112646706ae7bd73bf6393cb956466be3\n" }, { "commit": "961ea9fe42edcc2c57469bf451d1ca421da5cd59", "tree": "eb1930308f9f64a108e3f2387faca90cff869509", "parents": [ "6b7d2c09b4710503a72ff5de31bff5cb23a3a921" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Apr 01 12:02:58 2016 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Apr 01 12:02:58 2016 -0700" }, "message": "Revert \"Revert \"Revert \"ART: Improve JitProfile perf in arm/arm64 mterp\"\"\"\n\nThis reverts commit 4a8ac9cee4312ac910fabf31c64d28d4c8362836.\n\n570-checker-osr intermittently failing.\n\nBug: 27939339\n" }, { "commit": "4a8ac9cee4312ac910fabf31c64d28d4c8362836", "tree": "a9d662cb43a3fe1c071618ef78f772655bfc4442", "parents": [ "ec898396658bc6a36e2b3c32569fbcb034d60e22" ], "author": { "name": "Bill Buzbee", "email": "buzbee@google.com", "time": "Fri Mar 25 13:16:55 2016 +0000" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Tue Mar 29 15:48:45 2016 -0700" }, "message": "Revert \"Revert \"ART: Improve JitProfile perf in arm/arm64 mterp\"\"\n\nReady for review.\n\nThis reverts commit 6aef867f4d1a98a12bcdd65e9bf2ff894f0f2d7e.\n\nChange-Id: I5d53ed2bedc7e429ce7d3cdf80b6696a9628740e\n" }, { "commit": "6aef867f4d1a98a12bcdd65e9bf2ff894f0f2d7e", "tree": "cda140806a091805213edca2708e69337f6869ef", "parents": [ "1cb8689ccc23fd513c5472911dd07e44b55e9c8d" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Mar 25 11:27:48 2016 +0000" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Mar 25 11:27:48 2016 +0000" }, "message": "Revert \"ART: Improve JitProfile perf in arm/arm64 mterp\"\n\nThis reverts commit c1d6b341eed646e5adafc6c4fd4e3748f0292368.\n" }, { "commit": "42cd3cb6379fe3faf078dcce98610e90155bc8e0", "tree": "59ff8135fcfa13b8263afe0d77b58dce6cc2ea0b", "parents": [ "62e78a7ea8221f3fbe848fa883fb86c9d87ba50b", "c1d6b341eed646e5adafc6c4fd4e3748f0292368" ], "author": { "name": "Bill Buzbee", "email": "buzbee@google.com", "time": "Thu Mar 24 16:16:59 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Mar 24 16:16:59 2016 +0000" }, "message": "Merge \"ART: Improve JitProfile perf in arm/arm64 mterp\"" }, { "commit": "c1d6b341eed646e5adafc6c4fd4e3748f0292368", "tree": "ddee928954b03a919ce601f12876f1fe161d577a", "parents": [ "eecf60d51b481647c8508f22b3d6ce437773ea0c" ], "author": { "name": "buzbee", "email": "buzbee@google.com", "time": "Tue Mar 01 15:03:16 2016 -0800" }, "committer": { "name": "buzbee", "email": "buzbee@google.com", "time": "Wed Mar 23 11:11:26 2016 -0700" }, "message": "ART: Improve JitProfile perf in arm/arm64 mterp\n\nART currently requires two profiling-related things from the\ninterpreters: hotness updates and OSR switch checks. The hotness\nupdates previously used the existing instrumentation framework - which\nis flexible, but quite heavyweight. For most things, the\ninstrumentation framework overhead is acceptable, but because we do a\nhotness update on every backwards branch the overhead is unacceptable.\nPrior to this CL, branch profiling dominates interpreter cost.\n\nHere, we bypass the instrumentation framework for hotness updates\nand deliver a significant performance improvement. Running\ninterpreter-only (dalvikvm -Xint) on a Nexus 6, we see the logic\nsubtest of Caffeinemark improving from 2600 to 9200, and the\noverall score going from 1979 to over 3000. Compared to the\nC++ switch interpreter, we see a 6x improvement on the branchy logic\nsubtest and a 2.6x improvement overall.\n\nCompared with the previous mterp which did not have support for\njit profiling, we see a few (1% to 5%) performance loss on the\nstandard command-line benchmarks. I consider this acceptable\n(we could create an alternate non-profiling mterp which would\nhave no penalty, but I don\u0027t consider this overhead big enough to\njustify that).\n\nChange-Id: I50b5b8c5ed8ebda3c8b4e65d27ba7393c3feae04\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": "0b4cbd0c2a75b47ae09d21e5d73d2b1709cb5b9e", "tree": "72572a01a181c12272a5570ae8b4d41e9ef553f8", "parents": [ "c3214fbb66b5f4c436c44d027df5046c803f6432" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 08 16:49:58 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Mar 14 17:07:11 2016 -0700" }, "message": "Add oatdump support for app images\n\nExample usage on host:\noatdumpd --app-oat\u003dart/plus32.odex --app-image\u003dart/plus32.art\n--image\u003dart/oats/system@framework@boot.art --instruction-set\u003darm\n\nTODO: Add to oatdump test.\n\nBug: 27408512\nBug: 22858531\n\n(cherry picked from commit bcb6a72569a1401b36a3ad3b6aa4d13e29966cf0)\n\nChange-Id: I9d1aa7eaa16795e5fbabc6974d245849e16b1d03\n" }, { "commit": "3612149aee482ab7a17da68b0ef5fef3879729a2", "tree": "42180a35266333cbf37163a12185cf8824255a71", "parents": [ "a5502e31cb558c6e257d1dd1889c073a793e3d9d" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Feb 22 13:43:29 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Feb 23 13:24:05 2016 -0800" }, "message": "Make JNI work correctly with default methods.\n\nAlso adds some tests for JNI and DefaultMethods.\n\nBug: 27259142\nBug: 24618811\n\nChange-Id: I31222e3e41059d803be1dbb0f40e1144ac4bf457\n" }, { "commit": "fcea56f9cc51957161fe7a6e35e895fd8c4c4a7f", "tree": "b43c197e9370eab38a58e1d54ec319ea45df613d", "parents": [ "aaf56c4c95331d4dd8ac298e6c234d4d58d28308" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Feb 17 11:59:05 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Feb 17 13:02:11 2016 -0800" }, "message": "Fix issue with copied methods not being checked.\n\nIn several places we were using IsMiranda to check if a method is\ncopied. This misses cases involving default methods.\n\nBug: 27216437\nChange-Id: I8c800e3e622a9c0ca0f8752c3d5202f433af9a1c\n" }, { "commit": "867d63b65f653d27dc7ea87e924f47148cec22a7", "tree": "2038bbb25fe121bdcf3653cc32211c6e2cb88936", "parents": [ "6006e2ce92fd86fdf028cd7b3afe972815b0e0f3", "df707e406877e9c0426dd051c00933ebb331673e" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Feb 04 00:34:43 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Feb 04 00:34:43 2016 +0000" }, "message": "Merge \"runtime: Don\u0027t skip verification for -Xverify:soft-fail\"" }, { "commit": "df707e406877e9c0426dd051c00933ebb331673e", "tree": "db9c96c96d373935a83dda2fafbf3c7e1aef3377", "parents": [ "e3d4ff566f9545114bf9ed72421e49eaf9d37953" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Feb 02 16:56:50 2016 -0800" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Feb 03 13:39:19 2016 -0800" }, "message": "runtime: Don\u0027t skip verification for -Xverify:soft-fail\n\nWhen forcing the interpreter into access checks mode,\nmake sure that the regular verification is still run,\ngiving the verifier an opportunity to throw a VerifyError.\n\nIf verification would\u0027ve succeeded (without -Xverify:soft-fail flag),\noverride this and soft-fail, to force the interpreter-with-access-checks to be run\ninstead of the normal faster interpreter.\n\nThis fixes the following run-tests under the interpeter-access-checks:\n* 135\n* 412\n* 471\n* 506\n* 800\n\nBug: 22414682\nChange-Id: I5cb86a8bba71c7af9361a63c0802786c852b857b\n" }, { "commit": "e7f75f35371cf32954d94f9bb10fc96924f7edb4", "tree": "4968ea7f5dd02786e50bf4dc1873c30ff37cde8a", "parents": [ "c3cf1d2e35a45bd2e2c60fbc9c2a1a6a56dfc529" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 01 16:08:15 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 01 16:19:59 2016 -0800" }, "message": "Add read barrier option to UpdateEntrypoints\n\nAlso call this without read barrier for image relocation.\n\nBug: 26786304\nChange-Id: Ia718c1491b54cadb7283c62afc1eb9031a15b4ef\n" }, { "commit": "fbc31087932a65e036a153afab3049dc5298656a", "tree": "c728b9039a71db3be9cae888ad5e1afc31f34a13", "parents": [ "c7f4e3a5aeaa23342b4e03b0d751f60ac5c5815c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Jan 24 11:59:56 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jan 25 14:22:15 2016 -0800" }, "message": "Revert \"Revert \"Load app images\"\"\n\nThis reverts commit 1bc977cf2f8199311a97f2ba9431a184540e3e9c.\n\nBug: 22858531\n\nChange-Id: Ide00bf3a73a02cba3bb364177204ad1b13f70295\n" }, { "commit": "1bc977cf2f8199311a97f2ba9431a184540e3e9c", "tree": "580a02752d8e447f6dce7cce01386c7e2a9a87f4", "parents": [ "f7fd970244f143b1abb956e29794c446e4d57f46" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sat Jan 23 14:15:49 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sat Jan 23 14:15:49 2016 +0000" }, "message": "Revert \"Load app images\"\n\nFails when a method is duplicated (see test 097-duplicate-method)\n\nBug: 22858531\n\nThis reverts commit f7fd970244f143b1abb956e29794c446e4d57f46.\n\nChange-Id: Ib30ae5be00cc568e799290be6b3c8f29cbbe4c20\n" }, { "commit": "f7fd970244f143b1abb956e29794c446e4d57f46", "tree": "aac1f57ac70747957f609bb46305dfeca87645a1", "parents": [ "95005291d8ebdd1d2ac58ffc5181fef4fbbf2383" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 09 11:16:49 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jan 22 15:01:55 2016 -0800" }, "message": "Load app images\n\nSupport in-place patching of the app image based on boot image\nlocation and app oat location. Only loads for art run test so far\nsince we do not automatically generate app images for app installs.\n\nN5 maps launch time (~200 runs):\nBefore: 930ms\nAfter: 878.18ms\nAfter + image class table: 864.57ms\n\nTODO:\nOatdump support.\nStore class loaders as class roots in image.\n\nBug: 22858531\n\nChange-Id: I9cbc645645e62ea2ed1ad8e139e91af7d88514c1\n" }, { "commit": "705ad49f353d3f90d8b63625aca2c2035bacdbef", "tree": "ac70af53158a80bc35c057aefae11428281df9ac", "parents": [ "fae1db92d8433d0f75258c190bcf2c940731f036" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Sep 21 11:36:30 2015 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Jan 12 15:40:31 2016 -0800" }, "message": "Support directly invoking interface default methods\n\nWith the Java 8 Language one is allowed to directly call default\ninterface methods of interfaces one (directly) implements through the\nuse of the super keyword. We support this behavior through the\ninvoke-super opcode with the target being an interface.\n\nWe add 3 tests for this behavior.\n\nCurrently only supports slow-path interpreter.\n\nInvoke-super is currently extremely slow.\n\nBug: 24618811\n\nChange-Id: I7e06e17326f7dbae0116bd7dfefca151f0092bd2\n" }, { "commit": "6c8467ffc427a513dc2fdea7ed9eeb0bb54d30ef", "tree": "a3f0258c260e3818c32aa254d484724fde1d16d3", "parents": [ "b9bbbeb2b05d74cae0fe0682a26b19f5a7e68d8c" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Nov 20 15:03:26 2015 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Nov 30 22:39:43 2015 +0000" }, "message": "Make debugger support default methods.\n\nSupports breaking on, stepping into and through default methods.\n\nBug: 25549445\nBug: 24618811\n\nChange-Id: I1a47e3263a0624adbd9e83b99d0acd7bef7757a4\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": "a5891e81a2fb833307cf7c7e7267070dc0223dc8", "tree": "a7acb2a0f2836fee2729db2b795d15bb3f586863", "parents": [ "2fa684206b0a8bd203d07e1edf74a839b498f74f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Nov 06 14:18:27 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Nov 06 16:05:42 2015 +0000" }, "message": "Fix interaction between JIT and instrumentation.\n\n- The JIT needs to go through the instrumentation to update\n entry points.\n- The instrumention needs to know if a method got JITted\n to know if needs to deoptimize.\n\nbug:25438583\n\nChange-Id: I4b186a1da9f4a3fb329efd052a774d5502a902a1\n" } ], "next": "22cf3d361695ff1d585a8a412ebeade69749811f" }