)]}' { "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": "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": "e51ca8bfa8a193b64901ad214842f213adca92eb", "tree": "b07b2534edc592d68b5461a3bdb8a31241892d54", "parents": [ "eb9c58e88f8e0f4a0a413efce04b49b183ad23e3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 22 14:49:31 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 29 21:05:51 2016 +0000" }, "message": "Move inline caches GC handling in JitCodeCache.\n\nMake the classes they hold weak references and visit\nthem during SweepJitRoots.\n\nThis fixes the newly introduced deadlock:\n\nThread1:\n1) Lock JitCodeCache lock to create Profiling info for\n ArtMethod m.\n2) m is a copied method, we need to track the actual holder,\n needing to decode a weak reference.\n3) Weak references are not accessible due to GC.\n\nGC Thread:\n- Disallow weak reference access.\n- Wait for checkpoint.\n\nThread2:\n- Try to lock JitCodeCache lock\n- Deadlock, as Thread1 owns the JitCodeCache lock.\n\nTest: test-art-host\nbug: 31289185\nbug: 33198826\n\nChange-Id: I7ee17631015450ace8d2a0264415a81c5a902bb8\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": "c757decb04d8535fd806b9bce1c2fe5e52c228dc", "tree": "8800f743cdfc30a37279d918f3947d3ec694ef66", "parents": [ "11bbc7fd5b4ed64d13c5d21ae9c8b686cf24bcbb" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Nov 04 15:48:34 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Nov 09 15:14:51 2016 -0800" }, "message": "Do not inline loops without exit edges\n\nFixes an issue with LinearOrder after inlining a function containing a\nloop that has no exit edge. The failure is due to incorrect loop\ninformation being computed for blocks that are not on a path to the\ninlined function\u0027s return. They should not be considered part of the\ncaller\u0027s enclosing loop, but are today.\n\nBug: 32547653\nTest: run-test --host 478-checker-inline-noreturn\nChange-Id: I9694a1cb861430051c801d07f7ce29752332cba5\n" }, { "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": "2c45bc9137c29f886e69923535aff31a74d90829", "tree": "90c7762c2dcb1105bf5907d6a1d068f172ea6155", "parents": [ "168c58ce18b139d313cbc6131e3bb43539952c55" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 25 16:54:12 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Oct 27 14:30:29 2016 +0100" }, "message": "Remove H[Reverse]PostOrderIterator and HInsertionOrderIterator.\n\nUse range-based loops instead, introducing helper functions\nReverseRange() for iteration in reverse order in containers.\nWhen the contents of the underlying container change inside\nthe loop, use an index-based loop that better exposes the\ncontainer data modifications, compared to the old iterator\ninterface that\u0027s hiding it which may lead to subtle bugs.\n\nTest: m test-art-host\nChange-Id: I2a4e6c508b854c37a697fc4b1e8423a8c92c5ea0\n" }, { "commit": "709b070044354d9f47641f273edacaeeb0240ab7", "tree": "3a8ac051d7c35076303984d0d892cdd396b60427", "parents": [ "1a4de6a2453a3ad0310aca1a44e7e2d3b6f53bc1" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Oct 13 09:12:37 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Oct 18 14:10:04 2016 -0700" }, "message": "Remove mirror:: and ArtMethod deps in utils.{h,cc}\n\nThe latest chapter in the ongoing saga of attempting to dump a DEX\nfile without having to start a whole runtime instance. This episode\nfinds us removing references to ArtMethod/ArtField/mirror.\n\nOne aspect of this change that I would like to call out specfically\nis that the utils versions of the \"Pretty*\" functions all were written\nto accept nullptr as an argument. I have split these functions up as\nfollows:\n1) an instance method, such as PrettyClass that obviously requires\nthis !\u003d nullptr.\n2) a static method, that behaves the same way as the util method, but\ncalls the instance method if p !\u003d nullptr.\nThis requires using a full class qualifier for the static methods,\nwhich isn\u0027t exactly beautiful. I have tried to remove as many cases\nas possible where it was clear p !\u003d nullptr.\n\nBug: 22322814\nTest: test-art-host\nChange-Id: I21adee3614aa697aa580cd1b86b72d9206e1cb24\n" }, { "commit": "97d7e1cd7f733cb33a0e238bec6d7ed525638cd1", "tree": "d2f02420f82d76d10dc0d80f431c9f4aba03ac82", "parents": [ "eebe142e935e9444c7c947d02246e7fbe88b2cc4" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 04 14:44:28 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Oct 05 13:21:44 2016 +0100" }, "message": "Remove #include \"oat_file.h\" from class_linker.h .\n\nRefactor the OatClass and OatMethod related functions from\nClassLinker to OatFile and ArtMethod, respectively. Refactor\nthe remaining ClassLinker dependencies on OatFile to break\nthe #include dependency and reduce incremental build times.\n\nTest: m test-art-host\nChange-Id: Iebc5b9f81b48fbcf79821cc827a5d7c4a0261bf6\n" }, { "commit": "762869dee6e0eadab5be1c606792d6693bbabf4e", "tree": "8c986c621e8a5f3cf4e4e3b2cc13b400401ad89b", "parents": [ "b4cf427734c6839b46d0d6037e3189a5e8aa1bdb" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jul 15 15:28:35 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 30 10:44:31 2016 +0100" }, "message": "Simplify our intrinsic recognizer.\n\n- Use the modifiers for storing the intrinsic kind.\n- Delete dex_file_method_inliner and its associated map.\n\nThis work was also motivated by the fact that the inline\nmethod analyzer leaks intrinsic tables, and even worse, might re-use\na table from one dex file to another unrelated dex file in the presence\nof class unloading and the unlikely event of the dex files getting\nthe same address.\n\ntest: m test-art-host m test-art-target\n\nChange-Id: Ia653d2c72df13889dc85dd8c84997582c034ea4b\n" }, { "commit": "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": "5e4e11e171f90d9a3ea178fc8e72aac909de55d5", "tree": "53314d1139ac797d55258f39097ecfb5cef45920", "parents": [ "ca8bad9136d1389deeebc8652fb17063388de6b2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Sep 22 13:17:41 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 23 09:08:00 2016 +0100" }, "message": "Clean-up sharpening and compiler driver.\n\nRemove dependency on compiler driver for sharpening\nand dex2dex (the methods called on the compiler driver were\ndoing unnecessary work), and remove the now unused methods\nin compiler driver.\n\nAlso remove test that is now invalid, as sharpening always\nsucceeds.\n\ntest: m test-art-host m test-art-target\nChange-Id: I54e91c6839bd5b0b86182f2f43ba5d2c112ef908\n" }, { "commit": "31b12e32073f458950e96d0d1b44e48508cf67e4", "tree": "b6f818dfa6b45342d6b69283c10e1db9817b8b0e", "parents": [ "038cb84f792501ae01561fe5ea4e8144f1918b7e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 02 17:11:57 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 07 10:13:48 2016 -0700" }, "message": "Avoid read barrier for image HLoadClass\n\nConcurrent copying baker:\n\nX86_64 core-optimizing-pic.oat: 28583112 -\u003e 27906824 (2.4% smaller)\n\nAround 0.4% of 2.4% is from re-enabling kBootImageLinkTimeAddress,\nkBootImageLinkTimePcRelative, and kBootImageAddress.\n\nN6P boot.oat 32: 73042140 -\u003e 71891956 (1.57% smaller)\nN6P boot.oat 64: 83831608 -\u003e 82531456 (1.55% smaller)\n\nEAAC: 1252 -\u003e 1245 (32 samples)\n\nBug: 29516974\n\nTest: test-art-host CC baker, N6P booting\n\nChange-Id: I9a196cf0157058836981c43c93872e9f0c4919aa\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": "ba65cc4a71273904294245cb37ce70e5bce797e3", "tree": "d53a7a816ec4c8e5bdbf80729ac945787b27bf2c", "parents": [ "1a827a05afbffd5bee241f245f9aa3c40b4dbae4", "542451cc546779f5c67840e105c51205a1b0a8fd" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Aug 02 03:47:35 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Aug 02 03:47:35 2016 +0000" }, "message": "Merge \"ART: Convert pointer size to enum\"" }, { "commit": "542451cc546779f5c67840e105c51205a1b0a8fd", "tree": "11e09bb5abaee12dddffefbe7e425291076dfa7a", "parents": [ "85c4a4b8c9eabfe16e4e49f9b4aa78c1bf4be023" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 26 09:02:02 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Aug 01 18:54:48 2016 -0700" }, "message": "ART: Convert pointer size to enum\n\nMove away from size_t to dedicated enum (class).\n\nBug: 30373134\nBug: 30419309\nTest: m test-art-host\nChange-Id: Id453c330f1065012e7d4f9fc24ac477cc9bb9269\n" }, { "commit": "f1aedb126ad1cb56dc7a38c8e07113295b123f58", "tree": "1ed73536af4ce231ffeb884bb9a464bb9b57ea00", "parents": [ "e3051bf60783fa18e6f8c7ac6f73091f05af5665" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jul 28 03:49:14 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jul 28 17:16:39 2016 +0100" }, "message": "Pass the right class loader when inlining.\n\nOtherwise, method and type resolution can resolve to the wrong\nthings and as a side effect update the dex cache with wrong data.\n\nbug:30403437\ntest:./art/test/run-test --host --jit --dev --no-prebuild 613\n\n(cherry picked from commit 0a210d9b108c87c0e7c1d430a92ce6fc89790c95)\n\nChange-Id: I8fdca96fccae079c8e99b8e86e7b6935acfce89d\n" }, { "commit": "e4b1c86d13c3f60362708f4a128b62db156f5fde", "tree": "4920c33f3510a21dba212650659174a7bb44c855", "parents": [ "ce1ba111bfa122247ac186e7d2c6cf5943ba28cd", "465ecc86ff65ca546629630c9469deb6d2d8137e" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Jul 21 00:04:52 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jul 21 00:04:52 2016 +0000" }, "message": "Merge \"Revert \"Revert \"Refactor GetIMTIndex\"\"\"" }, { "commit": "465ecc86ff65ca546629630c9469deb6d2d8137e", "tree": "3a0ed3f2d3ab9fd2cffec44a659036efab35fb4e", "parents": [ "bae13af2fc2fa89985d0466cedf155cb96767910" ], "author": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Tue Jul 19 21:32:52 2016 +0000" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Tue Jul 19 21:48:06 2016 +0000" }, "message": "Revert \"Revert \"Refactor GetIMTIndex\"\"\n\nOriginally reverted in order to revert\nhttps://android-review.googlesource.com/#/c/244190/\nbut can now be merged again.\n\nThis reverts commit d4ceecc85a5aab2ec23ea1bd010692ba8c8aaa0c.\n\nTest: m test-art-host\n\nChange-Id: Id9205f2b77a378fc0f06088e78c66e81a49f712d\n" }, { "commit": "491617a612c8a1c890e72d8ba780a151ddee8e11", "tree": "3f10b3af3363d70173f43a4235a782e166a5afe2", "parents": [ "2be946bbf995496fe56364d9b7c4957fcb6aeec5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jul 19 17:06:23 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jul 19 17:46:29 2016 +0100" }, "message": "JIT: Don\u0027t update the dex cache of another class loader.\n\nThis only works for properly delegating class loaders. But Java allows\nnon-delegating ones :(\n\nbug:29964720\n\nChange-Id: I8b785e6cdfe9a2b77322521a02b8e59ec332ad83\ntest:612-jit-dex-cache\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": "83e4903f9f966b52d758b52ac1d522e13fd43e60", "tree": "966df75b9d69f43bb18ece89b1ef16db6c546e45", "parents": [ "dadc075149eb322bf28ba35a75a70157147c1c30", "a62cb9bb6cb2278cb41ab0664191623e178c6a4f" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Jul 08 20:06:58 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jul 08 20:06:58 2016 +0000" }, "message": "Merge \"Revert \"Revert \"Optimize IMT\"\"\"" }, { "commit": "a62cb9bb6cb2278cb41ab0664191623e178c6a4f", "tree": "62263dac644e2d80a34a04f2649e4741a1bed6f4", "parents": [ "bb8d501c9bb882a8927c6ceda07bf9577e06c3e1" ], "author": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Thu Jun 30 09:18:25 2016 +0000" }, "committer": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Thu Jul 07 10:01:04 2016 +0000" }, "message": "Revert \"Revert \"Optimize IMT\"\"\n\nThis reverts commit 88f288e3564d79d87c0cd8bb831ec5a791ba4861.\n\nChange-Id: I49605d53692cbec1e2622e23ff2893fc51ed4115\n" }, { "commit": "5bf7bacc0993a2efd9b0765813928ff72ea4bb9f", "tree": "c8d6df8d62ea03664199420d2866cb66c3fdf945", "parents": [ "a5f0a1ab3f632f1e9c0bc048604aa0c0e98a4a93" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 14:18:23 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 15:50:18 2016 +0100" }, "message": "Revert \"Revert \"Inline and optimize interface calls.\"\"\n\nThis reverts commit 19dc255bf94a4229de8627a2079ee6f0e9005e2d.\n\nChange-Id: Ifc3a92280878d4db1b460d486137497b3456beae\n" }, { "commit": "19dc255bf94a4229de8627a2079ee6f0e9005e2d", "tree": "38ce4ec6ab0ae5ed99f16c695441135fd36d26dd", "parents": [ "d9faceaa8da92f4a56c2907de949081bd42faf79" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 14:01:50 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 14:01:50 2016 +0000" }, "message": "Revert \"Inline and optimize interface calls.\"\n\nWent ahead too quickly.\n\nThis reverts commit d9faceaa8da92f4a56c2907de949081bd42faf79.\n\nChange-Id: I1610deaf89b38037cf2786d135a59c48b10ced0c\n" }, { "commit": "d9faceaa8da92f4a56c2907de949081bd42faf79", "tree": "2efc00cfe8d5211e420174db81e732f8e92cbb33", "parents": [ "74c0d1bb67f9c6ee8306f0318ab7251d56dc99d6" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 10:19:23 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 06 13:30:13 2016 +0100" }, "message": "Inline and optimize interface calls.\n\n- Support for inlining methods with invoke-interface (the\n previous performance limitation is now fixed with the new\n ImtConflictTable).\n- Turn non inlineable invoke-interface into invoke-virtual to\n avoid any potential execution of the conflict trampoline.\n\ntest:609-checker-inline-interface\n\nChange-Id: Ibe33bc945deaada8fb78541e6e0bf034ebb52420\n" }, { "commit": "88f288e3564d79d87c0cd8bb831ec5a791ba4861", "tree": "dd051a7b2985d1af3fea91ad6000de4bdc701a19", "parents": [ "e36389f20c83083c0aaba2dcac0888951e55cae1" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:17:52 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:41:25 2016 +0000" }, "message": "Revert \"Optimize IMT\"\n\nBug: 29188168 (for initial CL)\nBug: 29778499 (reason for revert)\n\nThis reverts commit badee9820fcf5dca5f8c46c3215ae1779ee7736e.\n\nChange-Id: I32b8463122c3521e233c34ca95c96a5078e88848\n" }, { "commit": "e36389f20c83083c0aaba2dcac0888951e55cae1", "tree": "3fe74b637f8dc90e3cbc8ad3949f3f340236537f", "parents": [ "abf64415cf99a9a7ee048ae26c76f7bbe972a6b9", "d4ceecc85a5aab2ec23ea1bd010692ba8c8aaa0c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:40:14 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jun 29 08:40:14 2016 +0000" }, "message": "Merge \"Revert \"Refactor GetIMTIndex\"\"" }, { "commit": "d4ceecc85a5aab2ec23ea1bd010692ba8c8aaa0c", "tree": "982948a67a88a1f9a734c935f919f8d307969f48", "parents": [ "50706437d8216e41f0fea1e413cda7891324d397" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:39:47 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:39:47 2016 +0000" }, "message": "Revert \"Refactor GetIMTIndex\"\n\nI need to revert this to get https://android-review.googlesource.com/#/c/244190/ to cleanly revert. Matthew, do you mind rewriting it?\n\nThis reverts commit 50706437d8216e41f0fea1e413cda7891324d397.\n\nChange-Id: I5c1435f5dffb46dbb5b613b22adb88c7770304f2\n" }, { "commit": "94ab38f01dc2cf3ed0c6e73e2a6b594c14758d67", "tree": "c6da419571e7bb8d3c83ffb4a80e15042e3667a0", "parents": [ "9a8620e45e6eedadaa476322841e26858d10eeee" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Jun 21 17:48:19 2016 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jun 23 14:55:42 2016 +0100" }, "message": "ART: Run RTP if invoke inputs have more specific type\n\nMoving RTP inside the graph builder introduced a regression where\nreplacing the inner parameters with the actual arguments of the\nHInvoke would not build the inner graph with types more specific\nthan the method\u0027s signature.\n\nThis patch runs RTP on the inner graph again when it is detected\nthat RTP may improve typing precision.\n\nBug: 29595335\nChange-Id: I351babc8497c83c2fba589aa51f46eaa0b7ab33c\n" }, { "commit": "50706437d8216e41f0fea1e413cda7891324d397", "tree": "23c7c5d1198b8cdde6261198626cfe443eab59ba", "parents": [ "badee9820fcf5dca5f8c46c3215ae1779ee7736e" ], "author": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Tue Jun 14 11:31:04 2016 -0700" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Wed Jun 15 09:08:35 2016 -0700" }, "message": "Refactor GetIMTIndex\n\nThis allows us to more easily maintain and experiment with\ninterface method table indexing and hashing.\n\nChange-Id: I719920fae7490dcedcda7c1c36db225c2b8b16df\n" }, { "commit": "badee9820fcf5dca5f8c46c3215ae1779ee7736e", "tree": "982948a67a88a1f9a734c935f919f8d307969f48", "parents": [ "614968198625a6693666bdc1e5609e2f663f5638" ], "author": { "name": "Nelli Kim", "email": "nelli.kim@samsung.com", "time": "Fri May 13 13:08:53 2016 +0300" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Tue Jun 14 10:05:47 2016 -0700" }, "message": "Optimize IMT\n\n* Remove IMT for classes which do not implement interfaces\n* Remove IMT for array classes\n* Share same IMT\n\nSaved memory (measured on hammerhead):\nboot.art:\nTotal number of classes: 3854\nNumber of affected classes: 1637\nSaved memory: 409kB\n\nChrome (excluding classes in boot.art):\nTotal number of classes: 2409\nNumber of affected classes: 1259\nSaved memory: 314kB\n\nGoogle Maps (excluding classes in boot.art):\nTotal number of classes: 6988\nNumber of affected classes: 2574\nSaved memory: 643kB\n\nPerformance regression on benchmarks/InvokeInterface.java benchmark\n(measured timeCall10Interface)\n1st launch: 9.6%\n2nd launch: 6.8%\n\nChange-Id: If07e45390014a6ee8f3c1c4ca095b43046f0871f\n" }, { "commit": "0f35e0b26d8c7738ca7b284357afa9dafb659d47", "tree": "b84167d8b952ab19287f9d87edbdb4272149d28f", "parents": [ "fa288cff8689f882aa99696bc1c9b2e826730736", "1be7cbd532a41981019f1e89c8ecf404bc9af809" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Apr 29 13:57:19 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Apr 29 13:57:19 2016 +0000" }, "message": "Merge \"Remove invoke after inlining.\"" }, { "commit": "1be7cbd532a41981019f1e89c8ecf404bc9af809", "tree": "f027ec1b2835a47abf64faff263b79772e186b35", "parents": [ "945dc321df3e69bb59c834f6399d34bbfd624e5c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Apr 29 13:56:01 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Apr 29 13:56:01 2016 +0100" }, "message": "Remove invoke after inlining.\n\nbug:28460178\nChange-Id: I0ce00667aab1e02bab9e04a4cea7ee5d278f6a71\n" }, { "commit": "ffc87076dda9878cb2cc098149bae441d38b9268", "tree": "e587208d6a8e62532792add3e1ace6b4e6d73e0f", "parents": [ "97cbc9206e9adc473a90650ebdb5d620f517ff04" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Apr 20 14:22:09 2016 +0100" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Apr 28 16:15:04 2016 +0100" }, "message": "Split profile recording from jit compilation\n\nWe still use ProfileInfo objects to record profile information. That\ngives us the flexibility to add the inline caches in the future and the\nconvenience of the already implemented GC.\n\nIf UseJIT is false and SaveProfilingInfo true, we will only record the\nProfileInfo and never launch compilation tasks.\n\nBug: 27916886\n\n(cherry picked from commit e5de54cfab5f14ba0b8ff25d8d60901c7021943f)\n\nChange-Id: I68afc181d71447895fb12346c1806e99bcab1de2\n" }, { "commit": "e30ee0a74f7ab723e81591e8a9c6c3be7260c08b", "tree": "5ec317b236ddf293f3604ea2ad2ffee44de09880", "parents": [ "4e377e58066ccef401276d7d0985521e5e63d1fb", "93a18c5d4160f632ecdb92af099574e9c7098c49" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Apr 27 11:31:23 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Apr 27 11:31:23 2016 +0000" }, "message": "Merge \"Forbid HDeoptimize instructions in OSR methods.\"" }, { "commit": "93a18c5d4160f632ecdb92af099574e9c7098c49", "tree": "328e28e160fe894fb9487a625cbbfc4ab94c093c", "parents": [ "f7cda8088ec57ab1422f85f08df78e217a9f7094" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Apr 22 13:16:14 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Apr 27 10:18:48 2016 +0100" }, "message": "Forbid HDeoptimize instructions in OSR methods.\n\nOtherwise dominated instructions will assume something that\nisn\u0027t necessarily correct if coming from the interpreter.\n\nbug:28335959\nbug:28249238\nbug:28348878\nbug:28080135\n\nChange-Id: I842bd1c6a919aff48cf6048d2ea51cf2d40f3c1d\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": "456307a47336e3d6576ed6d8563b67573a4238d3", "tree": "35d7c3d213b942a562538a8d25129b4e27cc712f", "parents": [ "814ae9448d7a620a1855d53c7b69b1b377944a67" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 19 14:12:13 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 19 19:25:00 2016 +0100" }, "message": "Revert \"Revert \"Use dex cache from compilation unit in RTP.\"\"\n\nThe exposed issue has been fixed by\n https://android-review.googlesource.com/215877\n\nBug:28210356\n\nThis reverts commit 34d9b04d8d0006967486c0ad1b221e7b632652af.\n\nChange-Id: I5288c923e45d9ef3190dabb89738350a1212a60d\n" }, { "commit": "8b3f9b246d5bdbf67faeb2b872b75b8d72777bc0", "tree": "6cceffd4ee15b9c67f67083851769980cc8988a7", "parents": [ "c4d445a40bf00ab497b5e4d43a6b43eaafb5fa50" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Apr 06 11:22:12 2016 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Mon Apr 18 20:29:08 2016 -0700" }, "message": "Avoid constructing types with errors.\n\nBUG\u003d27626735\n\nRationale:\nDo not construct classes with a link error. Without this,\nthe error type thought it was Object (mirror\u0027s method\nIsObjectClass() returns true if there is no superclass).\n\nChange-Id: I55ca8cc8cfc042210edf748aab10da4c6e345980\n" }, { "commit": "34d9b04d8d0006967486c0ad1b221e7b632652af", "tree": "cf00b5f4af53b6eadef0c3e3f8c05f743176271e", "parents": [ "27bb86edf60e2f9ca2c1075c0c86b9e79374f1d0" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Apr 15 17:01:11 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Apr 15 17:01:11 2016 +0000" }, "message": "Revert \"Use dex cache from compilation unit in RTP.\"\n\nbug:28210356\n\nThis reverts commit 27bb86edf60e2f9ca2c1075c0c86b9e79374f1d0.\n\nChange-Id: Ib27ee90a7e4d516fd2db67a9c4e454023737841a\n" }, { "commit": "27bb86edf60e2f9ca2c1075c0c86b9e79374f1d0", "tree": "5e2cb0caa0b4456fd55c93f6c863bb3b26d5cd31", "parents": [ "0108f3d945cd5483217d7bf1df75ab5108d4555c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Apr 14 18:07:55 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Apr 14 18:45:01 2016 +0100" }, "message": "Use dex cache from compilation unit in RTP.\n\nAvoid calling the costly ClassLinker::FindDexCache() from\nreference type propagation when the dex cache from the\ncompilation unit will do, i.e. almost always. Compiling\nthe Nexus 5 boot image on host under perf(1) shows that\nthe FindDexCache() hits drop from about 0.2% to almost\nnothing, though enabling inlining for the boot image will\nincrease it a bit to 0.03% due to unavoidable calls from\nthe inliner.\n\nAlso clean up the ScopedObjectAccess usage a bit.\n\nChange-Id: I426a5f9f5da9e64fad2ea57654240789a48d3871\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": "a3aef2e42f4e88ae556432bd4a8040456a42bf74", "tree": "03261fe4fd630bb18840f0397dcf19475deb8129", "parents": [ "5827507da8efdaf4536c70dfdc54407c28e37852" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Apr 06 17:45:58 2016 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Apr 06 17:45:58 2016 +0100" }, "message": "Move optimizations out of HInliner::TryBuildAndInlineHelper.\n\nThis is a followup CL to\nhttps://android-review.googlesource.com/#/c/212651.\n\nChange-Id: I37d1c3fc06d8e36c335dd62887936486f7deeba4\n" }, { "commit": "7d9a5b0fbceef5806f059060d47533b9e699ec28", "tree": "411f4f2a709efdbf2b3d54cb40ee271061cb30bb", "parents": [ "75c221dcffab45e3c1e4ba8b8cd048c87497334f", "60328910cad396589474f8513391ba733d19390b" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Apr 04 17:48:02 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Apr 04 17:48:02 2016 +0000" }, "message": "Merge \"Revert \"Refactor HGraphBuilder and SsaBuilder to remove HLocals\"\"" }, { "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": "a8013fd4f2630aa45b5a3c320660ac9590523537", "tree": "b8bbdcd6999e728d5faae712044ca541b22c3aba", "parents": [ "75ee5c28ab583c532066b64d6337a5f78d6aa2ad" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Apr 04 15:34:31 2016 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon Apr 04 15:34:31 2016 +0100" }, "message": "Please Clang wrt stack frames in HInliner::TryBuildAndInlineHelper.\n\nThis change enables Clang to compile ART on MIPS64 without\ncomplaining about stack frames larger than what\n`-Wframe-larger-than` allows.\n\nChange-Id: I72f424922d9cb3d01f7b8ba4e5cb9170b82870d9\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": "0ced281ae6216c29f57ca0f8b7388a722e8da97b", "tree": "b7f8273bb117c8ec8f8546ed937a8c0a96d2e5de", "parents": [ "843a65556616183a36792bbcc1632c6d8d0e78b2", "1a65388f1d86bb232c2e44fecb44cebe13105d2e" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Mar 24 10:25:51 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Mar 24 10:25:51 2016 +0000" }, "message": "Merge \"Clean up art::HConstant predicates.\"" }, { "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": "ab5327d3d57d0d3561e697f196da7f2ee2e50290", "tree": "ab08ed0f15b8166140baeabd7d4c96239b6a9037", "parents": [ "e4914ec0d83544243257e55d0f0b9da3ed7f147a" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 18 11:36:20 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 21 11:23:28 2016 +0000" }, "message": "Check if the type of an instruction is erroneous before inlining.\n\nWe can get HInstructions typed with a class that is in an error state.\nFor such classes, we cannot look at the vtable or imt table as they\nare not cleanly populated.\n\nbug:27683927\n\nChange-Id: I0d64ca470e1cb6cf9b40e9f02bb9b0bb12c2bac1\n" }, { "commit": "1a65388f1d86bb232c2e44fecb44cebe13105d2e", "tree": "515e3000b3ad6d195101f20f33f3c9498e536593", "parents": [ "f808e8a0cc218c2b98023ef0e91f3c5b74ad2962" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Mar 18 18:05:57 2016 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Mar 18 18:05:57 2016 +0000" }, "message": "Clean up art::HConstant predicates.\n\n- Make the difference between arithmetic zero and zero-bit\n pattern non ambiguous.\n- Introduce Boolean predicates in art::HIntConstant for when\n they are used as Booleans.\n- Introduce aritmetic positive and negative zero predicates\n for floating-point constants.\n\nBug: 27639313\nChange-Id: Ia04ecc6f6aa7450136028c5362ed429760c883bd\n" }, { "commit": "a17245013e17621ff1a629ab164a1284e2258ad0", "tree": "f39105d2bb8c885cd0624c678159272473e308e2", "parents": [ "8b5e04b3573f430dc0475c17f4b3c6ada8aaccf5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 16 17:01:39 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 16 17:01:39 2016 +0000" }, "message": "Forgot space.\n\nChange-Id: If9eced6d5bd6d8d0248bad0e7b12095fb5a84d26\n" }, { "commit": "8b5e04b3573f430dc0475c17f4b3c6ada8aaccf5", "tree": "584efbc108d09817c62e45dbf5638a50bce58e00", "parents": [ "713c519db15aaa8d6f33b744fd28adddb97a07c2", "44fd0e5e9cc384783398b1f3ad85937aa856651c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 16 16:58:22 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Mar 16 16:58:22 2016 +0000" }, "message": "Merge \"Fix inliner crash related to type propagation.\"" }, { "commit": "44fd0e5e9cc384783398b1f3ad85937aa856651c", "tree": "60414c63cdc0d2d84abc92ad8eb9fa8379bc32b9", "parents": [ "767c3b9779cfa877867ba0e8b9f66a726cf24ecf" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 16 15:16:06 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 16 15:17:55 2016 +0000" }, "message": "Fix inliner crash related to type propagation.\n\n- Do not agressively try to resolve a return type.\n- Deal with unresolved return type.\n\nbug:25492619\n\nChange-Id: Idc9c96a0b376fe5ee86b411c02ce7078c7f48c84\n" }, { "commit": "6915898b28cea6c9836ca1be6814d87e89cc6d76", "tree": "89eb0f498a958b72d78c8aee25b4de212c25348b", "parents": [ "fbc61e19578d281d05728bcd120e1ace57c2fbd8" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Mar 16 11:53:41 2016 +0000" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Mar 16 14:07:22 2016 +0000" }, "message": "Improve compiler stats\n\n- report the max size of arena alloc\n- report how many virtual or interface invokes were inlined\n\nChange-Id: I82f154a8e25b5e3890181a1aa11346cdc3f93e37\n" }, { "commit": "07e3ca9a46801990d662c48d0fddedc63c4c053c", "tree": "94fedc96b28c6a46d2b83c80b2cfefd4c2898f4a", "parents": [ "b7f257f353b1eb2db2732939a0404c118316891d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 11 09:57:57 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 11 09:57:57 2016 +0000" }, "message": "Fix thread race when fetching the ProfilingInfo object.\n\nProblem is:\n1) Compiler fetches the ProfilingInfo of A, it\u0027s null.\n2) Mutator creates the ProfilingInfo.\n3) Compiler notifies it\u0027s not using A anymore, calls\n ProfilingInfo::DecrementInlineUse -\u003e Crash as we expected\n ProfilingInfo::IncrementUse to be called before.\n\nAlso update some namings to better reflect what is going on.\n\nChange-Id: I55ea4c5d81988131467095e18a0d13a8be9d0ef7\n" }, { "commit": "b6e20ae17d0881a66c22532e4152ce6779454a92", "tree": "35d7fbfb812aef11ea51010794dce26eb91c1ae4", "parents": [ "8a8c84114b16b8971bc2d3c5c7e0e31470d1a68b" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 07 14:29:04 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 09 17:25:19 2016 +0000" }, "message": "Clear inline caches at each full GC.\n\nThis fixes occasional failures of 141-class-unload.\n\nAlso fix a bug where clearing inline caches also cleared the dex\npc associated with it.\n\nbug:26846185\nbug:23128949\nChange-Id: I77bf1dee229d7764c3cc21440829c7fba7b37001\n" }, { "commit": "3f52306b259caed1c654c4b3fd5b594d5ec8d46c", "tree": "8a94e537ee9c17ce8591e1775fb232ee1af09425", "parents": [ "757701fc80dfe35ff9677c858b3d85e946e1ae36" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Feb 29 16:53:33 2016 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Mar 01 22:54:29 2016 +0000" }, "message": "ART: Fix overlapping instruction IDs in inliner\n\nInliner creates the inner graph so that it generates instruction IDs\nhigher than the outer graph. This was broken because the inliner\nwould create instructions in the outer graph before the inner graph\nis inlined.\n\nThe bug cannot be triggered because the offending instruction would\nshare the same ID as the first inner HLocal, which is removed before\nthe inner graph is inlined. The added DCHECKs reveal the hidden problem\nand make it safe for HLocals to be removed in the future.\n\nChange-Id: I486eb0f3987e20c50cbec0fb06332229e07fbae9\n" }, { "commit": "c3bcf40e905c53ec857ec40a9ae45f4c86738929", "tree": "aa03a2a6d0f70faa20f3579458abde24f71abb04", "parents": [ "fb734b416d20e5c3d1c29576f5cf16a003960b55", "a1d8ddfaf09545f99bc326dff97ab604d4574eb6" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 29 14:30:18 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Feb 29 14:30:18 2016 +0000" }, "message": "Merge \"Bug fix for polymorphic inlining.\"" }, { "commit": "4f97a212b6ff801eb9f7f88a87f44e0584339cca", "tree": "ed2f46d965cb5f5e87b93836f0d94e72ab2cfd71", "parents": [ "eebfed8b2fc4d6efa614f98001a0dc918bd225c0" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 25 16:17:54 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 29 14:29:22 2016 +0000" }, "message": "Cleanup polymorphic inlining to the same target.\n\n- Don\u0027t do expensive lookups, just fetch the ArtMethods from the\n imtable or vtable.\n- Don\u0027t perform the optimization if we may end up in the conflicting\n trampoline.\n\nbug:27398183\n\nChange-Id: I327c9f7c3a85d570a057ff8f18602e8e52402fdc\n" }, { "commit": "a1d8ddfaf09545f99bc326dff97ab604d4574eb6", "tree": "c4a9893ac195928397c91a84223e2e011cd059bd", "parents": [ "a48224f32797b234f07d78fbbede5edbb212ceab" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 29 11:46:58 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 29 12:17:52 2016 +0000" }, "message": "Bug fix for polymorphic inlining.\n\nThe code used to wrongly propagate try/catch information to\nnew blocks. Since it has the same logic as Hraph::InlineInto,\nextract the code that updates loop and try/catch information\nto blocks to a shared method.\n\nbug:27330865\nbug:27372101\nbug:27360329\n\nChange-Id: I4386f724d8d412bde5bcc04fda6955bc3bacf5a9\n" }, { "commit": "a1de9188a05afdecca8cd04ecc4fefbac8b9880f", "tree": "a671c8aef814ccf194e5c3950a551f2711516c53", "parents": [ "950d063395c7cecbbe372fd607468018d661a35c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 25 11:37:38 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Feb 26 16:01:59 2016 +0000" }, "message": "Optimizing: Reduce memory usage of HInstructions.\n\nPack narrow fields and flags into a single 32-bit field.\n\nChange-Id: Ib2f7abf987caee0339018d21f0d498f8db63542d\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": "1fe26e1ca54e2f0eed6056ebdbc6b2e6c02ee0bb", "tree": "b821021b98e5afae4c1feebc9a45631db427fed7", "parents": [ "1d01ef7f0f79b032449c35203ca9543fb8a0b22f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 16:55:42 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 16:55:42 2016 +0000" }, "message": "Fix build aflter conflicting changes got in.\n\nChange-Id: Ia3846e9a0dad137fe797bf3f2ece0cb78c106ec0\n" }, { "commit": "1d01ef7f0f79b032449c35203ca9543fb8a0b22f", "tree": "78e44decc10d0155ddec585bba5500451e2daacb", "parents": [ "afe84004583488ded7f99bb703ed4c1a601ff034", "e4084a5eb46dc6b99c0e0b74bcdecccaceb28fe7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 16:33:37 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Feb 18 16:33:37 2016 +0000" }, "message": "Merge \"Small inlining improvements.\"" }, { "commit": "afe84004583488ded7f99bb703ed4c1a601ff034", "tree": "7b30a6244f1331be3bcba4552e33ddb3eb488e39", "parents": [ "f8a08735f6c60ddee2bb5646dcad561361cce079", "38e9e8046ea2196284bdb4638771c31108a30a4a" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Feb 18 16:16:34 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Feb 18 16:16:35 2016 +0000" }, "message": "Merge \"Add statistics support for some optimizations\"" }, { "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": "e4084a5eb46dc6b99c0e0b74bcdecccaceb28fe7", "tree": "d47f5a4ff9faae1736fc065e82d39110b9261e61", "parents": [ "a1f65135cd2315159ac302f904ba0c5ba0d7fd0e" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 14:43:42 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 15:14:30 2016 +0000" }, "message": "Small inlining improvements.\n\n- Use the type_index in the current dex file for classes not\n defined in the current dex file.\n- Make the loading of the vtable field of a class have no side effects\n to enable gvn\u0027ing it.\n\nNote that those improvements only affect the JIT, where we don\u0027t have\nchecker support.\n\nChange-Id: I519f52bd8270f2b828f0920a1214da33cf788f41\n" }, { "commit": "916cc1d504f10a24f43b384e035fdecbe6a74b4c", "tree": "a68f7276cba6cd75bc0b200337edde8eafccccc7", "parents": [ "442643920a6c539e98aad76594e99b932b5631ba" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 11:12:31 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 14:41:25 2016 +0000" }, "message": "Implement polymorphic inlining.\n\nFor example, before:\nHInvokeVirtual\n\nAfter:\nIf (receiver \u003d\u003d Foo) {\n // inlined code.\n} else if (receiver \u003d\u003d Bar) {\n // inlined code\n} else {\n // HInvokeVirtual or HDeoptimize(receiver !\u003d Baz)\n}\n\nChange-Id: I5ce305aef8f39f8294bf2b2bcfe60e0dddcfdbec\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": "55bd749991f9a0a73f612696e1a93e739380546b", "tree": "7bfe5a6565e728468fa67cb9ce7c62907d9ce0f8", "parents": [ "7dc277b171b4d8294c4d152d8a6020995dc95f98" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 16 15:37:12 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 16 16:05:11 2016 +0000" }, "message": "Refactor the inliner.\n\nIn preparation for more polymorphic inlining, refactor the inliner\na bit.\n\nChange-Id: Ie3fd6c1ef205f1089989c67a527e6f57ff3c8b5d\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": "d9994f069dfeaa32ba929ca78816b5b83e2a4134", "tree": "a826990f89d8f546921902933a4cc05b57196971", "parents": [ "be89a6f1d9e41b154be3fe1da97cedb6964fbd35" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 11 17:35:55 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 11 21:47:46 2016 +0000" }, "message": "Re-enable OSR.\n\nFixes two bugs:\n- Dealing with proxy methods, which the compiler and code cache\n does not handle.\n- Dealing with phi types, that may have been speculatively optimized\n but do not hold once jumping to the compiled code.\n\nChange-Id: I7dcd9976ef7b12128fff95d2b7ed3e69cc42e90a\n" }, { "commit": "927f40d925e2106598d413ac02e8d80463357258", "tree": "b43a85e62460bb83a0e2e82deba969d6d896714f", "parents": [ "dfcd2f43a6694482a4e691261905508f65d8dc9e", "354efa6cdf558b2331e8fec539893fa51763806e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 10 09:44:52 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Feb 10 09:44:52 2016 +0000" }, "message": "Merge \"Try to substitute constructor chains for IPUTs.\"" }, { "commit": "354efa6cdf558b2331e8fec539893fa51763806e", "tree": "b2d91993fc46011d1eeb416d411bf16fb4d25c20", "parents": [ "519281001dabe9060e2bfefc2382570bab26c96a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 04 19:46:56 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Feb 09 19:57:41 2016 +0000" }, "message": "Try to substitute constructor chains for IPUTs.\n\nMatch a constructor chain where each constructor either\nforwards some or all of its arguments to the next (i.e.\nsuperclass constructor or a constructor in the same class)\nand may pass extra zeros (of any type, including null),\nfollowed by any number of IPUTs on \"this\", storing either\narguments or zeros, until we reach the contructor of\njava.lang.Object.\n\nWhen collecting IPUTs from the constructor chain, remove\nany IPUTs that store the same field as an IPUT that comes\nlater. This is safe in this case even if those IPUTs store\nvolatile fields because the uninitialized object reference\nwasn\u0027t allowed to escape yet. Also remove any IPUTs that\nstore zero values as the allocated object is already zero\ninitialized.\n\nChange-Id: If93022310bf04fe38ee741665ac4a65d4c2bb25f\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": "2aba7cd68851d79643338f57fcf2dff3cbf81895", "tree": "449f65420cddb1becdb8c7cbd573b3c0cf5e1f81", "parents": [ "85fdcda926fbab0e5a7b3e8b3541f5d225b2bcdb" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Feb 03 12:27:20 2016 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Feb 03 12:27:20 2016 +0000" }, "message": "Implement first kind of polymorphic inlining on MIPS.\n\nChange-Id: Ie430d6c5821273143a1872b376e8e24683d00ab0\n" }, { "commit": "adda435ebb3b678b1920a99d18d63c78031bee83", "tree": "1d9ea376c8f8ca72adc92927f5bdaa47e54a132d", "parents": [ "10cd5eec804167dd67b4fd1096108f0837e151c6" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jan 29 10:24:41 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jan 29 10:55:13 2016 +0000" }, "message": "Optimizing: Use dex pc 0 for pattern substitution-generated IGET.\n\nIn the read barrier configuration, HInstanceFieldGet needs\na slow-path with a runtime call with an associated stack map\nand we assert that we have a valid dex pc for stack maps.\nFix the pattern substitution-generated HInstanceFieldGet to\nuse the valid, if bogus, dex pc 0.\n\nBug: 26854537\nBug: 12687968\n\nChange-Id: I9f379ea530ce3f89af8db40169a6c41b525abbd7\n" }, { "commit": "be10e8e99a78caae01fb65769218800d465144ae", "tree": "cfa5cd3264137febdba6e49cae0393c779c6439c", "parents": [ "fa72bf1823a39b43552fa1b9ae572023830a4ac1" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jan 22 12:09:44 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jan 28 18:13:53 2016 +0000" }, "message": "Optimizing: Try pattern substitution when we cannot inline.\n\nChange-Id: I7c01f4494bac8498accc0f087044ec509fee4c98\n" }, { "commit": "a42363f79832a6e14f348514664dc6dc3edf9da2", "tree": "bcd43acdf9903a704b566af00b5c740786284b7b", "parents": [ "9cea9139033a4d04437ebc5542e9466fd67137fe" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 17 14:57:09 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 28 15:55:46 2016 +0000" }, "message": "Implement first kind of polymorphic inlining.\n\nAdd HClassTableGet to fetch an ArtMethod from the vtable or imt,\nand compare it to the only method the profiling saw.\n\nChange-Id: I76afd3689178f10e3be048aa3ac9a97c6f63295d\n" }, { "commit": "788f2f05c3e5b0e5bda247b00e34f0094585546f", "tree": "7e8b578b60bed6e550b62767f1fbc43651755798", "parents": [ "c24b8df48be848af1f4cb54e9caef2b7d6afe680" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 22 12:41:38 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 22 13:33:57 2016 +0000" }, "message": "Revert \"Revert \"Inline methods with loops.\"\"\n\nBug: 26689526\n\nThis reverts commit 451ad8d1be9a1949ea3c3e3a713a9e76198a8b2d.\n\nChange-Id: If484fe4c0744254dd7568fd5006e574d621a1855\n" }, { "commit": "451ad8d1be9a1949ea3c3e3a713a9e76198a8b2d", "tree": "4a1bb8da95ff9b971f2ef9f3941b645bcaf211d3", "parents": [ "82fc9bb45dbf8ff728122fb7ab72d1eb7b2f4869" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 20 21:23:30 2016 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 20 21:23:30 2016 +0000" }, "message": "Revert \"Inline methods with loops.\"\n\nThis reverts commit 82fc9bb45dbf8ff728122fb7ab72d1eb7b2f4869.\n\nLoop inlining exposes issues with BCE.\n\nBug: 26689526\nChange-Id: Id9983d7f9d3c5579d91e56e4699d4d939517b2dc\n" }, { "commit": "82fc9bb45dbf8ff728122fb7ab72d1eb7b2f4869", "tree": "3a22cf5bb2e0012723ecdc7640e4db0dd5be2f0a", "parents": [ "17ccfff2de292fd2b4a78aef87d79b662381f920" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 19 14:08:02 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 20 16:28:39 2016 +0000" }, "message": "Inline methods with loops.\n\nCompiling Gms/Fb/Framework/Docs:\n- Overall compilation-time increase: 2.2%\n- Overall code size increase: 1.1%\n\nPerformance improvements:\n- Richards with jit: +6%\n- Takl: +11%\n\nChange-Id: I0a6fcf2a360e5ad193cd95b5c4fe92227ac6bd96\n" }, { "commit": "7c0f2e55456c39f92f0136844ebf17b2bbcf9a97", "tree": "70cceb1203e173bb594667872b91b60d2e14311c", "parents": [ "7d6872a2471a20a10fc99b02459db89a5fc2dc1d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 18 15:24:53 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 18 15:39:39 2016 +0000" }, "message": "Do HFieldGet first to avoid explicit null check.\n\nChange-Id: If22f85d502e1dce428f8d341fcb64e27a886fb89\n" }, { "commit": "15bd22849ee6a1ffb3fb3630f686c2870bdf1bbc", "tree": "a261601589163faa4538bcf1c9d156e8ec4a42b3", "parents": [ "5b7b5ddb515828c93f0c2aec67aa513c32d0de22" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 05 15:55:41 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 14 15:00:20 2016 +0000" }, "message": "Implement irreducible loop support in optimizing.\n\nSo we don\u0027t fallback to the interpreter in the presence of\nirreducible loops.\n\nImplications:\n- A loop pre-header does not necessarily dominate a loop header.\n- Non-constant redundant phis will be kept in loop headers, to\n satisfy our linear scan register allocation algorithm.\n- while-graph optimizations, such as gvn, licm, lse, and dce\n need to know when they are dealing with irreducible loops.\n\nChange-Id: I2cea8934ce0b40162d215353497c7f77d6c9137e\n" }, { "commit": "fd2140f815bd173c9d399b2b012ea6d0075e77fa", "tree": "a3bca4f5ed222574999e9ef1c5dedc0ffed264d0", "parents": [ "802207cbe3a39a89bcdfe687897711e0c8eb9f29" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Dec 23 16:30:44 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 05 10:24:51 2016 -0800" }, "message": "ART: Make opt inliner a little bit cleaner/faster\n\nRemove the double-if.\n\nChange-Id: Ic6551e30d016d4b2c3b8dfb8841d2db90fce45cd\n" }, { "commit": "5949fa0cb9a8d26ac20b9b02065a63b4b20443be", "tree": "7e4760bac3073fdae84e850637aa35dd8c4c519c", "parents": [ "b7371a5517f78f61759f7e6124f2d957d974d9cd" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Dec 18 10:57:10 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 04 12:53:06 2016 +0000" }, "message": "Revert \"Revert \"Tweak inlining heuristics.\"\"\n\nThis reverts commit b17d1ccff0ac26fc22df671907ba2b4f4c656ce4.\n\nChange-Id: I26f6f8702a448c3da12662cbc6bc0f6e562bc40b\n" }, { "commit": "4833f5a1990c76bc2be89504225fb13cca22bedf", "tree": "8e096f222368f30bf821b154bc78bac12d5cd2e5", "parents": [ "bb41b250153308ef51664d7a6cc26c6a2e588fb7" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Dec 16 10:37:39 2015 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Dec 31 09:58:33 2015 +0000" }, "message": "ART: Refactor SsaBuilder for more precise typing info\n\nThis reverts commit 68289a531484d26214e09f1eadd9833531a3bc3c.\n\nNow uses Primitive::Is64BitType instead of Primitive::ComponentSize\nbecause it was incorrectly optimized by GCC.\n\nBug: 26208284\nBug: 24252151\nBug: 24252100\nBug: 22538329\nBug: 25786318\n\nChange-Id: Ib39f3da2b92bc5be5d76f4240a77567d82c6bebe\n" }, { "commit": "dcdc85bbd569f0ee66c331b4219c19304a616214", "tree": "b5ab789248e279318f6c1e3f6c511703d7294476", "parents": [ "48944c760b196188b968b7af81439466cf987a75" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Dec 04 14:06:18 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Dec 23 21:14:23 2015 -0800" }, "message": "Dex2oat support for multiple oat file and image file outputs.\n\nMultiple changes to dex2oat and the runtime to support a --multi-image\noption. This generates a separate oat file and image file output for\neach dex file input.\n\nChange-Id: Ie1d6f0b8afa8aed5790065b8c2eb177990c60129\n" }, { "commit": "8e006762917a2ee193e12c74ba24e69a5e1b9144", "tree": "c5a7099c24ab7ac7782e7bbf64f26c75b2350216", "parents": [ "0ac7a8e505e14e65ff989c483a40348a3d4b6a86", "b17d1ccff0ac26fc22df671907ba2b4f4c656ce4" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 17 15:26:44 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Dec 17 15:26:44 2015 +0000" }, "message": "Merge \"Revert \"Tweak inlining heuristics.\"\"" }, { "commit": "b17d1ccff0ac26fc22df671907ba2b4f4c656ce4", "tree": "cf202a0e783e523c9a0e4d8b25f0dc7a1d4df118", "parents": [ "fcb7613d3aaa9a6802800b6e957aaad51cedf6dc" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 17 15:26:21 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 17 15:26:21 2015 +0000" }, "message": "Revert \"Tweak inlining heuristics.\"\n\nThis reverts commit fcb7613d3aaa9a6802800b6e957aaad51cedf6dc.\n\nChange-Id: Idc0df6a2f68e8b5aa740bb1259f19c2953811510\n" }, { "commit": "ff6ab45547a629b1e237a1bf4b8530a5c99377e5", "tree": "65f58353143515441c2097edee5d68bc8d677cd4", "parents": [ "2dbab9548b538cdb647e5819f7f93d43ae67116a", "fcb7613d3aaa9a6802800b6e957aaad51cedf6dc" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 17 13:59:47 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Dec 17 13:59:47 2015 +0000" }, "message": "Merge \"Tweak inlining heuristics.\"" }, { "commit": "fcb7613d3aaa9a6802800b6e957aaad51cedf6dc", "tree": "9ae15e4bfb0b3d90d69bc8538ba351c5ce9311bf", "parents": [ "ad94d64719c2a82cdd312b318b8e50d80e2956c5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 17 12:43:00 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Dec 17 12:52:39 2015 +0000" }, "message": "Tweak inlining heuristics.\n\ngo/lem driven:\n\nPerformance:\nRichards +41%\nCaffeineMethod +43%\nReversiBench: +52%\nTowers: +73%\nTak: +85%\n\nMemory use: 7% less memory\nCompileTime: 14% increase\nCodeSize: 8% increase\n\nLast three measures are now more acceptable given we JIT.\n\nChange-Id: Ic4aa6535d2b76cf3545ef00e9b2ae32330f10745\n" }, { "commit": "68289a531484d26214e09f1eadd9833531a3bc3c", "tree": "6f87852b9d14e479ea2c7ef92de35c3118a0fd1e", "parents": [ "bc90a0538e56f98b8e138cb622e6b9d834244ad9" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Dec 15 17:30:30 2015 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Dec 15 17:40:08 2015 -0800" }, "message": "Revert \"ART: Refactor SsaBuilder for more precise typing info\"\n\nThis reverts commit d9510dfc32349eeb4f2145c801f7ba1d5bccfb12.\n\nBug: 26208284\n\nBug: 24252151\nBug: 24252100\nBug: 22538329\nBug: 25786318\n\nChange-Id: I5f491becdf076ff51d437d490405ec4e1586c010\n" }, { "commit": "d9510dfc32349eeb4f2145c801f7ba1d5bccfb12", "tree": "9f4e4ffb5fae25c4f14059fd1d772726e9d96170", "parents": [ "ceec5a3c81925cf80a18954b2e585316450c575c" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Nov 04 23:30:22 2015 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Dec 14 13:03:19 2015 +0000" }, "message": "ART: Refactor SsaBuilder for more precise typing info\n\nThis patch refactors the SsaBuilder to do the following:\n\n1) All phis are constructed live and marked dead if not used or proved\nto be conflicting.\n\n2) Primitive type propagation, now not a separate pass, identifies\nconflicting types and marks corresponding phis dead.\n\n3) When compiling --debuggable, DeadPhiHandling used to revive phis\nwhich had only environmental uses but did not attempt to resolve\nconflicts. This pass was removed as obsolete and is now superseded\nby primitive type propagation (identifying conflicting phis) and\nSsaDeadPhiEliminiation (keeping phis live if debuggable + env use).\n\n4) Resolving conflicts requires correct primitive type information\non all instructions. This was not the case for ArrayGet instructions\nwhich can have ambiguous types in the bytecode. To this end,\nSsaBuilder now runs reference type propagation and types ArrayGets\nfrom the type of the input array.\n\n5) With RTP being run inside the SsaBuilder, it is not necessary to\nrun it as a separate optimization pass. Optimizations can now assume\nthat all instructions of type kPrimNot have reference type info after\nSsaBuilder (with the exception of NullConstant).\n\n6) Graph now contains a reference type to be assigned to NullConstant.\nAll reference type instructions therefore have RTI, as now enforced\nby the SsaChecker.\n\nBug: 24252151\nBug: 24252100\nBug: 22538329\nBug: 25786318\n\nChange-Id: I7a3aee1ff66c82d64b4846611c547af17e91d260\n" }, { "commit": "73be1e8f8609708f6624bb297c9628de44fd8b6f", "tree": "47a22745797a345dea8dd846aad00334da0d5bf1", "parents": [ "47229aa5848df7d45578dbdd9285f57dfa9399a8" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Sep 17 15:22:56 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Dec 04 12:08:02 2015 +0000" }, "message": "Inline monomorphic calls.\n\nChange-Id: If38171c2dc7d4a4378df5d050afc4fff4499c98f\n" } ], "next": "e523423a053af5cb55837f07ceae9ff2fd581712" }