)]}' { "log": [ { "commit": "28e012a4af2d710e5e5f824709ffd6432e4f549f", "tree": "576ebdbff9d5f9f098cd29d652215b8f67b6b042", "parents": [ "4388fb213ec746ee18a6bea38ee894f8c19990b9" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Dec 07 11:22:59 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Dec 08 15:27:27 2017 +0000" }, "message": "Determine HLoadClass/String load kind early.\n\nThis helps save memory by avoiding the allocation of\nHEnvironment and related objects for AOT references to\nboot image strings and classes (kBootImage* load kinds)\nand also for JIT references (kJitTableAddress).\n\nCompiling aosp_taimen-userdebug boot image, the most memory\nhungry method BatteryStats.dumpLocked() needs\n - before:\n Used 55105384 bytes of arena memory...\n ...\n UseListNode 10009704\n Environment 423248\n EnvVRegs 20676560\n ...\n - after:\n Used 50559176 bytes of arena memory...\n ...\n UseListNode 8568936\n Environment 365680\n EnvVRegs 17628704\n ...\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing --jit\nBug: 34053922\nChange-Id: I68e73a438e6ac8e8908e6fccf53bbeea8a64a077\n" }, { "commit": "d57fdd82fccfa15933033646c9436313b399d528", "tree": "20446e27ec69fa1f2588ac166a85b4e25f9cf861", "parents": [ "6f2960c633d6928fe8491a4bbb1e1eae82f25188", "dbb9aef046301940d0b253c918a5c78b277330ba" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 04 17:52:28 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Dec 04 17:52:28 2017 +0000" }, "message": "Merge \"Log at places we fail to compile.\"" }, { "commit": "dbb9aef046301940d0b253c918a5c78b277330ba", "tree": "8803fa971c0200d56aa8eba7386374eecda01c17", "parents": [ "65e0432cdaf3ee170045b21943c24eae5d6a0357" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Nov 23 10:44:11 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Dec 01 13:03:29 2017 +0000" }, "message": "Log at places we fail to compile.\n\nUseful when diagnosing some compiler issues / limitations.\n\nTest: test.py\n\nChange-Id: I8759d0e78b0682b300ddcadfe02793432cab2036\n" }, { "commit": "75bb2f3c85330b2aeba9e0a4a25f7eb059bcd754", "tree": "2e9c2884dc546540c66b057374822a45f7d9a414", "parents": [ "1f541bf57dfdfe4faf169b811aadb91298a886ca" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Nov 30 14:45:44 2017 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Nov 30 14:45:44 2017 -0800" }, "message": "Type conversion elimination of constants\n\nA better way of eliminating type conversion for constants.\n\nTest: run-test on host. 711-checker-type-conversion.\nChange-Id: I457bc091542a5ac4cc4e77cadb012ee7cb040ce8\n" }, { "commit": "68351ebf5b44989f7ccd75933f1f4093a8d3b670", "tree": "a0716ffc78d9bd115eca545e70c4ada0facbf3a8", "parents": [ "b360bff818ad0bf59668cd2bebaaeeaa8a3b5dfe", "cced8ba4245a061ab047a0a6882468d75d619dd9" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Tue Nov 21 16:56:28 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Nov 21 16:56:28 2017 +0000" }, "message": "Merge \"ART: Introduce individual HInstruction cloning.\"" }, { "commit": "46721ef33e8f5cd405c291d72e3f259e3085fb5f", "tree": "88709396bf7439400125f5d5c1883a42826cd5ca", "parents": [ "6b337da6e9498e62b4c3e399366be7ad3b99169e" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Oct 05 14:45:17 2017 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Nov 16 09:10:32 2017 -0800" }, "message": "Don\u0027t merge values for exit block in LSE.\n\nThis enables some additional optimizations since exit block doesn\u0027t\nreally merge values.\n\nTest: run-test on host.\nChange-Id: I21ed7e0e43a3bc5d9ed2dabfad8462129b904eb7\n" }, { "commit": "cced8ba4245a061ab047a0a6882468d75d619dd9", "tree": "b379abfa48689c108e1cacedd2b13d4b5394baf2", "parents": [ "96c76457d5c5af2d4243c78d74ada77de3223d88" ], "author": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Wed Jul 19 18:18:09 2017 +0100" }, "committer": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Tue Nov 07 12:56:17 2017 +0000" }, "message": "ART: Introduce individual HInstruction cloning.\n\nIntroduce API for HInstruction cloning, support it for a few\ninstructions. add a gtest.\n\nTest: cloner_test.cc, test-art-target, test-art-host.\n\nChange-Id: I8b6299be5d04a26390d9ef13a20ce82ee5ae4afe\n" }, { "commit": "69d310e0317e2fce97bf8c9c133c5c2c0332e61d", "tree": "fba05a1530e6fc4a2e6950303c1f7c6b0ffbb936", "parents": [ "e764d2e50c544c2cb98ee61a15d613161ac6bd17" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 09 14:12:23 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Oct 11 09:44:26 2017 +0100" }, "message": "Use ScopedArenaAllocator for building HGraph.\n\nMemory needed to compile the two most expensive methods for\naosp_angler-userdebug boot image:\n BatteryStats.dumpCheckinLocked() : 21.1MiB -\u003e 20.2MiB\n BatteryStats.dumpLocked(): 42.0MiB -\u003e 40.3MiB\nThis is because all the memory previously used by the graph\nbuilder is reused by later passes.\n\nAnd finish the \"arena\"-\u003e\"allocator\" renaming; make renamed\nallocator pointers that are members of classes const when\nappropriate (and make a few more members around them const).\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nBug: 64312607\nChange-Id: Ia50aafc80c05941ae5b96984ba4f31ed4c78255e\n" }, { "commit": "ca6fff898afcb62491458ae8bcd428bfb3043da1", "tree": "195a6b16d3a4b34acc2faf91ce56f448efb15e07", "parents": [ "aa7273e56fbafc2692c8d20a31b50d2f4bdd2aa1" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 03 14:49:14 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Oct 06 17:53:50 2017 +0100" }, "message": "ART: Use ScopedArenaAllocator for pass-local data.\n\nPasses using local ArenaAllocator were hiding their memory\nusage from the allocation counting, making it difficult to\ntrack down where memory was used. Using ScopedArenaAllocator\nreveals the memory usage.\n\nThis changes the HGraph constructor which requires a lot of\nchanges in tests. Refactor these tests to limit the amount\nof work needed the next time we change that constructor.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: Build with kArenaAllocatorCountAllocations \u003d true.\nBug: 64312607\nChange-Id: I34939e4086b500d6e827ff3ef2211d1a421ac91a\n" }, { "commit": "d5d2f2ce627aa0f6920d7ae05197abd1a396e035", "tree": "e8e780780c832e3614a22438a23fb60ee4960ca3", "parents": [ "efac0df8c738764823c637deeca1f3be33912064" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Sep 26 12:37:26 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 03 10:40:51 2017 +0100" }, "message": "ART: Introduce Uint8 compiler data type.\n\nThis CL adds all the necessary codegen for the Uint8 type\nbut does not add code transformations that use that code.\nVectorization codegens are modified to use Uint8 as the\npacked type when appropriate. The side effects are now\ndisconnected from the instruction\u0027s type after the graph has\nbeen built to allow changing HArrayGet/H*FieldGet/HVecLoad\nto use a type different from the underlying field or array.\n\nNote: HArrayGet for String.charAt() is modified to have\nno side effects whatsoever; Strings are immutable.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing --jit\nTest: testrunner.py --target --optimizing on Nexus 6P\nTest: Nexus 6P boots.\nBug: 23964345\nChange-Id: If2dfffedcfb1f50db24570a1e9bd517b3f17bfd0\n" }, { "commit": "0ebe0d83138bba1996e9c8007969b5381d972b32", "tree": "a5ee66ebc5b587ade97e56ac8fc7d832fbbed4af", "parents": [ "e1e347dace0ded83774999bb26c37527dcdb1d5a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 21 22:50:39 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Sep 25 15:45:01 2017 +0100" }, "message": "ART: Introduce compiler data type.\n\nReplace most uses of the runtime\u0027s Primitive in compiler\nwith a new class DataType. This prepares for introducing\nnew types, such as Uint8, that the runtime does not need\nto know about.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nBug: 23964345\nChange-Id: Iec2ad82454eec678fffcd8279a9746b90feb9b0c\n" }, { "commit": "9d9f2d7782545082a22b3630ec75aa5a7e887582", "tree": "1210668cbd5b35a0d1f6c1f83f766979c7a780b2", "parents": [ "fe30b3e2815c6140471338df5d903a2c1d242414", "94ec2db21332ee1dcdbbf254b99a9a999a304fe0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Sep 11 16:55:32 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Sep 11 16:55:32 2017 +0000" }, "message": "Merge \"Use mmapped boot image class table for PIC app HLoadClass.\"" }, { "commit": "94ec2db21332ee1dcdbbf254b99a9a999a304fe0", "tree": "6ced7e596731b61f95a3693f336527f55ea3cf3a", "parents": [ "6cfbdbc359ec5414d3e49f70d28f8c0e65b98d63" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Sep 06 17:21:03 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Sep 11 15:12:51 2017 +0100" }, "message": "Use mmapped boot image class table for PIC app HLoadClass.\n\nImplement new HLoadClass load kind for boot image classes\nreferenced by PIC-compiled apps (i.e. prebuilts) that uses\nPC-relative load from a boot image ClassTable mmapped into\nthe apps .bss. This reduces the size of the PIC prebuilts\nthat reference boot image classes compared to the kBssEntry\nas we can completely avoid the slow path and stack map\nunless we need to do the class initialization check.\n\nPrebuilt services.odex for aosp_angler-userdebug (arm64):\n - before: 20312800\n - after: 19775352 (-525KiB)\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: testrunner.py --host --pictest\nTest: testrunner.py --target on Nexus 6P.\nTest: testrunner.py --target --pictest on Nexus 6P.\nTest: Nexus 6P boots.\nBug: 31951624\nChange-Id: I13adb19a1fa7d095a72a41f09daa6101876e77a8\n" }, { "commit": "86ce50481f91e3be2e5f2686e18e11babde721ae", "tree": "3179155108b883d8c0c4bb3dbac0b70db0d7f698", "parents": [ "88f929ce72a5eabdb1ae1b32e5ec157d0e9f1ef8", "dd018df8a00e841fe38fabe38520b7d297a885c1" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Sep 08 16:16:46 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Sep 08 16:16:46 2017 +0000" }, "message": "Merge \"optimizing: add block-scoped constructor fence merging pass\"" }, { "commit": "dd018df8a00e841fe38fabe38520b7d297a885c1", "tree": "4974b4f718d9cbb5910df7f476f2581aff1a17e2", "parents": [ "6ef45677305048c2bf0600f1c4b98a11b2cfaffb" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Aug 09 10:38:31 2017 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Sep 08 14:06:32 2017 +0000" }, "message": "optimizing: add block-scoped constructor fence merging pass\n\nIntroduce a new \"Constructor Fence Redundancy Elimination\" pass.\nThe pass currently performs local optimization only, i.e. within instructions\nin the same basic block.\n\nAll constructor fences preceding a publish (e.g. store, invoke) get\nmerged into one instruction.\n\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nOptStat#ConstructorFenceGeneratedNew: 43825\nOptStat#ConstructorFenceGeneratedFinal: 17631 \u003c+++\nOptStat#ConstructorFenceRemovedLSE: 164\nOptStat#ConstructorFenceRemovedPFRA: 9391\nOptStat#ConstructorFenceRemovedCFRE: 16133 \u003c---\n\nRemoves ~91.5% of the \u0027final\u0027 constructor fences in RitzBenchmark:\n\n(We do not distinguish the exact reason that a fence was created, so\nit\u0027s possible some \"new\" fences were also removed.)\n\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nTest: art/test/run-test --host --optimizing 476-checker-ctor-fence-redun-elim\nBug: 36656456\nChange-Id: I8020217b448ad96ce9b7640aa312ae784690ad99\n" }, { "commit": "6cfbdbc359ec5414d3e49f70d28f8c0e65b98d63", "tree": "f92b309ddc43c2254b6067346a653170fbbf7316", "parents": [ "0f3c7003e08a42a4ed8c9f8dfffb1bee1118de59" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jul 25 13:26:39 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 07 17:52:35 2017 +0100" }, "message": "Use mmapped boot image intern table for PIC app HLoadString.\n\nImplement new HLoadString load kind for boot image strings\nreferenced by PIC-compiled apps (i.e. prebuilts) that uses\nPC-relative load from a boot image InternTable mmapped into\nthe apps .bss. This reduces the size of the PIC prebuilts\nthat reference boot image strings compared to the kBssEntry\nas we can completely avoid the slow path and stack map.\n\nWe separate the InternedStrings and ClassTable sections of\nthe boot image (.art) file from the rest, aligning the\nstart of the InternedStrings section to a page boundary.\nThis may actually increase the size of the boot image file\nby a page but it also allows mprotecting() these tables as\nread-only. The ClassTable section is included in\nanticipation of a similar load kind for HLoadClass.\n\nPrebuilt services.odex for aosp_angler-userdebug (arm64):\n - before: 20862776\n - after: 20308512 (-541KiB)\nNote that 92KiB savings could have been achieved by simply\navoiding the read barrier, similar to the HLoadClass flag\nIsInBootImage(). Such flag is now unnecessary.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: testrunner.py --host --pictest\nTest: testrunner.py --target on Nexus 6P.\nTest: testrunner.py --target --pictest on Nexus 6P.\nTest: Nexus 6P boots.\nBug: 31951624\nChange-Id: I5f2bf1fc0bb36a8483244317cfdfa69e192ef6c5\n" }, { "commit": "4ec14ff3d6126750de753f7d162df408f278e825", "tree": "9f37b3881e615211a4fd458ee36d67939c6da514", "parents": [ "c1bb1cd339b2ebea9c4770fb4d61bacd7d77746f", "16e528957869c7debb1f6758c9a364819e15ee1a" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Aug 14 09:19:03 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Aug 14 09:19:03 2017 +0000" }, "message": "Merge \"RFC: Generate select instruction for conditional returns.\"" }, { "commit": "e9b61bac34bea439d8ce39592973a66a32f43fb7", "tree": "aa0043f331844ba6083b764c7bce8c2a81671058", "parents": [ "675c779cb046bca49229e1e5268d0eb622159214", "6ef45677305048c2bf0600f1c4b98a11b2cfaffb" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Aug 11 19:29:31 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Aug 11 19:29:31 2017 +0000" }, "message": "Merge changes Ic119441c,I83b96b41\n\n* changes:\n optimizing: Add statistics for # of constructor fences added/removed\n optimizing: Refactor statistics to use OptimizingCompilerStats helper\n" }, { "commit": "6ef45677305048c2bf0600f1c4b98a11b2cfaffb", "tree": "9a8df6a3bebe4a6120403562c21817d775a6ef36", "parents": [ "1e065a54845da12541572f4f149e6ab0dcd20180" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Aug 08 13:59:55 2017 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Aug 11 10:23:30 2017 -0700" }, "message": "optimizing: Add statistics for # of constructor fences added/removed\n\nStatistics are attributed as follows:\n\nAdded because:\n* HNewInstances requires a HConstructorFence following it.\n* HReturn requires a HConstructorFence (for final fields) preceding it.\n\nRemoved because:\n* Optimized in Load-Store-Elimination.\n* Optimized in Prepare-For-Register-Allocation.\n\nTest: art/test.py\nBug: 36656456\nChange-Id: Ic119441c5151a5a840fc6532b411340e2d68e5eb\n" }, { "commit": "16e528957869c7debb1f6758c9a364819e15ee1a", "tree": "050d42754ce9bca3c3e23652093cd209ae59ea53", "parents": [ "1d2d8b1b6b359f5f476ef78662baa4b8782be530" ], "author": { "name": "Mads Ager", "email": "ager@google.com", "time": "Fri Jul 14 13:11:37 2017 +0200" }, "committer": { "name": "Mads Ager", "email": "ager@google.com", "time": "Thu Aug 10 14:04:04 2017 +0200" }, "message": "RFC: Generate select instruction for conditional returns.\n\nThe select generator currently only inserts select instructions\nif there is a diamond shape with a phi.\n\nThis change extends the select generator to also deal with the\npattern:\n\n if (condition) {\n movable instruction 0\n return value0\n } else {\n movable instruction 1\n return value1\n }\n\nwhich it turns into:\n\n moveable instruction 0\n moveable instruction 1\n return select (value0, value1, condition)\n\nTest: 592-checker-regression-bool-input\nChange-Id: Iac50fb181dc2c9b7619f28977298662bc09fc0e1\n" }, { "commit": "5aedbcd3d0a1a60643db3e904254e0a3060599bf", "tree": "3d136637c0ef0cb12093d6f412fe1ec7e7a26e7e", "parents": [ "c7c25d5b4cf243cbbf06f2e4302a0faa5eccb42a", "c73ee37b76494253862ee17933acfe2b88de1a01" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Aug 07 17:23:38 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Aug 07 17:23:38 2017 +0000" }, "message": "Merge \"ART: Fix loop header\u0027s predecessors reordering in SimplifyLoops.\"" }, { "commit": "c73ee37b76494253862ee17933acfe2b88de1a01", "tree": "f88685e10a1075748c7634c8ec913544f2b12068", "parents": [ "399492a86198b42fded9ac1f4aa61c82869328eb" ], "author": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Mon Jul 31 15:08:40 2017 +0100" }, "committer": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Thu Aug 03 17:12:31 2017 +0100" }, "message": "ART: Fix loop header\u0027s predecessors reordering in SimplifyLoops.\n\nFix the issue when after loop header\u0027s predecessors reordering in\nSimplifyLoops phi inputs are not reordered correspondingly.\n\nTest: loop_optimization_test.cc, test-art-host, test-art-target.\n\nChange-Id: I8a251a0a953d751f9bb67da58181e47d225d90e6\n" }, { "commit": "21c7e6fbcabef2f22b467e1e89f4abe1aa43e459", "tree": "1a62fe2485d9642ee3d8768c6edd005278b1014f", "parents": [ "2e53f8f69f8c4175085e337445ec42aa045a2f7f" ], "author": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Thu Jul 27 16:04:42 2017 +0100" }, "committer": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Tue Aug 01 11:35:23 2017 +0100" }, "message": "ART: Fix SimplifyInduction for an instruction with HEnvironment.\n\nAfter an instruction is removed during RemoveFromCycle its\nenvironment isn\u0027t properly cleaned: it still has input instructions\npresent and registered (those instructions still hold records for\nthat).\n\nTest: test-art-target, test-art-host.\nChange-Id: Iea315bdf735d75fe477f43671f05b40dfecc63a8\n" }, { "commit": "8cf9cb386cd9286d67e879f1ee501ec00d72a4e1", "tree": "88e86e214b425e444760fe4e0ffeee677e1558a2", "parents": [ "914b7b6a6c9f399b26b41e9160e9871ef749e0db" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 19 09:28:38 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 24 16:07:10 2017 -0700" }, "message": "ART: Include cleanup\n\nLet clang-format reorder the header includes.\n\nDerived with:\n\n* .clang-format:\n BasedOnStyle: Google\n IncludeIsMainRegex: \u0027(_test|-inl)?$\u0027\n\n* Steps:\n find . -name \u0027*.cc\u0027 -o -name \u0027*.h\u0027 | xargs sed -i.bak -e \u0027s/^#include/ #include/\u0027 ; git commit -a -m \u0027ART: Include cleanup\u0027\n git-clang-format -style\u003dfile HEAD^\n manual inspection\n git commit -a --amend\n\nTest: mmma art\nChange-Id: Ia963a8ce3ce5f96b5e78acd587e26908c7a70d02\n" }, { "commit": "a4b58ed1ef8dceafbffcfa88bf2c11144e302d18", "tree": "c1a931ba1bc1c710d1cdad9291c57aaa6b373597", "parents": [ "b96ed2c271a56fb8be0c8f30231710095e66a201" ], "author": { "name": "George Burgess IV", "email": "gbiv@google.com", "time": "Thu Jun 22 15:47:25 2017 -0700" }, "committer": { "name": "George Burgess IV", "email": "gbiv@google.com", "time": "Thu Jun 22 15:47:25 2017 -0700" }, "message": "Fix static analyzer warning\n\nnodes.cc: warning: Access to field \u0027next_\u0027 results in a dereference of a\nnull pointer (loaded from field \u0027last_instruction_\u0027)\n\nThis was split from\nhttps://android-review.googlesource.com/#/c/416101/. Please see the\ndiscussion nodes.cc (patch set 1) for why this warning is triggered.\n\nBug: 32619234\nTest: test-art-host. Rebuilt ART with the analyzer to verify that these\nissues are gone.\n\nChange-Id: Id5da00ceee0667441233153a7971d238ea8c8650\n" }, { "commit": "0eb882bfc5d260e8014c26adfda11602065aa5d8", "tree": "e66dbebfb1e9a254c20954a2f2f98541aebfd5af", "parents": [ "b5f5d746ac3f2c3088292395603cb1470e7749d2" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon May 15 13:39:18 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jun 07 15:39:24 2017 +0100" }, "message": "Use ArtMethod* .bss entries for HInvokeStaticOrDirect.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: testrunner.py --target\nTest: Nexus 6P boots.\nTest: Build aosp_mips64-userdebug.\nBug: 30627598\nChange-Id: I0e54fdd2e91e983d475b7a04d40815ba89ae3d4f\n" }, { "commit": "3c8a91250b3e4e87548ec16bf1ab1ea46dbb84a4", "tree": "b5da100b358d1335eab403372e4f616c5c2d607c", "parents": [ "0a87f31513e5f9da27856af054d2241452898b22", "e7197bf7d58c705a048e13e241d7ca320502cd40" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jun 07 10:38:10 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jun 07 10:38:11 2017 +0000" }, "message": "Merge \"Replace invoke kind kDexCacheViaMethod with kRuntimeCall.\"" }, { "commit": "e7197bf7d58c705a048e13e241d7ca320502cd40", "tree": "496032b40145660b40002b9d5b7a78f1c2eeb44f", "parents": [ "4ee8e291a7d5b7b98f35f495eb97705836910871" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 02 17:00:23 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jun 06 15:26:42 2017 +0100" }, "message": "Replace invoke kind kDexCacheViaMethod with kRuntimeCall.\n\nIn preparation for replacing the dex cache method array\nwith a hash-based array, get rid of one unnecessary use.\nThis method load kind is currently used only on mips for\nirreducible loops and OSR, so this should have no impact\non x86/x86-64/arm/arm64.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: Repeat the above tests with manually changing\n kDexCachePcRelative to kRuntimeCall in sharpening.cc.\n (Ignore failures in 552-checker-sharpening.)\nBug: 30627598\nChange-Id: Ifce42645f2dcc350bbb88c2f4642e88fc5f98152\n" }, { "commit": "847e6ce98b4b822fd94c631975763845978ebaa3", "tree": "760e26dea1597d8219d8c515317d978b0213cdc1", "parents": [ "cff1b21b3e19c5d3a2d726fdb60dacd7de2a6f0d" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 02 13:55:07 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 02 14:03:28 2017 +0100" }, "message": "Rename kDexCacheViaMethod to kRuntimeCall for HLoadClass/String.\n\nThe old name does not reflect the actual code anymore.\n\nTest: testrunner.py --host\nChange-Id: I2e13cf727bba9d901c4d3fc821bb526d38a775b8\n" }, { "commit": "19d7d50c0ae23f2041b1aee4aa8a5b789e3d2020", "tree": "c3483c46f7c967d21dcc1ace5d0af874f82e0288", "parents": [ "a4bf3c33e501758fa8e707ef30f9ee67ea20511e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 24 13:04:14 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 24 13:19:23 2017 +0100" }, "message": "ARM64: Fix IsAdrpPatch().\n\nkMethodRelative patch can be an ADRP patch. Note that this\nminor bug had no real consequences as the patch would later\nbe filtered out in NeedsErratum843419Thunk() because we\ngenerate the instruction sequence ADRP+ADD in this case.\n\nAlso change the output for kDirectAddress from \"Direct\" to\n\"DirectAddress\". This kind is currently unused in checker\ntests.\n\nTest: Rely on TreeHugger.\nBug: 30627598\nChange-Id: I4e7c033aa010bbff53a554bfadd2ba0f03f69528\n" }, { "commit": "6597946d29be9108e2cc51223553d3db9290a3d9", "tree": "cea6647a45cc59fa1423751179e647124f269990", "parents": [ "a654e0378a8d0bb149362399917e4da2959e6991" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 19 17:25:12 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon May 22 18:35:05 2017 +0100" }, "message": "Use PC-relative pointer to boot image methods.\n\nIn preparation for adding ArtMethod entries to the .bss\nsection, add direct PC-relative pointers to methods so that\nthe number of needed .bss entries for boot image is small.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: testrunner.py --target on Nexus 6P\nTest: Nexus 6P boots.\nTest: Build aosp_mips64-userdebug\nBug: 30627598\nChange-Id: Ia89f5f9975b741ddac2816e1570077ba4b4c020f\n" }, { "commit": "662ee25b95a908ab063d4a8ae136ab695ea22229", "tree": "1b8e89a0c1ac009490dd2e8c3a6c73ae30e4bf3e", "parents": [ "c1c3351492674f2e264f41d93ffc80b633726689", "79d8fa7c52c1810d4618c9bd1d43994be5abb53d" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu May 18 16:03:44 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu May 18 16:03:47 2017 +0000" }, "message": "Merge \"optimizing: Build HConstructorFence for HNewArray/HNewInstance nodes\"" }, { "commit": "79d8fa7c52c1810d4618c9bd1d43994be5abb53d", "tree": "411a76dec2adf4139328d5e607b498b72c9aa2af", "parents": [ "acac09dad3d5aa3922e6cdf54ff2e4fa6f176484" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Apr 18 09:37:23 2017 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu May 18 14:14:13 2017 +0000" }, "message": "optimizing: Build HConstructorFence for HNewArray/HNewInstance nodes\n\nAlso fixes:\n* LSE, code_sinking to keep optimizing new-instance if it did so before\n* Various tests to expect constructor fences after new-instance\n\nSidenote: new-instance String does not get a ConstructorFence; the\nspecial StringFactory calls are assumed to be self-fencing.\n\nMetric changes on go/lem:\n* CodeSize -0.262% in ART-Compile (ARMv8)\n* RunTime -0.747% for all (linux-armv8)\n\n(No changes expected to x86, constructor fences are no-op).\n\nThe RunTime regression is temporary until art_quick_alloc_* entrypoints have their\nDMBs removed in a follow up CL.\n\nTest: art/test.py\nBug: 36656456\nChange-Id: I6a936a6e51c623e1c6b5b22eee5c3c72bebbed35\n" }, { "commit": "764d454d1d51448deb81f6e8d2d7d317c7f4d1b4", "tree": "029f9e65cfe7e953df55b7af45e099924ffdbbac", "parents": [ "7974468d32b817be1d49dfcf513605f646c481bc" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 16 10:31:41 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 16 15:08:55 2017 +0100" }, "message": "Remove LoadString/Class kind kBootImageLinkTimeAddress.\n\nWe no longer support non-PIC boot image compilation.\n\nAlso clean up some obsolete code for method patches\nand make JIT correctly report itself as non-PIC.\n\nTest: testrunner.py --host\nTest: testrunner.py --target\nBug: 33192586\nChange-Id: I593289c5c1b0e88b82b86a933038be97bbb15ad2\n" }, { "commit": "4e92c3ce7ef354620a785553bbada554fca83a67", "tree": "42029deff4d3ba7f89b5fdbf79ff410da575f431", "parents": [ "549844e9ccf432d1396b19af890eedb602b8ba04" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 08 09:34:26 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed May 10 09:27:33 2017 +0100" }, "message": "Add runtime reasons for deopt.\n\nCurrently to help investigate. Also:\n1) Log when deoptimization happens (which method and what reason)\n2) Trace when deoptimization happens (to make it visible in systrace)\n\nbug:37655083\nTest: test-art-host test-art-target\nChange-Id: I0c2d87b40db09e8e475cf97a7c784a034c585e97\n" }, { "commit": "d01745ef88bfd25df574a885d90a1a7785db5f5b", "tree": "058eb1593dbb0fe8a8e26b901909bec8aa01d474", "parents": [ "a57c334075b193de9690fff97acf6c1b1d1283fc" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Apr 05 16:40:31 2017 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue May 02 09:45:45 2017 -0700" }, "message": "optimizing: constructor fence redundancy elimination - remove dmb after LSE\n\nPart one of a few upcoming CLs to optimize constructor fences.\n\nThis improves load-store-elimination; all singleton objects that are not\nreturned will have their associated constructor fence removed.\n\nIf the allocation is removed, so is the fence. Even if allocation is not\nremoved, fences can sometimes be removed.\n\nThis change is enabled by tracking the \"this\" object associated with the\nconstructor fence as an input. Fence inputs are considered weak; they do not keep\nthe \"this\" object alive; if the instructions for \"this\" are all deleted,\nthe fence can also be deleted.\n\nBug: 36656456\nTest: art/test.py --host \u0026\u0026 art/test.py --target\nChange-Id: I05659ab07e20d6e2ecd4be051b722726776f4ab1\n" }, { "commit": "c6ea7d00ad069a2736f603daa3d8eaa9a1f8ea11", "tree": "a6d3332a8592fb806841314d55b206b573d37d86", "parents": [ "d68677c5fde1ace16ea58d65733776c954e7acb4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Feb 01 16:46:28 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Apr 21 08:40:33 2017 -0700" }, "message": "ART: Clean up art_method.h\n\nClean up the header. Fix up other headers including the -inl file,\nin an effort to prune the include graph. Fix broken transitive\nincludes by making includes explicit. Introduce new -inl files\nfor method handles and reference visiting.\n\nTest: source build/envsetup.sh \u0026\u0026 lunch aosp_angler-userdebug \u0026\u0026 mmma art\nTest: source build/envsetup.sh \u0026\u0026 lunch aosp_mips64-userdebug \u0026\u0026 mmma art\nChange-Id: I8f60f1160c2a702fdf3598149dae38f6fa6bc851\n" }, { "commit": "b07d1bcf055d3eb8c6b4c45b359ad8ef30909af7", "tree": "e925d6df31a3da17757c5d3f1b4f32c341847ce5", "parents": [ "f6e11e9e218e5e70463b5b3a94c89309936601d7" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Apr 05 10:03:15 2017 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Apr 05 10:03:15 2017 -0700" }, "message": "Ensure environment is ready when populating loop.\n\nRationale:\nOSR requires the suspend check to already have an environment,\nalbeit just for testing irreducible loops. This CL fixes the\nomission. Note, the error is spurious on OSR and writing a\nunit or regression test for this is hard.\n\nTest: test-art-host\nBug: 36950873\nChange-Id: Ica89e18e10deb438dead79e2cc40dd00a60b529f\n" }, { "commit": "f8f5a16ed7bad1e18179e38453e59c96a944de10", "tree": "53369083a97103563467cc5910a439a1864dd0b1", "parents": [ "7298b1ae3e9af5fdb46d168302a26cfbf5d475f5" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Mon Feb 06 15:35:29 2017 -0800" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Fri Mar 31 10:58:11 2017 -0700" }, "message": "ART vectorizer.\n\nRationale:\nMake SIMD great again with a retargetable and easily extendable vectorizer.\n\nProvides a full x86/x86_64 and a proof-of-concept ARM implementation. Sample\nimprovement (without any perf tuning yet) for Linpack on x86 is about 20% to 50%.\n\nTest: test-art-host, test-art-target (angler)\nBug: 34083438, 30933338\n\nChange-Id: Ifb77a0f25f690a87cd65bf3d5e9f6be7ea71d6c1\n" }, { "commit": "7359649f638859e265f252b2301a15c4b7bbafa4", "tree": "5ea16cb6e50976e4754ea092f9761cb0eba97bd4", "parents": [ "da3f60aed24d6e804f3b6393e74c62b8de0b4a26", "6f8e2c9913b24f746a154dda700f609cee3095f9" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 28 07:55:30 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Mar 28 07:55:31 2017 +0000" }, "message": "Merge \"Make data dependency around HDeoptimize correct.\"" }, { "commit": "6f8e2c9913b24f746a154dda700f609cee3095f9", "tree": "463b5925a1c87ef75ceab807e4c238a3e44a555a", "parents": [ "3bd2346db702894ef6c61372e7a83930b1c613ac" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Mar 23 14:37:26 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 27 15:14:40 2017 +0100" }, "message": "Make data dependency around HDeoptimize correct.\n\nWe use HDeoptimize in a few places, but when it comes to data\ndependency we either:\n- don\u0027t have any (BCE, CHA), in which case we should make sure no\n code that the deoptimzation guards moves before the HDeoptimize\n- have one on the receiver (inline cache), in which case we can\n update the dominated users with the HDeoptimize to get the data\n dependency correct.\n\nbug:35661819\nbug:36371709\ntest: 644-checker-deopt\nChange-Id: I4820c6710b06939e7f5a59606971693e995fb958\n" }, { "commit": "b13c65bb46544821a84ff2106d0710d77b0fb463", "tree": "46ef54ce881e32c80901528cf21a7951cf219023", "parents": [ "2b864659d4399de6c17e93b8df8cdbf08c6a7ac9" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Tue Mar 21 20:14:07 2017 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Mar 22 20:11:05 2017 +0000" }, "message": "Saves full XMM state along suspend check\u0027s slow path.\n\nRationale:\nBreak-out CL of ART Vectorizer. We need to save 128-bit\nof data (default ABI of ART runtime only saves 64-bit)\nNote that this is *only* done for xmm registers that\nare live, so overhead is not too big.\n\nBug: 34083438\nTest: test-art-host\nChange-Id: Ic89988b0acb0c104634271d0c6c3e29b6596d59b\n" }, { "commit": "1eede6ae9b08d305d0c1123284ff958373916474", "tree": "7827df0c9fd45bfe68e29b4105ad0b16f0076662", "parents": [ "48b795efcb504b71fbf038fabdd3d05366793197" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Mar 02 16:14:53 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 08 10:17:57 2017 +0000" }, "message": "Don\u0027t inline methods that throw in graph with irreducible loops.\n\nRe-computing the loop information is not supported in graphs\nwith irreducible loops, as it is not deterministic, and the\nloop header of a loop could change. That would lead to having the\nsuspend check in the wrong block.\n\nTest: test-art-host\nTest: 641-irreducible-inline\nbug: 35757766\nChange-Id: I6a435885461fbeca035e4f5d94f055fc3262adca\n" }, { "commit": "69d75ffac23fe1e655b7e81f0454c2841280dc1f", "tree": "88432364dd096bdd5674f48fb26e77f08cbf0dc6", "parents": [ "d8a16ba050978e4fe1447089002040a0047a5b3a" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Tue Feb 07 13:06:06 2017 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Fri Feb 17 12:21:46 2017 -0800" }, "message": "Skip loop optimization if there is no loop in the graph.\n\nLinearizeGraph() does quite some allocations.\nAlso add some comments on the possible false positives of\nsome flags.\n\nTest: m test-art-host\nChange-Id: I80ef89a2dc031d601e7621d0b22060cd8c17fae3\n" }, { "commit": "fdb7d638c17dab47984e1d325d04796bb426d9b3", "tree": "a352c1d8e52e18551c16bfea5bb9c564695b5239", "parents": [ "8dc12b1546c7409be19a7b5dc48932011db13067" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Feb 08 15:07:18 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Feb 15 10:25:54 2017 +0000" }, "message": "Inline methods that throw.\n\nForked from https://android-review.googlesource.com/214292\n\ntest: 637-checker-throw-inline\nbug: 30933338\nChange-Id: I184be82dfab0710af3f3796e9e486c7817fa9c60\n" }, { "commit": "fe076a51b0498c2771341cc09a77db15b437328f", "tree": "6f6e0c250e2ecc450567f90a7792f51f8d6b3384", "parents": [ "8781fe65fe41d971173bb2d05afe0dc00b5c08ce", "83c8e27a292e6e002fb3b3def75cf6d8653378e8" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Feb 06 08:27:55 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Feb 06 08:27:56 2017 +0000" }, "message": "Merge \"Code refactoring around sharpening HLoadClass.\"" }, { "commit": "83c8e27a292e6e002fb3b3def75cf6d8653378e8", "tree": "f49ff5c239f318a0290a0d1e0a5b4d9a1ee1d2ba", "parents": [ "357dcb73934356239292c46d6fbedba734da5e00" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 31 14:36:37 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 03 15:12:46 2017 +0000" }, "message": "Code refactoring around sharpening HLoadClass.\n\nEven if the class is not accessible through the dex cache, we\ncan access it by other means (eg boot class, jit table). So rewrite\nstatic field access instruction builder to not bail out if a class\ncannot be accessed through the dex cache.\n\nbug:34966607\n\ntest: test-art-host test-art-target\nChange-Id: I88e4e09951a002b480eb8f271726b56f981291bd\n" }, { "commit": "22aa54bf8469689c7c6c33f15ff4df2ffba8fa15", "tree": "14204d55784dce3ffdd2641382a763afee85ced0", "parents": [ "8116003cc9dd1e74fac1682eec547b8bb0afb061" ], "author": { "name": "Alexandre Rames", "email": "alexandre.rames@linaro.org", "time": "Tue Oct 18 09:32:29 2016 +0100" }, "committer": { "name": "xueliang.zhong", "email": "xueliang.zhong@linaro.org", "time": "Wed Jan 25 14:25:20 2017 +0000" }, "message": "AArch64: Add HInstruction scheduling support.\n\nThis commit adds a new `HInstructionScheduling` pass that performs\nbasic scheduling on the `HGraph`.\n\nCurrently, scheduling is performed at the block level, so no\n`HInstruction` ever leaves its block in this pass.\n\nThe scheduling process iterates through blocks in the graph. For\nblocks that we can and want to schedule:\n1) Build a dependency graph for instructions. It includes data\n dependencies (inputs/uses), but also environment dependencies and\n side-effect dependencies.\n2) Schedule the dependency graph. This is a topological sort of the\n dependency graph, using heuristics to decide what node to schedule\n first when there are multiple candidates. Currently the heuristics\n only consider instruction latencies and schedule first the\n instructions that are on the critical path.\n\nTest: m test-art-host\nTest: m test-art-target\n\nChange-Id: Iec103177d4f059666d7c9626e5770531fbc5ccdc\n" }, { "commit": "1ea9efcbd22127b75865f9a7c2949e20f5553744", "tree": "983395f1b543939dc5672deab04a6cfc502ba462", "parents": [ "dcc7ab628c9d59bfab203ab752ff7e11bfd60181" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 16 22:57:39 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 17 14:17:46 2017 +0000" }, "message": "Acquire the mutator lock before comparing classes/strings.\n\nScratch my initial thought we woudldn\u0027t need it because the\nhandlescope is visited during the pause: as the compiler thread\nis in state native, the GC can concurrently update the handlescope,\nleading to false negatives when doing class/string equality.\n\nbug:34240874\ntest: test-art-host gcstress\nChange-Id: Icda0722fb49300a7de57e1c5d1efaa9e8dbda83f\n" }, { "commit": "dcc7ab628c9d59bfab203ab752ff7e11bfd60181", "tree": "b37f3f978c06d4205145eab948d51f86560f64b0", "parents": [ "9748d3d2094c1d3c443a350cf12b9d77b4c4d1e3", "5247c08fb186a5a2ac02226827cf6b994f41a681" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 17 10:25:57 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Jan 17 10:25:57 2017 +0000" }, "message": "Merge \"Put the resolved class in HLoadClass.\"" }, { "commit": "e037a736be43c1e8ba9340dcbf1d17722356a37b", "tree": "f0bc7b4a78d04376b042046afbfd0677d766b523", "parents": [ "db54cc42859a8cc24ed1cda7f9b2f64b27dcb34a", "5d37c152f21a0807459c6f53bc25e2d84f56d259" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 17 09:16:31 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Jan 17 09:16:31 2017 +0000" }, "message": "Merge \"Put inlined ArtMethod pointer in stack maps.\"" }, { "commit": "5247c08fb186a5a2ac02226827cf6b994f41a681", "tree": "8b1305f9fb918024302382b8e8aa43962098e9fa", "parents": [ "0d478f289f0e33f19693d135f1d562b57427ed32" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 13 14:17:29 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 16 23:42:09 2017 +0000" }, "message": "Put the resolved class in HLoadClass.\n\nTo avoid repeated lookups in sharpening/rtp/inlining.\n\nTest: test-art-host test-art-target\nChange-Id: I08d0da36a4bb061cdaa490ea2af3a3217a875bbe\n" }, { "commit": "5d37c152f21a0807459c6f53bc25e2d84f56d259", "tree": "7d8cbce0a55f258150a047def70244f79afc866d", "parents": [ "aa89a4c6fca095904521842c018399f1e3501a45" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 12 13:25:19 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 16 22:56:56 2017 +0000" }, "message": "Put inlined ArtMethod pointer in stack maps.\n\nCurrently done for JIT. Can be extended for AOT and inlined boot\nimage methods.\n\nAlso refactor the lookup of a inlined method at runtime to not\nrely on the dex cache, but look at the class loader tables.\n\nbug: 30933338\ntest: test-art-host, test-art-target\nChange-Id: I58bd4d763b82ab8ca3023742835ac388671d1794\n" }, { "commit": "6bec91c7d4670905cd67440991ec76fd54d0f000", "tree": "05f4ba288e629270773c65b34b71be7bae5e92ff", "parents": [ "4155998a2f5c7a252a6611e3926943e931ea280a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 09 15:03:12 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 12:09:53 2017 +0000" }, "message": "Store resolved types for AOT code in .bss.\n\nTest: m test-art-host\nTest: m test-art-target on Nexus 9.\nTest: Nexus 9 boots.\nTest: Build aosp_mips64-eng.\nBug: 30627598\nBug: 34193123\nChange-Id: I8ec60a98eb488cb46ae3ea56341f5709dad4f623\n" }, { "commit": "48886c2ee655a16224870fee52dc8721a52babcf", "tree": "debc8b7d9c99a83e2c056c47a8e0718be00c12c3", "parents": [ "58207cfd229d9f0b39fc634cff489dac83e1c010" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jan 06 11:45:47 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 12:09:53 2017 +0000" }, "message": "Remove HLoadClass::LoadKind::kDexCachePcRelative.\n\nTest: m test-art-host\nTest: m test-art-target-run-test-552-checker-sharpening\nBug: 30627598\nChange-Id: Ic809b0f3a8ed0bd4dc7ab67aa64866f9cdff9bdb\n" }, { "commit": "6b69e0acb0e4c506ce2587e362c38e36e41e34ab", "tree": "976f08c78d3c5efa2dac8ec0409f36fae51456cb", "parents": [ "93939824c7e6e16cf98941cd4724278e87d6259d" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Jan 11 10:20:43 2017 -0800" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Fri Jan 13 10:04:42 2017 -0800" }, "message": "Complete unrolling of loops with small body and trip count one.\n\nRationale:\nAvoids the unnecessary loop control overhead, suspend check,\nand exposes more opportunities for constant folding in the\nresulting loop body. Fully unrolls loop in execute() of\nthe Dhrystone benchmark (3% to 8% improvements).\n\nTest: test-art-host\n\nChange-Id: If30f38caea9e9f87a929df041dfb7ed1c227aba3\n" }, { "commit": "f0acfe7a812a332122011832074142718c278dae", "tree": "49c4fc481cebd03323aaf0109066859165508303", "parents": [ "91db41f315f6c2366b7098c531224bee01170364" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 09 20:54:52 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 10 21:26:23 2017 +0000" }, "message": "Keep resolved String in HLoadString.\n\nFor the following reasons:\n- Avoids needing to do a lookup again in CodeGenerator::EmitJitRoots.\n- Fixes races where we the string was GC\u0027ed before CodeGenerator::EmitJitRoots.\n- Makes it possible to do GVN on the same string but defined in different\n dex files.\n\nTest: test-art-host, test-art-target\nChange-Id: If2b5d3079f7555427b1b96ab04546b3373fcf921\n" }, { "commit": "c1a42cf3873be202c8c0ca3c4e67500b470ab075", "tree": "f2bffbd14e1f9d5429dd8514d19be4fa6dfa392f", "parents": [ "06ce6d4359ed897f1d1b39be4e748f0c4f3ca2ff" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sun Dec 18 15:52:36 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 20 13:29:45 2016 +0000" }, "message": "Remove soon to be obsolete call kinds for direct calls.\n\nAnd remove CompilerDriver::GetCodeAndMethodForDirectCall in\npreparation of removing non-PIC prebuild and non-PIC on-device\nboot image compilation.\n\nTest: test-art-host test-art-target\nbug:33192586\nChange-Id: Ic48e3e8b9d7605dd0e66f31d458a182198ba9578\n" }, { "commit": "b0b051ad6c9fab511346882650d5d689f805a980", "tree": "fe02f128018f1aa55be5c0425295ae0ef670de2c", "parents": [ "d54f43ca39dfa92f08c2d760123f185f0f65fb86" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Nov 17 09:04:53 2016 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Dec 19 14:26:33 2016 -0800" }, "message": "CHA guard optimization (elimination/hoisting).\n\nTest: manual by checking the dump-cfg output.\n\nChange-Id: I254e168b9a85d2d3d23e02eea7e129c1bc9ab920\n" }, { "commit": "a9dbe8333d4df5447157fe575a805003172af047", "tree": "cfc116d6196a39eafd9eca891d13c39126d56adc", "parents": [ "bdca0a27eb034152afba52f1d499d773b81908df" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Dec 15 12:02:53 2016 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu Dec 15 12:02:53 2016 -0800" }, "message": "Add HVariableInputSizeInstruction.\n\nMake HPhi and HInvoke subclasses of the new instruction.\n\nTest: m test-art-host-run-test\nChange-Id: I303c725876f1f4407b98702d92370be25193fc53\n" }, { "commit": "9b1583e7d799a3bb3c0036abb8a0b9fcbfad360a", "tree": "5b3f6d7719bcbbee3f88d7f5ffd4f699f872b574", "parents": [ "fb678bfaad7f432861d26bde8343d53fadd68c07" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 13 13:43:31 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 13 22:14:02 2016 +0000" }, "message": "Support GVN for HLoadClass::LoadKind::kJitTableAddress.\n\nFixes performance regressions seen in eg Dhrystone.\n\nAlso add comment on why a class may not be found when sharpening.\n\nTest: manual Dhrystone run, performance recovers\nTest: ART_TEST_JIT\u003dtrue test-art-host-run-test-jit\nChange-Id: I8e879f1c390f83e8bc930f343beb7b4a41c2f190\n" }, { "commit": "22384aeab988df7fa5ccdc48a668589c5f602c39", "tree": "daca06adfc92c93017618c3729af54ed40214ba4", "parents": [ "0ee6447c63e354131dec78743ccabcbc964129e3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 12 22:33:36 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Dec 12 23:06:38 2016 +0000" }, "message": "Revert \"Revert \"Add kJitTableAddress for HLoadClass.\"\"\n\nThis reverts commit d2d5262c8370309e1f2a009f00aafc24f1cf00a0.\n\nChange-Id: I6149d5c7d5df0b0fc5cb646a802a2eea8d01ac08\n" }, { "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": "be44dcf6b2338782c276483c6c79a4ab17ca98a9", "tree": "21f97e74dafb37f9a0382e30a6b0fea18f570ddb", "parents": [ "07ab8831ff6025f91f927cc9e60c04801583666c" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Nov 30 14:17:32 2016 -0800" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Nov 30 14:29:58 2016 -0800" }, "message": "Add LoadString kind of kJitTableAddress for dump-cfg.\n\nTest: manual\n\nChange-Id: Ifcae7b26f666930766635d8b3ed7a495494cddf7\n" }, { "commit": "48f3e29643233dd5383deca10c4c65690a3e0b47", "tree": "b7d3cf1d726fa56157321e2d380b497a1ba62af5", "parents": [ "5199c11040973475509456b186437b654946bb82", "c757decb04d8535fd806b9bce1c2fe5e52c228dc" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Nov 10 20:41:12 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Nov 10 20:41:13 2016 +0000" }, "message": "Merge \"Do not inline loops without exit edges\"" }, { "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": "b4e4c279a0d96c5c9bd6d038d97c3c9992ff8d09", "tree": "530664632e08ed8f82006e8899ff0de4a32256f7", "parents": [ "11bbc7fd5b4ed64d13c5d21ae9c8b686cf24bcbb", "661b69bd7b19f90b0c0cba7f354e61f7ff9cc583" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Nov 09 16:42:45 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Nov 09 16:42:46 2016 +0000" }, "message": "Merge \"Reduce arena memory usage when changing graph structure.\"" }, { "commit": "661b69bd7b19f90b0c0cba7f354e61f7ff9cc583", "tree": "85e69c696629cb63e14a04abb71b6ff4b0a14fa6", "parents": [ "2a2f713717ace8a026cfe76c91165a5ed0d80182" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 09 14:11:37 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 09 15:00:35 2016 +0000" }, "message": "Reduce arena memory usage when changing graph structure.\n\nUse swap() to reuse previously allocated memory. Use\nrange insertion instead of individual push_back()s.\n\nTest: m test-art-host.\nChange-Id: I7174f8cc66937f34b452ff7268a8806ca94e7573\n" }, { "commit": "54d6a207341ad45cb5eceed71a344073ed6d4e31", "tree": "d24fc641ca9ad2b1732a40f0989e3683cc0a5028", "parents": [ "2a2f713717ace8a026cfe76c91165a5ed0d80182" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 09 12:46:38 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 09 12:46:38 2016 +0000" }, "message": "Fix 552-checker-sharpening for PIC test.\n\nAnd remove obsolete HLoadString::LoadKind::kDexCacheAddress.\n\nTest: m ART_TEST_PIC_TEST\u003dtrue test-art-host\nChange-Id: I3e7a1a98c2c7eba5ea10954d7efcf743a807c300\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": "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": "9620230700d4b451097c2163faa70627c9d8088a", "tree": "695b96b9efeaa4c2cb3816e51904e19540fe3883", "parents": [ "4aa6a93c46a959df1ab71ee7a68ad345338046ef" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Tue Oct 04 17:33:56 2016 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Oct 05 11:50:42 2016 -0700" }, "message": "Refactoring of graph linearization and linear order.\n\nRationale:\nOwnership of graph\u0027s linear order and iterators was\na bit unclear now that other phases are using it.\nNew approach allows phases to compute their own\norder, while ssa_liveness is sole owner for graph\n(since it is not mutated afterwards).\n\nAlso shortens lifetime of loop\u0027s arena.\n\nTest: test-art-host\nChange-Id: Ib7137d1203a1e0a12db49868f4117d48a4277f30\n" }, { "commit": "26705cca4643385931f3b3568b6b82525400b52b", "tree": "433a9e1189ffccc70291e08b51ecec36a59cfc87", "parents": [ "d74bd3a9a48d34d7321f7de7a6f8e1d3467be58a", "281c681a0852c10f5ca99b351650b244e878aea3" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Tue Oct 04 20:40:23 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Oct 04 20:40:24 2016 +0000" }, "message": "Merge \"A first implementation of a loop optimization framework.\"" }, { "commit": "aad75c6d5bfab2dc8e30fc99fafe8cd2dc8b74d8", "tree": "c1b9e1eabcf35c5cbb5b4f46313a4e062f2d5d51", "parents": [ "82d4838d6bb3480cd25327cedc5179fb2d86881c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 03 08:46:48 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 04 17:51:46 2016 +0100" }, "message": "Revert \"Revert \"Store resolved Strings for AOT code in .bss.\"\"\n\nFixed oat_test to keep dex files alive. Fixed mips build.\nRewritten the .bss GC root visiting and added write barrier\nto the artResolveStringFromCode().\n\nTest: build aosp_mips-eng\nTest: m ART_DEFAULT_GC_TYPE\u003dSS test-art-target-host-gtest-oat_test\nTest: Run ART test suite on host and Nexus 9.\nBug: 20323084\nBug: 30627598\n\nThis reverts commit 5f926055cb88089d8ca27243f35a9dfd89d981f0.\n\nChange-Id: I07fa2278d82b8eb64964c9a4b66cb93726ccda6b\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": "5f926055cb88089d8ca27243f35a9dfd89d981f0", "tree": "8d87d400e36301eb648e19bcd225f13c469648ad", "parents": [ "9e5739aaa690a8529c104f4c05035a657616c310" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 30 17:04:49 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 30 18:08:09 2016 +0100" }, "message": "Revert \"Store resolved Strings for AOT code in .bss.\"\n\nThere are some issues with oat_test64 on host and aosp_mips-eng.\n\nAlso reverts \"compiler_driver: Fix build.\"\n\nBug: 20323084\nBug: 30627598\n\nThis reverts commit 63dccbbefef3014c99c22748d18befcc7bcb3b41.\nThis reverts commit 04a44135ace10123f059373691594ae0f270a8a4.\n\nChange-Id: I568ba3e58cf103987fdd63c8a21521010a9f27c4\n" }, { "commit": "428792553e59452d0b6fdbf2460072d697c966db", "tree": "c4d3c14944e772cd98f2fd76e4f41c2fbc3f6acf", "parents": [ "ce157a6864bd346152bef1b66d050cfb144f56b5", "63dccbbefef3014c99c22748d18befcc7bcb3b41" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 30 14:10:21 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Sep 30 14:10:21 2016 +0000" }, "message": "Merge \"Store resolved Strings for AOT code in .bss.\"" }, { "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": "63dccbbefef3014c99c22748d18befcc7bcb3b41", "tree": "60a498041bebff43bc1f43d438e3bc34a30887f7", "parents": [ "6bee25976782a063d6b44f7718a6302761bf6403" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Sep 21 13:51:10 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 29 15:58:43 2016 +0100" }, "message": "Store resolved Strings for AOT code in .bss.\n\nAnd do some related refactorings.\n\nBug: 20323084\nBug: 30627598\nTest: Run ART test suite including gcstress on host and Nexus 9.\nTest: Run ART test suite including gcstress with baker CC on host and Nexus 9.\nTest: Build aosp_mips64-eng.\nChange-Id: I1b12c1570fee8e5da490b47f231050142afcbd1e\n" }, { "commit": "20e9db6db787e007e7032878c9899b28ec43e93f", "tree": "9d2071fd7f286ccf408044b4366714a481351393", "parents": [ "26ead4975e1752e8ae2f5ed6fda73876c4f9ff59" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Sep 14 10:52:13 2016 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Sep 14 14:09:27 2016 -0700" }, "message": "Make LinearizeGraph() public (and move it to nodes files)\n\nRationale:\nIt is strange that HLinearOrderIterator is defined (and visible)\nin nodes.h, but clients have no way to build this order. This CL\nmakes the building available at the usual place.\n\nChange-Id: Ib66f2edf6dfc8edd6b429bd4bea3ac7e37440b28\nTests: m test-art\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": "75d2df275abedf90be8c8e31202389788c3b079b", "tree": "01d63985ad762f8fe22ffc82e8532d920bb3af36", "parents": [ "6369394e75b84ff3aefbd6523bb1f955bcc2323f" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 27 21:25:41 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 27 21:26:18 2016 -0700" }, "message": "ART: Use old operator\u003c\u003c for MemBarrierKind\n\nTo fix assumptions in the tests.\n\nTest: m test-art-host\nChange-Id: Ie2e738524ee8f75a3c41730f6a1449dab490e535\n" }, { "commit": "26de38bb7f2122417388809f4ff88a7cb5c4af5e", "tree": "878f432e2476f90201dd4695cfc8c3498c2c207f", "parents": [ "9755c262df1be7f5d5b98d038c8fd3734e974f9d" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 27 17:53:11 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 27 17:56:08 2016 -0700" }, "message": "ART: Delete old compiler_enums.h\n\nHoldover from the Quick days. Move the two enums that are still\nused closer to the actual users (and prune no longer used cases).\n\nTest: m test-art-host\nChange-Id: I88aa49961a54635788cafac570ddc3125aa38262\n" }, { "commit": "e90049140fdfb89080e5cc9b000b0c9be8c18bcd", "tree": "66b45c052b6778fabd7847a44af5e610808fa867", "parents": [ "a77ceae14a7be2494874d9256327efa8c522e234" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jun 16 16:50:52 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jul 01 13:26:24 2016 +0100" }, "message": "Create a typedef for HInstruction::GetInputs() return type.\n\nAnd some other cleanup after\n https://android-review.googlesource.com/230742\n\nTest: No new tests. ART test suite passed (tested on host).\nChange-Id: I4743bf17544d0234c6ccb46dd0c1b9aae5c93e17\n" }, { "commit": "dbb7f5bef10138ade0fb202da1d61f562b2df649", "tree": "f0aa4b390c534b215a6e000c865783cdd9852353", "parents": [ "b94b5706f0b8e2e1c7e1db22274f9f4bae0c4b5a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Mar 30 13:23:58 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jun 21 15:11:57 2016 +0100" }, "message": "Improve HLoadClass code generation.\n\nFor classes in the boot image, use either direct pointers\nor PC-relative addresses. For other classes, use PC-relative\naccess to the dex cache arrays for AOT and direct address of\nthe type\u0027s dex cache slot for JIT.\n\nFor aosp_flounder-userdebug:\n - 32-bit boot.oat: -252KiB (-0.3%)\n - 64-bit boot.oat: -412KiB (-0.4%)\n - 32-bit dalvik cache total: -392KiB (-0.4%)\n - 64-bit dalvik-cache total: -2312KiB (-1.0%)\n (contains more files than the 32-bit dalvik cache)\nFor aosp_flounder-userdebug forced to compile PIC:\n - 32-bit boot.oat: -124KiB (-0.2%)\n - 64-bit boot.oat: -420KiB (-0.5%)\n - 32-bit dalvik cache total: -136KiB (-0.1%)\n - 64-bit dalvik-cache total: -1136KiB (-0.5%)\n (contains more files than the 32-bit dalvik cache)\n\nBug: 27950288\nChange-Id: I4da991a4b7e53c63c92558b97923d18092acf139\n" }, { "commit": "206fbf51ccd2dbbcf106e480cc6c5bb96fc7befb", "tree": "38c57afab97083c1d805a381c1dfa0764605fec6", "parents": [ "834f4445e70543bfdbcdd6435ccceb7a6dd82c4d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jun 10 16:06:34 2016 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Jun 13 14:14:43 2016 +0100" }, "message": "Remove too aggressive DCHECKs.\n\nA class can move from a state greater or equal than resolved\nto erroneous concurrently to the verifier or the compiler.\n\nbug:29239283\nChange-Id: I89f3fe1c1d9556c6c99b8e005b3ec02de7f01b85\n(cherry picked from commit f7d994622aabcc689f62253a9253e0c67d9e787e)\n" }, { "commit": "d6c205eb9b04bcfa072cd5ffdd93deef167ec340", "tree": "ba2d71a03b266e317fe7278d1136098cb890dfee", "parents": [ "f1c25cf31d29c70b5102b249344bea76c374ecd8" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Jun 07 14:20:52 2016 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Jun 07 14:49:18 2016 +0100" }, "message": "ART: Remove redundant MoveInstructionBefore method\n\nChange-Id: If53d7011197cc6b9c1702a3d98ef11b59eb76f0c\n" }, { "commit": "6b631dc9662d39ca0f89ee4653d911f1f5291913", "tree": "076ece82cfbbd0b859a8e36cf1c4226dc122a801", "parents": [ "d1be2a34836fc32d938225c08eae722f50bbb175", "f89381fed12faf96c45a83a989ae2fff82c05f3b" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Mon Jun 06 20:56:49 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jun 06 20:56:49 2016 +0000" }, "message": "Merge \"ART: ArrayGet hoisting restriction added.\"" }, { "commit": "f89381fed12faf96c45a83a989ae2fff82c05f3b", "tree": "fafdf6e995bbea12c62220e9e8cda6a7d22fc139", "parents": [ "2e1c66e37db7eaa713459e15b115d5c217d6e414" ], "author": { "name": "Anton Shamin", "email": "anton.shamin@intel.com", "time": "Mon May 16 16:44:13 2016 +0600" }, "committer": { "name": "Anton Shamin", "email": "anton.shamin@intel.com", "time": "Tue Jun 07 01:11:44 2016 +0600" }, "message": "ART: ArrayGet hoisting restriction added.\n\nCurrently if we hoist ArrayGet from loop there is no guarantee\nthat insn will be executed at runtime. Because of that we could\nface issues like crashes in generated code.\n\nThis patch introduces restriction for ArrayGet hoisting. We say\nthat ArrayGet execution is guaranteed at least one time if its bb\ndominates all exit blocks.\n\nChange-Id: I9f72c0f4c33b358341109238bea46cb5a82f490f\nSigned-off-by: Anton Shamin \u003canton.shamin@intel.com\u003e\n" }, { "commit": "372f10e5b0b34e2bb6e2b79aeba6c441e14afd1f", "tree": "1f29c2467c8909ef0e0147f37f176caa1bcd2ccc", "parents": [ "1b66fdf3f33c72dfdda4d31f6f17b6a0d8607402" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 17 16:30:10 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jun 02 19:04:20 2016 +0100" }, "message": "Refactor handling of input records.\n\nIntroduce HInstruction::GetInputRecords(), a new virtual\nfunction that returns an ArrayRef\u003c\u003e to all input records.\nImplement all other functions dealing with input records as\nwrappers around GetInputRecords(). Rewrite functions that\npreviously used multiple virtual calls to deal with input\nrecords, especially in loops, to prefetch the ArrayRef\u003c\u003e\nonly once for each instruction. Besides avoiding all the\nextra calls, this also allows the compiler (clang++) to\nperform additional optimizations.\n\nThis speeds up the Nexus 5 boot image compilation by ~0.5s\n(4% of \"Compile Dex File\", 2% of dex2oat time) on AOSP ToT.\n\nChange-Id: Id8ebe0fb9405e38d918972a11bd724146e4ca578\n" }, { "commit": "d7c2fdc939bb7efb3e7204d62e54c6a3f7d77f9b", "tree": "692eb754d2cf5fdb81809529f02a50f2e4747a62", "parents": [ "b2b55596e605bef315b615cb89e4515f360548f2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue May 10 14:35:34 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu May 12 08:58:12 2016 +0100" }, "message": "Fix another case of live_in at irreducible loop entry.\n\nGVN was implicitly extending the liveness of an instruction across\nan irreducible loop.\n\nFix this problem by clearing the value set at loop entries that contain\nan irreducible loop.\n\nbug:28252896\n\n(cherry picked from commit 77ce6430af2709432b22344ed656edd8ec80581b)\n\nChange-Id: Ie0121e83b2dfe47bcd184b90a69c0194d13fce54\n" }, { "commit": "3ea5a97d27468cec846d958c38d0d706ef7ec67e", "tree": "8895f0adeaff161be3a7101d7f11fa744ff739c6", "parents": [ "60be30a353aa8515c72aa5240508128bb3ad9829" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon May 09 20:23:34 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon May 09 20:33:31 2016 +0100" }, "message": "Attribute arena allocations previously marked as STL.\n\nBug: 28603175\nChange-Id: I488e39b23afb86f3ff5a2df16d2df59eb3adaa0f\n" }, { "commit": "5bcbdd59c066f46642f6ddf5833d58ce4170c1e0", "tree": "aa50b637f192f96e9a3afe562e03ebfaf13e8ccc", "parents": [ "af3ec0a7d3428d02492dd41ecf19bc33e6807828", "ace7a000a433ce4ecf94f30adea39c01a76fa936" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 06 14:46:47 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri May 06 14:46:47 2016 +0000" }, "message": "Merge \"Optimizing: LoadString may not have any side effects.\"" }, { "commit": "af3ec0a7d3428d02492dd41ecf19bc33e6807828", "tree": "49704c702114598e6e78203a33a38d90dd7b4fb4", "parents": [ "9858f04577d2ca7400660dba9b3abe8b33bab96c", "3f4a522cc39f5c651e7c718196e989bc81d8c6ef" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri May 06 13:45:50 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri May 06 13:45:50 2016 +0000" }, "message": "Merge \"ART: Fix dominance for irreducible loops\"" }, { "commit": "3f4a522cc39f5c651e7c718196e989bc81d8c6ef", "tree": "37820ae22c0e96817f6278ebe71f4588f26eab15", "parents": [ "bc177b3f3461f1db013d5511344ebe52270e136f" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri May 06 12:46:21 2016 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri May 06 13:10:16 2016 +0100" }, "message": "ART: Fix dominance for irreducible loops\n\nComputation of dominance was broken in the presence of irreducible\nloops because the algorithm assumed that back edges are always\ndominated by their respective headers and a fix-point iteration is\ntherefore unnecessary.\n\nThis is not true for irreducible loops, forcing us to revisit their\nloop headers and all dependent blocks. This patch adds a fix-point\niteration if a back edge not dominated by its header is found.\n\nBug: 28611485\nChange-Id: If84044e49d5b9c682949648033d2861628d7fe05\n" }, { "commit": "5a620590e5cf6d6817693edffd661371555de88b", "tree": "16f93fac9b5ec42bd8574acb6535ddd1dcc9f61f", "parents": [ "a69ae8ab13d07fb3b03596a9b9049b3292dd0b20" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu May 05 11:27:03 2016 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu May 05 16:44:29 2016 +0100" }, "message": "Stop populating irreducible loop at header\n\nRecent CL modified the (previously exponential) algorithm for\npopulating irreducible loops so as to not visit blocks multiple times.\nHowever, the new condition for an early exit did not take into account\nthat it is not necessary to visit predecessors of the loop header.\n\nThe algorithm remains correct but does unnecessary work.\n\nBug: 27856014\nChange-Id: If72f8be82e79838f0dd9678082a3bbaabb51e43b\n" }, { "commit": "f417ff44d1eb111854d7a213f106912b3dd9e3d4", "tree": "be69b00348b8d296a5bae7dd2c17d9f9a2b8ba0e", "parents": [ "dbea8b4eacb62b357bce1bc299c8442788b288a1" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Mon Apr 25 12:51:37 2016 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Mon Apr 25 16:21:34 2016 -0700" }, "message": "Test component type for errors too.\nWith regression test.\n\nRationale:\nMoved erroneous check in convenience method, so we\nput all the same logic in one place. When testing\nfor erroneous T[], check both the array type\nas well at the component type T for errors\n(it is possible T[] is not marked erroneous\neven though T is eventually).\n\nBUG\u003d28358598\n\nChange-Id: I11339a976dc83e0493a99e6bb97f3a058ca3f796\n" }, { "commit": "c6b5627c25ff5653e97ccff8c5ccf6ac967b6f83", "tree": "8cec0d66bdd5e5a9be14bbb881eb36d09868ab39", "parents": [ "a584db5460a31198bf621cdacf5bc304a984efa4" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Apr 20 18:45:25 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Apr 20 19:40:35 2016 +0100" }, "message": "Fix HInstruction::ReplaceInput(), allow no-op.\n\nAllow HInstruction::ReplaceInput() to be called with\na `replacement` being the same as the old input and\ndo nothing in that case.\n\nThis is a follow-up to\n https://android-review.googlesource.com/216923\nwhere I erroneously assumed that it never happens.\n\nAlso adhere to the standard C++ std::forward_list\u003c\u003e\nsemantics in the single-element overload of\n`IntrusiveForwardList\u003c\u003e::splice_after()`.\n\nBug: 28173563\nChange-Id: I5cea14c212b1083f90ffe6b5b53324ad663d57d8\n" } ], "next": "3c19d3e029a9fcc123d2c6fd1e5e13867d2cfe1f" }