)]}' { "log": [ { "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": "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" }, { "commit": "3c19d3e029a9fcc123d2c6fd1e5e13867d2cfe1f", "tree": "6cd3dcb90f5ee163b08d933299d5b821c03c6a73", "parents": [ "6435910a65067fb38477232b2620cc53978b13c7" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 19 14:36:35 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 19 19:43:31 2016 +0100" }, "message": "Reuse HUseListNode\u003c\u003es when replacing instruction or input.\n\nCompiling the Nexus 5 boot image with the 64-bit dex2oat\non host this CL reduces the memory used for compiling the\nmost hungry method, BatteryStats.dumpLocked(), by ~5.6MiB:\n\nBefore:\n MEM: used: 44393040, allocated: 45361248, lost: 968208\n Number of arenas allocated: 319,\n Number of allocations: 815492, avg size: 54\n ...\n UseListNode 10308480\n ...\nAfter:\n MEM: used: 38554536, allocated: 39463008, lost: 908472\n Number of arenas allocated: 274,\n Number of allocations: 572221, avg size: 67\n ...\n UseListNode 4469976\n ...\n\nWith 32-bit dex2oat, the UseListNode would be 2/3 of the\nvalues for 64-bit dex2oat (both before and after).\n\nBug: 28173563\nChange-Id: Ia4fabe03568f0e0dbf2cdf2b031863602aea3530\n" }, { "commit": "6435910a65067fb38477232b2620cc53978b13c7", "tree": "d607c9103f46242bce5ed75297c20087c0e3f8cd", "parents": [ "814ae9448d7a620a1855d53c7b69b1b377944a67", "46817b876ab00d6b78905b80ed12b4344c522b6c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 19 18:26:22 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Apr 19 18:26:22 2016 +0000" }, "message": "Merge \"Use iterators \"before\" the use node in HUserRecord\u003c\u003e.\"" }, { "commit": "46817b876ab00d6b78905b80ed12b4344c522b6c", "tree": "6715bee60b0682a10437866c9617cb442146aa2f", "parents": [ "f59149a151ee694484e21da7b3b207920dead5a6" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 29 12:21:58 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 19 18:33:06 2016 +0100" }, "message": "Use iterators \"before\" the use node in HUserRecord\u003c\u003e.\n\nCreate a new template class IntrusiveForwardList\u003c\u003e that\nmimicks std::forward_list\u003c\u003e except that all allocations\nare handled externally. This is essentially the same as\nboost::intrusive::slist\u003c\u003e but since we\u0027re not using Boost\nwe have to reinvent the wheel.\n\nUse the new container to replace the HUseList and use the\niterators to \"before\" use nodes in HUserRecord\u003c\u003e to avoid\nthe extra pointer to the previous node which was used\nexclusively for removing nodes from the list. This reduces\nthe size of the HUseListNode by 25%, 32B to 24B in 64-bit\ncompiler, 16B to 12B in 32-bit compiler. This translates\ndirectly to overall memory savings for the 64-bit compiler\nbut due to rounding up of the arena allocations to 8B, we\ndo not get any improvement in the 32-bit compiler.\n\nCompiling the Nexus 5 boot image with the 64-bit dex2oat\non host this CL reduces the memory used for compiling the\nmost hungry method, BatteryStats.dumpLocked(), by ~3.3MiB:\n\nBefore:\n MEM: used: 47829200, allocated: 48769120, lost: 939920\n Number of arenas allocated: 345,\n Number of allocations: 815492, avg size: 58\n ...\n UseListNode 13744640\n ...\nAfter:\n MEM: used: 44393040, allocated: 45361248, lost: 968208\n Number of arenas allocated: 319,\n Number of allocations: 815492, avg size: 54\n ...\n UseListNode 10308480\n ...\n\nNote that while we do not ship the 64-bit dex2oat to the\ndevice, the JIT compilation for 64-bit processes is using\nthe 64-bit libart-compiler.\n\nBug: 28173563\nChange-Id: I985eabd4816f845372d8aaa825a1489cf9569208\n" }, { "commit": "5f6b90824628504813cd174b90eaaad8c8031726", "tree": "53f7230957dc30bbc5505a946d7281d4a8d97863", "parents": [ "35a38d1e250fc3237c79fafa96ec69d6f07c1f3d", "8b3f9b246d5bdbf67faeb2b872b75b8d72777bc0" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Tue Apr 19 16:21:46 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Apr 19 16:21:47 2016 +0000" }, "message": "Merge \"Avoid constructing types with errors.\"" }, { "commit": "fd66c50d64c38e40bafde83b4872e27bbff7546d", "tree": "e77ea1374211042d5eca3d1a62a81293a29eb108", "parents": [ "c4d445a40bf00ab497b5e4d43a6b43eaafb5fa50" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Apr 18 15:37:01 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 19 13:59:15 2016 +0100" }, "message": "Fix inlining loops in OSR mode.\n\nWhen compiling a method in OSR mode and the method does not\ncontain a loop (arguably, a very odd case) but we inline\nanother method with a loop and then the final DCE re-runs\nthe loop identification, the inlined loop would previously\nbe marked as irreducible. However, the SSA liveness analysis\nexpects irreducible loop to have extra loop Phis which were\nalready eliminated from the loop before the inner graph was\ninlined to the outer graph, so we would fail a DCHECK().\n\nWe fix this by not marking inlined loops as irreducible when\ncompiling in OSR mode.\n\nBug: 28210356\nChange-Id: If10057ed883333c62a878ed2ae3fe01bb5280e33\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": "c2e8af9659db7e456b26febb1b971900057ad427", "tree": "af35981588f2995d8bb3a570e2f7244c8c22373a", "parents": [ "04c2b2de69460d41e0c39c1f7d9fb77376c3aa8b" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Apr 05 17:15:19 2016 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Apr 12 15:17:28 2016 +0100" }, "message": "ART: Speed up HGraph::PopulateIrreducibleRecursive\n\nPopulating an irreducible loop can potentially traverse all possible\npaths through the HGraph, leading to an exponential algorithm.\nThis patch adds a bit vector of nodes whose membership in the loop\nhas been decided and need not be revisited again.\n\nBug: 27856014\nChange-Id: I3696f08c846e6f40e5de44cb771811bac7e3e08a\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": "ace7a000a433ce4ecf94f30adea39c01a76fa936", "tree": "9d41d8974c1feda1af134a536eee8064028d670f", "parents": [ "75563e34a4e64a8e805e6810b13e07f1cafec137" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 05 11:18:49 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Apr 05 12:27:58 2016 +0100" }, "message": "Optimizing: LoadString may not have any side effects.\n\nLoadString does not have any side effects if the string is\nknown to be in the dex cache or it\u0027s a boot image string\nreferenced directly, as specified by the string load kind.\nWe need to clear the side effects for these cases to avoid\na DCHECK() failure when such LoadString instruction ends up\nbetween a ClinitCheck and an instruction to which we want to\nmerge that ClinitCheck. This may happen as a consequence of\ninlining, LICM and DCE as shown by a regression test.\n\nBug: 27929914\nChange-Id: I7b3bddf7d8c79ce1828a4a751f1270cf2e3d61f0\n" }, { "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": "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": "cac5a7e871f1f346b317894359ad06fa7bd67fba", "tree": "2df7d4892216a7c976dfe848c9893cb5eb8fa6a5", "parents": [ "6f51d7756a9c66007fe7666b19399e1f60ff6092" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Feb 22 10:39:50 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 29 19:27:40 2016 +0100" }, "message": "Optimizing: Improve const-string code generation.\n\nFor strings in the boot image, use either direct pointers\nor pc-relative addresses. For other strings, use PC-relative\naccess to the dex cache arrays for AOT and direct address of\nthe string\u0027s dex cache slot for JIT.\n\nFor aosp_flounder-userdebug:\n - 32-bit boot.oat: -692KiB (-0.9%)\n - 64-bit boot.oat: -948KiB (-1.1%)\n - 32-bit dalvik cache total: -900KiB (-0.9%)\n - 64-bit dalvik cache total: -3672KiB (-1.5%)\n (contains more files than the 32-bit dalvik cache)\nFor aosp_flounder-userdebug forced to compile PIC:\n - 32-bit boot.oat: -380KiB (-0.5%)\n - 64-bit boot.oat: -928KiB (-1.0%)\n - 32-bit dalvik cache total: -468KiB (-0.4%)\n - 64-bit dalvik cache total: -1928KiB (-0.8%)\n (contains more files than the 32-bit dalvik cache)\n\nBug: 26884697\nChange-Id: Iec7266ce67e6fedc107be78fab2e742a8dab2696\n" }, { "commit": "62e78a7ea8221f3fbe848fa883fb86c9d87ba50b", "tree": "665cac33ba8de588707155f0ac9450cf11b9d5c5", "parents": [ "308640fd439f6e0867770d74600ad4d8edda7093", "9eeebf646b118269b3f5bd6fc6c1b0a58c26c6d4" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Mar 24 15:35:10 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Mar 24 15:35:10 2016 +0000" }, "message": "Merge \"ART: Fix order of operations in HBasicBlock::DisconnectAndDelete\"" }, { "commit": "9eeebf646b118269b3f5bd6fc6c1b0a58c26c6d4", "tree": "90dfa9c619cdf18935e9acfd045a977a8e0ae48c", "parents": [ "843a65556616183a36792bbcc1632c6d8d0e78b2" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Mar 24 11:18:15 2016 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Mar 24 15:33:49 2016 +0000" }, "message": "ART: Fix order of operations in HBasicBlock::DisconnectAndDelete\n\nThe method would remove predecessors before successors. As a result,\ninstructions used by dead loop phis would see dangling uses, causing\na DCHECK to fail.\n\nSteps were reordered to remove dependencies in post order.\n\nBug: 27683071\nChange-Id: I8e0e976443fb410908321a065276f1340b757c41\n" }, { "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": "c9b21f83c2eab8bdd16241992193c3049dc68e43", "tree": "00a4b12cac09c7ba84db7508ce22ab1fd939c571", "parents": [ "7ee5bc54fbe1bf5218696e63a80e76cd0c8ea387" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Mar 23 16:36:59 2016 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Mar 23 16:38:24 2016 +0000" }, "message": "Fix some typos in art/compiler/optimizing/nodes.cc.\n\nChange-Id: I11be5a9b73da207c9eb497bcaffc49d614c1ca89\n" }, { "commit": "f6a35de9eeefb20f6446f1b4815b4dcb0161d09c", "tree": "cf484acbd6889b92a7fe3e8615611129088c3894", "parents": [ "459898dc4470559ba1e1d578bc52a914d1f573f5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Mar 21 12:01:50 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Mar 21 16:50:08 2016 +0000" }, "message": "Optimizing: Fix register allocator validation memory usage.\n\nAlso attribute ArenaBitVector allocations to appropriate\npasses. This was used to track down the source of the\nexcessive memory alloactions.\n\nBug: 27690481\n\nChange-Id: Ib895984cb7c04e24cbc7abbd8322079bab8ab100\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": "18401b748a3180f52e42547ede22d1b184fe8c43", "tree": "aebee923635cc2563c74033b00ffc90ceadafed4", "parents": [ "b7f257f353b1eb2db2732939a0404c118316891d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 11 13:35:51 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 11 13:36:19 2016 +0000" }, "message": "Fix invariant in reference type propagation.\n\nAlso some cleanups.\n\nChange-Id: I7f0ec7d06b4bab10dbfa230c757447d311658f93\n" }, { "commit": "7ba9966b76bbf818513018fa1da72c89330fe384", "tree": "adb082660168b9c50319d861cba664374d55d5a5", "parents": [ "e4cec517faf1c74d6895e0e5b512a6e2f83a0e06" ], "author": { "name": "Serguei Katkov", "email": "serguei.i.katkov@intel.com", "time": "Wed Mar 02 16:25:36 2016 +0600" }, "committer": { "name": "Serguei Katkov", "email": "serguei.i.katkov@intel.com", "time": "Thu Mar 03 16:54:31 2016 +0600" }, "message": "ART: cleanup exit_block_ in graph if exit block is removed\n\nIf we remove the exit block from the graph (for example method\ncontains infinite loop) we should also clean the field exit_block_\nin graph. At least inliner expects it.\n\nChange-Id: Icda668da2233cdd6cd673635a1949f5ed34cf270\nSigned-off-by: Serguei Katkov \u003cserguei.i.katkov@intel.com\u003e\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": "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": "e53bd8160ad2892f33849108d3b1099992a311fd", "tree": "2e2c33e50fa871174273799d09689d669f4c0561", "parents": [ "d15ede2df7d157ea5480614fd18c2bf0d37a6c2a" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Feb 24 14:54:18 2016 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Feb 24 14:54:45 2016 +0000" }, "message": "Remove unreachable code paths in constant folding.\n\nChange-Id: I7ffb361711c87f6b1b98d172d2cfdf9b2ba65607\n" }, { "commit": "f8a08735f6c60ddee2bb5646dcad561361cce079", "tree": "36111e04c122c0c4f58dc1ea24a327c9069f72d6", "parents": [ "99da8cd8818e015b21d1c2af73ac40749698c930", "916cc1d504f10a24f43b384e035fdecbe6a74b4c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Feb 18 15:56:59 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Feb 18 15:56:59 2016 +0000" }, "message": "Merge \"Implement polymorphic inlining.\"" }, { "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": "31dd3d60491148d345c1edae1ccd090a1b67dd2b", "tree": "57e6fc0663a39dd5abd6284adc822e8782c4bdf0", "parents": [ "88f38e9b4e9ba0aca3d6fd4823be3a7226c87f33" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Feb 16 12:21:02 2016 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Feb 17 13:52:33 2016 +0000" }, "message": "Extend constant folding to float and double operations.\n\nChange-Id: I2837064b2ceea587bc171fc520507f13355292c6\n" } ], "next": "55bd749991f9a0a73f612696e1a93e739380546b" }