)]}' { "log": [ { "commit": "d2d5262c8370309e1f2a009f00aafc24f1cf00a0", "tree": "15b542ac079f30043cd3654cf5d3c40ae3ea34d0", "parents": [ "5b12f7973636bfea29da3956a9baa7a6bbe2b666" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 12 16:28:54 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 12 16:28:54 2016 +0000" }, "message": "Revert \"Add kJitTableAddress for HLoadClass.\"\n\nOne test failure after merge.\n\nThis reverts commit 5b12f7973636bfea29da3956a9baa7a6bbe2b666.\n\nChange-Id: I120c49e53274471fc1c82a10d52e99c83f5f85cc\n" }, { "commit": "5b12f7973636bfea29da3956a9baa7a6bbe2b666", "tree": "a2cd41c1d3c09abc594a76af11b7bebc302a2870", "parents": [ "0dd27eb2b51d030866c25dbf8e7bb737eb35a888" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Dec 09 11:26:35 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 12 11:50:24 2016 +0000" }, "message": "Add kJitTableAddress for HLoadClass.\n\nThis new kind loads classes from the root table associated with\nJIT compiled code.\n\nAlso remove kDexCacheAddress, which is replaced by kJitTableAddress.\n\ntest: ART_TEST_JIT\u003dtrue test-art-host-jit test-art-target-jit\nChange-Id: Ia23029688d1a60c178bf2ffa7463927c5d5de4d0\n" }, { "commit": "d4cc5b2127894475160b5d2bba9791dd21af5953", "tree": "04120b6077cacb5d16cb40161dfddd6170d8a64e", "parents": [ "3acf36142594f0afb60e2eef82fdfab1b17b2463" ], "author": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Fri Nov 04 11:19:09 2016 +0000" }, "committer": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Mon Dec 05 13:01:29 2016 +0000" }, "message": "ARM: VIXL32: Implement Invoke, LoadClass, LoadString dispatch.\n\nImplemented dispatch optimizations for InvokeStaticOrDirect,\nLoadClass, LoadString (excluding cases that use Literals).\nPerformed a cleanup of VIXL backend.\n\nTest: export ART_USE_VIXL_ARM_BACKEND\u003dtrue \u0026\u0026 \\\n mma test-art-host dist \u0026\u0026 mma test-art-target dist\n\nChange-Id: Ib37a6b7e7657196b13caec999d190be747857c1d\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": "f46501c6ad11025843682267c10f221323a206b1", "tree": "a5b77de81f5ab31e66082b02b225e7fe56877f98", "parents": [ "71d15102b52af67e8fe1193192aa2b4cd1956ae0" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 22 13:45:36 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 22 14:21:31 2016 +0000" }, "message": "Clear the right data pointer.\n\nInternally, stack_map_data is an offset of the roots_data allocation.\nPass both to the ClearData method, which will use the right pointer\nto deallocate.\n\nTest: test-art-host in debug mode\nChange-Id: Ibfe38e7f1853870076048427f2efe7121c11c136\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": "dac7ad17c78387d15d7aefae0f852dddf5f37e34", "tree": "603b3ca74ba9144d217497417314fe863e2201ef", "parents": [ "af524438b8a76264a23ec2d748e11eeded02abc7", "132d8363bf8cb043d910836672192ec8c36649b6" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 21 10:14:06 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Nov 21 10:14:07 2016 +0000" }, "message": "Merge \"Revert \"Revert \"Revert \"Revert \"JIT root tables.\"\"\"\"\"" }, { "commit": "132d8363bf8cb043d910836672192ec8c36649b6", "tree": "42ee8b08f0323e9762675333b50a4c218ecdefc7", "parents": [ "e89667815b9d0a1eacb91678fed2a7518bb07cc2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 16 09:19:42 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Nov 18 14:31:30 2016 +0000" }, "message": "Revert \"Revert \"Revert \"Revert \"JIT root tables.\"\"\"\"\n\nTest: 626-set-resolved-string, test-art-host, test-art-target\nTest: run-libcore-tests.sh\nTest: phone boots and runs\n\nThis reverts commit 3395fbc20bcd20948bec8958db91b304c17cacd8.\n\nChange-Id: I104b73d093e3eb6a271d564cfdb9ab09c1c8cf24\n" }, { "commit": "2bbc953357ac5f62cc4ab2eb055e4392e47d0c3d", "tree": "aaaa0ea192d66bf71722fd71ec046b9abf3e884c", "parents": [ "ecd8300b94938f22fa3988c58187acce9f091da8" ], "author": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Fri Oct 21 11:51:50 2016 +0100" }, "committer": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Wed Nov 16 18:40:36 2016 +0000" }, "message": "ARM: VIXL32: Arch specific operations.\n\nTests which now start to pass:\n 550-checker-multiply-accumulate\n 564-checker-negbitwise\n\nTest: export ART_USE_VIXL_ARM_BACKEND\u003dtrue \u0026\u0026 \\\n mma test-art-host dist \u0026\u0026 mma test-art-target dist\n\nChange-Id: I0b057486a8e0e48afbc52dd18b61ea16c671bec2\n" }, { "commit": "81cae78d1853893ff9c3ecea4b5100002a538eb7", "tree": "d32c8f7e57449381511ffb206f4a335c157ae597", "parents": [ "e8fc2cedb85ce4a6747cddbbf4cf33288e0ba5b9", "3395fbc20bcd20948bec8958db91b304c17cacd8" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 14 12:41:12 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Nov 14 12:41:12 2016 +0000" }, "message": "Merge \"Revert \"Revert \"Revert \"JIT root tables.\"\"\"\"" }, { "commit": "3395fbc20bcd20948bec8958db91b304c17cacd8", "tree": "e0d00b412961e2aaa0dff91317a9f4168189e5fb", "parents": [ "75afcdd3503a8a8518e5b23d21b6e73306ce39ce" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 14 12:40:52 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 14 12:40:52 2016 +0000" }, "message": "Revert \"Revert \"Revert \"JIT root tables.\"\"\"\n\nlibcore failures:\ndalvikvm32 F 11-14 03:04:06 14870 14870 jit_code_cache.cc:310] Check failed: new_string !\u003d nullptr\n\nThis reverts commit 75afcdd3503a8a8518e5b23d21b6e73306ce39ce.\n\nChange-Id: I5a6b6b48aa79a763d1ff1ba4d85d63811254787d\n" }, { "commit": "4b050e817b929169e57661376793cff4c3ef421a", "tree": "56ee6c72dd7ed64fe4bcf701a0733c50b0289bc9", "parents": [ "7fad15efba22817e9abb0f35b15aa2dd4715884c", "75afcdd3503a8a8518e5b23d21b6e73306ce39ce" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 14 09:13:46 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Nov 14 09:13:47 2016 +0000" }, "message": "Merge \"Revert \"Revert \"JIT root tables.\"\"\"" }, { "commit": "75afcdd3503a8a8518e5b23d21b6e73306ce39ce", "tree": "b49afc4772fa72319475f96ed66d744bb03e803c", "parents": [ "ee48b0f11df812d98de5c989e1f6d95a21515564" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Nov 10 10:38:11 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Nov 11 13:18:24 2016 +0000" }, "message": "Revert \"Revert \"JIT root tables.\"\"\n\nAlso contains Revert \"Support kJitTableAddress in x86/arm/arm64.\"\n\nThis reverts commit 4acd03638fcdb4e5d1666f8eec7eb3bf6d6be035.\nThis reverts commit 997d1217830c0a18b70faeabd53c04700a87d7d9.\n\nTest: ART_USE_READ_BARRIER\u003dtrue/false test-art-host test-art-target\n\nChange-Id: I77cb1e9bf8f1b4c58b72d3cf5ca31ced2aaa1ea3\n" }, { "commit": "00c141aaa07b95a8c2c492b617871abd3cf7162a", "tree": "e4777db1006e6ac42a0669bf5383b61235835dce", "parents": [ "ee59d46595bb860fea26122f47f8797d2d0fe5f4" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Nov 10 15:19:15 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Nov 10 16:55:22 2016 +0000" }, "message": "Dump c1visualizer output before and after each pass.\n\nIn order to get the last state of a graph before a crash.\n\nTest: dex2oat --dump-cfg\nChange-Id: I461a26a47a170b95fadb9da21a443af26fe52181\n" }, { "commit": "20c56a8cded87092d40097323fcf9eaf50f64e4b", "tree": "1dd41fe4afd736aa0cfb8d6ef8ed4835a4dbe4af", "parents": [ "2e7f085990423a706137c1d42673226ee7a7194f", "4acd03638fcdb4e5d1666f8eec7eb3bf6d6be035" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 09 17:30:49 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Nov 09 17:30:49 2016 +0000" }, "message": "Merge \"Revert \"JIT root tables.\"\"" }, { "commit": "4acd03638fcdb4e5d1666f8eec7eb3bf6d6be035", "tree": "1220262d0ed22d91777c68a8a568348cddf820cb", "parents": [ "ac3ebc3150760425ed00abd56da48f9a6e0666bc" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 09 17:30:31 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 09 17:30:31 2016 +0000" }, "message": "Revert \"JIT root tables.\"\n\nMay be the offender for jit-gcstress failure of 902.\n\nThis reverts commit ac3ebc3150760425ed00abd56da48f9a6e0666bc.\n\nChange-Id: I9ea6c9236fd1729fed7d1868dd8a111172932308\n" }, { "commit": "0864c98b10cb8ca306671cabaa9be2a4068a2711", "tree": "56d9bb3097c51d5efbf67cd7516b0bc3c8e16616", "parents": [ "4c45ffaf6329d40e3249894cfb078d3d274ddfd5", "ac3ebc3150760425ed00abd56da48f9a6e0666bc" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 09 09:55:45 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Nov 09 09:55:45 2016 +0000" }, "message": "Merge \"JIT root tables.\"" }, { "commit": "ca620d7bc03b23a0bcf0ef58df58603ee000dca0", "tree": "07cb026075b70a958d14ae84b4e213178a6ba0b4", "parents": [ "b02b8d7df48ea3314cfcb3c08d84ac9556363833" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 08 08:09:33 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 08 15:04:48 2016 -0800" }, "message": "ART: Fix tidy warnings\n\nSwitch to char versions of find variants.\n\nAdd \"explicit\" constructor variants or refactor and\nremove defaults.\n\nUse const references.\n\nBug: 32619234\nTest: m test-art-host\nChange-Id: I970cc2f47d6cf8f0c74104b994b075b2fafb3d45\n" }, { "commit": "ac3ebc3150760425ed00abd56da48f9a6e0666bc", "tree": "3c5c923bb00a898e035c0e7c4b8a6a684a578419", "parents": [ "76ad2405b40547e0ab4fc401fb8303189ff783eb" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 05 13:13:50 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 08 17:21:30 2016 +0000" }, "message": "JIT root tables.\n\nImplement root tables for the JIT. Each JIT compiled method\ngets a table allocated before the stack maps.\n\nThe table gets visited through Runtime::SweepSystemWeaks.\n\nImplement String roots for x86_64 as an example.\n\nTest: test-art-host test-art-target\n\nChange-Id: Id3d5bc67479e08b52dd4b253e970201203a0f0d2\n" }, { "commit": "2767f4ba2df934fea4c50a016e2955c2cf3f6b19", "tree": "2676ce7b335cc0ce197a4754ed62fc114647ba4d", "parents": [ "77a0e4541a29c26dba21ed8c1ed1c29ef8d0ce4c" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Fri Oct 28 15:03:53 2016 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Nov 02 14:45:36 2016 -0700" }, "message": "New instruction simplifications. Extra dce pass. Allow more per block repeats.\n\nRationale:\nWe were missing some obvious simplifications, which left performance\nat the table for e.g. CaffeineLogic compiled with dx (4200us-\u003e2700us).\nThe constant for allowing a repeat on a BB seemed very low, at the\nvery least it should depend on the BB size.\n\nTest: test-art-host\n\nChange-Id: Ic234566e117593e12c936d556222e4cd4f928105\n" }, { "commit": "b0103ca545c285f07e30ba5fd1cc007332fef66b", "tree": "b9adcb3e16c9e9bc2de17571b3f03b6fd583ca61", "parents": [ "b9b8cab87a275fad382fefbddd18daf495a58772" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Nov 01 14:48:47 2016 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Nov 01 14:48:47 2016 +0000" }, "message": "Flush the CFG visualizer output after writing to it.\n\nThis fixes test failures exposed by the ART Buildbot in\n608-checker-unresolved-lse (AOT) on the hammerhead-ndebug\nconfiguration. The test used to fail because the CFG file\ndumped by second invocation of dex2oat on device would be\ntruncated (at the beginning of the disassembly section\nof the second compiled Dex file\u0027s contents) because of\ndex2oat\u0027s fast exit.\n\nInterestingly enough, this is the only case where this\nfailure has been observed, which seems to be due to a\ncombination of:\n- targeting (32-bit) ARM;\n- using a secondary Dex file (used to create an unresolved\n access for that test);\n- compiling that secondary Dex file with dex2oat (ndebug\n mode) instead of dex2oatd;\n- supporting multithread CFG graph dumping (by having the\n write-to-file operation in a critical section) since\n https://android-review.googlesource.com/#/c/296224/.\n\nTest: art/test/run-test -O --debuggable 608-checker-unresolved-lse\nChange-Id: Ifc1a23a3708b8645fd36c148312074bb9fe00cfc\n" }, { "commit": "a0d81af89349dada4586a2770597e27e5c9f0f90", "tree": "4ae75d10be17c6fc45a1471a77ad9df0f41bff64", "parents": [ "a908348b4388854dab0b655b55dbeac1ecec2949" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Oct 27 12:04:57 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 28 08:10:50 2016 -0700" }, "message": "ART: Make graph visualizer thread-safe\n\nStore visualizer to a temporary string stream during compilation.\nPrint to the final output on destruction, and do so under a lock.\n\nNote: the order of methods in the output is only guaranteed to be\n deterministic with single-threaded compilation (-j1).\n\nTest: m\nTest: Compile multithread\nTest: Check output file that begin_compilation and end_compilation\n are interleaved\nTest: Check that names of compiled methods match\n\nChange-Id: I7eb3be2f88333cf1f116fe71b77cffa1ac702208\n" }, { "commit": "a8188191477b7b5b01a3c4426c51c48cd55f6678", "tree": "794ab1f6abf125263589acc037f01e8ba97004ef", "parents": [ "6494dae35b15cc5b771daa7871e4621e6dec1744", "639cc8c7bbb7d8c341173bcf24604ccb4328acb8" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Oct 20 22:50:12 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Oct 20 22:50:12 2016 +0000" }, "message": "Merge \"Improve recognition of select-based period induction.\"" }, { "commit": "639cc8c7bbb7d8c341173bcf24604ccb4328acb8", "tree": "ae8181b94ddd145edaff141a2b351478161f1559", "parents": [ "3941c882ea7b54772dec36a9a1b33e0b8a7474f7" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Tue Oct 18 13:03:31 2016 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Thu Oct 20 14:14:17 2016 -0700" }, "message": "Improve recognition of select-based period induction.\n\nRationale:\nSimilar to the previous CL, this helps to eliminate more dead induction.\nNow, CaffeineLogic, when compiled with dx (rather than jack) improves\nby a 1.5 speedup (9000us -\u003e 6000us).\n\nNote:\nWe need to run the simplifier before induction analysis to trigger\nthe select simplification first. Although a bit of a compile-time hit,\nit seems a good idea to run a simplifier here again anyway.\n\nTest: test-art-host\nChange-Id: I93b91ca40a4d64385c64393028e8d213f0c904a8\n" }, { "commit": "2f2533f686f759ccd29d2712da2c7610382fb59f", "tree": "b4853d93d2427928bbf28aac135b939017360c9e", "parents": [ "fc7a96464c58f2e3432cba2a3cfb19d84b033fdf", "a7812ae7939b199392f874b24a52454bbd0c13f2" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Oct 20 11:59:42 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Oct 20 11:59:43 2016 +0000" }, "message": "Merge \"ARM: VIXL32: Pass initial ART tests with new code generator.\"" }, { "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": "a7812ae7939b199392f874b24a52454bbd0c13f2", "tree": "fcaff0417676c74ad7bf88e7c2571948e7eeef81", "parents": [ "a497095b67c4ba2969d0b2a13f6959125c89d2bc" ], "author": { "name": "Scott Wakeling", "email": "scott.wakeling@linaro.org", "time": "Mon Oct 17 10:03:36 2016 +0100" }, "committer": { "name": "Scott Wakeling", "email": "scott.wakeling@linaro.org", "time": "Mon Oct 17 13:41:40 2016 +0100" }, "message": "ARM: VIXL32: Pass initial ART tests with new code generator.\n\n- Implement enough codegen to pass ~70 art/tests.\n\n- When ART_USE_VIXL_ARM_BACKEND is defined:\n - Blacklist known-to-fail target tests\n - interpret-only everything except the tests themselves\n - Set a flag to use the VIXL based ARM backend\n\nTest: export ART_USE_VIXL_ARM_BACKEND\u003dtrue \u0026\u0026 mma test-art-target \u0026\u0026 mma test-art-host\n\nChange-Id: Ic8bc095e8449f10f97fa0511284790f36c20e276\n" }, { "commit": "e8a3c576301fd531d5f73a65fc8b84a63619d580", "tree": "a00e488f153505ee890c28873308d373e6ab231b", "parents": [ "e8317d90c61dde07c12e404a2bc1fabf584905c1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 11 16:52:17 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 12 13:30:54 2016 -0700" }, "message": "Replace StackHandleScopeCollection with VariableSizedHandleScope\n\nVariableSizedHandleScope\u0027s internal handle scopes are not pushed\ndirectly on the thread. This means that it is safe to intermix with\nother types of handle scopes.\n\nAdded test.\n\nTest: clean-oat-host \u0026\u0026 test-art-host\n\nChange-Id: Id2fd1155788428f394d49615d337d9134824c8f0\n" }, { "commit": "2f61867045ffbd0c38a4ecec5f59632004f7efff", "tree": "c7b4393d7cd7cbcd3272ff83d8424be02828251a", "parents": [ "4aa6a93c46a959df1ab71ee7a68ad345338046ef", "97d7e1cd7f733cb33a0e238bec6d7ed525638cd1" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Oct 05 17:00:41 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Oct 05 17:00:41 2016 +0000" }, "message": "Merge \"Remove #include \"oat_file.h\" from class_linker.h .\"" }, { "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": "ebe167422f6197b9df0698bbeb944a0e4eea5f2d", "tree": "60bb0084030cadfed694261c36c5722f4f945869", "parents": [ "2a5c5160771ad528bc9b3ac36ac5785a184c956d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 05 09:55:42 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 05 09:55:42 2016 +0100" }, "message": "Properly scope HLoopOptimization\u0027s allocator.\n\nHOptimization classes do not get their destructor called,\nas they are arena objects. So the scope for the optimization\nallocator needs to be the Run method.\n\nAlso anticipate bisection search breakage by adding\nHLoopOptimization to the list of recognized optimizations.\n\nChange-Id: I7770989c39d5700a3b6b0a20af5d4b874dfde111\n" }, { "commit": "281c681a0852c10f5ca99b351650b244e878aea3", "tree": "33036cbfb76ee497eedf60e0e5785a2267c9dd02", "parents": [ "a845d07bbd57f8beaea8b4fb47192a3382ef25b2" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Fri Aug 26 11:31:48 2016 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Mon Oct 03 15:15:27 2016 -0700" }, "message": "A first implementation of a loop optimization framework.\n\nRationale:\nWe are planning to add more and more loop related optimizations\nand this framework provides the basis to do so. For starters,\nthe framework optimizes dead induction, induction that can be\nreplaced with a simpler closed-form, and eliminates dead loops\ncompletely (either pre-existing or as a result of induction\nremoval).\n\nSpeedup on e.g. Benchpress Loop is 73x (17.5us. -\u003e 0.24us.)\n[with the potential for more exploiting outer loop too]\n\nTest: 618-checker-induction et al.\n\nChange-Id: If80a809acf943539bf6726b0030dcabd50c9babc\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": "0795f23920ee9aabf28e45c63cd592dcccf00216", "tree": "ff3f880c5e84f3316532b47d0e9a7729ade848ac", "parents": [ "d1224dce59eb0019507e41da5e10f12dda66bee4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 27 18:43:30 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 29 17:31:09 2016 -0700" }, "message": "Clean up ScopedThreadStateChange to use ObjPtr\n\nAlso fixed inclusion of -inl.h files in .h files by adding\nscoped_object_access-inl.h and scoped_fast_natvie_object_access-inl.h\n\nChanged AddLocalReference / Decode to use ObjPtr.\n\nChanged libartbenchmark to be debug to avoid linkage errors.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I4d2e160483a29d21e1e0e440585ed328b9811483\n" }, { "commit": "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": "68c981fad87720fae9c799b240141ce3c12cd5bf", "tree": "e55099aca20d8e7be13cf8eb3e48f37b0afd38bb", "parents": [ "c4db1d10c586627d6c3af01e399b333a15344969" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Aug 26 13:13:33 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Aug 26 15:35:53 2016 +0100" }, "message": "ARM/MIPS: Avoid dead dex cache arrays base for intrinsics.\n\nTest: Run ART test suite on host and Nexus 6.\nChange-Id: Ie2ad70f1e3f125eae5dad53a6384d405e0311505\n" }, { "commit": "a518c150add36b71aaaf9b904d9f5b4ad61b8c8c", "tree": "7daa4cc425a5e29179e310646f9c99ce2ebf84b1", "parents": [ "c8cbbf518d8a23fecaedb22c71e44ef3736b08e2", "ba6b679bd34449ec56508966706ca1b8d5e7cb17" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Aug 25 12:19:36 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Aug 25 12:19:36 2016 +0000" }, "message": "Merge \"ARM: Purge Arm32Assembler.\"" }, { "commit": "b6a12832943fcf838d89134158208c8a2e47f49f", "tree": "c85a7b7f0469447f3ed41ca6c4a4a6b975a4fbbc", "parents": [ "82a4c22ef4277faa08bf588c560c3dddcca559d2" ], "author": { "name": "Wojciech Staszkiewicz", "email": "staszkiewicz@google.com", "time": "Tue Aug 16 16:33:00 2016 -0700" }, "committer": { "name": "Wojciech Staszkiewicz", "email": "staszkiewicz@google.com", "time": "Tue Aug 23 00:45:36 2016 +0000" }, "message": "Use full pass name when building optimizations\n\nIf possible, use full pass name provided in --run-passes rather\nthan its base version.\n\nTest: m test-art-host -j32\n\n1. Prepare a run-passes file with content:\ndead_code_elimination$initial\ninstruction_simplifier\nx86_memory_operand_generation\n2. Run art for a dex file like:\nart -Xcompiler-option --run-passes\u003drun-passes -Xcompiler-option\n--dump-passes -classpath classes.dex Test\n3. Verify that dead_code_elimination$initial string is present in\ndump-passes output.\n\nChange-Id: I92d9ed0c8b919ea03f625f549123f546dffe546b\n" }, { "commit": "ba6b679bd34449ec56508966706ca1b8d5e7cb17", "tree": "ec5985cc0f9e8a75c717513954d25d69e526065e", "parents": [ "f606c3a687e3eae94296ba74d2d820b6e37692ff" ], "author": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Mon Aug 15 14:22:07 2016 +0100" }, "committer": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Fri Aug 19 10:39:23 2016 +0100" }, "message": "ARM: Purge Arm32Assembler.\n\nUse Thumb2Assembler always. This originated from finding out that\nthe JNI tests are run using the Arm32Assembler however in real\nworld Thumb2Assembler is used for JNI. Therefore Arm32Assembler\ncode is dead except its own tests and the illegitimate use in\nJNI tests.\n\nChange-Id: I9ca6b83582bf97149a46690518ccb9312b1a3b68\n" }, { "commit": "35938f90ef69fb030b18f2f7fb21f9b78620914b", "tree": "79619376bb6920c4a1b84e8f974a2abb54916eb7", "parents": [ "3049b2a54b5b55d094fea603236f492c31b842e1", "e706070b271945d60229736a15bb712f3061fd15" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Thu Aug 18 23:35:09 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Aug 18 23:35:09 2016 +0000" }, "message": "Merge \"Log optimization passes in verbose mode\"" }, { "commit": "e28c7d022b73cff58d0a9bcbaa5cfdc0950fe003", "tree": "479dbf47819e5b0ae5a6aabcceafe4e08e90d90c", "parents": [ "c6efcaa17319e9f81def246c277fec523f5b85e0" ], "author": { "name": "Wojciech Staszkiewicz", "email": "staszkiewicz@google.com", "time": "Wed Aug 17 19:15:51 2016 -0700" }, "committer": { "name": "Wojciech Staszkiewicz", "email": "staszkiewicz@google.com", "time": "Thu Aug 18 02:50:05 2016 +0000" }, "message": "Fix optimizing compiler not building for svelte\n\nFixes compiler not building when some of the codegen paths\nare disabled.\n\nTest: mmma -j art ART_TARGET_CODEGEN_ARCHS\u003dsvelte\nm -j32 test-art-host\n\nBUG\u003d30928847\n\nChange-Id: I52c78e8a4e507f74b1f2a39352970079721b737e\n" }, { "commit": "e706070b271945d60229736a15bb712f3061fd15", "tree": "b8951b25bf06733b5aa022aeff444033831f1654", "parents": [ "3cc35c324b748c41be92a51eae68b77846b4c243" ], "author": { "name": "Wojciech Staszkiewicz", "email": "staszkiewicz@google.com", "time": "Tue Aug 16 17:31:19 2016 -0700" }, "committer": { "name": "Wojciech Staszkiewicz", "email": "staszkiewicz@google.com", "time": "Thu Aug 18 01:36:04 2016 +0000" }, "message": "Log optimization passes in verbose mode\n\nIntroduce verbose logging of optimization passes run during\ncompilation.\n\nTest: m test-art-host -j32\nart -Xcompiler-option --runtime-arg -Xcompiler-option -verbose:compiler\n-classpath classes.dex Test\nChange-Id: Iae98ce9dcafc252f2d0eec138aa05b34e424bd2a\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": "5319d3cca5a9b8e9e3f59421818272b966575172", "tree": "a90bd83b7e69bbff0be601088bb1c764125d8cf6", "parents": [ "9cff32df754c428ef69ddb61e7600abfd4c75266" ], "author": { "name": "Wojciech Staszkiewicz", "email": "staszkiewicz@google.com", "time": "Mon Aug 01 17:48:59 2016 -0700" }, "committer": { "name": "Wojciech Staszkiewicz", "email": "staszkiewicz@google.com", "time": "Mon Aug 08 11:24:26 2016 -0700" }, "message": "Implement running user defined list of passes\n\nThis change introduces new dex2oat switch --run-passes\u003d. This switch\naccepts path to a text file with names of passes to run.\nCompiler will run optimization passes specified in the file rather\nthen the default ones.\n\nThere is no verification implemented on the compiler side. It is user\u0027s\nresponsibility to provide a list of passes that leads to successful\ngeneration of correct code. Care should be taken to prepare a list\nthat satisfies all dependencies between optimizations.\n\nWe only take control of the optional optimizations. Codegen (builder),\nand all passes required for register allocation will run unaffected\nby this mechanism.\n\nChange-Id: Ic3694e53515fefcc5ce6f28d9371776b5afcbb4f\n" }, { "commit": "2cd05b7d9976c0c7fa74d58cb1608c809e5c37d3", "tree": "5cb7751e491aeae7357ed357f2a52702bf16854a", "parents": [ "11a59a48474caa818ddf344575aa6afc51f45590" ], "author": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Wed Aug 03 16:57:37 2016 -0700" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Sat Aug 06 01:21:14 2016 +0000" }, "message": "Add a register allocation strategy compiler option\n\nTest: manually, on device.\n\nChange-Id: If007a1657dd5769ddef03691e0a19dbbe6ba1a29\n" }, { "commit": "06a46c44bf1a5cba6c78c3faffc4e7ec1442b210", "tree": "fa82ec7a787200e475e567a51c2839349a513021", "parents": [ "9755c262df1be7f5d5b98d038c8fd3734e974f9d" ], "author": { "name": "Alexey Frunze", "email": "Alexey.Frunze@imgtec.com", "time": "Tue Jul 19 15:00:40 2016 -0700" }, "committer": { "name": "Alexey Frunze", "email": "Alexey.Frunze@imgtec.com", "time": "Wed Jul 27 20:02:48 2016 -0700" }, "message": "MIPS32: Improve string and class loads\n\nTested:\n- MIPS32 Android boots in QEMU\n- test-art-host-gtest\n- test-art-target-run-test-optimizing in QEMU, on CI20\n- test-art-target-gtest on CI20\n\nChange-Id: I70fd5d5267f8594c3b29d5a4ccf66b8ca8b09df3\n" }, { "commit": "a69e790ad9751807d4a660bb4432b7ea79f0ad3b", "tree": "ce885e1cc3379d7f32c286b987d7d741ab571663", "parents": [ "edbecee66ff9c550cb82d4a5310b81c9def1e80e", "f5fb09035da52a07e9df81cdcb4b025edb0db113" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Mon Jul 25 17:53:51 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jul 25 17:53:51 2016 +0000" }, "message": "Merge \"Make static helper methods member functions of OptimizingCompiler class\"" }, { "commit": "f5fb09035da52a07e9df81cdcb4b025edb0db113", "tree": "469a91dd42323b63b72895d844de0e11455939c0", "parents": [ "633c22de95fe6f80c0dd3176e15de4de3ee4bc79" ], "author": { "name": "Wojciech Staszkiewicz", "email": "staszkiewicz@google.com", "time": "Fri Jul 22 13:33:11 2016 -0700" }, "committer": { "name": "Wojciech Staszkiewicz", "email": "staszkiewicz@google.com", "time": "Fri Jul 22 15:42:56 2016 -0700" }, "message": "Make static helper methods member functions of OptimizingCompiler class\n\nMake RunOptimizations, MaybeRunInliner and RunArchOptimizations member\nfunctions of OptimizingCompiler class.\n\nBoth versions of RunOptimizations are protected in preparation for\nbisection bug search CL.\n\nChange-Id: I596efa9ed3fccd1ed3798c6427cc166e2a5d28bd\n" }, { "commit": "a92938a17b75eed3213030faaff9cfb321d47f37", "tree": "c6db649f7c1f6b6a86eb568212a721aac4ee6d96", "parents": [ "89b03e0cfb03e3194e39f07fa8c1bd46e3b28a34", "328429ff48d06e2cad4ebdd3568ab06de916a10a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 21 09:17:15 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jul 21 09:17:16 2016 +0000" }, "message": "Merge \"ARM: Port instr simplification of array accesses.\"" }, { "commit": "328429ff48d06e2cad4ebdd3568ab06de916a10a", "tree": "6290ac8afc3e93488382727f6765f548a2cfff04", "parents": [ "79e73245140f4115039a7284b3797d701f368fe6" ], "author": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Wed Jul 06 16:23:04 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 21 09:16:43 2016 +0000" }, "message": "ARM: Port instr simplification of array accesses.\n\nAfter changing the addressing mode for array accesses (in\nhttps://android-review.googlesource.com/248406) the \u0027add\u0027\ninstruction that calculates the base address for the array can be\nshared across accesses to the same array.\n\nBefore https://android-review.googlesource.com/248406:\n add IP, r[Array], r[Index0], LSL #2\n ldr r0, [IP, #12]\n add IP, r[Array], r[Index1], LSL #2\n ldr r0, [IP, #12]\n\nBefore this CL:\n add IP. r[Array], #12\n ldr r0, [IP, r[Index0], LSL #2]\n add IP. r[Array], #12\n ldr r0, [IP, r[Index1], LSL #2]\n\nAfter this CL:\n add IP. r[Array], #12\n ldr r0, [IP, r[Index0], LSL #2]\n ldr r0, [IP, r[Index1], LSL #2]\n\nLink to the original optimization:\n https://android-review.googlesource.com/#/c/127310/\n\nTest: Run ART test suite on Nexus 6.\nChange-Id: Iee26f9a0a7ca46abb90e3f60d19d22dc8dee4d8f\n" }, { "commit": "8f49d4b04bab40bfd32ed7c8dfe501dea172bd79", "tree": "53ebbc7573f6ebd9c53e00f62e93358e9c4405af", "parents": [ "360b4b0137ce5f0bb771e2ddbfd4735cae932565" ], "author": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Thu Jul 14 13:24:00 2016 -0700" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Wed Jul 20 09:33:48 2016 -0700" }, "message": "Refactor register allocation to be pluggable\n\nAllow alternate register allocation strategies to be implemented\nin subclasses of a common register allocation base class.\n\nTest: m test-art-host\n\nChange-Id: I7c5866aa9ddff8f53fcaf721bad47654ab221b4f\n" }, { "commit": "e9288851eb6b62ece28f510d978d3793723b9a51", "tree": "3d422c4dfa7437b4891504ee452bf50b4852a9c5", "parents": [ "161c866ca742049f5c916696e1503c697be30e87" ], "author": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Thu Jul 14 14:08:16 2016 -0700" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Fri Jul 15 18:09:46 2016 -0700" }, "message": "Rename current register allocator implementation\n\nThis will allow a cleaner commit in an upcoming\nrefactoring of register allocation.\n\nTest: m test-art-host\n\nChange-Id: If420c97b088b3c934411ff83373e024003120746\n" }, { "commit": "ee8d971b14edfc1e0af9436fd223277df966e4c2", "tree": "2dc16ca40980cc716aba24d793eaa8731846a180", "parents": [ "dedde3f10d7801ad862d1ca1de89135decff6a60" ], "author": { "name": "Mark Mendell", "email": "mark.p.mendell@intel.com", "time": "Tue Jul 12 11:13:15 2016 -0400" }, "committer": { "name": "Mark Mendell", "email": "mark.p.mendell@intel.com", "time": "Wed Jul 13 10:40:45 2016 -0400" }, "message": "X86: Use memory to do array range checks\n\nCurrently, an HBoundsCheck is fed by an HArrayLength, causing a load of\nthe array length, followed by a register compare.\n\nAvoid the load when we can by comparing directly with the array length\nin memory. Implement this by marking the HArrayLength as \u0027emitted at\nuse site\u0027, and then generating the code in the HBoundsCheck.\n\nOnly do this replacement when we are the only user of the ArrayLength\nand it isn\u0027t visible to the environment.\n\nHandle the special case where the array is \u0027null\u0027 and where an implicit\nnull check can\u0027t be eliminated.\n\nThis code moves the load of the length to the slow code for the failed\ncheck, which is what we want.\n\nTest: 609-checker-x86-bounds-check\n\nChange-Id: I9cdb183301e048234bb0ffeda940eedcf4a655bd\nSigned-off-by: Mark Mendell \u003cmark.p.mendell@intel.com\u003e\n" }, { "commit": "e3fb245fbdb5e91cf8a9750504df40bd629e0080", "tree": "a3882db92b7942b2edd6add3090b5c875fef2d09", "parents": [ "1fdb340de4e608a88e8683c857cad5d0da2c16de" ], "author": { "name": "Alexey Frunze", "email": "Alexey.Frunze@imgtec.com", "time": "Tue May 10 16:08:05 2016 -0700" }, "committer": { "name": "Alexey Frunze", "email": "Alexey.Frunze@imgtec.com", "time": "Fri Jul 01 14:10:14 2016 -0700" }, "message": "MIPS32: Improve method invocation\n\nImprovements include:\n- CodeGeneratorMIPS::GenerateStaticOrDirectCall() supports:\n - MethodLoadKind::kDirectAddressWithFixup (via literals)\n - CodePtrLocation::kCallDirectWithFixup (via literals)\n - MethodLoadKind::kDexCachePcRelative\n- 32-bit literals to support the above (not ready for general-\n purpose applications yet because RA is not saved in leaf\n methods, but is clobbered on MIPS32R2 when simulating\n PC-relative addressing (MIPS32R6 is OK because it has\n PC-relative addressing with the lwpc instruction))\n- shorter instruction sequences for recursive static/direct\n calls\n\nTested:\n- test-art-host-gtest\n- test-art-target-gtest and test-art-target-run-test-optimizing on:\n - MIPS32R2 QEMU\n - CI20 board\n - MIPS32R6 (2nd arch) QEMU\n\nChange-Id: Id5b137ad32d5590487fd154c9a01d3b3e7e044ff\n" }, { "commit": "3aaf9645639ff56fbfe49a981242cbda01d6796e", "tree": "822de2f62ddeb8576ab6567e5cab723b5731dadf", "parents": [ "97b1e6633be800827145c4ffeded1c834082730f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jun 07 14:14:37 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jun 07 16:12:59 2016 +0100" }, "message": "Revert \"Revert \"Fix ArtMethod::GetInvokeType for static methods on interfaces.\"\"\n\nbug:29089267\nbug:27521545\n\nWe were hitting a compiler DCHECK that a class would never require to\ndo access checks on itself. The reason was that the compiler driver\nwas not trying to resolve a type, but instead relied on the verifier\nfor pre-populating the dex cache. However, the verifier doesn\u0027t\nnecessarily run in JIT mode.\n\nThis reverts commit 12abcbd950bd0ff4528e2e0d27ca5e881c7b0467.\n\nChange-Id: I59204c16927084f6605a2a3f999ca529f949e1ad\n" }, { "commit": "12abcbd950bd0ff4528e2e0d27ca5e881c7b0467", "tree": "2788e5aec7b022694f7e7d41c1e40c205e46596e", "parents": [ "808067335b228d7b50ad84123d3c8ecb7aeeb200" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jun 06 15:51:58 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jun 06 15:51:58 2016 +0000" }, "message": "Revert \"Fix ArtMethod::GetInvokeType for static methods on interfaces.\"\n\nbug:29089267\nbug:27521545\n\nFails some DHECKs.\n\nThis reverts commit 808067335b228d7b50ad84123d3c8ecb7aeeb200.\n\nChange-Id: I0e768ce85be593e3f50fd02abc29aa34f2be3562\n" }, { "commit": "808067335b228d7b50ad84123d3c8ecb7aeeb200", "tree": "8642091a7775c18ac03a079f1befea3261805346", "parents": [ "4b53f5795a289b40aa81b9c42ede5c896ea0e4ef" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jun 02 15:55:48 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jun 06 11:29:47 2016 +0100" }, "message": "Fix ArtMethod::GetInvokeType for static methods on interfaces.\n\nAnd avoid calling ResolveMethod in the JIT, since it already\nknows that method.\n\nbug:29089267\nbug:27521545\n\n(cherry picked from commit 2dc77ecf375882f51ff7c09712c05b80e58abb6b)\n\nChange-Id: I36084b1f207317452c42fdfc8ffa4d8c721d2f76\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": "93205e395f777c1dd81d3f164cf9a4aec4bde45f", "tree": "1d08efd9b7bca9fe23df9ae9489c5dd575d3c6df", "parents": [ "6990775e323cd9164d6cc10955a047b9d9f15f32" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Apr 13 11:59:46 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Apr 13 17:50:16 2016 +0100" }, "message": "Move Assemblers to the Arena.\n\nAnd clean up some APIs to return std::unique_ptr\u003c\u003e instead\nof raw pointers that don\u0027t communicate ownership.\n\nChange-Id: I3017302307a0253d661240750298802fb0d9585e\n" }, { "commit": "dee58d6bb6d567fcd0c4f39d8d690c3acaf0e432", "tree": "5a2f20546ca3c1544c44bee560062580e22dc79c", "parents": [ "391e155a6936a05bd39b171031ec21d2dee62133" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Apr 07 09:54:26 2016 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Apr 07 16:03:16 2016 +0000" }, "message": "Revert \"Revert \"Refactor HGraphBuilder and SsaBuilder to remove HLocals\"\"\n\nThis patch merges the instruction-building phases from HGraphBuilder\nand SsaBuilder into a single HInstructionBuilder class. As a result,\nit is not necessary to generate HLocal, HLoadLocal and HStoreLocal\ninstructions any more, as the builder produces SSA form directly.\n\nSaves 5-15% of arena-allocated memory (see bug for more data):\n GMS 20.46MB \u003d\u003e 19.26MB (-5.86%)\n Maps 24.12MB \u003d\u003e 21.47MB (-10.98%)\n YouTube 28.60MB \u003d\u003e 26.01MB (-9.05%)\n\nThis CL fixed an issue with parsing quickened instructions.\n\nBug: 27894376\nBug: 27998571\nBug: 27995065\n\nChange-Id: I20dbe1bf2d0fe296377478db98cb86cba695e694\n" }, { "commit": "75563e34a4e64a8e805e6810b13e07f1cafec137", "tree": "30bc7341b8b2c49e0fc56f783de86777abada138", "parents": [ "a07be75830af60ce50aba357e8de066e849aa21c", "9d07e3d128ccfa0ef7670feadd424a825e447d1d" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 05 09:23:52 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Apr 05 09:23:53 2016 +0000" }, "message": "Merge \"Clean up OatQuickMethodHeader after Quick removal.\"" }, { "commit": "60328910cad396589474f8513391ba733d19390b", "tree": "01702f6df5c39925b354a3152dd04289e7d97062", "parents": [ "e3ff7b293be2a6791fe9d135d660c0cffe4bd73f" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Apr 04 17:47:42 2016 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Apr 04 17:47:42 2016 +0000" }, "message": "Revert \"Refactor HGraphBuilder and SsaBuilder to remove HLocals\"\n\nBug: 27995065\nThis reverts commit e3ff7b293be2a6791fe9d135d660c0cffe4bd73f.\n\nChange-Id: I5363c7ce18f47fd422c15eed5423a345a57249d8\n" }, { "commit": "9d07e3d128ccfa0ef7670feadd424a825e447d1d", "tree": "dfb677fd75f0f297fef9bc49311cf1d22c770f56", "parents": [ "eb98c0ded592cfca8187c744393c82efd1020b2a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Mar 31 12:02:28 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Apr 04 17:50:20 2016 +0100" }, "message": "Clean up OatQuickMethodHeader after Quick removal.\n\nThis reduces the size of the pre-header by 8 bytes, reducing\noat file size and mmapped .text section size. The memory\nneeded to store a CompiledMethod by dex2oat is also reduced,\nfor 32-bit dex2oat by 8B and for 64-bit dex2oat by 16B. The\naosp_flounder-userdebug 32-bit and 64-bit boot.oat are each\nabout 1.1MiB smaller.\n\nDisable the broken StubTest.IMT, b/27991555 .\n\nChange-Id: I05fe45c28c8ffb7a0fa8b1117b969786748b1039\n" }, { "commit": "e3ff7b293be2a6791fe9d135d660c0cffe4bd73f", "tree": "d578d27cb78e6d2caef683cd8ac94c9a9752b192", "parents": [ "86ea7eeabe30c98bbe1651a51d03cb89776724e7" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Mar 02 16:48:20 2016 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Apr 04 11:21:30 2016 +0100" }, "message": "Refactor HGraphBuilder and SsaBuilder to remove HLocals\n\nThis patch merges the instruction-building phases from HGraphBuilder\nand SsaBuilder into a single HInstructionBuilder class. As a result,\nit is not necessary to generate HLocal, HLoadLocal and HStoreLocal\ninstructions any more, as the builder produces SSA form directly.\n\nSaves 5-15% of arena-allocated memory (see bug for more data):\n GMS 20.46MB \u003d\u003e 19.26MB (-5.86%)\n Maps 24.12MB \u003d\u003e 21.47MB (-10.98%)\n YouTube 28.60MB \u003d\u003e 26.01MB (-9.05%)\n\nBug: 27894376\nChange-Id: Iefe28d40600c169c5d306fd2c77034ae19476d90\n" }, { "commit": "86ea7eeabe30c98bbe1651a51d03cb89776724e7", "tree": "01702f6df5c39925b354a3152dd04289e7d97062", "parents": [ "0cb5c86b7021e70cbf584c1455aad1ef383af786" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Feb 16 09:26:07 2016 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Apr 04 11:21:20 2016 +0100" }, "message": "Build dominator tree before generating HInstructions\n\nSecond CL in the series of merging HGraphBuilder and SsaBuilder. This\npatch refactors the builders so that dominator tree can be built\nbefore any HInstructions are generated. This puts the SsaBuilder\nremoval of HLoadLocals/HStoreLocals straight after HGraphBuilder\u0027s\nHInstruction generation phase. Next CL will therefore be able to\nmerge them.\n\nThis patch also adds util classes for iterating bytecode and switch\ntables which allowed to simplify the code.\n\nBug: 27894376\nChange-Id: Ic425d298b2e6e7980481ed697230b1a0b7904526\n" }, { "commit": "2e384346d93a747e19fb74f32eba2d1e9403721f", "tree": "1aa4a05a06298a5c3c741414e859b0d37143c84e", "parents": [ "a872368b065af7cd42a1be02742a60877e096bae", "df739841b781dbd49c247e3795696389c4799020" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Mar 24 13:35:08 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Mar 24 13:35:08 2016 +0000" }, "message": "Merge \"Post-Quick cleanup: Remove CompilationUnit.\"" }, { "commit": "df739841b781dbd49c247e3795696389c4799020", "tree": "0e7dfce8d23f22aac9bf2b2849a087f36d5f58c3", "parents": [ "f8a3dea642fe638f54d181b7ca280b79ee519398" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Mar 23 16:59:07 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Mar 24 09:29:18 2016 +0000" }, "message": "Post-Quick cleanup: Remove CompilationUnit.\n\nChange-Id: I309411b0fffaaed1e218e2c34394bdf6e2f75b48\n" }, { "commit": "29d38e77c553c6cf71fc4dafe2d22b4e3f814872", "tree": "92f34f589af205af56189d221ded293234935c26", "parents": [ "2e89e901b9f303549f3ba64e45d87292e9c986c8" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Mar 23 15:31:51 2016 +0000" }, "committer": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Wed Mar 23 11:30:29 2016 -0700" }, "message": "Revert \"Revert \"Use compiler filter to determine oat file status.\"\"\n\nThis reverts commit 845e5064580bd37ad5014f7aa0d078be7265464d.\n\nAdd an option to change what OatFileManager considers up-to-date.\nIn our tests we\u0027re allowed to write to the dalvik-cache, so it\ncannot be kSpeed.\n\nBug: 27689078\nChange-Id: I0c578705a9921114ed1fb00d360cc7448addc93a\n" }, { "commit": "845e5064580bd37ad5014f7aa0d078be7265464d", "tree": "1e621756ba6a1fd345f2fb468eed88cdc81886e7", "parents": [ "a62d2f04a6ecf804f8a78e722a6ca8ccb2dfa931" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 23 06:42:05 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 23 06:42:05 2016 +0000" }, "message": "Revert \"Use compiler filter to determine oat file status.\"\n\nBots are red. Tentative reverting as this is likely the offender.\n\nBug: 27689078\n\nThis reverts commit a62d2f04a6ecf804f8a78e722a6ca8ccb2dfa931.\n\nChange-Id: I3ec6947a5a4be878ff81f26f17dc36a209734e2a\n" }, { "commit": "a62d2f04a6ecf804f8a78e722a6ca8ccb2dfa931", "tree": "d8624bb85016a3301b473b6503324db4d4cf24a3", "parents": [ "48e722432bb6e19df7bba02427e4a707e671af06" ], "author": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Fri Mar 18 15:05:30 2016 -0700" }, "committer": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Tue Mar 22 15:38:36 2016 -0700" }, "message": "Use compiler filter to determine oat file status.\n\nRecord the compiler filter in the oat header. Use that to determine\nwhen the oat file is up-to-date with respect to a target compiler\nfilter level.\n\nNew xxx-profile filter levels are added to specify if a profile should\nbe used instead of testing for the presence of a profile file.\n\nThis change should allow for different compiler-filters to be set for\ndifferent package manager use cases.\n\nBug: 27689078\nChange-Id: Id6706d0ed91b45f307142692ea4316aa9713b023\n" }, { "commit": "3a40bf2fc1b9823e3bc8fcf96d5242668b5c088b", "tree": "7dab31da65fc38c0b05cb236aa7722169a7d4210", "parents": [ "0339bee5b1be39669606c4ba57edd42a0bacf0ff" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 22 16:26:33 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 22 16:26:33 2016 +0000" }, "message": "ART: Clean up arena allocation memory tracking.\n\nRemove unused allocation types, mostly from removed Quick.\nMove logging one level up to capture memory used by stack\nmaps during AOT compilation. Raise the reporting threshold\nto 8MiB to limit the output to the worst offenders.\n\nChange-Id: I8c7a01bfa90bc8ec5eab66187eb6850a022f3543\n" }, { "commit": "914d71ead70bb6f2084b2ed39a9fd58fd014f67d", "tree": "50f30d6e47d18aa6c3ccec9f05727b4898268b20", "parents": [ "1583e624d4c970d8e571b265b9a8f08402d91f82", "2ae48182573da7087bffc2873730bc758ec29696" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Mar 16 14:40:08 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Mar 16 14:40:08 2016 +0000" }, "message": "Merge \"Clean up NullCheck generation and record stats about it.\"" }, { "commit": "933330a192d43d7c7304120999907857a89c2f96", "tree": "a413b4df2a4b9fdb804983204168ee23cd9c9eb7", "parents": [ "f808e8a0cc218c2b98023ef0e91f3c5b74ad2962" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 16 14:20:06 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 16 14:20:26 2016 +0000" }, "message": "Add more statistics for memory use of the JIT.\n\nCollect data for stack maps, profiling info, and compiled code.\n\nbug:27520994\n\nChange-Id: Ic87361230c96ce0090027a37d750e948d806c597\n" }, { "commit": "2ae48182573da7087bffc2873730bc758ec29696", "tree": "d6955329ad876aefd477f7ef8905b070b9ab95dd", "parents": [ "6915898b28cea6c9836ca1be6814d87e89cc6d76" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Mar 16 14:05:09 2016 +0000" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Mar 16 14:10:27 2016 +0000" }, "message": "Clean up NullCheck generation and record stats about it.\n\nThis removes redundant code from the generators and allows for easier\nstat recording.\n\nChange-Id: Iccd4368f9e9d87a6fecb863dee4e2145c97851c4\n" }, { "commit": "a4f81546373f4cb5fa6dfc135307ee0a1d930872", "tree": "bfc592fc0ec418ac8bbc270a31c6e741849c3276", "parents": [ "e70cda6147e2e49384a219b167a6c734a8db28f5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 08 16:57:48 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 16 11:16:49 2016 +0000" }, "message": "Collect memory use for the JIT.\n\nbug:27520994\n\nChange-Id: I67b0c5b822001bfde8738a988c1ade69f1a26e3f\n" }, { "commit": "09c2a6be63337ee060e2d54bd01cf18be7301d29", "tree": "c623c5352bc908410f03b20b772c3dad47df207c", "parents": [ "fdaa583e785383cca68df63c7ef53fcaf667b8e2" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Mar 11 17:11:44 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Mar 11 17:35:19 2016 +0000" }, "message": "Add trampoline symbols to native debug info.\n\nThe debugger needs them to unwind through the trampolines and to\nunderstand what is happening in the call stack.\n\nChange-Id: Ia554058c3796788adcd7336d620a7734eb366905\n" }, { "commit": "cc22d41d4de2514cc3cb4ed569764a35f15da48d", "tree": "49809c99b991eece44dad9ca26c89bdcf0f9d911", "parents": [ "1a008a127730710f8f56c1ca66385e1b062a2dcb", "fe736b775c7774bb2f7072e022099ca47dd21a3c" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Mar 11 14:25:56 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Mar 11 14:25:57 2016 +0000" }, "message": "Merge \"Allow generation of native debug info for multiple methods.\"" }, { "commit": "1a008a127730710f8f56c1ca66385e1b062a2dcb", "tree": "b3d33325c5e5ef2f34aa9a497c83863cf2481abb", "parents": [ "2af7213a4e0d395fe22dcdce6ec10a3bd131023d", "197160d47f34238cb5e7444fa4c2de300db8e2c6" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Mar 11 14:25:40 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Mar 11 14:25:41 2016 +0000" }, "message": "Merge \"Refactor MethodDebugInfo (input of DWARF writer).\"" }, { "commit": "d1c4045fb4d4703642f3f79985727b9a12cf5c49", "tree": "5fda881ed412b71013e1f2445ed5ddd1a7b37470", "parents": [ "2048b01dfec8877d672a4da41a158f68b12b5337" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Mar 02 16:06:13 2016 -0800" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Thu Mar 10 09:22:49 2016 -0800" }, "message": "Avoid generating dead code on frame enter/exit.\nThis includes stack operations and, on x86, call/pop to read PC.\n\nbug\u003d26997690\n\nRationale:\n(1) If method is fully intrinsified, and makes no calls in slow\n path or uses special input, no need to require current method.\n(2) Invoke instructions with HasPcRelativeDexCache() generate code\n that reads the PC (call/pop) on x86. However, if the invoke is\n an intrinsic that is later replaced with actual code, this PC\n reading code may be dead.\n\nExample X86 (before/after):\n\n0x0000108c: 83EC0C sub esp, 12\n0x0000108f: 890424 mov [esp], eax \u003c-- not needed\n0x00001092: E800000000 call +0 (0x00001097)\n0x00001097: 58 pop eax \u003c-- dead code to read PC\n0x00001098: F30FB8C1 popcnt eax, ecx\n0x0000109c: F30FB8DA popcnt ebx, edx\n0x000010a0: 03D8 add ebx, eax\n0x000010a2: 89D8 mov eax, ebx\n0x000010a4: 83C40C add esp, 12 \u003c-- not needed\n0x000010a7: C3 ret\n\n0x0000103c: F30FB8C1 popcnt eax, ecx\n0x00001040: F30FB8DA popcnt ebx, edx\n0x00001044: 03D8 add ebx, eax\n0x00001046: 89D8 mov eax, ebx\n0x00001048: C3 ret\n\nExample ARM64 (before/after):\n\n0x0000103c: f81e0fe0 str x0, [sp, #-32]!\n0x00001040: f9000ffe str lr, [sp, #24]\n0x00001044: dac01020 clz x0, x1\n0x00001048: f9400ffe ldr lr, [sp, #24]\n0x0000104c: 910083ff add sp, sp, #0x20 (32)\n0x00001050: d65f03c0 ret\n\n0x0000103c: dac01020 clz x0, x1\n0x00001040: d65f03c0 ret\n\nChange-Id: I8377db80c9a901a08fff4624927cf4a6e585da0c\n" }, { "commit": "fe736b775c7774bb2f7072e022099ca47dd21a3c", "tree": "92a7850c812dc0373738c6c5cfdff5a32d5e489c", "parents": [ "197160d47f34238cb5e7444fa4c2de300db8e2c6" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Mar 09 11:44:44 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Mar 10 10:29:27 2016 +0000" }, "message": "Allow generation of native debug info for multiple methods.\n\nSlight tweak in the API which I will need in the near future.\n\nChange-Id: I45954ae16710bc941a9a06a3a17c70798315ca53\n" }, { "commit": "197160d47f34238cb5e7444fa4c2de300db8e2c6", "tree": "4c07b9e97e8ffaffb274d03b63701780baf0b72f", "parents": [ "ef0119a0f329906ba1e8cbd8e53ce5a759fd4f99" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Mar 07 17:33:57 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Mar 10 10:11:31 2016 +0000" }, "message": "Refactor MethodDebugInfo (input of DWARF writer).\n\nDo not pass CompiledMethod pointer through since it is only available\nduring AOT compile but not during JIT compile or at runtime. Creating\nmock CompiledMethod just pass data is proving increasingly tricky, so\ncopy the fields that we need to MethodDebugInfo instead.\n\nChange-Id: I820297b41e769fcac488c0ff2d2ea0492bb13ed8\n" }, { "commit": "91cc06c1814bd1d0fd6635bc3d7632a2bb7b0e7c", "tree": "7c57374bd28d245fd0f071ec66d9aad8501a878a", "parents": [ "5d8112029d0e085c5a0099257daa4c7e29c12310" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Mar 07 16:13:58 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Mar 08 15:48:02 2016 +0000" }, "message": "Mark DWARF lines in non-debuggable methods as \"not a statement\".\n\nThis is a hint to the debugger that breakpoints and stepping\nmight not function as intended (since we have limited information).\n\nChange-Id: I23c4a816182cc7548fcd69fbd00112225e7b1710\n" }, { "commit": "5d8112029d0e085c5a0099257daa4c7e29c12310", "tree": "b98a1b83095ebd34feb3140e6910ec7e830bce0a", "parents": [ "8a8c84114b16b8971bc2d3c5c7e0e31470d1a68b" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Mar 08 13:21:22 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Mar 08 13:46:37 2016 +0000" }, "message": "Propagate InstructionSetFeatures to ElfBuilder.\n\nThis is subset of CL171665 and it separates it into two.\nIt will be needed to generate .MIPS.abiflags ELF section.\n\nChange-Id: I5557e7cb98d0fa1dc57c85cf6161e119c6d50a1a\n" }, { "commit": "700347e9bc6c2ed29046e0c13122a5ad57d2fc51", "tree": "e27ba6800f58515a3b19b12b32a6ee414faa35bf", "parents": [ "a71ec6419d9e80b354de721003c8b7929d224a49" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Mar 02 14:59:32 2016 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Mar 02 14:59:32 2016 -0800" }, "message": "Add a PassScope for PrepareForRegisterAllocation\n\nThis pass does transform the graph so make it part of cfg-dumping.\n\nChange-Id: I42e361382c85c97b974faad8bb0fcf2cb0750355\n" }, { "commit": "25e0456b6ea13eba290b63ea88b6b7120ed89413", "tree": "212c7433997a9e498ce68136ab511ae366e983e9", "parents": [ "47bce25b3eaf5c0e1d46f0555eaf4676f958a7a7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 01 13:17:58 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 01 13:17:58 2016 +0000" }, "message": "Give the JIT its own arena pool to avoid lock contentions.\n\nSharing it with the verifier and the class loader is not ideal,\nespecially at startup time.\n\nbug:27398183\nbug:23128949\nChange-Id: I1b91663a13f6c5b33ad3b4be780d93eb7fe445b4\n" }, { "commit": "4a0dad67867f389e01a5a6c0fe381d210f687c0d", "tree": "91f1e70f4a2d0bd32aa7eb51e546f5330d72f772", "parents": [ "d15ede2df7d157ea5480614fd18c2bf0d37a6c2a" ], "author": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Tue Jan 26 12:28:31 2016 +0300" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 25 10:14:30 2016 +0000" }, "message": "Revert \"Revert \"ARM/ARM64: Extend support of instruction combining.\"\"\n\nThis reverts commit 6b5afdd144d2bb3bf994240797834b5666b2cf98.\n\nChange-Id: Ic27a10f02e21109503edd64e6d73d1bb0c6a8ac6\n" }, { "commit": "219830939ac94af8bbf019cbf4fdb65f05992713", "tree": "5d29487608f838f5651fcb0edebbf2968569a58b", "parents": [ "05d77c2673082b371129f8d854439688533cfff2", "5b82d339955d1a0dc23eeb8d2d5659459ff987ba" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 19 16:01:49 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Feb 19 16:01:49 2016 +0000" }, "message": "Merge \"The JIT does not need VerifedMethod nor CompilerCallbacks.\"" }, { "commit": "38e9e8046ea2196284bdb4638771c31108a30a4a", "tree": "a2fecce9af616bebd73ce17d9f5954f84f291817", "parents": [ "eda4dfae59eec2db945cf032aee3a9aeaa2eb494" ], "author": { "name": "Jean-Philippe Halimi", "email": "jean-philippe.halimi@intel.com", "time": "Thu Feb 18 16:42:03 2016 +0100" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Feb 18 16:15:27 2016 +0000" }, "message": "Add statistics support for some optimizations\n\nThis patch adds support for the --dump-stats facility with some\noptimizations\nand fixes all build issues introduced by the patch:\nI68751b119a030952a11057cb651a3c63e87e73ea (which got reverted)\n\nChange-Id: I5af1f2a8cced0a1a55c2bb4d8c88e6f0a24ec879\nSigned-off-by: Jean-Philippe Halimi \u003cjean-philippe.halimi@intel.com\u003e\n" }, { "commit": "5b82d339955d1a0dc23eeb8d2d5659459ff987ba", "tree": "79540d831ec7ac82f5520dc6bc53ac8c39f47972", "parents": [ "d9c05309e8fab26a6087abd5773859f112cdefe9" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 14:22:32 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 14:33:13 2016 +0000" }, "message": "The JIT does not need VerifedMethod nor CompilerCallbacks.\n\nbug:27173201\n\nChange-Id: I971659f9ff6a8b780c94a7bed84de90fa9fc3456\n" }, { "commit": "badd826664896d4a9628a5a89b78016894aa414b", "tree": "a30e8b3e62126ae1e1df1152ac643cfc5f2b074a", "parents": [ "47a2a45a6673ddf3322115ff5058763f82a9368f" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Feb 02 16:28:56 2016 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Feb 15 10:21:07 2016 +0000" }, "message": "ART: Run SsaBuilder from HGraphBuilder\n\nFirst step towards merging the two passes, which will later result in\nHGraphBuilder directly producing SSA form. This CL mostly just updates\ntests broken by not being able to inspect the pre-SSA form.\n\nUsing HLocals outside the HGraphBuilder is now deprecated.\n\nBug: 27150508\nChange-Id: I00fb6050580f409dcc5aa5b5aa3a536d6e8d759e\n" }, { "commit": "c5bfa97c47d656b76f297af8abcd5f7502987399", "tree": "84da9d0f39096d8b857760a0848c0231aff0ed6d", "parents": [ "a0f2bc5b09cdaf8a8ff5cec3c40a9f18885e8796" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Feb 05 15:49:10 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Feb 08 12:47:07 2016 +0000" }, "message": "Split elf_writer_debug.cc to several files.\n\nRefactoring only. The file has grown significantly over time,\nand it is time to split it so it can be better managed.\n\nChange-Id: Idce0231718add722292f4701df353d5baf31de5f\n" }, { "commit": "a0f2bc5b09cdaf8a8ff5cec3c40a9f18885e8796", "tree": "f7be0f7f42b3a79eef3f1f97f39ad73f5b269b72", "parents": [ "e003beb849386236c1fe6e5edea6aaf5f58b7919", "4fda4eb799c95be266f52aaf3461a440ea86b841" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Feb 08 12:46:10 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Feb 08 12:46:10 2016 +0000" }, "message": "Merge \"Move code related to debug info generation to its own directory.\"" }, { "commit": "4fda4eb799c95be266f52aaf3461a440ea86b841", "tree": "54a7e56b5633caf2b868f6028010b488eb1923ce", "parents": [ "3c258f4d7a6492af733a9351ba430d876a3e5ccf" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Feb 05 13:34:46 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Feb 08 11:33:50 2016 +0000" }, "message": "Move code related to debug info generation to its own directory.\n\ndebug/dwarf/ contains helper classes which hide the details\nof the DWARF file format. It acts as independent DWARF library.\n\ndebug/ contains ART-specific code which generates ELF debug\nsections (which includes non-DWARF sections like .symtab).\n\nChange-Id: Id351f604e4e64be2ca395a78324ea02e30481497\n" }, { "commit": "b331febbab8e916680faba722cc84b66b84218a3", "tree": "35f985b021e476914bfe91492da23fee218014a7", "parents": [ "586996afc905518ed926e4680aab67bedabec9b7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 05 16:51:53 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 05 17:11:54 2016 +0000" }, "message": "Revert \"Revert \"Implement on-stack replacement for arm/arm64/x86/x86_64.\"\"\n\nThis reverts commit bd89a5c556324062b7d841843b039392e84cfaf4.\n\nChange-Id: I08d190431520baa7fcec8fbdb444519f25ac8d44\n" }, { "commit": "74eb1b264691c4eb399d0858015a7fc13c476ac6", "tree": "0b6fc4f3003d50bf6c388601013cdfc606e53859", "parents": [ "75fd2a8ab9b4aff59308034da26eb4986d10fa9e" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Dec 14 11:44:01 2015 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jan 28 15:50:27 2016 +0000" }, "message": "ART: Implement HSelect\n\nThis patch adds a new HIR instruction to Optimizing. HSelect returns\none of two inputs based on the outcome of a condition.\n\nThis is only initial implementation which:\n - defines the new instruction,\n - repurposes BooleanSimplifier to emit it,\n - extends InstructionSimplifier to statically resolve it,\n - updates existing code and tests accordingly.\n\nCode generators currently emit fallback if/then/else code and will be\nupdated in follow-up CLs to use platform-specific conditional moves\nwhen possible.\n\nChange-Id: Ib61b17146487ebe6b55350c2b589f0b971dcaaee\n" }, { "commit": "5b1c2ca30dad519be285f0a1e839c23cc4e3a51d", "tree": "330ebac2bd2592ccced6a17b069516afd45fa220", "parents": [ "d112827911122ba6412fd90b8d8b9a9d6db9d513" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Jan 25 17:32:41 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Jan 26 12:50:23 2016 +0000" }, "message": "Revert \"Revert \"Add option to generate compressed backtrace info.\"\"\n\nThis reverts commit 8546cc9aeb05e866e1fb6a9e4130d53ea330baa8.\n\nChange-Id: I676fdf9af27fa3b16fa8921778ff8832ab8c437d\n" }, { "commit": "8546cc9aeb05e866e1fb6a9e4130d53ea330baa8", "tree": "580a02752d8e447f6dce7cce01386c7e2a9a87f4", "parents": [ "5fdcc3c931b70204fd8c491afa66f57f8428490f" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Jan 25 17:31:49 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Jan 25 17:31:49 2016 +0000" }, "message": "Revert \"Add option to generate compressed backtrace info.\"\n\nThis reverts commit 5fdcc3c931b70204fd8c491afa66f57f8428490f.\n\nChange-Id: I9c1f5aad6933a46af6717e3a90a51f76111f9c8a\n" }, { "commit": "5fdcc3c931b70204fd8c491afa66f57f8428490f", "tree": "729f1433c75dda8a1f0e578a1c5297eed31f7852", "parents": [ "95005291d8ebdd1d2ac58ffc5181fef4fbbf2383" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Jan 19 14:33:43 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Jan 22 18:10:37 2016 +0000" }, "message": "Add option to generate compressed backtrace info.\n\nAdd flag --generate-mini-debug-info which generates\nLZMA compressed .symtab and .debug_frame, which are\nsufficient to print java backtraces in libunwind.\n\nIf enabled, it increases the size of boot.oat by about 3.5%.\n\nChange-Id: Ic3c2ef7704c05fa328720c6781ca2a9b8e3935a3\n" }, { "commit": "6b5afdd144d2bb3bf994240797834b5666b2cf98", "tree": "d536cd7b3aaf55c563e82c2c522521a91b2bb953", "parents": [ "debeb98aaa8950caf1a19df490f2ac9bf563075b" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 22 09:31:52 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 22 09:31:52 2016 +0000" }, "message": "Revert \"ARM/ARM64: Extend support of instruction combining.\"\n\nThe test fails its checker parts.\n\nThis reverts commit debeb98aaa8950caf1a19df490f2ac9bf563075b.\n\nChange-Id: I49929e15950c7814da6c411ecd2b640d12de80df\n" }, { "commit": "debeb98aaa8950caf1a19df490f2ac9bf563075b", "tree": "b2a7a7cc6fb2f56d4bcc6cecaa80035668f38dc4", "parents": [ "6aadaef35ea52506db61e463910c2520b702ca5e" ], "author": { "name": "Ilmir Usmanov", "email": "i.usmanov@samsung.com", "time": "Fri Dec 11 11:39:44 2015 +0300" }, "committer": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Thu Jan 21 11:07:38 2016 +0300" }, "message": "ARM/ARM64: Extend support of instruction combining.\n\nCombine multiply instructions in the following way:\nARM64:\nMUL/NEG -\u003e MNEG\nARM32 (32-bit integers only):\nMUL/ADD -\u003e MLA\nMUL/SUB -\u003e MLS\n\nChange-Id: If20f2d8fb060145ab6fbceeb5a8f1a3d02e0ecdb\n" } ], "next": "955d24c4221aa514067dc13d8a40c8b5071f467d" }