)]}' { "log": [ { "commit": "9575c124c3d77e9f07e8c101571776ac9270af83", "tree": "e3477e5c407249cd6e02dfcc5d039bfdc0375b7a", "parents": [ "fa73acca98ddcad8cf208e9a8bce714744b70a6c" ], "author": { "name": "Mythri Alle", "email": "mythria@google.com", "time": "Fri Nov 12 12:04:41 2021 +0000" }, "committer": { "name": "Mythri Alle", "email": "mythria@google.com", "time": "Mon Nov 15 09:48:20 2021 +0000" }, "message": "Revert^2 \"Cleanup the code to determine instrumentation level\"\n\nThis reverts commit cb8f8c12872d36304dbac80fbd08d8400a757fe5.\n\nReason for revert: Reland commit 21ef5a80704da06bdb2945be1c735ca86e8f1860 with fixes\n\nFixes:\nIn JITed code when we compare a bool variable\n(instrumentation_stubs_installed_) we should use cmpb and not cmpw. This\nwasn\u0027t caught earlier because the next three variables are bool which\nare related to instrumentation are false when instrumentation is disabled.\n\nTest: testrunner.py --host --redefine-stress -t 421-large-frame,\ntestrunner.py --host --redefine-stress -t 545-tracing-and-jit\n\nChange-Id: Iba363fb62d0cb41bcbc86af202eae73a833ba267\n" }, { "commit": "6545ee31288b72e117450ea77b07f3593bff6180", "tree": "60085d1b253758ba32d8b585e493f634d95e422a", "parents": [ "a0232ad060c65d0ac3e022c9f721b41c02fd0b71" ], "author": { "name": "Shalini Salomi Bodapati", "email": "shalini.salomi.bodapati@intel.com", "time": "Tue Nov 02 20:01:06 2021 +0530" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 11 11:32:25 2021 +0000" }, "message": "Support Math.fma Intrinsic for x86 and x86_64\n\nIntrinsic Implementation is 2287x faster than base\nTime for base: 188.25s\nTime for patch: 0.0822s\n\nTest: ./run-test --host --64 --optimizing 082-inline-execute\nTest: ./run-test --host --64 --optimizing --instruction-set-features sse4.1 082-inline-execute\nTest: ./run-test --host --64 --optimizing --instruction-set-features sse4.1,avx2 082-inline-execute\nSigned-off-by: Shalini Salomi Bodapati \u003cshalini.salomi.bodapati@intel.com\u003e\nChange-Id: I68b96a35f41f3bb23d7e26d87fee1dd2a5ebf6a6\n" }, { "commit": "a0232ad060c65d0ac3e022c9f721b41c02fd0b71", "tree": "7fcc4bcac3d271342262e03e69cac6dd1deb7a88", "parents": [ "6806d3c9e41e0e8b187d5e4c76d48ea6cc90b613" ], "author": { "name": "Santiago Aboy Solanes", "email": "solanes@google.com", "time": "Mon Nov 08 17:00:06 2021 +0000" }, "committer": { "name": "Santiago Aboy Solanes", "email": "solanes@google.com", "time": "Thu Nov 11 09:12:07 2021 +0000" }, "message": "Add .bss support for inlining across dexfiles within Oat\n\nWe can safely reference other dexfiles within the same oat file for cross-dex inlining.\n\nThis CL makes the OptStat#NotInlinedBss drop to less than 1% of the not-inlining cases.\n\nTest: ART tests\nChange-Id: I676d48d973abf7a6f8412cf3b7bb73afd7747f31\n" }, { "commit": "61673dc4bdee2fa4770ee72fbc347cca595dd231", "tree": "4a3a3f1b712e72bc7b9be3c84c5c79ce3588bdbf", "parents": [ "f726d20976dff58db4e0111860f6b23b77be61a7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sat Nov 06 13:58:31 2021 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 10 09:19:19 2021 +0000" }, "message": "Reland \"Change hotness counting.\"\n\nThis reverts commit ce131feb7c60ffdf28c315c5d67f9cac33a077ce.\n\nBug: 203810169\nTest: test.py\nTest: health/microbench/startup/hscapps/compile-speed-profile/open-clock\n\nReason for revert: Kept logic from before on what methods to save in the\nprofile.\n\nChange-Id: Id67cd47a9fe31b4c6b154db20f632015238016d2\n" }, { "commit": "5097f83c4719a76fdfab1044ab745273841aca45", "tree": "adca5e53b3ef1807aa7dd9c3465ac3305fedc4fd", "parents": [ "d1e6bea2c502a72dc5d0c3d16deb591ba61c9d76" ], "author": { "name": "Mythri Alle", "email": "mythria@google.com", "time": "Tue Nov 02 14:52:30 2021 +0000" }, "committer": { "name": "Mythri Alle", "email": "mythria@google.com", "time": "Tue Nov 09 14:22:50 2021 +0000" }, "message": "Revert^2 \"Add support for calling entry / exit hooks directly from JIT code\"\"\n\nThis reverts commit 72be14ed06b76cd0e83392145cec9025ff43d174.\n\nReason for revert: A reland of\ncommit 2d4feeb67912d64b9e980e6687794826a5c22f9d with a fix for no-image\ntests\n\nChange-Id: I79f719f0d4d9b903db301a1636fde5689da35a29\n" }, { "commit": "72be14ed06b76cd0e83392145cec9025ff43d174", "tree": "aa186f70ea70011d428ab91a1f4a3fb0c2e39de5", "parents": [ "667584d95926af5f0f6e392ab2dc575cc2906c51" ], "author": { "name": "Mythri Alle", "email": "mythria@google.com", "time": "Mon Nov 01 11:48:06 2021 +0000" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Mon Nov 01 14:31:37 2021 +0000" }, "message": "Revert \"Add support for calling entry / exit hooks directly from JIT code\"\n\nThis reverts commit 2d4feeb67912d64b9e980e6687794826a5c22f9d.\n\nReason for revert: This breaks no-image tests. Example failure: https://android-build.googleplex.com/builds/submitted/7871904/art-no-image/latest/view/logs/build_error.log\n\nChange-Id: I0f97c672c2d48f125931171ee1041a7c1cf20127\n" }, { "commit": "2d4feeb67912d64b9e980e6687794826a5c22f9d", "tree": "7ab2071bbf5d5907d205b8e2a092ea9869974ba1", "parents": [ "60abdd9c89525a277d75df19ff2792614651e1ff" ], "author": { "name": "Mythri Alle", "email": "mythria@google.com", "time": "Wed Oct 13 15:39:37 2021 +0000" }, "committer": { "name": "Mythri Alle", "email": "mythria@google.com", "time": "Mon Nov 01 08:31:54 2021 +0000" }, "message": "Add support for calling entry / exit hooks directly from JIT code\n\nThe idea of this CL is to avoid maintaining the instrumentation stack\nand manipulating the return addresses on the stack to call the entry /\nexit hooks. This Cl only addresses this for JITed code. In follow up\nCLs, we will extend this to others (native, nterp). Once we have\neverything in place we could remove the complexity of instrumentation\nstack.\n\nThis CL introduces new nodes (HMethodEntry / HMethodExit(Void)) that\ngenerate code to call the trace entry / exit hooks when\ninstrumentation_stubs are installed. Currently these are introduced for\nJITed code in debuggable mode. The entry / exit hooks roughly do the\nsame this as instrumentation entry / exit points.\nWe also extend the JITed frame slots by adding a ShouldDeoptimize slot.\nThis will be used to force deoptimization of frames when requested by\njvmti (for ex: structural re-definition).\n\nTest: art/testrunner.py\nChange-Id: Id4aa439731d214a8d2b820a67e75415ca1d5424e\n" }, { "commit": "ce131feb7c60ffdf28c315c5d67f9cac33a077ce", "tree": "c0d95161dc6ba2ed5cad0fc5cc4c131c3eed15d0", "parents": [ "9c924e89e5f52da9faadd1e4e60c73eaed6bc9d1" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 26 20:03:35 2021 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Oct 26 22:56:55 2021 +0000" }, "message": "Revert \"Change hotness counting.\"\n\nThis reverts commit c86869ab894c05e3181e7d15eb1893fa8a3fcd47.\n\nReason for revert: App startup performance regression.\n\nBug: 203810169\nChange-Id: Ie816969fffd7740fcdfa330aeb645399f5351865\n" }, { "commit": "c86869ab894c05e3181e7d15eb1893fa8a3fcd47", "tree": "a0a000eea11bddba4c822d4e30e06456c12ccaa4", "parents": [ "d1da3ae0aa3ddde0b129364b5e1c6073f7bdb7b7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Oct 01 12:15:08 2021 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 20 17:32:57 2021 +0000" }, "message": "Change hotness counting.\n\nDecrement instead of increment to:\n- Simplify nterp\n- Support better future optimizations on memory savings.\n\nAlso clean up Jit::AddSamples which is now only used for the C++\ninterpreter.\n\nTest: test.py\nChange-Id: I2edcee3affc3e5205e253fc7824ccdc862565931\n" }, { "commit": "4a4696ad2a1403139828e5e6703479f320f06fee", "tree": "e40c40ba014e4ba411f2345cd4b898a91717e298", "parents": [ "b0cc5923bf0b0315fa181542c765bfa8b8083cad" ], "author": { "name": "Sorin Basca", "email": "sorinbasca@google.com", "time": "Sat Oct 09 07:14:40 2021 +0000" }, "committer": { "name": "Sorin Basca", "email": "sorinbasca@google.com", "time": "Mon Oct 11 07:30:36 2021 +0000" }, "message": "Fixing Unsafe.getObjectAcquire code generation\n\nThis fixes the intrinsic code generation for\nUnsafe.getObjectAcquire which was introduced in\nIacd39ea1750c92a828d99241bff6b609acadb025\n\nTest: art/test/testrunner/testrunner.py -t 2235-JdkUnsafeTest\nBug: 188889082\nBug: 190791083\nChange-Id: I456126ce5f8355aa944b7ba6e3be9405531d3778\n" }, { "commit": "282795ca98d955697823aea6fd9c6b3f51780045", "tree": "b8d95a385c18684e8fa7481bae6088dfbe365581", "parents": [ "03ce70a18e860abe5ab2e2198cb928e43a732e9c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 24 18:16:41 2021 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Sep 28 07:59:34 2021 +0000" }, "message": "Don\u0027t do a recursive call when there are CHA guards.\n\nOtherwise we would continue execute the method with invalid inlining\noptimizations.\n\nTest: 832-cha-recursive\nBug: 19381779\nChange-Id: I57d73828d2a9c30f429cf32906f94244346c1310\n" }, { "commit": "2f01e8ed55663b73787e079955d890e1ffd2983e", "tree": "804c2a605a0aed384f10ff809d3967afaca568ec", "parents": [ "33a312992c86bdc718783496a9d42a18d7b342b0" ], "author": { "name": "Sorin Basca", "email": "sorinbasca@google.com", "time": "Fri Jun 18 06:44:07 2021 +0000" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Sep 24 10:21:16 2021 +0000" }, "message": "Copying sun.misc.Unsafe to jdk.internal.misc\n\nFrom OpenJDK 10 onward, sun.misc.Unsafe has been moved to\njdk.internal.misc. Trying to merge in changes from OpenJDK 11\ndepends on having jdk.internal.misc.Unsafe available. As\nsun.misc.Unsafe is still used throughout libcore code, it cannot\nbe changed yet, so a copy is made to jdk.internal.misc.Unsafe for\nnow.\n\nAs OpenJDK 11 is merged in and sun.misc.Unsafe references will\ndisappear, it can be deprecated and eventually made to call into\njdk.internal.misc.Unsafe.\n\nTest: m\nTest: art/test/testrunner/testrunner.py -t 004-UnsafeTest\nTest: art/test/testrunner/testrunner.py -t 2235-JdkUnsafeTest\nBug: 190470684\nChange-Id: I472a9778a1001fbd7c3ecce0dfa58ea6d632f158\n" }, { "commit": "893e2edbd5aa14ab2ccb1800ccd3154f21a2d8a7", "tree": "6093926743fa150ab6ad972e1ef106e22025f975", "parents": [ "33ed33bcc8bd08fe743508a08b0b5f7e4be0d805" ], "author": { "name": "Ulya Trafimovich", "email": "skvadrik@google.com", "time": "Thu Jun 10 16:18:12 2021 +0100" }, "committer": { "name": "Ulyana Trafimovich", "email": "skvadrik@google.com", "time": "Mon Jun 14 10:01:03 2021 +0000" }, "message": "x86: fix helper function that displaces address by a given amount.\n\nChange the function from a non-const method to a static member to avoid\ngiving an impression that it modifies the old address.\n\nAdd a gtest that constructs addresses with some initial displacement and\nchecks that they are equal to addresses constructed in the same way only\nwithout displacement, and then displaced.\n\nThe added gtest reveals errors in the displace function. Some of the\nadded test cases crashed on checks, others failed test assertions. This\nshouldn\u0027t affect real world as the use of the helper function in the\ncompiler is limited to a few working cases.\n\nChecks failed because some of the address constructors do not expect ESP\nas base register. It is possible to construct such addresses with\nanother constructor like `x86::Address(x86::ESP, 0))`, but an attempt to\ndisplace this address would try to reassemble it using a forbidden\nconstructor and hit the check.\n\nSome of the failed test assertions were for Address::Absolute, such as:\n\n Expected equality of these values:\n x86::Address::displace(x86::Address::Absolute(0), 42)\n Which is: 42(%ebp)\n x86::Address::Absolute(42)\n Which is: (%ebp)\n\nOther failed test assertions were due to the fact that one and the same\naddress can be encoded in more than one way, e.g. 32-bit displacement\nnormally requires mod 11b, but can also be achieved with mod 00b if EBP\nis used as r/m or base. Example of a test failure:\n\n Expected equality of these values:\n x86::Address::displace(x86::Address(EAX, TIMES_1, 42), -42)\n Which is: 0(%ebp,%eax,1)\n x86::Address(EAX, TIMES_1, 0)\n Which is: 0(,%eax,1)\n\nBug: 65872996\nTest: m test-art-host-gtest # changes covered by the new test\nTest: art/test.py --host -r\nChange-Id: I8dcdc968d7bd9da2c0a16ef0afeb13cf9a168359\n" }, { "commit": "c8451cb4302e028d4e106c1a2a44749d5cb9bb31", "tree": "c59b852b27ef441c5a09c4755e33628315d2e678", "parents": [ "322eced250f893a85dce0f78c74ef91eba5fdb1a" ], "author": { "name": "Ulya Trafimovich", "email": "skvadrik@google.com", "time": "Wed Jun 02 17:35:16 2021 +0100" }, "committer": { "name": "Ulya Trafimovich", "email": "skvadrik@google.com", "time": "Thu Jun 03 14:00:34 2021 +0100" }, "message": "Factor out arch-independent ScaleFactor definition.\n\nBug: 65872996\nTest: m test-art-host-gtest\nTest: art/test.py --host -r --optimizing\nChange-Id: I27763286847b45a5a3a493c3dba48418575b3eb6\n" }, { "commit": "322eced250f893a85dce0f78c74ef91eba5fdb1a", "tree": "e3a544f1012b236c6f42e90ac9de3134b452f37a", "parents": [ "6034c0755a12c4c3836d49282269a2670e6ee264" ], "author": { "name": "Ulya Trafimovich", "email": "skvadrik@google.com", "time": "Wed Jun 02 15:39:36 2021 +0100" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Jun 03 12:47:34 2021 +0000" }, "message": "x86: Deduplicate load from memory for non-reference types.\n\nDeduplicate common code in HandleFieldGet, VisitArrayGet and\nGenerateVarHandleGet. Reference types are handled individually in each\ncase because these functions have subtle differences.\n\nBug: 65872996\n\nTest: art/test.py --host -r --optimizing --32\nChange-Id: Idcbe0bdb98290732816deec0cb1c42c530d5026f\n" }, { "commit": "3a73ffb70151dbc99fa41f300a237f8c29783e0e", "tree": "8debef1b372e5bea27d5265a9019446229955e3d", "parents": [ "e585964df42e9fd2fab6f209810cb03e1b261ab1" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Jan 25 14:11:05 2021 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Jan 25 07:01:32 2021 -0800" }, "message": "Revert^4 \"Partial Load Store Elimination\"\n\nThis reverts commit 791df7a161ecfa28eb69862a4bc285282463b960.\nThis unreverts commit fc1ce4e8be0d977e3d41699f5ec746d68f63c024.\nThis unreverts commit b8686ce4c93eba7192ed7ef89e7ffd9f3aa6cd07.\n\nWe incorrectly failed to include PredicatedInstanceFieldGet in a few\nconditions, including a DCHECK. This caused tests to fail under the\nread-barrier-table-lookup configuration.\n\nReason for revert: Fixed 2 incorrect checks\n\nBug: 67037140\nTest: ./art/test/testrunner/run_build_test_target.py -j70 art-gtest-read-barrier-table-lookup\n\nChange-Id: I32b01b29fb32077fb5074e7c77a0226bd1fcaab4\n" }, { "commit": "791df7a161ecfa28eb69862a4bc285282463b960", "tree": "a43b022307ea2df82a360164bdf2627cd07eb5b1", "parents": [ "805769bb3800c1e47b0a76ce2c5ebb270a4ac69d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sat Jan 23 13:28:56 2021 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sun Jan 24 17:18:55 2021 +0000" }, "message": "Revert \"Revert^2 \"Partial Load Store Elimination\"\"\n\nThis reverts commit fc1ce4e8be0d977e3d41699f5ec746d68f63c024.\n\nBug: 67037140\n\nReason for revert: Fails read-barrier-table-lookup tests.\n\nChange-Id: I373867c728789bc14a4370b93a045481167d5f76\n" }, { "commit": "fc1ce4e8be0d977e3d41699f5ec746d68f63c024", "tree": "b656aa7c9e62aa181dfbf7fd4f2a0d32b8bf0704", "parents": [ "c6da1be58086e873c9695f8c4c1a3a8ca718696e" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Jan 22 14:05:13 2021 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Jan 22 07:15:51 2021 -0800" }, "message": "Revert^2 \"Partial Load Store Elimination\"\n\nThis reverts commit 47ac53100303e7e864b7f6d65f17b23088ccf1d6.\n\nThere was a bug in LSE where we would incorrectly record the\nshadow$_monitor_ field as not having a default initial value. This\ncaused partial LSE to be unable to compile the Object.identityHashCode\nfunction, causing crashes. This issue was fixed in a parent CL. Also\nupdated all Offsets in LSE_test to be outside of the object header\nregardless of configuration.\n\nTest: ./test.py --host\nBug: 67037140\n\nReason for revert: Fixed issue with shadow$_monitor_ field and offsets\n\nChange-Id: I4fb2afff4d410da818db38ed833927dfc0f6be33\n" }, { "commit": "47ac53100303e7e864b7f6d65f17b23088ccf1d6", "tree": "95730c6ba84ef0eebca6236428c8ba51c5eaf280", "parents": [ "51d771db35cc0647bc1fd2a32ed6348d181a7a41" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 22 08:41:08 2021 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 22 08:41:39 2021 +0000" }, "message": "Revert \"Partial Load Store Elimination\"\n\nThis reverts commit b8686ce4c93eba7192ed7ef89e7ffd9f3aa6cd07.\n\nBug: 67037140\n\nReason for revert: Fails a few tests.\n\nChange-Id: Icf0635bffbfbba93bf0a5b854a9582c418198136\n" }, { "commit": "b8686ce4c93eba7192ed7ef89e7ffd9f3aa6cd07", "tree": "1721ee940f978736a2212d693271ee698897cb0b", "parents": [ "625048049558d394d50b6e98885b8c210e481bf1" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Nov 02 08:48:33 2020 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Jan 21 17:58:10 2021 +0000" }, "message": "Partial Load Store Elimination\n\nAdd partial load-store elimination to the LSE pass. Partial LSE will\nmove object allocations which only escape along certain execution\npaths closer to the escape point and allow more values to be\neliminated. It does this by creating new predicated load and store\ninstructions that are used when an object has only escaped some of the\ntime. In cases where the object has not escaped a default value will\nbe used.\n\nTest: ./test.py --host\nTest: ./test.py --target\nBug: 67037140\n\nChange-Id: Idde67eb59ec90de79747cde17b552eec05b58497\n" }, { "commit": "d6bd107ed83502eb0bbaf66911ab036ecf74612d", "tree": "81686cdcb1e29b7fc077a9ba31db4c448ac7d007", "parents": [ "f6332e8b09599bea16666e9bbc6bdfd0c3fb2cef" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 30 18:42:01 2020 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 30 18:46:19 2020 +0000" }, "message": "Fix invokeinterface sharpened with kRuntimeCall.\n\nBug: 174260111\nBug: 173677667\n\nTest: 728-imt-conflict-zygote\nTest: atest com.android.bootimageprofile.BootImageProfileTest#testSystemServerProfile\nTest: adb install com.google.android.art.apex\nChange-Id: Ie600a0c8c8eb38d9084b796bac9184c06ea0a2f4\n" }, { "commit": "94d2c81caaa0a7155b0ec45ac2bd55247a50fb9a", "tree": "128954d088110272a2e2d8baf3f994a1a9e96183", "parents": [ "1960c42c5df95f3011c020b07328d03716532037" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 05 10:04:45 2020 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 05 10:18:14 2020 +0000" }, "message": "Fix DCHECK()s in read barrier slow paths.\n\nThis is a follow-up to\n https://android-review.googlesource.com/1478798\nto fix non-Baker read barrier configurations.\n\nTest: ART_READ_BARRIER_TYPE\u003dTABLELOOKUP m test-art-host-gtest\nBug: 170286013\nChange-Id: I6044d4fba720e0ed11b254990021f5b8236964a3\n" }, { "commit": "de91ca90389e4b41ed27b320a6c43ff56a6d75ff", "tree": "2e18ff33d30fce88d578ea68b8b1037755aececc", "parents": [ "9ca92fb4646eccff9f972f6a2a21709676b65460" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 27 13:41:40 2020 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Oct 29 15:42:52 2020 +0000" }, "message": "Refactor Integer.valueOf() intrinsic implementation.\n\nPrepare for Reference.getReferent() intrinsic implementation\nby a refactoring to separate the retrieval of an intrinsic\nmethod\u0027s declaring class to its own helper function, rather\nthan being a part of a larger one.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing --jit\nTest: aosp_blueline-userdebug boots.\nTest: run-gtests.sh\nTest: testrunner.py --target --optimizing --jit\nBug: 170286013\nChange-Id: Ib6c0e55d0c6fcc932999428f21c51afe32ab7ef2\n" }, { "commit": "eb9eb00868106af52386d7113a8aafaa6d44e8b6", "tree": "05b1243b30b31d6e2d3215ebdaa329c90d9875f8", "parents": [ "5fa36f99fdb5617d1ced977c637dcaa2762704fc" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Oct 02 13:54:19 2020 +0100" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Mon Oct 19 17:01:09 2020 +0000" }, "message": "Faster @CriticalNative for boot image.\n\nThe @CriticalNative call does not need the target method, so\nwe can avoid one instruction on x86, x86-64 and arm64. The\ncurrent approach for arm does not allow such optimization.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: aosp_blueline-userdebug boots.\nTest: run-gtests.sh\nTest: testrunner.py --target --64 --optimizing\nBug: 112189621\nChange-Id: I11b7e415be2697757cbb11c9cccf4058d1d72d7d\n" }, { "commit": "8d34a182fea1b24f7b8361b55e930cb953cf3fb2", "tree": "4f5ed9d9ac417dfd69fd18f64412b2272c448e05", "parents": [ "8ecbc4e844fc3b73e6a5c5151eda914d53297180" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Sep 16 09:46:58 2020 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 07 08:32:52 2020 +0000" }, "message": "Change interface conflict stub to take the interface method.\n\nTo avoid doing dex cache lookup, pass the interface method instead. This\ncosts a few hundred KBs on speed compiled APKs (\u003c 0.5% code size), but\nimproves performance when hitting a conflict (as seen on dogfood data).\n\nFor nterp, we currently pass the conflict method instead of the\ninterface method. We need to handle default methods before optimizing\nit.\n\nThis removes our last use of dex cache in compiled code. A follow-up CL\nwill remove the NeedsDexCacheOfDeclaringClass from HInvokeInterface.\n\nTest: test.py\n\nChange-Id: I3cdd4543ad7d904b3e81950af46a48a48af6991a\n" }, { "commit": "8f63f1084b013a129f66cf8a7ed8ab1cae9f02aa", "tree": "6e9bbf5ad71a55f701f740e2995e0b84e9b87307", "parents": [ "7aa2bfc09541ea5d2516738de84c24cd0269fed0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Sep 28 12:10:28 2020 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Sep 30 13:58:32 2020 +0000" }, "message": "Faster access to unresolved classes from compiled code.\n\nAdd two new load kinds to LoadClass, similar to kBssEntry\nbut using the access-checking entrypoint on the slow-path.\nOne is used for classes that are in the literal package and\nthe other for classes outside the literal package of the\ncompiling class. Associate new .bss entries with these load\nkinds and update them from entrypoints based on the resolved\nclass properties. If the resolved class is public, both\ntypes of entries can be updated, otherwise only the package\nlocal entry can be updated and only if the defining class\nloader of the class is the same as the caller\u0027s defining\nclass loader (which is identical for all code in an oat\nfile) because the run time access check for same package\nrequires both class loader and literal package name match.\n\nTest: Additional tests in 727-checker-unresolved-class.\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: aosp_blueline-userdebug boots.\nTest: run-gtests.sh\nTest: testrunner.py --target --optimizing\nBug: 161898207\nChange-Id: I281e06ac2825caf81c6d7ee3128af833abd39992\n" }, { "commit": "370948e9b91bd1a63d548b434a5d155b794c47c9", "tree": "9b191c71fd7e37d6c4dea54376b8402679f68e8a", "parents": [ "c616530e7290ad21693b9ec626c55811c1f2c473" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Wed Sep 23 08:07:25 2020 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 24 11:54:30 2020 +0000" }, "message": "X86: Implement VarHandle.compareAndExchange\n\nTest: ART_HEAP_POISONING\u003dtrue art/test.py --host -r -t 712-varhandle-invocation --32\nTest: ART_HEAP_POISONING\u003dfalse art/test.py --host -r -t 712-varhandle-invocation --32\nTest: ART_USE_READ_BARRIER\u003dtrue art/test.py --host -r -t 712-varhandle-invocation --32\nTest: ART_USE_READ_BARRIER\u003dfalse art/test.py --host -r -t 712-varhandle-invocation --32\nBug: 65872996\nChange-Id: I880ba6176050591e7398f95b92fc3f74f566b042\n" }, { "commit": "6d69b52f331f788cbd1f21ffd5b87cb3b39965e4", "tree": "2bca1001afb4c3fc45cbb198765c51aa65e45c42", "parents": [ "e407d97b0f25562110511d025058dcefdacca623" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Sep 23 14:47:28 2020 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Sep 23 14:47:28 2020 +0100" }, "message": "Move MethodLoadKind out of HInvokeStaticOrDirect.\n\nTo prepare for using it in HInvokeInterface. For consistency, also move\nCodePtrLocation.\n\nTest: test.py\nChange-Id: I84f973764275002e2adb71080ebc833b2bafb975\n" }, { "commit": "9dfb1a9d642b9eb627895ef3364736fc839b4fe4", "tree": "776f1e58c13f620829bde196163ebab089506d1b", "parents": [ "c372314fe478afe6aec5573c26bca378d686072d" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Tue Sep 22 13:27:18 2020 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Sep 23 08:05:32 2020 +0000" }, "message": "X86: Implement VarHandle.getAndSet, getAndSetAcquire, getAndSetRelease\n\nThis commit implements VarHandle getAndSet variants. This implied adding\nxchgb and xchgw instructions and tests for them.\n\nAlso, all the varType checks were moved to GenerateVarHandleCommonChecks.\n\nTest: ART_HEAP_POISONING\u003dtrue art/test.py --host -r -t 712-varhandle-invocation --32\nTest: ART_HEAP_POISONING\u003dfalse art/test.py --host -r -t 712-varhandle-invocation --32\nTest: ART_USE_READ_BARRIER\u003dtrue art/test.py --host -r -t 712-varhandle-invocation --32\nTest: ART_USE_READ_BARRIER\u003dfalse art/test.py --host -r -t 712-varhandle-invocation --32\nTest: m test-art-host-gtest\nBug: 65872996\n\nChange-Id: I675f47e1dbb51a5ece42f20ad8ce552cfef63ffe\n" }, { "commit": "d0f71f26f0f23d7a054ccdde552a9c6003fd33a6", "tree": "4a86250064448fc85678969ba70e9c28d487e65a", "parents": [ "fdb81da69234b482ddcf5089d6821bb60dc93868" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Thu Sep 17 09:00:15 2020 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Sep 22 11:52:55 2020 +0000" }, "message": "X86: Add the other get VarHandles (getVolatile, getAcquire, getOpaque)\n\nThis commit implements VarHandle.getVolatile, getAcquire and getOpaque\nintrinsics.\n\nTest: ART_HEAP_POISONING\u003dtrue art/test.py --host -r -t 712-varhandle-invocation --32\nTest: ART_HEAP_POISONING\u003dfalse art/test.py --host -r -t 712-varhandle-invocation --32\nTest: ART_USE_READ_BARRIER\u003dtrue art/test.py --host -r -t 712-varhandle-invocation --32\nTest: ART_USE_READ_BARRIER\u003dfalse art/test.py --host -r -t 712-varhandle-invocation --32\nBug: 65872996\nChange-Id: I38501c226c9d5af0a9e5a1230abcb3114aad4737\n" }, { "commit": "8b8d93d16ce2b409b27b4655bad0ee41987cb06d", "tree": "947ca0d8507b5860de45659c79e1cfafa7b1c305", "parents": [ "eb2c9dd52679d590428a33556f45fb49eaef5e0e" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Sep 17 14:30:01 2020 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 18 08:37:07 2020 +0000" }, "message": "Use kTieredHotnessMask in arm32 and x86 baseline backends.\n\nThose backends were missed when introducing kTieredHotnessMask.\n\nTest: test.py\n\nChange-Id: If72c00f4c1c7d668e0cd360773313c3717fd9456\n" }, { "commit": "cde98193476cc295339e54a608110f2546eb8308", "tree": "cea46ad451e18a677a2380dc4c0e7c8ad76d2b5f", "parents": [ "9dec90a069386a5e538f5cfb9ff7ef789bdbafdb" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Sun Sep 13 12:32:09 2020 +0000" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Sep 17 09:00:53 2020 +0000" }, "message": "X86: Add the other set VarHandles (setVolatile, setRelease, setOpaque)\n\nThis commit implements VarHandle.setVolatile, setRelease and setOpaque\nintrisics. This also implied refactoring HandleFieldSet to be reused in\nall set VarHandles, as the code is very similar.\n\nTest: ART_HEAP_POISONING\u003dtrue art/test.py --host --all-compiler -r --32\nTest: ART_HEAP_POISONING\u003dfalse art/test.py --host --all-compiler -r --32\nTest: ART_USE_READ_BARRIER\u003dtrue art/test.py --host --all-compiler -r --32\nTest: ART_USE_READ_BARRIER\u003dfalse art/test.py --host --all-compiler -r --32\nBug: 65872996\nChange-Id: I9a1d5fec6c5086c1e77ba65c3337da1133b3e3f1\n" }, { "commit": "52d2c0c3af51b9da9e20cfe6b10d72b816ddb7af", "tree": "73545ea3b9948923bcd94ec7cdbf9b1b17d2edfb", "parents": [ "afad9f95c5321e2d6fe312c377dc8a11336f5fa4" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Tue Sep 15 14:27:21 2020 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Sep 16 14:47:30 2020 +0000" }, "message": "X86: Add the other variants of VarHandle.compareAndSet\n\nThis commit implements VarHandle WeakCompareAndSet,\nWeakCompareAndSetPlain, WeakCompareAndSetAcquire,\nWeakCompareAndSetRelease.\n\nTest: art/test.py --host -r -t 712-varhandle-invocation --32\nChange-Id: I1e0f5e7dcb21cd0f107d5a74cb534cef992ef9fc\n" }, { "commit": "c992e42601a9dc99a8ddcaecbc7f9d5048d77a38", "tree": "56e192bc533d7d97f1954d1ee50d9333363757da", "parents": [ "ff6d5fa6add68a6ab2b699444c3539ee1950d491" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Wed Sep 16 08:12:02 2020 +0000" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Sep 16 11:11:46 2020 +0000" }, "message": "X86: Clarify HandleFieldSet constant volatile Float64 case\n\nIn case of a constant Float64 value code generation, we ensure that the\nfield is not volatile.\n\nTest: art/test.py --host --all-compiler --optimizing --32 -r\nChange-Id: I512c89dffc446fd9a44d2f409c6d2ddac756e111\n" }, { "commit": "5e13d453acc03fda08dae23e085f7161a73f7032", "tree": "713b60fff222f3ed1b4ad518ee7e95c9cba041ce", "parents": [ "8376a6543b8500c5321ddd6f6ead04e1397ceab8" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Tue Sep 08 14:35:09 2020 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 10 12:48:50 2020 +0000" }, "message": "X86: Implement VarHandle.compareAndSet() for fields.\n\nThis commit implements VarHandle compareAndExchange access mode for\nfields (both static and instance).\n\nInt64 and Float64 are not implemented because ParallelMove might fail\nwhen moving register pairs.\n\nTest: ART_HEAP_POISONING\u003dtrue art/test.py --host --32 -r -t 712-varhandle-invocations\nTest: ART_HEAP_POISONING\u003dfalse art/test.py --host --32 -r -t 712-varhandle-invocations\nBug: 65872996\nChange-Id: I92e51c348f076c23413e419948f03197c286a619\n" }, { "commit": "e6c0f2a75bd969253279580e2e4772e54787034b", "tree": "79598b2b6b457d9c718ddb18e29a0211a73ae85a", "parents": [ "63c0c2d9da31d26781f5e77aba6125f0d0988795" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Sep 07 08:30:52 2020 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Sep 08 13:51:53 2020 +0000" }, "message": "Pass a full MethodReference of the invoke in HInvoke nodes.\n\nCleanup to ensure we don\u0027t make mistakes when passing a dex method index\nto the HInvoke constructor, and we know which dex file it relates to.\n\nTest: test.py\nChange-Id: I625949add88a6b97e1dafeb7aed37961e105d6aa\n" }, { "commit": "73c3180e6355ce82fc40b3ded2c6eec0fee199f9", "tree": "b793b5066fa9f3fd3fe3297e867a11be207d2541", "parents": [ "f96ef4d4253adbc1418ed02386463e37027f998c" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Tue Sep 01 13:17:05 2020 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Sep 07 10:31:32 2020 +0000" }, "message": "X86: Implement VarHandle.set() for fields.\n\nThis commit implements VarHandle set access mode for fields (both static\nand instance). This also implied refactorizing the common parts of get\nand set access modes (e.g. coordinate type checks).\n\nNon-constant Int64 set value is not implemented.\n\nTest: ART_HEAP_POISONING\u003dtrue art/test.py --host --32 -r -t 712-varhandle-invocations\nTest: ART_HEAP_POISONING\u003dfalse art/test.py --host --32 -r -t 712-varhandle-invocations\nBug: 65872996\nChange-Id: I384c09e08f7a772fc7ec372b1ebb5f882ecc5586\n" }, { "commit": "095dc4611b8001861f8d0e621f9df704a933754a", "tree": "edbbd5b116d8caaceb4ce4605343f3e0d59c11c1", "parents": [ "ed29dcee8b5d7e62bb119d2366f3b95dd5f96163" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Aug 17 16:40:28 2020 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Aug 24 09:06:03 2020 +0000" }, "message": "Move the profiling info out of ArtMethod.\n\nInstead, keep a map in JitCodeCache.\n\nBug: 112676029\nTest: test.py\nChange-Id: I5ab769a9b7b3214af7832478d1b06c9e9adbf8b8\n" }, { "commit": "1ca6f32dd26f97e2617c45627b470d548d573975", "tree": "ebb64a8b0507ff1b245e77aa435d4ca183874baf", "parents": [ "02fd071c03c8587ff2bbea639c165b24c11d9a2f" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Wed Aug 12 08:58:07 2020 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Aug 13 09:41:48 2020 +0000" }, "message": "Revert^2 \"Revert^2 \"X86: VarHandle.Get() intrinsic for static primitive fields.\"\"\n\nThis reverts commit 0e5e1772fb6ed3dcbd1cdaf09e3c8cd2020091d5.\n\nReason for revert: Relanding the change. It was reverted because it\nseemed to be the source of a test failure, but it turned out to be a\nknown flaky test (Bug: 147572335).\n\nChange-Id: I572d2d1951b0909641a73df0b3ef2fd453f62d8b\n" }, { "commit": "0e5e1772fb6ed3dcbd1cdaf09e3c8cd2020091d5", "tree": "53daaedc7d7a07bec449723521dcfaad3782de83", "parents": [ "c3c7dff8784ab409fe0c3e47d9516ed0bc6f6cc9" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Aug 11 23:03:59 2020 +0000" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Aug 12 00:56:06 2020 +0000" }, "message": "Revert \"Revert^2 \"X86: VarHandle.Get() intrinsic for static primitive fields.\"\"\n\nThis reverts commit ed0acfdaffdbf3c76fa57b15150390dabb4c69aa.\n\nReason for revert: Failing buildbot test\n\nChange-Id: Ib43f526de492f9a09075c6c57658b7a14d05b168\n" }, { "commit": "ed0acfdaffdbf3c76fa57b15150390dabb4c69aa", "tree": "c353c4191ba3d61ad5afda3ccef173f73121a79a", "parents": [ "04bc725e1eafb567be44f64f27e441efcc0bc150" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Tue Aug 11 10:10:40 2020 +0000" }, "committer": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Tue Aug 11 12:05:00 2020 +0000" }, "message": "Revert^2 \"X86: VarHandle.Get() intrinsic for static primitive fields.\"\n\nTest: art/test.py --host --32 -r -t 712-varhandle-invocations\nTest: ART_READ_BARRIER_TYPE\u003dTABLELOOKUP ART_HEAP_POISONING\u003dtrue\nSOONG_ALLOW_MISSING_DEPENDENCIES\u003dtrue TARGET_BUILD_UNBUNDLED\u003dtrue\nart/test/testrunner/testrunner.py --interpreter --optimizing --host\n--dex2oat-jobs 4 -b --dist --verbose -t 712-varhandle-invocations\n\nBug: 65872996\n\nThis reverts commit e74df4c3f269f2f624fa6f093c48f901fe971002.\n\nReason for revert: Relanding the change. Support only Baker-style\nread barriers for VarHandleGet.\n\nChange-Id: Id02ff69350fb2a2f701a96a591b7efd52f1060e7\n" }, { "commit": "e74df4c3f269f2f624fa6f093c48f901fe971002", "tree": "31366eed138d6971725433da4349d0b048f9963e", "parents": [ "fc12a6cb415b2eacee0459fd170490a407c2548a" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Mon Aug 10 09:35:51 2020 +0000" }, "committer": { "name": "Ulyana Trafimovich", "email": "skvadrik@google.com", "time": "Mon Aug 10 12:52:14 2020 +0000" }, "message": "Revert \"X86: VarHandle.Get() intrinsic for static primitive fields.\"\n\nThis reverts commit d9af38a3b76fc54909148456da2701666bade6e0.\n\nReason for revert: Broken build git_master-art-host/art-read-barrier-table-lookup @ 6749888\n\nChange-Id: I17f5717e3a9a204568526fc72bce5d56a383e38d\n" }, { "commit": "d9af38a3b76fc54909148456da2701666bade6e0", "tree": "91e4d6c4307fd79738f7f89ab25a7d922854175d", "parents": [ "bf78fb558fc6c8a2cb71890f9981ad64d10d486e" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Wed Aug 05 13:17:54 2020 +0000" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Aug 07 14:20:51 2020 +0000" }, "message": "X86: VarHandle.Get() intrinsic for static primitive fields.\n\nTest: art/test.py --host --32 -r -t 712-varhandle-invocations\nBug: 65872996\nChange-Id: I8f1eeb1c9041ae55b3a159a6eb1e356687e574e9\n" }, { "commit": "a0130e8d2842a9a82e4fd4e811ee699272eb2e0b", "tree": "1468e015b7c4b001e40d847cf1448311706516e7", "parents": [ "75c8b635178d0c59691c2bc22f3bd1101d5516b5" ], "author": { "name": "Andra Danciu", "email": "andradanciu@google.com", "time": "Thu Jul 23 12:34:56 2020 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Jul 24 10:11:54 2020 +0000" }, "message": "Prepare compiler for adding VarHandle support.\n\nThis commit prepares the ground for adding VarHandle support\nin the compiler. The intrinsic locations builder and code\ngenerator are now triggered for HInvokePolymorphic nodes.\nVarHandle and MethodHandle intrinsics are marked as unimplemented\nrather than unreachable.\n\nSince the Varhandle intrinsics are not implemented yet, the\nfunctionality is not changed (i.e. the intrinsics are evaluated\nat runtime and not compiled). I manually tested that the intrinsic\nVisit* methods are triggered for the VarHandle methods.\n\nBug: b/65872996\nTest: art/test.py --host -r -t 713-varhandle-invokers\nTest: art/test.py --host --all-compiler -r\n\nChange-Id: I3333728c5f16d8dc4f92ceae2738ed59b3e31e6a\n" }, { "commit": "4ef451aecd4008ccee2ebf6136a70ed6a5c14717", "tree": "d9f2155873bf0c040aad3cc372c3b3509fe4b583", "parents": [ "659e8d6f76d5ce487c46710291e143bc897b5071" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 23 09:54:27 2020 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 23 11:58:05 2020 +0000" }, "message": "Rename kDummy32BitOffset to kPlaceholder32BitOffset.\n\nAnd drop an obsolete reference to Quick compiler.\n\nTest: m test-art-host-gtest\nBug: 161336379\nChange-Id: If08cc0ca83cfeb1c6923c41f9d61766f878a295a\n" }, { "commit": "dec7817522eeaf8f88dcae9ce065969aeebda3b3", "tree": "a15fd16ccb4a1929ec60584ead8f095b565c9e3e", "parents": [ "ea4d7d2d52dd9795cf39eccd46cb07551c62392f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 19 15:31:23 2020 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 22 08:05:28 2020 +0000" }, "message": "Optimizing: Introduce {Increase,Decrease}Frame().\n\nAnd use it to clean up code generators.\n\nAlso fix CFI in MaybeIncrementHotness() for arm/arm64/x86.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: testrunner.py --host --debuggable --ndebuggable \\\n --optimizing --jit --jit-on-first-use -t 178\nTest: aosp_cf_x86_phone-userdebug boots.\nTest: aosp_cf_x86_phone-userdebug/jitzygote boots.\nTest: # On blueline:\n testrunner.py --target --debuggable --ndebuggable \\\n --optimizing --jit --jit-on-first-use -t 178\nBug: 112189621\nChange-Id: I524e6c3054ffe1b05e2860fd7988cd9995df2963\n" }, { "commit": "86c8752f64629325026945cd4eabd1dcea224acb", "tree": "9dc2be978f9e784a3ce16fa29d46941a94ac1c94", "parents": [ "f97a859e85f703644d897f0e3e1bc54315557aaa" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon May 11 16:55:55 2020 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 19 08:26:46 2020 +0000" }, "message": "Direct calls to @CriticalNative methods.\n\nEmit direct calls from compiled managed code to the native\ncode registered with the method, avoiding the JNI stub.\n\nGolem results:\nart-opt-cc x86 x86-64 arm arm64\nNativeDowncallStaticCritical +12.5% +62.5% +75.9% +41.7%\nNativeDowncallStaticCritical6 +55.6% +87.5% +72.1% +35.3%\nart-opt x86 x86-64 arm arm64\nNativeDowncallStaticCritical +28.6% +85.6% +76.4% +38.4%\nNativeDowncallStaticCritical6 +44.6% +44.6% +74.6% +32.2%\n\nTest: Covered by 178-app-image-native-method.\nTest: m test-art-host-gtest\nTest: testrunner.py --host --debuggable --ndebuggable \\\n --optimizing --jit --jit-on-first-use\nTest: run-gtests.sh\nTest: testrunner.py --target --optimizing\nTest: testrunner.py --target --debuggable --ndebuggable \\\n --optimizing --jit --jit-on-first-use -t 178\nTest: aosp_cf_x86_phone-userdebug boots.\nTest: aosp_cf_x86_phone-userdebug/jitzygote boots.\nBug: 112189621\nChange-Id: I8b37da51e8fe0b7bc513bb81b127fe0416068866\n" }, { "commit": "695348f4b0541f4373b46eac5830cdd87f71c076", "tree": "f2f6019f0c394f99aaaf9f2f7deec16bf6116b0f", "parents": [ "1f5300a211202442a07607830c6550773ca50b50" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 19 14:42:02 2020 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 20 08:41:09 2020 +0000" }, "message": "Add compiler type to CompilerOptions.\n\nLet CompilerOptions hold the information whether it is AOT\nor JIT compilation, or Zygote JIT for shared code.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing --jit\nTest: aosp_taimen-userdebug boots.\nChange-Id: Id9200572406f8e43d99b8b61ef0e3edf43b52fff\n" }, { "commit": "796aa2cfcde9c88fa0a3176899e25bab3468ebd2", "tree": "9f4be44ef08e5abbdcb16ed6e8d15459bb743222", "parents": [ "57cacb720e6f995aa1a42df6e2e6470a9ec57261" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 17 10:20:05 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 17 11:31:30 2019 +0000" }, "message": "[baseline] Check that the profiling info is not null.\n\nZygote cannot allocate profiling infos.\n\nTest: 689-zygote-jit-deopt\nChange-Id: I85e8b7f16b81ba4de435a5417dbb2588c34414b0\n" }, { "commit": "57cacb720e6f995aa1a42df6e2e6470a9ec57261", "tree": "bb73a113c94bc397cd7c99a4c64e033bf29b9803", "parents": [ "013d1ee96b928f3bda9031e94d4a69f827133ce6" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sun Dec 08 22:07:08 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 17 09:48:00 2019 +0000" }, "message": "Refactor OSR related code to prepare for \"true\" OSR.\n\n- Make the compiler restore all callee-save registers.\n- Make the compiler return any value in a core register: this simplifies\n the current stub, and will also avoid having to look at the return\n type (and reading the shorty) when returning to an nterp frame.\n- Add OsrData and offsets of its members to be used by nterp.\n\nTest: test.py\nBug: 27094810\nChange-Id: Ifa4f4877ab8b1f0c6a96feccea30c909942eb2fa\n" }, { "commit": "9b5271e53a76cbe3d269d1b70da7f13b9d944db1", "tree": "ff89e3a40d274e812f5727d7ff7930d19d447d35", "parents": [ "a00b54b74bee06c006b8bebfbef85e2801de293c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 04 14:39:46 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 04 14:39:46 2019 +0000" }, "message": "Get the baseline information from the graph.\n\nBaseline could be set by the compiler options or the JIT.\n\nTest: test.py\nBug: 119800099\nChange-Id: I702bd7642dfd3353c9ad99cb6ac425c090e16101\n" }, { "commit": "a59af8aeaad8fe7d68d8f8de63eab9cf85b6ab31", "tree": "83195c74b135731cc4555254763a8f449691c1b0", "parents": [ "5c8cc64b5f1580faf510f27527e7e22987174963" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 27 17:42:32 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 03 14:32:09 2019 +0000" }, "message": "JIT baseline: trigger optimized compilation on hotness threshold.\n\n- Add a new hotness count in the ProfilingInfo to not conflict with\ninterpreter hotness which may use it for OSR.\n- Add a baseline flag in the OatQuickMethodHeader to identify baseline\ncompiled methods.\n- Add a -Xusetieredjit flag to experiment and test.\n\nBug: 119800099\nTest: test.py with Xusetieredjit to true\n\nChange-Id: I8512853f869f1312e3edc60bf64413dee9143c52\n" }, { "commit": "17a39babb7f42cbe108d6fab2760cbdc68b821a2", "tree": "5a01000d3ffe684343bae6e69b31a74c9cb6b151", "parents": [ "49af4cae5674c68d492659ab4b4015c3a6562970" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 27 20:57:48 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 27 20:58:28 2019 +0000" }, "message": "JIT baseline: Don\u0027t update the inline cache for an intrinsic.\n\nWe already know its target.\n\nTest: 597-deopt-busy-loop with heap poisoning\nBug: 119800099\nChange-Id: Ic0bd780b48e6ee31e2007c04528476fcb7fcc5bf\n" }, { "commit": "e2a3aa988630b3c2952ac44943f03dde60454195", "tree": "acee7012af6e2b161c91e6cd8b7b4d12eb5aa927", "parents": [ "a2c4d61e482a15974e3e220bcd62a64043ee536f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 25 17:52:58 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 27 14:04:43 2019 +0000" }, "message": "Baseline JIT: update inline caches in compiled code.\n\nIn trying to remove profiling from interpreter, to speed up\ninterpreter performance.\n\nBug: 119800099\nTest: test.py --baseline\nChange-Id: Ica1fa41a889b31262d9f5691b30a31fbcec01b34\n" }, { "commit": "0a51605ddd81635135463dab08b6f7c21b58ffb0", "tree": "820f338333010f4d6e4b543ffea47e9ff7d2dd3f", "parents": [ "e2727154f25e0db9a5bb92af494d8e47b181dfcf" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 14 13:00:44 2019 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 14 13:00:44 2019 +0000" }, "message": "Revert \"Make compiler/optimizing/ symbols hidden.\"\n\nThis reverts commit e2727154f25e0db9a5bb92af494d8e47b181dfcf.\n\nReason for revert: Breaks ASAN tests (ODR violation).\nBug: 142365358\n\nChange-Id: I38103d74a1297256c81d90872b6902ff1e9ef7a4\n" }, { "commit": "e2727154f25e0db9a5bb92af494d8e47b181dfcf", "tree": "0ddc6eec3ad9508d7eb1f2b941786dd0ed7d2cd8", "parents": [ "c78860b91ae07eed92f86876e7a03132adea6fcd" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Oct 10 10:46:42 2019 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 14 08:22:00 2019 +0000" }, "message": "Make compiler/optimizing/ symbols hidden.\n\nMake symbols in compiler/optimizing hidden by a namespace\nattribute. The unit intrinsic_objects.{h,cc} is excluded as\nit is needed by dex2oat.\n\nAs the symbols are no longer exported, gtests are now linked\nwith the static version of the libartd-compiler library.\n\nlibart-compiler.so size:\n - before:\n arm: 2396152\n arm64: 3345280\n - after:\n arm: 2016176 (-371KiB, -15.9%)\n arm64: 2874480 (-460KiB, -14.1%)\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing --jit\nBug: 142365358\nChange-Id: I1fb04a33351f53f00b389a1642e81a68e40912a8\n" }, { "commit": "7f8678ec4d2abec1f540fb441be60604bec86b6e", "tree": "e36b4d32dfc47fcebadf0ee5c7e4d1e3d51412a6", "parents": [ "84e5bb990d48263849bab132d80d753495bc7204" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 30 16:22:28 2019 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 06 18:40:59 2019 +0000" }, "message": "Revert^2 \"Prevent overflow for AOT hotness counters\"\n\nFixed bug where sbc usage was incorrect. sbc does -1 + carry.\n\nTest: test/run-test --always-clean --runtime-option -Xcheck:jni --64 674-hotness-compiled\nTest: test/run-test --always-clean --runtime-option -Xcheck:jni 674-hotness-compiled\nBug: 139883463\n\nThis reverts commit 7ab07777b08db86dda2891f3e7ae15df8f25a599.\n\nChange-Id: I6f8ac0320592a94314386b04cdb0c7e0e6da6994\n" }, { "commit": "7ab07777b08db86dda2891f3e7ae15df8f25a599", "tree": "1b0b2fa585e49e4a7913c09d67794763197c6490", "parents": [ "154445799432cb53d23cd011485132be07c39b5a" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Aug 30 08:26:59 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Aug 30 08:57:20 2019 +0000" }, "message": "Revert \"Prevent overflow for AOT hotness counters\"\n\nThis reverts commit 79e6eb8b79be6249358b7801bc511290dacf10d0.\n\nBug: 139883463\n\nReason for revert: 674-hotness-compiled fails on target.\n\nChange-Id: I02fce74d70a4ae69dd5b4ae3924aa11728d9e16f\n" }, { "commit": "79e6eb8b79be6249358b7801bc511290dacf10d0", "tree": "1a04d214dd6223423abd442d8d9b0b61a3db2336", "parents": [ "bae88c0759d48acf29b58d960ad2665e3462dfda" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Aug 26 12:33:46 2019 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Aug 29 16:05:52 2019 +0000" }, "message": "Prevent overflow for AOT hotness counters\n\nPrevious, the addition did not have a check for overflow and might wrap\naround since the counter is only 16 bits.\n\nModified the test to exercise this.\n\nThe slowdown from fixing the overflow is 2% average on golem arm32/64.\nOverall this brings the slowdown from the counter to ~15% from ~13%.\n\nThe benchmarks that regress the most are loopy ones that I would\nconsider non-representative. Code size increases by 0.6%.\n\nBug: 139883463\nTest: test/run-test --host --64 --prebuild 674-hotness-compiled\nTest: test/run-test --host --prebuild 674-hotness-compiled\nTest: test/run-test --64 --prebuild 674-hotness-compiled\nTest: test/run-test ---prebuild 674-hotness-compiled\n\nChange-Id: Icf0ab2aedbc40ab10c9d952ce0f9c7b5e5feaf15\n" }, { "commit": "6a0b657a1875b4fbb020b806169e2f73fcb2578b", "tree": "955bb0e3413e18f2b13b7fee7fa3e6e48a214597", "parents": [ "61f071630083775fe64d177455a056daa7071eca" ], "author": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Fri Jul 26 20:38:37 2019 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Aug 02 13:31:43 2019 +0000" }, "message": "ART: ARM64: Optimize frame size for SIMD graphs.\n\nFor SIMD graphs allocate 64 bit instead of 128 bit on stack for\neach FP register to be preserved by the callee in the frame entry\nas ABI suggests (currently 64-bit registers are preserved but\nmore space on stack is allocated).\n\nNote: slow paths still require spilling full 128-bit Q-Registers\nfor SIMD graphs due to register allocator restrictions.\n\nTest: test-art-target.\nChange-Id: Ie0b12e4b769158445f3d0f4562c70d4fb0ea7744\n" }, { "commit": "44ca0754b3c6f11303bac876a9175bbfa95609ef", "tree": "6f669466e6b0e750a075108cc6f8fe0a67b66658", "parents": [ "f3677471a58c2738a3d9dd05f07f01c18a5e61be" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jul 29 10:18:25 2019 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jul 30 14:49:54 2019 +0000" }, "message": "Compiler changes for boot image extension.\n\nTest: m test-art-host-gtest\nTest: testrunnner.py --host --optimizing\nTest: aosp_taimen-userdebug boots.\nTest: run-gtest.sh\nTest: testrunner.py --target --optimizing\nChange-Id: I8e999c96ec908f26d8c529edc9d2a3be49a9379a\n" }, { "commit": "bf12191214c0d6215a98dfe846a51230d995dad9", "tree": "085c1059014ce66fdb7eceb6e8502d906f0eacbd", "parents": [ "78342419743cb6d0f17dc2d4c0cd99d18d9c83d6" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jun 04 13:49:05 2019 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jul 26 11:10:26 2019 +0000" }, "message": "Implement ClassStatus::kVisiblyInitialized.\n\nPreviously all class initialization checks involved a memory\nbarrier to ensure appropriate memory visibility. We change\nthat by introducing the kVisiblyInitialized status which can\nbe checked without a memory barrier. Before we mark a class\nas visibly initialized, we run a checkpoint on all threads\nto ensure memory visibility. This is done in batches for up\nto 32 classes to reduce the overhead.\n\nAvoiding memory barriers in the compiled code reduces code\nsize and improves performance. This is also the first step\ntoward fixing a long-standing synchronization bug 18161648.\n\nPrebuilt sizes for aosp_taimen-userdebug:\n - before:\n arm/boot*.oat: 19150696\n arm64/boot*.oat: 22574336\n oat/arm64/services.odex: 21929800\n - after:\n arm/boot*.oat: 19134508 (-16KiB)\n arm64/boot*.oat: 22553664 (-20KiB)\n oat/arm64/services.odex: 21888760 (-40KiB)\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: aosp_taimen-userdebug boots\nTest: run-gtests.sh -j4\nTest: testrunner.py --target --optimizing\nTest: Manually diff `m dump-oat-boot` output from before\n with output after this CL without codegen changes,\n with `sed` replacements for class status. Check that\n only checksums and the oatdump runtime values of\n DexCache.dexFile differ.\nBug: 18161648\nBug: 36692143\nChange-Id: Ida10439d347e680a0abf4674546923374ffaa957\n" }, { "commit": "b45a435e25d69592f27084a615b351760f040875", "tree": "6920830856dbe3cceb648b365b0d057ea3ea93ec", "parents": [ "5132e0d1433b0962f9609dbbd68427e1fb5a4b9d" ], "author": { "name": "Shalini Salomi Bodapati", "email": "shalini.salomi.bodapati@intel.com", "time": "Wed Jul 10 16:09:41 2019 +0530" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jul 17 11:12:22 2019 +0000" }, "message": "Add AVX support for packed mul/div instructions.\n\nThis is a follow up for the below patch:\nhttps://android-review.googlesource.com/c/platform/build/+/830841\n\nTest: ./test.py --host --64, test-art-host-gtest\nChange-Id: Id2aa473035556ee230e66addeb69707df8530e75\nSigned-off-by: Shalini Salomi Bodapati \u003cshalini.salomi.bodapati@intel.com\u003e\n" }, { "commit": "2d06e029b1c84916154b5960d2acd1c84706dc04", "tree": "31dca979adebd2ed3a058b23a12a3c91ce2874d1", "parents": [ "7cde45800e21c270945b43a8989334ffc7422c32" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jul 08 15:45:19 2019 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jul 16 08:22:46 2019 +0000" }, "message": "Clean up linker patches in codegens.\n\nIn preparation for introducing boot image extension, make\nsure that we can use both kBootImageLinkTimePcRelative and\nkBootImageRelRo load kinds at the same time.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: aosp_taimen-userdebug boots\nTest: run-gtests.sh\nTest: testrunner.py --target --optimizing\nChange-Id: I340f2d7d19e1c20699b37b0304d2e487d497da98\n" }, { "commit": "d5fd5c3bbb44880e440c6920ce5ed56b5383c788", "tree": "2cf8a6354e5509c5a5e0bc2937c24fe7e03a12b1", "parents": [ "1a6f9fcce199f437a5945dfe0163188b923adb28" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jul 02 14:46:32 2019 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 04 08:17:30 2019 +0000" }, "message": "Make .bss stores atomic release operations.\n\nAnd rely on architecture-dependent behavior for the .bss\nentry loads.\n\nThis fixes theoretical races when one thread updates the\n.bss entry and another uses it immediately thereafter;\npreviously we did not ensure correct memory visibility.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: aosp_taimen-userdebug boots.\nTest: run-gtests.sh\nTest: testrunner.py --target --optimizing\nChange-Id: Ie7b7969eb355025b9c9205f8c936e702861943f4\n" }, { "commit": "8fa839cfe6f72adabdf79f938c57300e589e0803", "tree": "b940832441ef1b0c724c1383228225845e5d72d1", "parents": [ "3cddf4538a8df429a6084f7176c07d55e1e0ec67" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 16 12:50:47 2019 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 23 11:47:28 2019 +0000" }, "message": "Revert^2 \"Improve ArraySet codegen.\"\n\nThis reverts commit 0dda8c84938d6bb4ce5a1707e5e109ea187fc33d.\n\nThe original change had two issues that have been fixed.\nFirst, for heap poisoning, the null branch skipped over the\nreference poisoning instructions which copy and poison the\nvalue, thus writing whatever was left in the register.\nSecond, the change erroneously assumed that the slow path\nperformed only the assignability check and bound the slow\npath exit label before the actual array store, unnecessarily\nre-doing the store.\n\nChange-Id: I9f380efa12aa807b4f566a932dbc9dae824fb25a\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: aosp_taimen-userdebug boots.\nTest: testrunner.py --target --optimizing\nTest: Repeat the above with\n ART_USE_READ_BARRIER\u003dfalse ART_HEAP_POISONING\u003dtrue\nBug: 32489401\n" }, { "commit": "552a13415573da19eafa46e1ac00fb0eb68f2b23", "tree": "8cae5f3602d8f8e65cd3cbc349af17d785128605", "parents": [ "0dda8c84938d6bb4ce5a1707e5e109ea187fc33d" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 31 10:56:47 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 16 14:44:09 2019 +0000" }, "message": "ART: Optimize StringBuilder append pattern.\n\nRecognize appending with StringBuilder and replace the\nentire expression with a runtime call that perfoms the\nappend in a more efficient manner.\n\nFor now, require the entire pattern to be in a single block\nand be very strict about the StringBuilder environment uses.\nAlso, do not accept StringBuilder/char[]/Object/float/double\narguments as they throw non-OOME exceptions and/or require a\ncall from the entrypoint back to a helper function in Java;\nthese shall be implemented later.\n\nBoot image size for aosp_taimen-userdebug:\n - before:\n arm/boot*.oat: 19653872\n arm64/boot*.oat: 23292784\n oat/arm64/services.odex: 22408664\n - after:\n arm/boot*.oat: 19432184 (-216KiB)\n arm64/boot*.oat: 22992488 (-293KiB)\n oat/arm64/services.odex: 22376776 (-31KiB)\nNote that const-string in compiled boot image methods cannot\nthrow, but for apps it can and therefore its environment can\nprevent the optimization for apps. We could implement either\na simple carve-out for const-string or generic environment\npruning to allow this pattern to be applied more often.\n\nResults for the new StringBuilderAppendBenchmark on taimen:\n timeAppendLongStrings: ~700ns -\u003e ~200ns\n timeAppendStringAndInt: ~220ns -\u003e ~140ns\n timeAppendStrings: ~200ns -\u003e 130ns\n\nBug: 19575890\nTest: 697-checker-string-append\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: aosp_taimen-userdebug boots.\nTest: run-gtests.sh\nTest: testrunner.py --target --optimizing\nTest: vogar --benchmark art/benchmark/stringbuilder-append/src/StringBuilderAppendBenchmark.java\nChange-Id: I51789bf299f5219f68ada4c077b6a1d3fe083964\n" }, { "commit": "0dda8c84938d6bb4ce5a1707e5e109ea187fc33d", "tree": "3d0b4f35ef4d00aa18eba0e417655d43bc44bf5a", "parents": [ "0ece86491008837a9814f7a2e0d7961c74ef4195" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 16 12:47:40 2019 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 16 12:47:40 2019 +0000" }, "message": "Revert \"Improve ArraySet codegen.\"\n\nThis reverts commit 0ece86491008837a9814f7a2e0d7961c74ef4195.\n\nReason for revert: Breaks heap poisoning tests.\n\nBug: 32489401\nChange-Id: Ied4150829eea848d0f967866d87c6aa7dafd39a1\n" }, { "commit": "0ece86491008837a9814f7a2e0d7961c74ef4195", "tree": "590aa02d76255d9e9c4c1329b4a7278c4c8ff018", "parents": [ "3a8ab36cdfc9535bf79057cb9efe787ec8a491d2" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon May 13 11:49:17 2019 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 16 08:23:57 2019 +0000" }, "message": "Improve ArraySet codegen.\n\nSimplify the reference case to emit fewer instructions and\ntake at most one branch in the main path.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: aosp_taimen-userdebug boots.\nTest: testrunner.py --target --optimizing\nBug: 32489401\nChange-Id: I9d76b7795ec01e6245ed3184cd8d384389e5070d\n" }, { "commit": "20d1c942c0e841920eac92f68c6d3e7f2a2135b4", "tree": "931aed7f4639a4b0a6ab8e3cd7765295a1883be3", "parents": [ "d32f8aadd37aab5b89ffccc86f7d8d07447a213a" ], "author": { "name": "jaishank", "email": "jaishankar.rajendran@intel.com", "time": "Fri Mar 08 15:08:17 2019 +0530" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Apr 25 09:05:09 2019 +0000" }, "message": "Patch supports Intel(R) AVX/AVX2 MOV Instruction\n\nThis patch enhances the existing ART-Compiler\nto generate Intel(R) AVX/AVX2 MOV Instructions for\ndoing SIMD Operations on Intel(R) Architecture CPUs.\nIt also provides the framework for AVX/AVX2 Instruction\nencoding and dissassembly\n\nBUG: 127881558\nTest: run-test gtest\nChange-Id: I9386aecc134941a2d907f9ec6b2d5522ec5ff8b5\n" }, { "commit": "403aafa9f286e13ee2a64748514d33af39b55ab0", "tree": "fe9e5a5eed0081c2ac03dec0d616ed32d162b99c", "parents": [ "8809c9cd8d7f477e0a74f68537c864e3b7a35db3" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Mar 06 18:04:14 2019 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 12 14:11:25 2019 +0000" }, "message": "Fix non-deterministic compilation for const-string...\n\n... in inlined methods that are not in the boot profile.\nIf such string is not in the boot image for other reasons,\ndo not resolve the string and use the kBssEntry load kind.\n\nBoot image sizes for aosp_taimen-userdebug:\n - before:\n arm/boot*.art: 12349440\n arm/boot*.oat: 19862024\n arm64/boot*.art: 16609280\n arm64/boot*.oat: 23568592\n - after:\n arm/boot*.art: 12324864 (-24KiB)\n arm/boot*.oat: 19936612 (+73KiB)\n arm64/boot*.art: 16580608 (-28KiB)\n arm64/boot*.oat: 23642120 (+72KiB)\n\nTest: aosp_taimen-userdebug boots.\nTest: m test-art-host-gtest\nBug: 26687569\nChange-Id: I3e0b72cd5e8c67904517856208f25a6c379ab601\n" }, { "commit": "3db70689e3e1c92344d436a8ea4265046bdef449", "tree": "3db08743e968062ed5bdc143233cdb3c4564696b", "parents": [ "1650dafad62578a1766bd617d78458a4cf1e2a9a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Dec 26 15:12:03 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 27 12:56:39 2018 -0800" }, "message": "ART: Refactor for bugprone-argument-comment\n\nHandles compiler.\n\nBug: 116054210\nTest: WITH_TIDY\u003d1 mmma art\nChange-Id: I5cdfe73c31ac39144838a2736146b71de037425e\n" }, { "commit": "c1896c9a0e15df3a1b9a3a19bcd2a933b654fe06", "tree": "b68a5f5163f8da0da87d671a225addaa2a13095f", "parents": [ "f2970cd870948a6ee1c8ecd30c9c3147d05aa0be" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Nov 29 11:33:18 2018 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Nov 29 11:33:18 2018 -0800" }, "message": "C++17 compatibility: make WITH_TIDY\u003d1 happy again.\n\nBug: http://b/111067277\nTest: builds\nChange-Id: I8b69ea3815e14bb6eb27f40c0dd01a85b340a355\n" }, { "commit": "e5ed5bafc97e65e8d2b60ddaca48754a50ca5692", "tree": "89d2da51e5e95eefbf7159e40f9fbbd6fa233b97", "parents": [ "d1f1ccbd8c29962727ee859f71f11c8cfabcfaa1", "a66784b09f10d847b49bc878f10c45690e212f0b" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Nov 12 18:17:23 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Nov 12 18:17:23 2018 +0000" }, "message": "Merge \"Optimize mod power 2 for x86\"" }, { "commit": "a66784b09f10d847b49bc878f10c45690e212f0b", "tree": "a18ee4ec39a44256223b9f756159bfd535ddfde3", "parents": [ "fe59955fc41e277bf1c60378202ba785abb1e4a8" ], "author": { "name": "Shalini Salomi Bodapati", "email": "shalini.salomi.bodapati@intel.com", "time": "Tue Nov 06 13:05:44 2018 +0530" }, "committer": { "name": "Shalini Salomi Bodapati", "email": "shalini.salomi.bodapati@intel.com", "time": "Mon Nov 12 21:09:32 2018 +0530" }, "message": "Optimize mod power 2 for x86\n\nTest: 411-checker-hdiv-hrem-pow2, test.py --host\n\nChange-Id: I9334a3eb2cb50df439b56c0161379fef46e58603\nSigned-off-by: Shalini Salomi Bodapati \u003cshalini.salomi.bodapati@intel.com\u003e\n" }, { "commit": "dd121f637509a50d6744a0ad596f5dc627e0c60a", "tree": "1e20c636cd40a5a87cae1cd0d0b55940c8111a20", "parents": [ "fe59955fc41e277bf1c60378202ba785abb1e4a8" ], "author": { "name": "Shalini Salomi Bodapati", "email": "shalini.salomi.bodapati@intel.com", "time": "Fri Oct 26 15:03:53 2018 +0530" }, "committer": { "name": "Shalini Salomi Bodapati", "email": "shalini.salomi.bodapati@intel.com", "time": "Thu Nov 08 14:50:29 2018 +0530" }, "message": "Emit bit manipulation instructions for x86 and x86_64\n\nThis patch performs instruction simplification to\ngenerate instructions andn, blsmsk and blsr on\ncpus that have avx2.\n\nTest: test.py --host --64, test-art-host-gtest\nChange-Id: Ie41a1b99ac2980f1e9f6a831a7d639bc3e248f0f\nSigned-off-by: Shalini Salomi Bodapati \u003cshalini.salomi.bodapati@intel.com\u003e\n" }, { "commit": "b546163926889130354ccdbcccb80c0331c13f3c", "tree": "b4a3fb30e11e2abc671fb0b4b8098acd8fc49ce2", "parents": [ "8db807252e1d4d0bab7785be231e20a1e5fd8e74" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 15 14:24:21 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 16 10:47:24 2018 +0100" }, "message": "Fix HNewArray with unresolved primitive array type.\n\nAnd enable test 920-objects that was crashing because\nof this bug.\n\nTest: testrunner.py --host --jit-on-first-use -t 920\nTest: testrunner.py --host --optimizing\nTest: m test-art-host-gtest\nBug: 117638896\nChange-Id: I47dc893b121c82de537b3147c86d37a6eecf2d62\n" }, { "commit": "a2da9b99fa1ea3d25d52da71308a623b2aae216c", "tree": "5533be23eee7c24b68b1b72272cbae3f35a708cb", "parents": [ "dc3b4670b170b39a8bd6498d4de69c1513af1db2" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Oct 10 14:21:55 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Oct 10 15:25:26 2018 +0100" }, "message": "ART: Completely remove the --compile-pic option.\n\nAnd the PIC-related fields from image header.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nTest: testrunner.py --target --optimizing\nBug: 77856493\nChange-Id: I3787369378f12d8cd9003bebeae62830a67def33\n" }, { "commit": "c55bb390d88b8eb62a50932f7f9b47c2a3733f16", "tree": "ec2e74f3f8404dffcb9bfde4e7ffa3416a35543d", "parents": [ "89468aeaa6a67ae4025b30d427391186306d6522" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Sep 21 00:02:02 2018 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 20 18:37:12 2018 -0700" }, "message": "Revert^2 \"ART: Refactor typedef to using\"\n\nThis reverts commit ee07743e03042c2ca36e0c9513847a9e7d2509f1.\n\nReason for revert: fixed attributes.\n\nBug: 32619234\nTest: m test-art-host\nTest: m test-art-target-gtest-unstarted_runtime_test\nChange-Id: I6f0a775adfdf6ecd132b470f7c5446e949872e20\n" }, { "commit": "d5245a743df43830baf9fbc2233f6b5cebad8f32", "tree": "40a6269dfa24f31fe20caf5ae4df02e3812ae213", "parents": [ "98399b8667f1fd210c66b340a6b5de7d3693ad1c", "ee07743e03042c2ca36e0c9513847a9e7d2509f1" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Sep 20 20:05:41 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Sep 20 20:05:41 2018 +0000" }, "message": "Merge \"Revert \"ART: Refactor typedef to using\"\"" }, { "commit": "ee07743e03042c2ca36e0c9513847a9e7d2509f1", "tree": "66867d7443a3798f6d882ef2715ddeb7baf3e771", "parents": [ "9a20ff06f7ccee08a742c315ec6d351ab56ba1cd" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 20 15:05:17 2018 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 20 15:05:17 2018 +0000" }, "message": "Revert \"ART: Refactor typedef to using\"\n\nThis reverts commit 9a20ff06f7ccee08a742c315ec6d351ab56ba1cd.\n\nReason for revert: Attributes on the wrong side.\n\nBug: 32619234\nChange-Id: I8fd2abef619b22c02ccfbf5ae629339f1a60918b\n" }, { "commit": "c1ecc9b64a3f03e5a721081215c812c66183d2b8", "tree": "7af3fe28640d9f5094967806ff2e4dda1739f42c", "parents": [ "b32535defe6fcec752f4e78f8cfa2746037a1f70", "bdb2ecc8cfd0d6fc2f3f4fa4c65cca84b358cd61" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Sep 20 07:07:12 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Sep 20 07:07:12 2018 +0000" }, "message": "Merge \"Remove sharpening as an optimization pass.\"" }, { "commit": "9a20ff06f7ccee08a742c315ec6d351ab56ba1cd", "tree": "7e55557e118eb5fffe03ad3456e00d8469a36232", "parents": [ "70dfb6983492aff1c52cc1b52044258aef51f47f" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Sep 19 10:58:18 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Sep 19 21:56:32 2018 -0700" }, "message": "ART: Refactor typedef to using\n\nAdd clang-tidy\u0027s modernize-use-using.\n\nBug: 32619234\nTest: WITH_TIDY\u003d1 mmma art\nChange-Id: If50d37b5152af4270784e3cde7951292a9e19033\n" }, { "commit": "bdb2ecc8cfd0d6fc2f3f4fa4c65cca84b358cd61", "tree": "a7660c98c22d28bf508fe208845957418e0dee40", "parents": [ "4bd4d2c199c9e0e522526c40303652e29bc7c631" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Sep 18 14:33:55 2018 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Sep 19 14:46:04 2018 +0100" }, "message": "Remove sharpening as an optimization pass.\n\nMake the last sharpening helper (methods) like the other\nhelpers: being invoked by the instruction builder.\n\nTest: test.py\nChange-Id: Ic80a454f9b59b0b4ef7825590b24402500ba851c\n" }, { "commit": "bbc6e7edb5fca4a61ac53dd9bce79cb4f0bb3403", "tree": "0fbce767bc383358cf4cd65aafc74140e1850982", "parents": [ "19379b58bd433da91230e4fe6cd96e7416d16adc" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Aug 24 16:58:47 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 28 11:06:07 2018 +0100" }, "message": "Use \u0027final\u0027 and \u0027override\u0027 specifiers directly in ART.\n\nRemove all uses of macros \u0027FINAL\u0027 and \u0027OVERRIDE\u0027 and replace them with\n\u0027final\u0027 and \u0027override\u0027 specifiers. Remove all definitions of these\nmacros as well, which were located in these files:\n- libartbase/base/macros.h\n- test/913-heaps/heaps.cc\n- test/ti-agent/ti_macros.h\n\nART is now using C++14; the \u0027final\u0027 and \u0027override\u0027 specifiers have\nbeen introduced in C++11.\n\nTest: mmma art\nChange-Id: I256c7758155a71a2940ef2574925a44076feeebf\n" }, { "commit": "14e5a29a8c5dcd971376a4a04b3c3b05100b3f86", "tree": "81c607cde36b6481ed2cd2d8b41293f62a5521f8", "parents": [ "e0943873483cb2169e5360e1f746931a3371aa24" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Jun 28 12:00:56 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 14 18:01:43 2018 +0100" }, "message": "Rename art::ReadBarrier::WhiteState as art::ReadBarrier::NonGrayState.\n\nThe read barrier state recorded in object\u0027s lockword used to be a\nthree-state value (white/gray/black), which was turned into a\ntwo-state value (white/gray), where the \"black\" state was conceptually\nmerged into the \"white\" state. This change renames the \"white\" state\nas \"non-gray\" and adjusts corresponding comments.\n\nTest: art/test.py\nChange-Id: I2a17ed15651bdbbe99270c1b81b4d78a1c2c132b\n" }, { "commit": "c73f05242a6688c8edec46c1ff257a1efbd4b519", "tree": "68b95952ec9710da3aabf7a686725692a9fd5cf5", "parents": [ "9e113dd00d94526d7e6e546ac9bd4f066db3a019" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 14 15:16:50 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 14 15:20:33 2018 +0100" }, "message": "Document the use of the biased card table in ART\u0027s code generators.\n\nTest: n/a\nChange-Id: Ie03a6f6dc87fd0766fc2b685ec39a0a0ebe3fb57\n" }, { "commit": "9d479254d0dc4043a15ab26205f40439eca15493", "tree": "af8a9c9c6f2c28e723a971c9d39c9d1cebd1f814", "parents": [ "ca20fb6cc4dda392e63bdc8ec9de54d89793373e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jul 24 11:35:20 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Aug 03 11:43:30 2018 +0000" }, "message": "Rename type resolution entrypoints.\n\nRename the InitializeType and InitializeTypeAndVerifyAccess\nentrypoints to Resolve* to better match their semantics.\nKeep the InitializeStaticStorage name for now as the most\nappropriate name InitializeType would clash with the old\nname of the ResolveType entrypoint.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nChange-Id: Ide55b58c490d085ab37d8536f90699f7ed571d59\n" }, { "commit": "3232dbb6df866985089b13a36c56e2b39dd473ab", "tree": "055f3e8888bfb3cfd072a981e4733cfaad7b202c", "parents": [ "b27d874ebc0c067d96994a6ebe3c10eaeb2e4a75" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jul 25 15:42:46 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Aug 03 08:59:07 2018 +0000" }, "message": "Do not save/restore regs in ClinitCheck slow path.\n\nThe entrypoint is kSaveEverything, so the only register that\nneeds to be saved is the argument/return value register.\n\nThe size of the aosp_taimen-userdebug prebuilts:\n - before:\n arm/boot*.oat: 16811692\n arm64/boot*.oat: 19801032\n oat/arm64/services.odex: 20232208\n - after:\n arm/boot*.oat: 16798804 (-12.6KiB, -0.08%)\n arm64/boot*.oat: 19804392 (+3.3KiB, +0.02%)\n oat/arm64/services.odex: 20227784 (-4.3KiB, -0.02%)\nNote that though there is less code, the metadata for the\narm64/boot*.oat outweighs the code size reduction because of\nthe register map encoding as value+shift introduced in\n https://android-review.googlesource.com/695682\nwhich it\u0027s ill-suited for kSaveEverything entrypoints. We\nshould reconsider that encoding.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nTest: testrunner.py --target --optimizing\nChange-Id: I5cd1deb90332a3b88a0a59d87925c557d9bff1ab\n" }, { "commit": "a9f303c089aa2b2fc82d97201352945678ef54ae", "tree": "0df0eb5294a3ee72aea8ca670762c02ca9ffa8dd", "parents": [ "1bfd891d06e276d602b4a6ccf1a9f70967195218" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jul 20 16:43:56 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Aug 02 17:41:21 2018 +0100" }, "message": "Rewrite Class init entrypoint to take a Class arg.\n\nFixes invalid type index being passed to the entrypoint for\nclass init check across dex files when the target type does\nnot have a TypeId in the compilation unit\u0027s DexFile.\n\nThe size of the aosp_taimen-userdebug prebuilts:\n - before:\n arm/boot*.oat: 16782748\n arm64/boot*.oat: 19764400\n oat/arm64/services.odex: 20162432\n - after:\n arm/boot*.oat: 16811692 (+28.3KiB, +0.17%)\n arm64/boot*.oat: 19801032 (+35.8KiB, +0.19%)\n oat/arm64/services.odex: 20232208 (+68.1KiB, +0.35%)\nThis increase comes from doing two runtime calls instead of\none for HLoadClass/kBssEntry that MustGenerateClinitCheck().\n\nTest: Additional test in 476-clinit-inline-static-invoke\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing --jit\nTest: Pixel 2 XL boots.\nTest: testrunner.py --target --optimizing --jit\nTest: testrunner.py --jvm\nBug: 111433619\nChange-Id: I2fccd6944480ab4dac514f60d38e72c1014ae7b2\n" }, { "commit": "8e524ad3c690c183b1a71f6114796974a107c5dd", "tree": "6ee124814e8f33fd4706eb15fcad2cae0019546b", "parents": [ "5991b184a40e4ce181d67d683ced46caa6143b53" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jul 13 10:27:43 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jul 16 10:12:51 2018 +0100" }, "message": "Always produce PIC code for AOT compilation.\n\nChange sharpening to use PIC load kinds for AOT compilation\nand add \"Jit\" to the direct addressing load kind names. Use\nPIC code also for the Integer.valueOf() intrinsic codegen.\nRemove all support for non-PIC linker patches.\n\nThe dex2oat --compile-pic option is retained for now but\nignored by the compiler.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nTest: testrunner.py --target --optimizing\nBug: 77856493\nChange-Id: I54d666f6522f160a1b6ece4045a15d19363acbb6\n" }, { "commit": "d109e30eab8ba25f8d89be2a83d9036e2d541af2", "tree": "24df91603efe9ce8c4a2efd09ac402aceb10df4e", "parents": [ "c916736ca1e375c276df251446baf2ac8ff3eb13" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Jun 27 10:25:41 2018 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Jul 10 08:44:51 2018 -0700" }, "message": "Don\u0027t use StringFactory.newEmptyString in compiled code\n\nWhen compiling debuggable code we would compile a new-instance String\ninstruction into a StringFactory.newEmptyString invoke. This\nadditional invoke could be observed using tracing and is inconsistent\nwith the interpreter, where the string is simply allocated directly.\nIn order to bring these two modes into alignment we added a new\nAllocStringObject quick entrypoint that will be used instead of the\nnormal AllocObject\u003c...\u003e entrypoints when allocating a string. This\nentrypoint directly allocates a new string in the same manner the\ninterpreter does.\n\nNeeds next CL for test to work.\n\nBug: 110884646\nTest: ./test/testrunner/testrunner.py --host --runtime-option\u003d-Xjitthreshold:0 --jit\nTest: Manual inspection of compiled code.\nChange-Id: I7b4b084bcf7dd9a23485c0e3cd2cd04a04b43d3d\n" }, { "commit": "6fd1606a3f3fc2dd53ab4f8b371e420b3e33c74f", "tree": "9f944d267ce3616eb969c027665e4a451c2b3879", "parents": [ "bb089b6bf850c87e0e42917a383cc7298dcb09c5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jun 26 11:02:04 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 29 14:39:00 2018 +0100" }, "message": "Implement Integer.valueOf() intrinsic for boot image.\n\nAnd generate only one \"boot image live objects\" array rather\nthan one per boot*.art file.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nTest: testrunner.py --target --optimizing\nBug: 71526895\nChange-Id: I23af7f47fea5150805f801cd2512f2d152ee5b73\n" }, { "commit": "a043111e3a2c09b549708a6227a1f54d91da76aa", "tree": "393fe11cfceccebf474e4bdf36ff79b70b97f589", "parents": [ "213ee2da6a1c58d0fc12c937bbd9c9974ca00aca" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 25 09:32:54 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 25 18:43:19 2018 +0100" }, "message": "Move instruction_set_ to CompilerOptions.\n\nRemoves CompilerDriver dependency from ImageWriter and\nseveral other classes.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nTest: m test-art-target-gtest\nTest: testrunner.py --target --optimizing\nChange-Id: I3c5b8ff73732128b9c4fad9405231a216ea72465\n" }, { "commit": "eebb821b1adaf2db7662fc1c3ff4e9fcfe59a694", "tree": "a3d3cf5f8c20d03fccdc0808537904da63e74938", "parents": [ "7e56bd41cde4e489a11050d9e340bf8b5692d9e8" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jun 05 14:57:24 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jun 21 16:12:28 2018 +0100" }, "message": "Implement Integer.valueOf() intrinsic for PIC.\n\nAnd fix the intrinsic for JIT even in case when someone\nmesses up the IntegerCache using reflection. Two cases are\nexposed with a regression test (one that previously failed\nrandomly and one that failed 100%) but other crashes were\npossible; for example, we would need a read barrier for\narray reads when elements are not guaranteed to be in the\nboot image.\n\nThe new approach loads references only from the boot image\nlive objects array which cannot be touched by reflection.\nThe referenced objects and IntegerCache.cache are exposed\nand can lead to weird behavior but not crashes.\n\nOn x86, the pc_relative_fixups_86 actually checks the cache\nan additional time but discrepancies between this check and\nthe location building at the beginning of codegen should be\nOK as the HIsX86ComputeBaseMethodAddress should be added\nfor PIC regardless of whether pc_relative_fixups_86 thinks\nthe method is intrinsified or not.\n\nTest: 717-integer-value-of\nTest: Pixel 2 XL boots.\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing --pictest --npictest\nTest: testrunner.py --host --jit\nTest: testrunner.py --target --optimizing --pictest --npictest\nTest: testrunner.py --target --jit\nBug: 71526895\nChange-Id: I89b3245a62aba22980c86a99e2af480bfa250af1\n" }, { "commit": "4c8e12e66968929b36fac6a2237ca4b04160161e", "tree": "d8bbfd72a978c69ef2eef98c37e7869673c52295", "parents": [ "20c64f8d802cc575cc9a1a1f6c493a611b23e2ee" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri May 18 08:33:20 2018 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Jun 11 10:57:17 2018 +0100" }, "message": "ART: Adds an entrypoint for invoke-custom\n\nAdd support for the compiler to call into the runtime for\ninvoke-custom bytecodes.\n\nBug: 35337872\nTest: art/test.py --host -r -t 952\nTest: art/test.py --target --64 -r -t 952\nTest: art/test.py --target --32 -r -t 952\nChange-Id: I821432e7e5248c91b8e1d36c3112974c34171803\n" }, { "commit": "dbaa5c7ba8935cf87ceb40a4054f9842929e9a51", "tree": "5037625c80cb97a0e13026dc450db28e59ff72ca", "parents": [ "51dda39549033b3c50a7fce5522ffc81325db54b" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu May 10 08:22:46 2018 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri May 11 11:55:30 2018 +0100" }, "message": "ART: Compiler support for const-method-handle\n\nImplemented as a runtime call.\n\nBug: 66890674\nTest: art/test.py --target -r -t 979\nTest: art/test.py --target --64 -r -t 979\nTest: art/test.py --host -r -t 979\nChange-Id: I67f461c819a7d528d7455afda8b4a59e9aed381c\n" }, { "commit": "18259d7fb7164a5e029df4f883b3a79ccc2403e8", "tree": "ba378bfdef4127bb0607215186e3b150fd38bcdf", "parents": [ "922501b4bbf724e4259477a27764291684eedffb" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Apr 12 11:18:23 2018 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu May 10 15:04:09 2018 +0100" }, "message": "ART: Compiler support for const-method-type\n\nImplemented as a runtime call.\n\nBug: 66890674\nTest: art/test.py --target -r -t 979\nTest: art/test.py --target --64 -r -t 979\nTest: art/test.py --host -r -t 979\nChange-Id: I4b3d3969d455d0198cfe122eea8abd54e0ea20ee\n" }, { "commit": "9f8d312a91314b0f9985a0db7cfd5aac3a52a34c", "tree": "bb9f593037772b4339f7eac5caa3187510e7c6d1", "parents": [ "c14f54b6d328eb23763391f98dfd776a720c4f54" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Apr 06 13:47:59 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Apr 06 16:03:48 2018 +0100" }, "message": "x86: Remove unnecessary temp from CheckCast locations.\n\nThis should have been removed in\n https://android-review.googlesource.com/300816 .\nAlso clean up related comments and refactor type checks\nin the x86-64 codegen in line with the refactoring for\nother architectures in that CL.\n\nTest: testrunner.py --host --optimizing\nChange-Id: I721338985e5388ecd9216e11bcd1c772de8416e6\n" }, { "commit": "175e7862dbdb44089ef327fc43ba00c791fd3838", "tree": "7b7ff4327b51b57e47e4b22af8d771edb9d462c1", "parents": [ "77c6fc7341143dd27c74cddd786398688d7b4c91" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 27 09:03:13 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 27 10:04:25 2018 +0100" }, "message": "Revert^4 \"Compiler changes for bitstring based type checks.\"\n\nDisabled the build time flag. (No image version bump needed.)\n\nBug: 26687569\nBug: 64692057\nBug: 76420366\n\nThis reverts commit 3fbd3ad99fad077e5c760e7238bcd55b07d4c06e.\n\nChange-Id: I5d83c4ce8a7331c435d5155ac6e0ce1c77d60004\n" } ], "next": "3fbd3ad99fad077e5c760e7238bcd55b07d4c06e" }