)]}' { "log": [ { "commit": "d3e9c62976780e830da79ae32be4192dee196db2", "tree": "bf7855545f49ea039c6824d340ce2a162ad40ebd", "parents": [ "60ef3997cbcd866c505e51ecde7f06a0535110a0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Aug 05 12:20:28 2020 +0100" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Mon Aug 10 09:17:34 2020 +0000" }, "message": "ARM: Allow FP args in core regs for @CriticalNative.\n\nIf a float or double argument needs to be passed in core\nregister to a @CriticalNative method due to soft-float\nnative ABI, insert a fake call to Float.floatToRawIntBits()\nor Double.doubleToRawLongBits() to satisfy type checks in\nthe compiler.\n\nWe cannot do that for intrinsics that expect those inputs in\nactual FP registers, so we still prevent such intrinsics\nfrom using `kCallCriticalNative`. This should be irrelevant\nif an actual intrinsic implementation is emitted. There are\ncurrently two unimplemented intrinsics that are affected by\nthe carve-out, namely MathRoundDouble and FP16ToHalf, and\nfour intrinsics implemented only when ARMv8A is supported,\nnamely MathRint, MathRoundFloat, MathCeil and MathFloor.\n\nTest: testrunner.py --target --32 -t 178-app-image-native-method\nBug: 112189621\nChange-Id: Id14ef4f49f8a0e6489f97dc9588c0e6a5c122632\n" }, { "commit": "2446feb60919ceb09b5cab388291a86ccdc9be10", "tree": "c5293d846db8b93e3a3fc660ba5d07e5a56be4af", "parents": [ "2091019e6c3ca07454b97730f29bb7200fa9d46e" ], "author": { "name": "Evgeny Astigeevich", "email": "evgeny.astigeevich@linaro.org", "time": "Mon Jul 27 12:25:49 2020 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Jul 28 13:36:12 2020 +0000" }, "message": "Dump ISA into .cfg\n\nThe optimizing compiler dumps ISA features into .cfg which can be used\nby the checker. A new tool perf2cfg needs to know ISA a CFG corresponds\nto. The tool uses this to check that input perf data has the same ISA.\n\nThis CL implements in the compiler dumping ISA in addition to ISA features:\n\nbegin_compilation\n name \"isa:some_isa isa_features:feature1,-feature2\"\n method \"isa:some_isa isa_features:feature1,-feature2\"\n date 1580721972\nend_compilation\n\nBug: 147876827\nTest: ./art/tools/checker/run_unit_tests.py\nTest: test.py --target --optimizing --jit --interpreter\nTest: test.py --host --optimizing --jit --interpreter\nChange-Id: I189eae0f4de61e6a49c01d925e7136b5f7027c91\n" }, { "commit": "52d5354a399b4581222d0f02f2677927b48985f7", "tree": "c6e9beb18d514c0d586fe8b578a15ddfd9ce532e", "parents": [ "40b0614be3296e163654c4e293793d00bcf36a5a" ], "author": { "name": "Fabio Rinaldi", "email": "fabio.rinaldi@linaro.org", "time": "Mon Feb 10 17:28:06 2020 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Jul 22 14:37:34 2020 +0000" }, "message": "Dump instruction set features in .cfg\n\nThis commit adds a compilation block at the beginning of the .cfg\ndumped by the optimizing compiler when --dump-cfg is enabled.\n\nThe compilation block appears in the following form:\n\nbegin_compilation\n name \"isa_features:feature1,-feature2\"\n method \"isa_features:feature1,-feature2\"\n date 1580721972\nend_compilation\n\nThis compilation block dump is passed to checker script (see\nhttps://android-review.googlesource.com/c/platform/art/+/1290997)\nfor checking if a certain instruction set feature was used at compile\ntime.\n\nAuthor: Fabio Rinaldi\nCommitter: Artem Serov\n\nBug: 147876827\nTest: ./art/tools/checker/run_unit_tests.py\nTest: test.py --target --optimizing\nChange-Id: If4309af4bab892f715aad1d3bd338f8ee11e497c\n" }, { "commit": "0d60a2b1eaa2cd2ec3481e49578b77405353efa1", "tree": "91f16b9321ef7631aff731b10da2024c6c9d2029", "parents": [ "289bd1cccdb3aa37e2d129980f5c151f52f84897" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 17 14:31:56 2020 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jun 18 08:37:10 2020 +0000" }, "message": "Introduce an enum for the compilation kind.\n\nTest: test.py\nChange-Id: I5329e50a6b4521933b6b171c8c0fbc618c3f67cd\n" }, { "commit": "6587d9110bd7f836e43db16f3f676da996218aef", "tree": "437d06a8e60fd70aaafaf2b167dfe636a303c68a", "parents": [ "1912a5c7b9400009e361b0db52da77cc78f1cd77" ], "author": { "name": "Evgeny Astigeevich", "email": "evgeny.astigeevich@linaro.org", "time": "Fri Jun 12 10:51:43 2020 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 17 08:00:58 2020 +0000" }, "message": "ART: Simplify HRem to reuse existing HDiv\n\nA pattern seen in libcore and SPECjvm2008 workloads is a pair of HRem/HDiv\nhaving the same dividend and divisor. The code generator processes\nthem separately and generates duplicated instructions calculating HDiv.\n\nThis CL adds detection of such a pattern to the instruction simplifier.\nThis optimization affects HInductionVarAnalysis and HLoopOptimization\npreventing some loop optimizations. To avoid this the instruction simplifier\nhas the loop_friendly mode which means not to optimize HRems if they are in a loop.\n\nA microbenchmark run on Pixel 3 shows the following improvements:\n\n | little cores | big cores\narm32 Int32 | +21% | +40%\narm32 Int64 | +46% | +44%\narm64 Int32 | +27% | +14%\narm64 Int64 | +33% | +27%\n\nTest: 411-checker-instruct-simplifier-hrem\nTest: test.py --host --optimizing --jit --gtest --interpreter\nTest: test.py --target --optimizing --jit --interpreter\nTest: run-gtests.sh\n\nChange-Id: I376a1bd299d7fe10acad46771236edd5f85dfe56\n" }, { "commit": "077188411c692f82b0785597fee030810a2a5841", "tree": "f74ced58d91dcb215601175dc7d29854d46aee0d", "parents": [ "1715efa0b46d57d587237829d1c0695aaca2c344" ], "author": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Mon Feb 24 18:51:42 2020 +0000" }, "committer": { "name": "Ulyana Trafimovich", "email": "skvadrik@google.com", "time": "Tue Jun 09 13:11:45 2020 +0000" }, "message": "ART: Introduce predicated vector instructions.\n\nThis CL introduces a minimal changes to the IR to support\nautovectorization with use of predicated execution of SIMD\ninstructions (e.g. Arm SVE).\n\nTest: test-art-target, test-art-host.\nChange-Id: Ibb7c5520fec6b858fb29f0dde19ec65501831a3a\n" }, { "commit": "ef898425c975f150caaed077ca204fa86b951e7f", "tree": "1ad038b90bb860fe1b9a20872b990c7918fcd1e1", "parents": [ "f7290cac4af6a981d98122af1a6d48b0e80da574" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 08 10:26:06 2020 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 08 12:41:04 2020 +0000" }, "message": "Run LSA as a part of the LSE pass.\n\nMake LSA a helper class, not an optimization pass. Move all\nits allocations to ScopedArenaAllocator to reduce the peak\nmemory usage a little bit.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nChange-Id: I7fc634abe732d22c99005921ffecac5207bcf05f\n" }, { "commit": "695348f4b0541f4373b46eac5830cdd87f71c076", "tree": "f2f6019f0c394f99aaaf9f2f7deec16bf6116b0f", "parents": [ "1f5300a211202442a07607830c6550773ca50b50" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 19 14:42:02 2020 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 20 08:41:09 2020 +0000" }, "message": "Add compiler type to CompilerOptions.\n\nLet CompilerOptions hold the information whether it is AOT\nor JIT compilation, or Zygote JIT for shared code.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing --jit\nTest: aosp_taimen-userdebug boots.\nChange-Id: Id9200572406f8e43d99b8b61ef0e3edf43b52fff\n" }, { "commit": "02ca05a5a6e3f5028c6c2987a81be481d07bc617", "tree": "a364c4a46c573fdfddf607b0e78e5fd3f455c17f", "parents": [ "5868adaefe72cc8bcdcd8325c40f712375a506d1" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 12 13:58:51 2020 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 13 08:00:22 2020 +0000" }, "message": "Move HandleCache to HGraph.\n\nThis avoids passing the `VariableSizedHandleScope*` argument\naround and eliminates HGraph::inexact_object_rti_ and its\ninitialization. The latter shall allow running Optimizing\ngtests that do not require type information without creating\na Runtime in future. (To be implemented in a separate CL.)\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optmizing\nTest: aosp_taimen-userdebug boots.\nChange-Id: I36fe9bc556c6d610d644c8c14cc74c9985a14d64\n" }, { "commit": "5868adaefe72cc8bcdcd8325c40f712375a506d1", "tree": "a1d4328902c4e860fe69c4e4bb34052de2530df3", "parents": [ "5a62af5dc9e9bafeffcac7820e1a5b7586e58477" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 12 11:50:34 2020 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 13 08:00:22 2020 +0000" }, "message": "Move implementations from class_root.h to -inl.h .\n\nMake it possible to include the definition of enum ClassRoot\nwithout pulling in a lot of other headers.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: aosp_taimen-userdebug boots.\nChange-Id: Ic90fdd70bfe0c5428a5c9a0d7901ea7e15b03488\n" }, { "commit": "4fa07a5727551018e2dcd93d41dac98f20212e99", "tree": "8214358e517601c67706a4fbbb4010408a45780a", "parents": [ "d31def587a914c1d306355a7331c24d7b13ad5ca" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Mar 31 20:52:09 2020 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Apr 01 17:19:07 2020 +0100" }, "message": "Add explicit compiler option to enable test-specific features.\n\nWe can no longer rely on checking the \"core.art\" image name,\nsince we plan to remove it and use the shipped boot image.\n\nThis option enables test-specific features, such as $noinline$.\n\nTest: ./art/test.py -r --optimizing --64\nBug: 147817558\nChange-Id: Iadac6e1b0f46e83efd0551fb8462a6b268ad33d8\n" }, { "commit": "41617b18f1c09e3031710d58fdb93c5aa43399ac", "tree": "36a0f3e3dc27e97980b96e1150ede718aee775fa", "parents": [ "842555d72ee7511c193a65f34841cc92170a1850" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Mar 18 21:19:06 2020 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Mar 25 14:10:23 2020 +0000" }, "message": "Add more logging and sanity checks for JIT mini-debug-info.\n\nUsed when diagnosing b/151137723. Keep it around.\n\nBug: 151137723\nTest: test.py -r --jit\nChange-Id: I10cc613c7396607e221fdc1f5972d26c1ac03fa8\n" }, { "commit": "30fd85157260c91327c6b5a0816d312dd505c0e0", "tree": "355823ab233177d529baa873911cf08bb5b5deec", "parents": [ "80dc7dc20855bf680fa598127f26e6047821bdd0" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Feb 20 20:27:58 2020 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Mar 12 12:10:49 2020 +0000" }, "message": "Refactor RemoveNativeDebugInfoForJit.\n\nThis is partial revert of CL/1099280 (Remove global maps).\n\nIt somewhat resurrects the lazy method removal.\n\nThe original goal was to only remove methods from the GC,\nand do all of them in bulk for simplicity and efficiency.\n\nHowever, this is proving infeasible since we have several\ncorner cases which remove methods outside the GC code path.\n\nThe behaviour for the GC code path is preserved by this CL.\nInstead of passing method array, the methods are individually\nmarked for removal and then repacking is immediately forced.\nThe only difference is that coroner cases are done lazily.\n\nTest: ./art/test.py -b -r --host --jit --64\nChange-Id: I42729545d6b51df788d92f9cf149a6e065b90c68\n" }, { "commit": "6a8e66c753abdb53847107c1cb2f13e9114c811d", "tree": "ea8ca2171ca63818e4d9c3dcdd1d9d80af16d96e", "parents": [ "725da8fb9665abfb9c9c6aaca147120e46381b2d" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Feb 20 19:15:49 2020 +0000" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Sat Feb 22 07:24:40 2020 +0000" }, "message": "Remove JIT native debug info on failed commits.\n\nThis could happen if single-implementation assumptions are invalid,\nand would result in temporary double entries in JIT debug info.\n\nTest: ./art/test.py -b -r --host --gcstress --jit --32 -t 068\nChange-Id: I765c9310b6cc4fa440e4121780f8b9053388bd3d\n" }, { "commit": "54f4fbd1a6834f06dc9b644b865423fdc03afb15", "tree": "4f55b2a196453a8a197a1787a688cc299682e55d", "parents": [ "2d3de3a40015af07f7645a298f77b398af0c6c2c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 12 10:52:22 2020 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 13 09:14:02 2020 +0000" }, "message": "Remove MIPS support from Optimizing.\n\nTest: aosp_taimen-userdebug boots.\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 147346243\nChange-Id: I97fdc15e568ae3fe390efb1da690343025f84944\n" }, { "commit": "002eac6f818dc89c75487ba928fbe5901e4f82e8", "tree": "f49de3d98802b5bd95ed51a3b46b735f58c77f6d", "parents": [ "75bca74ab46799f0b9a2663184efe4735b3bd8c1" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 04 10:16:51 2020 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 04 13:22:16 2020 +0000" }, "message": "Only flush visualizer output at the end of the compilation.\n\nOtherwise multiple threads compiling will mix their output.\n\nTest: run-tests\nBug: 147094528\nChange-Id: I905c8874cbe2c02c2d79577024d4606a3784b67a\n" }, { "commit": "a59af8aeaad8fe7d68d8f8de63eab9cf85b6ab31", "tree": "83195c74b135731cc4555254763a8f449691c1b0", "parents": [ "5c8cc64b5f1580faf510f27527e7e22987174963" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Nov 27 17:42:32 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 03 14:32:09 2019 +0000" }, "message": "JIT baseline: trigger optimized compilation on hotness threshold.\n\n- Add a new hotness count in the ProfilingInfo to not conflict with\ninterpreter hotness which may use it for OSR.\n- Add a baseline flag in the OatQuickMethodHeader to identify baseline\ncompiled methods.\n- Add a -Xusetieredjit flag to experiment and test.\n\nBug: 119800099\nTest: test.py with Xusetieredjit to true\n\nChange-Id: I8512853f869f1312e3edc60bf64413dee9143c52\n" }, { "commit": "7c9cfe8b3f986e7cbc18350ad7b9b72f58f5846c", "tree": "838641fd0cecba43abb2a1382aa41c4e2ccc00f1", "parents": [ "d3ed4ada0d2bf6ffa0e997692734fe5016d365e3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Nov 25 17:40:21 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 26 10:31:02 2019 +0000" }, "message": "Honor the --baseline flag for the JIT.\n\nTest: art -Xcompiler-option --baseline uses the baseline compiler.\nChange-Id: I2a5a3ed0038ca965ab1f0cebffb7900e90bae43a\n" }, { "commit": "0a51605ddd81635135463dab08b6f7c21b58ffb0", "tree": "820f338333010f4d6e4b543ffea47e9ff7d2dd3f", "parents": [ "e2727154f25e0db9a5bb92af494d8e47b181dfcf" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 14 13:00:44 2019 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 14 13:00:44 2019 +0000" }, "message": "Revert \"Make compiler/optimizing/ symbols hidden.\"\n\nThis reverts commit e2727154f25e0db9a5bb92af494d8e47b181dfcf.\n\nReason for revert: Breaks ASAN tests (ODR violation).\nBug: 142365358\n\nChange-Id: I38103d74a1297256c81d90872b6902ff1e9ef7a4\n" }, { "commit": "e2727154f25e0db9a5bb92af494d8e47b181dfcf", "tree": "0ddc6eec3ad9508d7eb1f2b941786dd0ed7d2cd8", "parents": [ "c78860b91ae07eed92f86876e7a03132adea6fcd" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Oct 10 10:46:42 2019 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 14 08:22:00 2019 +0000" }, "message": "Make compiler/optimizing/ symbols hidden.\n\nMake symbols in compiler/optimizing hidden by a namespace\nattribute. The unit intrinsic_objects.{h,cc} is excluded as\nit is needed by dex2oat.\n\nAs the symbols are no longer exported, gtests are now linked\nwith the static version of the libartd-compiler library.\n\nlibart-compiler.so size:\n - before:\n arm: 2396152\n arm64: 3345280\n - after:\n arm: 2016176 (-371KiB, -15.9%)\n arm64: 2874480 (-460KiB, -14.1%)\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing --jit\nBug: 142365358\nChange-Id: I1fb04a33351f53f00b389a1642e81a68e40912a8\n" }, { "commit": "adb66f967432ff1f0d51726923a64943c9f1d35b", "tree": "1444900ef83edb837c446361ef38bbf17c17f50c", "parents": [ "ff258063f2748aadf0ea83d0d284b1bedaf792d9" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Oct 10 12:59:43 2019 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Oct 10 17:34:37 2019 +0000" }, "message": "Revert^2 \"JIT: Separate code allocation and initialization.\"\n\nThis reverts commit 63b0c26aae3e7237166dd781eb7a15fbc7c091c2.\n\nTest: ./art/test.py -b -r --host --all-gc -t 708\nReason for revert: Reland after bug fix.\nChange-Id: Ic13e2799bf4bdd8ca468f72cc0f3b72f224f2b08\n" }, { "commit": "63b0c26aae3e7237166dd781eb7a15fbc7c091c2", "tree": "5713f52331c0cd3b881df80f4314f9ea2a0f09db", "parents": [ "e1b36f09cb478a39ba443f6acb11cd1901c01c1d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 09 20:12:15 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 09 20:12:15 2019 +0000" }, "message": "Revert \"JIT: Separate code allocation and initialization.\"\n\nThis reverts commit e1b36f09cb478a39ba443f6acb11cd1901c01c1d.\n\nReason for revert: Crashes in art::jit::JitMemoryRegion::FreeCode\nhttps://logs.chromium.org/logs/art/buildbucket/cr-buildbucket.appspot.com/8900060997388741808/+/steps/test_libcore/0/stdout\n\nChange-Id: I40259baec9acbb7889732548f8b31e0a48651cd9\n" }, { "commit": "e1b36f09cb478a39ba443f6acb11cd1901c01c1d", "tree": "75642c703fee9fce7cd09af344dd26d485379b13", "parents": [ "449011217844401ed491ad0f89dcba21cc26bfc4" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Oct 04 17:44:33 2019 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Oct 09 15:51:56 2019 +0000" }, "message": "JIT: Separate code allocation and initialization.\n\nAllocate(reserve) both code and data first,\nand then initialise(commit) both of them.\n\nThis is useful since we know the address sooner\nfor the purpose of debug info generation.\n\nTest: ./art/test.py -b -r --jit --host --64\nChange-Id: I4971a8801004efbc6c2b27884834dda775b72664\n" }, { "commit": "444e998f5987007bd2783b4d88a9970d1a17736f", "tree": "ef8a93058e973c2c0cf022ea2265679cbb783580", "parents": [ "44ab2cd97c7f4086013a766f442d2f9f619efcc8" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Oct 02 17:59:23 2019 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Oct 08 13:43:16 2019 +0000" }, "message": "JIT mini-debug-info: Generate the debug info sooner.\n\nWe need to add the debug info before the entry-point is set,\notherwise there is window of time when we are missing it.\n\nTest: test.py -b -r --optimizing --jit --host --64\nChange-Id: I3bbad0c96d68e9603bac131faee498b8b9f62699\n" }, { "commit": "87fb032ee1e7ae98df26c646c450ef44e23fc805", "tree": "6754b2d2a0e38277885a691d1be6f0d796478cc7", "parents": [ "a86a5d162e6b59a32e8ea7991e6c8a157aca5a0a" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Aug 20 10:34:02 2019 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Sun Sep 01 21:08:43 2019 +0000" }, "message": "Fix JIT data dual mapping for apps.\n\nWe don\u0027t use it now, but it would be nice to make it functional.\n\nMark the read-only memory as const, and fix the compile errors.\n\nTest: test.py -b --host --jit\nBug: 119800099\nChange-Id: Ic1c45072f3c97f560e843f95fb87b95f754c6e03\n" }, { "commit": "9ac8e4327bd732e4b3b9b8dc6a29560013338d21", "tree": "b56840a0c7b0874f73e5db7259861c04f87df796", "parents": [ "29e740fe64a8caeb04e0d686b4d1e290ba5ad1e6" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Aug 13 13:16:13 2019 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Aug 22 20:00:55 2019 +0000" }, "message": "JIT mini-debug-info: Allocate entries in the JIT data space.\n\nTest: test.py -b --host --jit\nTest: device boots\nBug: 119800099\nChange-Id: I7efa1e6e6660239cbd6438b829e08dd9cd079343\n" }, { "commit": "8fc2f95291206806599d4f2a50da529da85155b6", "tree": "c3c9de9a9a925d6ab790f6db466d1f9c17ed1010", "parents": [ "9ce340f829f836560278ecd078fbefcf19c9d629" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Jul 31 18:40:09 2019 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Aug 16 09:10:55 2019 +0000" }, "message": "JIT mini-debug-info: Remove global maps.\n\nKeep the extra bookkeeping information in JITCodeEntry.\n\nAlso do the compression eagerly during GC rather then lazily.\n\nTest: test.py -b --host --jit\nBug: 119800099\nChange-Id: Ie6cc682033a32c01d4c2cac242d8a4201116f940\n" }, { "commit": "7cfc8f5b2a7c1af68de5f2e79652cf55954794dc", "tree": "bef388a5083ed2b9cf497cd6689489828f2ae13e", "parents": [ "70b93ffca22d665db661a7f15f2f1b59dc60839a" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Aug 07 10:41:09 2019 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Aug 07 15:49:47 2019 +0000" }, "message": "Don\u0027t compile OSR methods that have phi equivalents at loop entry.\n\nWe currently don\u0027t handle this in the stack map, where we only encode\none stack slot for a dex register.\n\nBug: 136698025\nTest: 721-osr\nChange-Id: Ib395ed1165387ad5446a463c307cc0a45e365885\n" }, { "commit": "a48c3dfcebece7ee4b893757b715319f0b5b1fa7", "tree": "eb846bcf242f7e9f4243196b5f4546b0a222cfa0", "parents": [ "1459929a7e9ea96932b6c24597e1692b9b05606c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jun 27 13:11:12 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jun 28 06:37:34 2019 +0000" }, "message": "Revert \"Revert \"Make the JIT zygote memory shared.\"\"\n\nThis reverts commit 2fef66b294417d447630f9d98de68227eef476d3.\n\nBug: 119800099\nBug: 136110523\n\nReason for revert: Fixed webview_zygote case.\n\nChange-Id: Iaae8c999463d77b7b1e62b55458493bdbc97a104\n" }, { "commit": "2fef66b294417d447630f9d98de68227eef476d3", "tree": "59fc4ea6cf13217eb92fbdd4e39561b5ad518cf0", "parents": [ "21d5994583c679cd5d8573b5d35dbd659bdca2c7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 26 22:00:02 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 26 22:00:38 2019 +0000" }, "message": "Revert \"Make the JIT zygote memory shared.\"\n\nThis reverts commit 05f87217ddc9b4b9186710c0135b918f456c5aef.\n\nBug: 119800099\nBug: 136110523\n\nReason for revert: testWebview flaking\n\nChange-Id: I96afa6bc9c56c4aaf5ed72ae370f6f69c096c559\n" }, { "commit": "05f87217ddc9b4b9186710c0135b918f456c5aef", "tree": "6e7bc0d2a3d8faa94c303d7d753319f3850fadcd", "parents": [ "8d335b61d637fa9b040eb9d559dbac98067467f1" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 19 10:00:00 2019 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 26 12:24:15 2019 +0000" }, "message": "Make the JIT zygote memory shared.\n\nTest: boots\nBug: 119800099\nChange-Id: I75ff8a58eea4de5cb833139641b4e15b8394d9b1\n" }, { "commit": "00a37fff76278cd19cf59568df87495f6663c4b6", "tree": "8dc0f95daf800d36ac5da12faa658416b0dcc6a0", "parents": [ "6c0c7c8226a80d0e56f071118f46e4491cfa7754" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jun 20 14:27:22 2019 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jun 24 15:40:45 2019 +0000" }, "message": "Move code writing to data portion of the JIT cache in JitMemoryRegion.\n\nSo there is only one method that needs to write to the data portion\nof the region.\n\nTest: test.py --jit\nChange-Id: I5b7dc442526da54f83a208cb70c8cf86fad6ebcd\n" }, { "commit": "7f7539b8ed0a31de1d1b6e1f0b730d55e37666a6", "tree": "d1611f06e3e468ff61c4d4bc349777b85b3a2e16", "parents": [ "93adcb53c77f4f04dfebd30b94e8ea9936aa8abb" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jun 06 16:20:54 2019 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jun 10 07:32:49 2019 +0000" }, "message": "Pass the memory region to allocate into to the compiler / allocation.\n\nTest: test.py\nBug: 119800099\nChange-Id: Ie3cba5abe3dd4f8756af5ecfd6c26320de314fe8\n" }, { "commit": "36ec6c7314b824f03f955a5c0c2cbce56ef63412", "tree": "cb3b72b2eefc25dd6ce59219d4d685145566d48e", "parents": [ "c36a8ccf44454f1a4b8b57420205faf11dec7046" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Apr 26 12:46:08 2019 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Apr 29 16:33:51 2019 +0000" }, "message": "Fix DWARF line-number generation for JITed methods.\n\nDon\u0027t strip, repack or compress debug-info if explicitly\nrequested by the developer (using the -g compiler flag).\n\nIf enabled, the DWARF debug info has about 1:1 size\noverhead relative to JIT code + data.\n\nBug: 131422204\nTest: Check that gdb shows line numbers for JITed method.\nTest: Hard-code always-enable generation and run maps.\nChange-Id: If06de8ae2317af4d57d84e8a8bfae86a597dd4e4\n" }, { "commit": "fe1d952a9912bf597174dc636d0f87db06a8d925", "tree": "db7f3fd5d0b5be8de106426f0bd9316b18594e3e", "parents": [ "34857b5fd3be5ee8cf1935bd1fbaad37a8e42768" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Mar 25 17:19:11 2019 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Mar 27 15:10:09 2019 +0000" }, "message": "Use generic name for JNI trampolines.\n\nJNI trampolines tend to be deduplicated which seems to lead to confusion.\n\nBug: 32949969\nTest: test.py -b --host -r -t 137-cfi\nChange-Id: Ic39998448a05f98fc1e068c4f384f35085121911\n" }, { "commit": "e7e195db8eb04a58c4c467bbe357841fdf621347", "tree": "bbefb03122251b826c644e196519422449157f6a", "parents": [ "557feceeb438946e4a005471bc80935e5c0af195" ], "author": { "name": "Greg Kaiser", "email": "gkaiser@google.com", "time": "Tue Mar 26 08:13:00 2019 -0700" }, "committer": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Mar 26 17:35:30 2019 +0000" }, "message": "optimizing_compiler: Avoid extra std::string copy\n\nThe function OptimizationPassByName() takes a const std::string\nreference, so it\u0027s inefficient to pass it the result of\nstd::string::c_str().\n\nTest: TreeHugger\nChange-Id: I84fc717285b0278264cba3c9504290876025a01d\n" }, { "commit": "bf5f0f3f1eea2e712e7269ff1e1f9b1bcc4cbaee", "tree": "030ef55b56ede3dbf42b6b5f19c59bfc0583636f", "parents": [ "7e057ec0e023910f6bc42bc552a8463343cf346f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 05 15:41:50 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Mar 06 14:21:51 2019 +0000" }, "message": "Revert \"Revert^2 \"Add code size to CodeInfo\"\"\n\nThis reverts commit e1412dacbf1d2a809bd1fca658cc8cb8f61f8ee6.\n\nBug: 123510633\nBug: 127305289\n\nReason for revert: b/127305289\n\nChange-Id: I54557b05a44777f1fa2c15bde4fa648980f42eed\n" }, { "commit": "038924b75f06b91c2a7e944196ca11f118ce182f", "tree": "fb3073918686ec6f54dce65d2697c0e46f05a272", "parents": [ "03c6b174a4e412e9bb62935c4ee15b5445e517f2" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Feb 19 15:09:35 2019 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 20 10:40:05 2019 +0000" }, "message": "ART: Reduce dependencies on CompilerDriver.\n\nPreparation for moving CompilerDriver and other stuff\nfrom libart-compiler.so to dex2oat.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nChange-Id: Ic221ebca4b8c79dd1549316921ace655f2e3f0fe\n" }, { "commit": "e1412dacbf1d2a809bd1fca658cc8cb8f61f8ee6", "tree": "bb3f56ecefe08b66f7a330a02caef0b089a0c2af", "parents": [ "8f20a23a35fa6fbe4dcb4ff70268a24dc7fb2a24" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Feb 13 17:27:17 2019 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Feb 14 09:43:00 2019 +0000" }, "message": "Revert^2 \"Add code size to CodeInfo\"\n\nThis temporarily adds 0.25% to oat file size.\nThe space will be reclaimed back in follow-up CL.\n\nThis reverts commit 8f20a23a35fa6fbe4dcb4ff70268a24dc7fb2a24.\n\nReason for revert: Reland as-is after CL/903819\n\nBug: 123510633\nTest: DCHECK compare the two stored code sizes.\nChange-Id: Ia3ab31c208948f4996188764fcdcba13d9977d19\n" }, { "commit": "8f20a23a35fa6fbe4dcb4ff70268a24dc7fb2a24", "tree": "84dc682b40f38d0377639befe6f3d43e5655a43c", "parents": [ "68efa7b1128486e08ae60cd27181645b27bbd2e4" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Feb 12 19:42:54 2019 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Feb 12 19:42:54 2019 +0000" }, "message": "Revert \"Add code size to CodeInfo\"\n\nThis reverts commit 68efa7b1128486e08ae60cd27181645b27bbd2e4.\n\nReason for revert: Breaks tests\n\nChange-Id: I28fb143990f58e0d5f0b106bea9d9a159f19297e\n" }, { "commit": "68efa7b1128486e08ae60cd27181645b27bbd2e4", "tree": "bb3f56ecefe08b66f7a330a02caef0b089a0c2af", "parents": [ "794350fd0e21aa9e259b6c45394494871e7fdb13" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Jan 28 18:59:23 2019 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Feb 11 13:23:53 2019 +0000" }, "message": "Add code size to CodeInfo\n\nThis temporarily adds 0.25% to oat file size.\nThe space will be reclaimed back in follow-up CL.\n\nBug: 123510633\nTest: DCHECK compare the two stored code sizes.\nChange-Id: I15340824ca637fd075a4cef87771b06cb96bb9f4\n" }, { "commit": "206348cea8b086a484b8bde37b2e281e5f7db638", "tree": "1e83b6484dbd9218ace7c94d266dc90ed7e66505", "parents": [ "58a268aa3569d8ad4f0e831de578620e1079ed59" ], "author": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Wed Dec 05 11:11:33 2018 -0800" }, "committer": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Mon Jan 28 09:37:57 2019 -0800" }, "message": "Selectively allow dead reference elimination\n\nAllow dead reference elimination in methods not containing\n@ReachabilitySensitive accesses or calls, when the class is marked\n@DeadReferenceSafe.\n\nAdd 1339-dead-reference-safe to aggressively check that everything\nworks as intended.\n\nBug: 111453875\n\nTest: art/test/testrunner/testrunner.py --host --64 -t 1339-dead-reference-safe\n\nDetect ReachabilitySensitive annotations.\n\nChange-Id: I70c20431fdbcfcfd2692b2255d12ad59e37cb669\n" }, { "commit": "43ae7791007d6e7bdc9dbf8cb7c9a3f3aec40efd", "tree": "a283f976c1d0a8f5596787313135cb341afd2b97", "parents": [ "a966376ac9ea3bf3b52ae52d47985d4967a99c3d" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Jan 09 17:34:01 2019 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Jan 22 14:01:41 2019 +0000" }, "message": "Add timing logger for mini-debug-info generation.\n\nTest: Check ANR report.\nChange-Id: I7e1b67bf8e1b950d5f67937fd29645b570d0465a\n" }, { "commit": "3f1dcd39e134d994ac88dcc4f30ec8cabcd8decf", "tree": "365d20ad6b68ff1dbd4903764b63880324136e4d", "parents": [ "0f0a4e40667c87fbd4ae5480eddbfd701bfabfa2" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Dec 28 09:39:56 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 02 10:32:25 2019 -0800" }, "message": "ART: Move dex structs into own header\n\nSeparating out the structs from DexFile allows them to be forward-\ndeclared, which reduces the need to include the dex_file header.\n\nBug: 119869270\nTest: m\nChange-Id: I32dde5a632884bca7435cd584b4a81883de2e7b4\n" }, { "commit": "3db70689e3e1c92344d436a8ea4265046bdef449", "tree": "3db08743e968062ed5bdc143233cdb3c4564696b", "parents": [ "1650dafad62578a1766bd617d78458a4cf1e2a9a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Dec 26 15:12:03 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 27 12:56:39 2018 -0800" }, "message": "ART: Refactor for bugprone-argument-comment\n\nHandles compiler.\n\nBug: 116054210\nTest: WITH_TIDY\u003d1 mmma art\nChange-Id: I5cdfe73c31ac39144838a2736146b71de037425e\n" }, { "commit": "0b21e41260bea6c078c18eba745d43e549230fd0", "tree": "25e41920b8a38d01ec904a2d1a1498c1364c07fb", "parents": [ "889e29aae6a7cbd3c6f8dae44f67e054e083a8b6" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Dec 05 13:24:06 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Dec 14 16:47:38 2018 +0000" }, "message": "Pack JIT mini-debug-infos together.\n\nWe currently produce one in-memory ELF file per JITed method,\nwhich has significant overheads due to the ELF file headers.\n\nPack several of those files together regularly to save space.\nSimilarly, prune information about methods which were GCed.\n\nThis reduces the size of JIT mini-debug-info by factor of 10.\nThe final overhead is 5% to 10% relative to the JIT code size.\n\nTest: ./art/test.py -b -r -t 137\nChange-Id: Idfaff8ed9a209e871e815e527f24f36c63a57bbf\n" }, { "commit": "afc60cd6ddcd15353698bb453716313dea241ef8", "tree": "708bbf11e425ddb7a75874f27d272fd5484d0bf1", "parents": [ "58431a7cf863b2eb7fe25eff6a28ec376f2f151d" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Dec 05 11:59:31 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Dec 11 11:32:37 2018 +0000" }, "message": "Cleanup native debug interface api.\n\nStrongly type the DEX file arguments.\n\nRemove the need for callers to worry about locks.\n\nUse std::map instead of std::unordered_map internally.\n\nTest: ./art/test.py -b -r -t 137\nChange-Id: I8bd79b796b5c7da265afc43a07ed227f550116c7\n" }, { "commit": "ed8b4ac3d7a78cc4976fa24a34bac92f4fd47a27", "tree": "8292de7f2a4e8b9c0526d05a604668da11e1980e", "parents": [ "a9a0973ff03ce2c2d1ec65695ad0f4cd8ed170a7", "2afaff7e9171992b0e2e95d0f485782d28e1b9dc" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Dec 07 08:11:24 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Dec 07 08:11:24 2018 +0000" }, "message": "Merge \"Refactor CompilerDriver::CompileAll().\"" }, { "commit": "2afaff7e9171992b0e2e95d0f485782d28e1b9dc", "tree": "96c7ea4a13e8a9a191d58002de89fb005231980e", "parents": [ "6a98f89c4ad645b04d6c80d3d7e260c59bf6f193" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Nov 30 17:01:50 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Dec 06 16:26:43 2018 +0000" }, "message": "Refactor CompilerDriver::CompileAll().\n\nTreat verification results and image classes as mutable\nonly in CompilerDriver::PreCompile(), and treat them as\nimmutable during compilation, accessed through the\nCompilerOptions. This severs the dependency of the inliner\non the CompilerDriver.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nChange-Id: I594a0213ca6a5003c19b4bd488af98db4358d51d\n" }, { "commit": "be50f9ae077b9d5fb4f12f693ba36cc69e9d4073", "tree": "edcd00b15d95aee101dd85574d1ce91b3cb35c0c", "parents": [ "6a98f89c4ad645b04d6c80d3d7e260c59bf6f193" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Dec 05 10:48:42 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Dec 05 11:35:29 2018 +0000" }, "message": "Simplify MakeElfFileForJIT\n\nWe only ever pass single method info to the method and\nI don\u0027t plan to use the method for anything more complex.\n\nThe JIT-related methods also know the relevant ElfTypes\nat compile time, so some of the helpers are not needed.\n\nTest: ./art/test.py -b -r -t 137\nChange-Id: Ie7815a6427ac05be15a79adeb05dccc9150441d2\n" }, { "commit": "85f1c576d228c2c43e9a0dc37f0cbb91fad59c45", "tree": "67d17ea5a7eb6021dabb335730402bd28d72694a", "parents": [ "79c693b3133da397cec7eaff19de631b65a0cf70" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Nov 21 13:52:48 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Nov 21 15:22:07 2018 -0800" }, "message": "ART: Some iwyu for logging.h\n\nRemove over-broad use in headers. Fix up transitive includes.\n\nBug: 119869270\nTest: mmma art\nChange-Id: I518fa7c8bee014b260818fca1fbde6ec47d126da\n" }, { "commit": "33f7c8aa87831945c567f57c8b860f216c4371ae", "tree": "9022cb2fe8bf889c0a32f4478b249145318a7a65", "parents": [ "54c7da9c50ee85ade636605cd6ea18b4c2bc69fa" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Nov 19 10:22:01 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Nov 19 10:45:13 2018 +0000" }, "message": "Remove CompiledMethod dependency on CompilerDriver.\n\nTest: m test-art-host-gtest\nChange-Id: Ibee78d5c54d3ff8162258963fde25065b579a000\n" }, { "commit": "6be1dbd3bc34b2cfa0fe257acbea273f1b6f4a53", "tree": "1840224989e4cbd538081951d948ed6a19bc61b4", "parents": [ "dbcb48fc0d0841f327d108246487a4ef3df556ed" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 13 13:09:51 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Nov 14 10:21:55 2018 +0000" }, "message": "ART: Rewrite core image detection.\n\nRemove unused compiler option `core_image_` and cache\nthe IsCompilingWithCoreImage in the CompilerOptions instead.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nChange-Id: Idcf3b6c96dab9a39f823a16778e7698589442cd8\n" }, { "commit": "dd121f637509a50d6744a0ad596f5dc627e0c60a", "tree": "1e20c636cd40a5a87cae1cd0d0b55940c8111a20", "parents": [ "fe59955fc41e277bf1c60378202ba785abb1e4a8" ], "author": { "name": "Shalini Salomi Bodapati", "email": "shalini.salomi.bodapati@intel.com", "time": "Fri Oct 26 15:03:53 2018 +0530" }, "committer": { "name": "Shalini Salomi Bodapati", "email": "shalini.salomi.bodapati@intel.com", "time": "Thu Nov 08 14:50:29 2018 +0530" }, "message": "Emit bit manipulation instructions for x86 and x86_64\n\nThis patch performs instruction simplification to\ngenerate instructions andn, blsmsk and blsr on\ncpus that have avx2.\n\nTest: test.py --host --64, test-art-host-gtest\nChange-Id: Ie41a1b99ac2980f1e9f6a831a7d639bc3e248f0f\nSigned-off-by: Shalini Salomi Bodapati \u003cshalini.salomi.bodapati@intel.com\u003e\n" }, { "commit": "c1c3452e465e473df499196b387330a793b89680", "tree": "1b144182126bfc73c5d82d1941e1673826d39343", "parents": [ "3b5062083d3e203d1c72b066150b4fe441588a25" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Oct 31 13:56:49 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 01 17:01:31 2018 +0000" }, "message": "Do not cache RequiresConstructorBarrier() results.\n\nAvoid caching the results. Caching was broken for JIT in the\npresence of class unloading; entries for unloaded dex files\nwere leaked and potentially used erroneously with a newly\nloaded dex file.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nTest: Pixel 2 XL boots.\nTest: m test-art-target-gtest\nTest: testrunner.py --target\nBug: 118808764\nChange-Id: Ic1163601170364e060c2e3009752f543c9bb37b7\n" }, { "commit": "a2c211ca97cce4d900a6b2047049ffbe46fd5890", "tree": "4fc383d217f9de000f11b99d2d5e9c834c5c8c70", "parents": [ "0708fc0c096fb4126c2029d0172830cf0c8e159b" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 01 09:50:52 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 01 12:17:35 2018 +0000" }, "message": "Add compiling class to the DexCompilationUnit.\n\nUse it to simplify HInstructionBuilder and reduce the\nnumber of class resolution calls.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nChange-Id: Ib4f9b4ea61235841e653a03a40755a39c7de7504\n" }, { "commit": "acc56ac6a33a23abbf4a9a62528e352ea28c5407", "tree": "e1dd4569fd5b49879ec8dfd1b5ef5204a1084407", "parents": [ "4ca6cc7ddcbd568b365c428371ea9675799775af" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 09 08:45:24 2018 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 10 11:10:08 2018 +0100" }, "message": "Introduce a \u0027baseline\u0027 variant for the compiler.\n\nImplemented as a stripped down version of the optimizing compiler,\nnot running any optimization.\n\nAdjust code to still work with expectations in code generators.\n\nbug: 111397239\n\nTest: test.py --baseline\nChange-Id: I4328283825f9a890616e7496ed4c1e77d6bcc5dd\n" }, { "commit": "0846a8ffafffdf4c18d52f9896b13f40a1c9828d", "tree": "299636dbde6772b4d11ce9ae3a3cd5675f3d0feb", "parents": [ "9ec8559bc2e18075cdcd9a2f09206b698d3c1993" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Sep 12 15:21:07 2018 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Oct 03 10:21:24 2018 +0000" }, "message": "Fix ReplaceUninitializedStringPhis.\n\nMove the analysis after redundant phi and dead phi elimination,\nknowing that only graphs with irreducible loops may still have\na phi as input of the invoke. In such a case, we bail.\n\nbug: 112537407\nTest: 563-checker-fake-string\nChange-Id: Ib9eefa4ce905b7fb418ca9b2a3c26ea4df74ce8f\n" }, { "commit": "0de9c08aefc2465fd1d2fcedec3d3fa2164056fe", "tree": "4c88b795d07b57ebef23accb87523600e8e217fe", "parents": [ "91be58aaa16ecf36642d9c274d48885c84f09f9d", "76d4bb0f3d460671982bf40b9a9365e236501d80" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 28 12:09:52 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Sep 28 12:09:52 2018 +0000" }, "message": "Merge \"Remove need for intrinsic recognizer to be a pass.\"" }, { "commit": "76d4bb0f3d460671982bf40b9a9365e236501d80", "tree": "30cef155340112a3c3628cfd95b9cc6ea49348a3", "parents": [ "8f676995c00154f48d6b68b1656b654d0cd4593c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 21 12:58:45 2018 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 28 10:36:00 2018 +0100" }, "message": "Remove need for intrinsic recognizer to be a pass.\n\nInstead just recognize the intrinsic when creating an invoke\ninstruction.\n\nAlso remove some old code related to compiler driver sharpening.\n\nTest: test.py\nChange-Id: Iecb668f30e95034970fcf57160ca12092c9c610d\n" }, { "commit": "48ba197beaffe64646e21cc06923d8b019c7aa6d", "tree": "fe21827494e10a8d83aea93fb23fa7e67b11de9e", "parents": [ "26f048f48cdb1e884aab2b6fddf26d58346d29ad" ], "author": { "name": "Stephen Hines", "email": "srhines@google.com", "time": "Mon Sep 24 13:35:54 2018 -0700" }, "committer": { "name": "Stephen Hines", "email": "srhines@google.com", "time": "Mon Sep 24 16:31:41 2018 -0700" }, "message": "Fix some performance-unnecessary-value-param tidy and performance-for-range warnings.\n\nart/profman/profile_assistant_test.cc:119:54: error: the const qualified parameter \u0027hot_methods\u0027 is copied for each invocation; consider making it a reference [performance-unnecessary-value-param,-warnings-as-errors]\n const std::vector\u003cuint32_t\u003e hot_methods,\n ^\n \u0026\nart/profman/profile_assistant_test.cc:120:54: error: the const qualified parameter \u0027startup_methods\u0027 is copied for each invocation; consider making it a reference [performance-unnecessary-value-param,-warnings-as-errors]\n const std::vector\u003cuint32_t\u003e startup_methods,\n ^\n \u0026\nart/profman/profile_assistant_test.cc:121:54: error: the const qualified parameter \u0027post_startup_methods\u0027 is copied for each invocation; consider making it a reference [performance-unnecessary-value-param,-warnings-as-errors]\n const std::vector\u003cuint32_t\u003e post_startup_methods,\n ^\n \u0026\nart/runtime/subtype_check_info_test.cc:134:56: error: the parameter \u0027sc\u0027 is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]\n static SubtypeCheckInfo CopyCleared(SubtypeCheckInfo sc) {\n ~~~~~~~~~~~~~~~~ ^\n const \u0026\nart/runtime/class_linker.cc:6451:62: error: the parameter \u0027to_process\u0027 is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]\n std::vector\u003cObjPtr\u003cmirror::Class\u003e\u003e to_process)\n ~~~ ^\n const \u0026\nart/runtime/trace.cc:1127:13: error: loop variable is copied but only used as const reference; consider making it a const reference [performance-for-range-copy,-warnings-as-errors]\n for (auto it : exited_threads_) {\n ~~~~ ^\n const \u0026\nart/runtime/oat_file_manager.cc:154:41: error: the parameter \u0027spaces\u0027 is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]\n std::vector\u003cgc::space::ImageSpace*\u003e spaces) {\n ~~~ ^\n const \u0026\nart/test/004-JniTest/jni_test.cc:707:72: error: the parameter \u0027methods\u0027 is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]\n void TestCalls(const char* declaring_class, std::vector\u003cconst char*\u003e methods) {\n ~~~ ^\n const \u0026\nart/compiler/optimizing/optimizing_compiler.cc:1409:89: error: the parameter \u0027info\u0027 is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param,-warnings-as-errors]\nvoid OptimizingCompiler::GenerateJitDebugInfo(ArtMethod* method, debug::MethodDebugInfo info) {\n ~~~~~ ^\n const \u0026\n\nBug: http://b/32619234\nBug: http://b/110779387\nTest: Build using WITH_TIDY\u003d1\nChange-Id: I911d838b8c26ddab3d6a64024f3220000f078cba\n" }, { "commit": "ac3ac681a1d1d5d5d4662b7e4d3cf8e0b4d53d27", "tree": "803b02eabfb6af718561d0dfd83d61e4bccbfa28", "parents": [ "8402166d0e3b5d91ab867a7dcd7218f824afe3ff" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 20 11:01:43 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 20 15:48:38 2018 +0100" }, "message": "Pass JIT roots as a vector\u003cHandle\u003c\u003e\u003e to JIT cache.\n\nThis avoids creating an object on the heap and thus\nprevents issues for the 904-object-allocation in the\nJIT-at-first-use configuration.\n\nTest: run_build_test_target.py -j48 art-jit-on-first-use\n (test 904 passes; test 1935 still failing).\nBug: 116189667\nChange-Id: I58c0c8cb2d78edc63dab7d72e69b882abbfb79fd\n" }, { "commit": "bdb2ecc8cfd0d6fc2f3f4fa4c65cca84b358cd61", "tree": "a7660c98c22d28bf508fe208845957418e0dee40", "parents": [ "4bd4d2c199c9e0e522526c40303652e29bc7c631" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Sep 18 14:33:55 2018 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Sep 19 14:46:04 2018 +0100" }, "message": "Remove sharpening as an optimization pass.\n\nMake the last sharpening helper (methods) like the other\nhelpers: being invoked by the instruction builder.\n\nTest: test.py\nChange-Id: Ic80a454f9b59b0b4ef7825590b24402500ba851c\n" }, { "commit": "fa6a1b0a1d9d186a0ad9bb413a4e49f1ffd187d8", "tree": "6d3bef945030868d7e78162dc3f4f35a07ce410a", "parents": [ "a0878aa9f4f4102e56a6381b28a8d7c38dec80b9" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Sep 07 08:11:55 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Sep 07 09:13:50 2018 -0700" }, "message": "ART: Continue adding override annotations\n\nUse Clang-tidy\u0027s modernize-use-override to add more annotations. Ignore\ninferred annotations on destructors.\n\nBug: 32619234\nTest: mmma art\nChange-Id: Ic432c928e398d44df9171e42db04ee19946e6887\n" }, { "commit": "bbc6e7edb5fca4a61ac53dd9bce79cb4f0bb3403", "tree": "0fbce767bc383358cf4cd65aafc74140e1850982", "parents": [ "19379b58bd433da91230e4fe6cd96e7416d16adc" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Aug 24 16:58:47 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 28 11:06:07 2018 +0100" }, "message": "Use \u0027final\u0027 and \u0027override\u0027 specifiers directly in ART.\n\nRemove all uses of macros \u0027FINAL\u0027 and \u0027OVERRIDE\u0027 and replace them with\n\u0027final\u0027 and \u0027override\u0027 specifiers. Remove all definitions of these\nmacros as well, which were located in these files:\n- libartbase/base/macros.h\n- test/913-heaps/heaps.cc\n- test/ti-agent/ti_macros.h\n\nART is now using C++14; the \u0027final\u0027 and \u0027override\u0027 specifiers have\nbeen introduced in C++11.\n\nTest: mmma art\nChange-Id: I256c7758155a71a2940ef2574925a44076feeebf\n" }, { "commit": "61ba8d2421a98e9b16510be4f9af7ca7bc4c9055", "tree": "9a79331dd5ebd46ed46f853b24ab072f43debf28", "parents": [ "eb369ce3669be74dd89b21f8b3ab31ace4a47086" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Aug 07 09:55:57 2018 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Aug 20 17:18:31 2018 +0100" }, "message": "Fix longstanding bug around implicit NPEs and GC, version 2.\n\nThe TODO has been there since M (so forever :)):\nhttps://android-review.googlesource.com/c/platform/art/+/122794/13//COMMIT_MSG#13\n\nWe hardly see the issue in our tests as we need to have:\n1) A GC happening while creating the NPE object.\n2) ParallelMoves between the NullCheck and implicit null check operation\n that moves references.\n\nThe CL piggy backs on the \"IsEmittedAtUseSite\" flag, to set implicit\nnull checks with it. The liveness analysis then special cases implicit\nnull checks to record environment uses at the location of the actual\ninstruction that will do the implicit null check.\n\nTest: test.py --gcstress\nTest: run-libcore-tests --gcstress\nbug: 111545159\nChange-Id: I3ecea4fe0d7e483e93db83281ca10db47da228c5\n" }, { "commit": "e7a9194558379ee24fde82fcaeaee29542df1520", "tree": "30f3c43f2d64f159503bb8c3442aa39cc6cb612e", "parents": [ "f50419eee7290542db07bff23c4f65e693ba037f" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Aug 01 17:23:53 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Aug 03 12:48:49 2018 +0100" }, "message": "Simplify StackMapStream encoding.\n\nTest: test-art-host-gtest-stack_map_test\nChange-Id: Ife021d03e4e486043ec609f9af8673ace7bde497\n" }, { "commit": "ced04835d8e3cd3f68576cfffc1d21283ca151b4", "tree": "125ddd1d222f4fb1710e17c76803ad3e92572a5c", "parents": [ "350b6a312222b9b27bfee0e72ce261a45cb60e1c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jul 26 14:42:17 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Aug 02 15:47:02 2018 +0100" }, "message": "Reuse arena memory for each block in scheduler.\n\nThis reduces the peak memory used for large methods with\nmultiple blocks to schedule.\n\nCompiling the aosp_taimen-userdebug boot image, the most\nmemory hungry method BatteryStats.dumpLocked has the\nScheduler memory allocations in ArenaStack hidden by the\nregister allocator:\n - before:\n MEM: used: 8300224, allocated: 9175040, lost: 197360\n Scheduler 8300224\n - after:\n MEM: used: 5914296, allocated: 7864320, lost: 78200\n SsaLiveness 5532840\n RegAllocator 144968\n RegAllocVldt 236488\nThe total arena memory used, including the ArenaAllocator\nnot listed above, goes from 44333648 to 41950324 (-5.4%).\n(Measured with kArenaAllocatorCountAllocations\u003dtrue,\nkArenaAllocatorPreciseTracking\u003dfalse.)\n\nAlso remove one unnecessary -Wframe-larger-than\u003d workaround\nand add one workaround for large frame with the above arena\nalloc tracking flags.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nBug: 34053922\nChange-Id: I7fd8d90dcc13b184b1e5bd0bcac072388710a129\n" }, { "commit": "8cd54547cec8a4537db5682c2da8be22843b1310", "tree": "fb1158993bab2e027984cedab59b402c051a45a7", "parents": [ "91f0fdb4372d3f2bcfcd9db67afcbe7ee1901048" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Sun Jul 15 23:58:44 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Aug 01 14:49:40 2018 +0100" }, "message": "Move MethodInfo to CodeInfo.\n\nThere is no need to treat it specially any more,\nbecause of the de-duplication at BitTable level.\n\nThis saves 0.6% of oat file size.\n\nTest: test-art-host-gtest\nChange-Id: Ife7927d736243879a41d6f325d49ebf6930a63f6\n" }, { "commit": "8808756b8fba036a9c73a45c800a56be09872364", "tree": "9b28f314ff516d1d9b85d07f71bdc77fad465e48", "parents": [ "dda4e8b5f2ffed24a2dcdb99d2301d9fc0eb91e2" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Sat Jun 23 22:05:56 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Jul 27 15:03:06 2018 +0100" }, "message": "Remove frame info from OatQuickMethodHeader.\n\nThe information has been moved to CodeInfo,\nwhere it is stored in much more compact way.\n\nThe old CL which added the data to CodeInfo cost 0.7%.\nThis CL saves 2.5% of .oat file size so a win overall.\n\nTest: test-art-host-gtest\nChange-Id: I07fcf6f2776c96218f995ba3b57a1e6ccbf5e317\n" }, { "commit": "cdfc942e60032622b5a4379d0dd5ca914ba6393a", "tree": "823e767f6eba6fb2831f69753936bc0c35eea84e", "parents": [ "ec1f1a91328f44d93cfc16e39160dbdfce2f7b9a", "f5f56c791c5853f43a2a9781c98d5776c7dd5a59" ], "author": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Fri Jul 13 16:35:30 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jul 13 16:35:30 2018 +0000" }, "message": "Merge \"Revert \"Emit vector mulitply and accumulate instructions for x86.\"\"" }, { "commit": "f5f56c791c5853f43a2a9781c98d5776c7dd5a59", "tree": "ed8270e3a5d0161ebe5bec0606a24cd5e3123e59", "parents": [ "61908880e6565acfadbafe93fa64de000014f1a6" ], "author": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Fri Jul 13 00:05:27 2018 +0000" }, "committer": { "name": "Hans Boehm", "email": "hboehm@google.com", "time": "Fri Jul 13 00:05:27 2018 +0000" }, "message": "Revert \"Emit vector mulitply and accumulate instructions for x86.\"\n\nThis reverts commit 61908880e6565acfadbafe93fa64de000014f1a6.\n\nReason for revert: By failing to round multiply results, it does not follow Java rounding rules.\n\nChange-Id: Ic0ef08691bef266c9f8d91973e596e09ff3307c6\n" }, { "commit": "afc97bca07c85d4c3b46801c557edd12d681fc96", "tree": "fd53094370a28aa4ba4b9d419e05ff7797cf51ae", "parents": [ "564fa8a1f3e3c39793c9b146ed5f21650617dc3f" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Jul 05 08:14:35 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Jul 05 13:04:43 2018 +0100" }, "message": "Revert^2 \"Add CodeInfo to JNI methods.\"\n\nIt has no stack maps. We only store the frame info there.\n\nAs a consequence of having CodeInfo, the JNI methods are\nnow classified as IsOptimized().\n\nThis does not have any real effect on .oat file size.\n\nThis reverts commit 564fa8a1f3e3c39793c9b146ed5f21650617dc3f.\n\nTest: test-art-host-gtest\nTest: test-art-target-gtest-jni_compiler_test32\nChange-Id: Ic7a1949027d89ba97cfedfc8ea453f041193b6a7\n" }, { "commit": "564fa8a1f3e3c39793c9b146ed5f21650617dc3f", "tree": "b9e960a55f5643e65ecd05e258f9ac1662288bcc", "parents": [ "85f3934f364e1674bfd2a6813274918576acdf60" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Jul 05 00:01:10 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Jul 05 00:01:10 2018 +0000" }, "message": "Revert \"Add CodeInfo to JNI methods.\"\n\nThis reverts commit 85f3934f364e1674bfd2a6813274918576acdf60.\n\nReason for revert: breaks gtests on target\n\nChange-Id: I9f97fe4e7557027677824974568120f658bbd53a\n" }, { "commit": "85f3934f364e1674bfd2a6813274918576acdf60", "tree": "7dde44e022bfe43439982e6d2c699f5acc73efcf", "parents": [ "0162450a86c1586c8e0d28710ea0280bfa64ae32" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Sat Jun 23 22:05:52 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Jul 04 13:51:50 2018 +0100" }, "message": "Add CodeInfo to JNI methods.\n\nIt has no stack maps. We only store the frame info there.\n\nAs a consequence of having CodeInfo, the JNI methods are\nnow classified as IsOptimized().\n\nThis does not have any real effect on .oat file size.\n\nTest: test-art-host-gtest\nChange-Id: I8d03a3af347dc7239719bb3c39ecf84c6eaece80\n" }, { "commit": "a8d5a5f709cc7611352d58fb6f2fb50411563653", "tree": "be3f0cf1d59e5797100cae0d859e12605ac0b378", "parents": [ "7583d915800ad330da0232dfdb0a4d1bda2d074c", "8a62a4c9570b345b715a022d33d443413a634392" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jul 04 07:50:56 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Jul 04 07:50:56 2018 +0000" }, "message": "Merge \"Handle String.\u003cinit\u003e with a phi input.\"" }, { "commit": "8a62a4c9570b345b715a022d33d443413a634392", "tree": "4b34426f14301f230e9d6b5f23486a038cf6e548", "parents": [ "b5271dd44a30f498689e503340d3c8d01bf31f07" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jul 03 09:39:07 2018 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jul 03 11:22:34 2018 +0100" }, "message": "Handle String.\u003cinit\u003e with a phi input.\n\nWe wrongly assumed only irreducible loops could lead\nto this situation, but any loop can actually be in between\na String NewInstance and a String.\u003cinit\u003e.\n\nbug: 109666561\nTest: 563-checker-fakestring\nChange-Id: I018a22f7e22c15e140252544415f51d544f7cc13\n" }, { "commit": "61908880e6565acfadbafe93fa64de000014f1a6", "tree": "40b535db9175f3d959364d5bc30eaab4e2c4b4c4", "parents": [ "b5271dd44a30f498689e503340d3c8d01bf31f07" ], "author": { "name": "Gupta Kumar, Sanjiv", "email": "sanjiv.kumar.gupta@intel.com", "time": "Fri Jun 29 13:06:35 2018 +0530" }, "committer": { "name": "Gupta Kumar, Sanjiv", "email": "sanjiv.kumar.gupta@intel.com", "time": "Mon Jul 02 15:37:38 2018 +0530" }, "message": "Emit vector mulitply and accumulate instructions for x86.\n\nThis patch adds a new cpu vaiant named kabylake and performs\ninstruction simplification to generate VectorMulitplyAccumulate.\n\nTest: ./test.py --host --64\n\nChange-Id: Ie6cc882dadf1322dd4d3ae49bfdb600b0c447765\nSigned-off-by: Gupta Kumar, Sanjiv \u003csanjiv.kumar.gupta@intel.com\u003e\n" }, { "commit": "a043111e3a2c09b549708a6227a1f54d91da76aa", "tree": "393fe11cfceccebf474e4bdf36ff79b70b97f589", "parents": [ "213ee2da6a1c58d0fc12c937bbd9c9974ca00aca" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 25 09:32:54 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jun 25 18:43:19 2018 +0100" }, "message": "Move instruction_set_ to CompilerOptions.\n\nRemoves CompilerDriver dependency from ImageWriter and\nseveral other classes.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nTest: m test-art-target-gtest\nTest: testrunner.py --target --optimizing\nChange-Id: I3c5b8ff73732128b9c4fad9405231a216ea72465\n" }, { "commit": "b4eb1b19e1dd35d12a408358656c1421f507d231", "tree": "03ba65d206b5e6222dcca0f62a973bcbb1bf8a50", "parents": [ "ff7ff426e136aa8fd6e33d873f6259311982f6bb" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 24 11:09:38 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 25 11:32:31 2018 +0100" }, "message": "Refactor ClassRoot/GetClassRoot().\n\nMove it outside the ClassLinker, into its own header file,\nand add retrieval based on a mirror class template argument.\nKeep the SetClassRoot() as a private member of ClassLinker.\n\nMake the new GetClassRoot()s return ObjPtr\u003c\u003e.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: Icbc6b62b41f6ffd65b437297a21eadbb0454e2b7\n" }, { "commit": "2da52b084b940a5330aa985b687c852495bc38ff", "tree": "61d01d2391bc6e65ecfcbc636159369e9425c59c", "parents": [ "b865d9af3bb5f4777dbe9cbd60cdb2a8fe5f0557" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 08 16:31:34 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 08 17:06:26 2018 +0100" }, "message": "ART: Add --dump-pass-timings compiler option.\n\nThis replaces the old --dump-passes option removed in\n https://android-review.googlesource.com/549200 .\nUsing --dump-timing for timing optimization passes makes\nthe high level timings useless, so return to using two\ndifferent options for these.\n\nTest: Manually run dex2oat with --dump-timings,\n --dump-pass-timings and both.\nChange-Id: Iddc4cfee35652fb493656e7d6081a898c2894f72\n" }, { "commit": "a8360cd6b858906f20558552f7bf3b3876c72ec4", "tree": "357c6916cb832b9a27ff6bbcf2d9344fb818e569", "parents": [ "d637fdac0ea396780b12052c7ab9de61744bcd4b" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed May 02 16:07:51 2018 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Thu May 03 09:26:25 2018 -0700" }, "message": "Perform rudimentary check on graph size for no-change assertions.\n\nRationale:\nThis will find blatant violations of asserting a no-change\npass change if the graph size changed nevertheless.\n\nBug: 78171933\n\nTest: test-art-host,target\nChange-Id: I07b38e71c75dd6f728246d096976c8333b363329\n" }, { "commit": "2e14868d43b4f8c21f8567f336a9bf1a2576dc1b", "tree": "3f17fc543c912e66ea937cd77b760f92faa4ea32", "parents": [ "5503d264ac1b5dd23df4079dc14f26919de42bec" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Apr 18 16:11:12 2018 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Mon Apr 30 08:58:32 2018 -0700" }, "message": "Step 2 of 2: conditional passes.\n\nRationale:\nThe change introduces actual conditional passes\n(dependence on inliner). This ensures more\ncases are optimized downstream without\nneedlessly introducing compile-time.\n\nNOTE:\nSome checker tests needed to be rewritten\ndue to subtle changes in the phase ordering.\nNo optimizations were harmed in the process,\nthough.\n\nBug: b/78171933, b/74026074\n\nTest: test-art-host,target\n\nChange-Id: I335260df780e14ba1f22499ad74d79060c7be44d\n" }, { "commit": "2477320a8d9de58ede68e2645ea53c10f71dcd57", "tree": "f428a6856e10d8ebaff0bb2da544a8d41c35ab77", "parents": [ "5a87e19e4bf1b6719c2aad3effde1b38d2c3085c" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Thu Apr 26 10:28:51 2018 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Thu Apr 26 10:28:51 2018 -0700" }, "message": "Step 1 of 2: conditional passes.\n\nRationale:\nThe change adds a return value to Run() in preparation of\nconditional pass execution. The value returned by Run() is\nbest effort, returning false means no optimizations were\napplied or no useful information was obtained. I filled\nin a few cases with more exact information, others\nstill just return true. In addition, it integrates inlining\nas a regular pass, avoiding the ugly \"break\" into\noptimizations1 and optimziations2.\n\nBug: b/78171933, b/74026074\n\nTest: test-art-host,target\nChange-Id: Ia39c5c83c01dcd79841e4b623917d61c754cf075\n" }, { "commit": "6d05700c620d2bca95fd046969753f71aa015ab4", "tree": "a4fe0409be49298541d4a508ab7ffd0869c71bfb", "parents": [ "8f669504a1c4646501a2cf5d090597f9fed59f70" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Mon Apr 09 15:39:58 2018 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Tue Apr 17 12:55:50 2018 -0700" }, "message": "Run GVN earlier.\n\nRationale:\nRunning GVN earlier allows for better subsequent\ninstruction simplifation. For example, running GVN\nbefore select generation also finds the MIN in:\n if (x \u003e a[i])\n x \u003d a[i];\n\nBug: b/74026074\n\nTest: test-art-host,target\n\nChange-Id: I633046375637c7809a3603fdf7c5cf77e8f21167\n" }, { "commit": "ca1e038eb94694f0f1f94ed3781572411c85d365", "tree": "90ad2c2494821c2f3d904eb42e61fbecd7acaf9d", "parents": [ "3f967b25650e44cd61f5a1112727a8218f2b0804" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Apr 11 09:58:41 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Apr 11 11:10:10 2018 +0100" }, "message": "Revert^2 \"Compile link-time thunks in codegen.\"\n\nThe linker crash (the reason for revert) is flaky and maybe\nwe shall not see it with this CL now that unrelated source\ncode has changed.\n\nTest: Rely on TreeHugger\nBug: 36141117\nBug: 77581732\n\nThis reverts commit 5806a9ec99b5494b511e84c74f494f0b3a8ebec5.\n\nChange-Id: I3a4a058847dff601681ba391abf45833424fa06d\n" }, { "commit": "5806a9ec99b5494b511e84c74f494f0b3a8ebec5", "tree": "bb50d00ff0890c2e10f351f462b47b56b01e78ea", "parents": [ "c9dd2207dfdab42586b1d6a5e7f11cf2fcea3a7a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Apr 04 17:23:28 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Apr 04 17:23:28 2018 +0000" }, "message": "Revert \"Compile link-time thunks in codegen.\"\n\nReason for revert: This caused clang linker crash\nin several branches.\n\nBug: 77581732\n\nThis reverts commit c9dd2207dfdab42586b1d6a5e7f11cf2fcea3a7a.\n\nChange-Id: I1923809083cf41c4f19e3e60df03ae80517aaedb\n" }, { "commit": "c9dd2207dfdab42586b1d6a5e7f11cf2fcea3a7a", "tree": "879df31fd10658093b8931117ee617064ce82519", "parents": [ "30a2d9c61da75359dee4ce90236d19fc6341b07a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 23 16:05:19 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Apr 04 10:34:36 2018 +0100" }, "message": "Compile link-time thunks in codegen.\n\nPrepare for experimenting with Baker read barrier marking\nintrospection entrypoints for JIT.\n\nTest: m test-art-host-gtest\nTest: Compare compiled boot*.oat before and after (no diff).\nTest: Pixel 2 XL boots.\nBug: 36141117\nChange-Id: Idb413a31b158db4bf89a8707ea46dd167a06f110\n" }, { "commit": "440a9b3b3e6aa60c14e73a97dd85c9ada0898569", "tree": "26dcc48d91e3eec18ab1e4dd987962a0939f6531", "parents": [ "d1652d14fbf477b3091eec3764b9890f2c072055" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Feb 15 17:47:29 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Feb 21 12:57:12 2018 +0000" }, "message": "Add timestamps to JIT/DEX native debug info.\n\nThis a forward-looking change intended to allow simpleperf to\nreliably correlate samples and native debug information.\n\nI have added the timestamps to both JIT and DEX, and refactored\nthe code in the process to avoid code duplication.\n\nTest: testrunner.py -t 137\nChange-Id: I45fa4310305aff540e036db9af15a86c5b8b7aff\n" }, { "commit": "fb3de3d72832177e4a8d1f322ed11cbe58e45c9f", "tree": "de8cded0436b54b2e5b1142b07fa75f0ed9791b0", "parents": [ "2827ff64c7a385cfb9d6e01e6385708461eb65fc" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Jan 29 16:11:49 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Feb 02 14:14:08 2018 +0000" }, "message": "Create list of open dex files for libbacktrace.\n\nThis fixes unwinds after recent changes (oob apks; cdex data sharing).\n\nBug: 72520014\nTest: m test-art-host-gtest\nChange-Id: Ie2a02657b2afbe899acd2e61f0a57d207e688b99\n" }, { "commit": "c9e020852645b24ce9e3db7d1480aa9e72057146", "tree": "7dbdb71d2513d0885d9698f266f5ef728e6458fd", "parents": [ "39d7589745f559b84b58d008f67c3aaf364b0370" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Jan 24 16:44:02 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Jan 24 16:52:06 2018 +0000" }, "message": "Log JIT mini-debug-info memory usage.\n\nChange-Id: I0ffa6fb466b1635e724b0e782702303b92355408\n" }, { "commit": "c684f337eae6bd3d77fa44cb568d2bab113e4cee", "tree": "21c3e40cc150da71d91aef03f33010813f3a8368", "parents": [ "26037dd60df16d84fbd5c0ee3215b78aa2bcb408" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Jan 19 17:38:06 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Jan 19 17:42:54 2018 +0000" }, "message": "Refactor jit debugger interface and its ELF creation.\n\nMake it possible to store more then one method per entry,\nand ref-count the number of live methods per entry.\n\nTest: m test-art-host-gtest\nChange-Id: I45d69185e85e47fbee88a8d1f549ede9875a3c0a\n" }, { "commit": "8892c6bd9235e7ae697039c901aaeea1597a7473", "tree": "fa10893a270bad91d3503a761f140cbe9c97a74c", "parents": [ "210531f8775c89feb90d430cd5b6026b4cf8ef89" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 09 15:10:17 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sat Jan 13 01:40:47 2018 +0000" }, "message": "Move debug info offsets into a side table\n\nAdd a compact side table for figuring out the debug info offsets\nfor a given method index. This reduces dex size by ~1.2%.\n\nThe debug table is keyed by method index and has leb encoded\noffsets for the offsets. This means the table is smaller if debug\ninfos are encoded by method index order.\n\nTo prevent expansion for method indicies without debug info, there\nis a bitmap that specifies if a method index has a debug info offset.\n\nMotivation: Reduce code item size and allow more deduping in the\nfuture.\n\nTest: test-art-host\nBug: 63756964\n\nChange-Id: Ib983e85c1727f58c97676bde275f4a9756314da0\n" }, { "commit": "210531f8775c89feb90d430cd5b6026b4cf8ef89", "tree": "3c23fffcc0d1d7876f15f64672e0de98b1f051ca", "parents": [ "3c9e0f69b04b614c44448e66e15c6f8a78da4b2c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jan 12 10:15:51 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sat Jan 13 01:40:03 2018 +0000" }, "message": "Revert \"Revert \"Move quickening info logic to its own table\"\"\n\nBug: 71605148\nBug: 63756964\n\nTest: test-art-target on angler\n\nThis reverts commit 6716941120ae9f47ba1b8ef8e79820c4b5640350.\n\nChange-Id: Ic01ea4e8bb2c1de761fab354c5bbe27290538631\n" }, { "commit": "6716941120ae9f47ba1b8ef8e79820c4b5640350", "tree": "76dd7d32703505d4ab44c0b55a7ecaf3ff66a94e", "parents": [ "7a26f948204377130be7b738d70d7365c86a804b" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 12 09:06:14 2018 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 12 09:06:14 2018 +0000" }, "message": "Revert \"Move quickening info logic to its own table\"\n\nBug: 71605148\nBug: 63756964\n\nSeems to fail on armv7.\n\nThis reverts commit f5245188d9c61f6b90eb30cca0875fbdcc493b15.\n\nChange-Id: I37786c04a8260ae3ec4a2cd73710126783c3ae7e\n" }, { "commit": "7a26f948204377130be7b738d70d7365c86a804b", "tree": "dd3ab45b73127fdcbcdca64bbac484bfe2c0a8cd", "parents": [ "29504326da13831fa3060ac507927c4d76aa556a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 04 18:23:55 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 11 10:58:22 2018 -0800" }, "message": "Move quickening info logic to its own table\n\nAdded a table that is indexed by dex method index. To prevent size\noverhead, there is only one slot for each 16 method indices. This\nmeans there is up to 15 loop iterations to get the quickening info\nfor a method. The quickening infos are now prefixed by a leb\nencoded length. This allows methods that aren\u0027t quickened to only\nhave 1.25 bytes of space overhead.\n\nThe value was picked arbitrarily, there is little advantage to\nincreasing the value since the table only takes 1 byte per 4 method\nindices currently. JIT benchmarks do not regress with the change.\n\nThere is a net space saving from removing 8 bytes from each\nquickening info since most scenarios have more quickened methods than\ncompiled methods.\n\nFor getting quick access to the table, a 4 byte preheader was added\nto each dex in the vdex file\n\nRemoved logic that stored the quickening info in the CodeItem\ndebug_info_offset field.\n\nThe change adds a small quicken table for each method index, this\nmeans that filters that don\u0027t quicken will have a slight increase in\nsize. The worst case scenario is compiling all the methods, this\nresults in 0.3% larger vdex for this case. The change also disables\ndeduping since the quicken infos need to be in dex method index\norder.\n\nFor filters that don\u0027t compile most methods like quicken and\nspeed-profile, there is space savings. For quicken, the vdex is 2%\nsmaller.\n\nBug: 71605148\nBug: 63756964\nTest: test-art-host\n\nChange-Id: I89cb679538811369c36b6ac8c40ea93135f813cd\n" }, { "commit": "698ebbca3c54d17c696e87b9e5838df6a5ac9b08", "tree": "7887b2cd4662410114f205bbe1896ff094d51407", "parents": [ "4b9a11f1d43df69c50e93ba1793fce1c2216682b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jan 05 11:00:42 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jan 08 09:11:49 2018 -0800" }, "message": "Clean up CodeItemAccessors and Compact/StandardDexFile\n\nChange constructor to use a reference to a dex file.\n\nRemove duplicated logic for GetCodeItemSize.\n\nBug: 63756964\nTest: test-art-host\nChange-Id: I69af8b93abdf6bdfa4454e16db8f4e75883bca46\n" }, { "commit": "9e734c7ab4599d7747a05db0dc73c7b668cb6683", "tree": "dce1d1993734a947fb2e6f626eb1b425cb72143b", "parents": [ "b496af808eaf3af5ebac50aef4fbec33323b5016" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Jan 04 17:56:19 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Jan 05 11:07:19 2018 -0800" }, "message": "Create dex subdirectory\n\nMove all the DexFile related source to a common subdirectory dex/ of\nruntime.\n\nBug: 71361973\nTest: make -j 50 test-art-host\nChange-Id: I59e984ed660b93e0776556308be3d653722f5223\n" }, { "commit": "808c7a57bb913b13c22884f57cdacd59bf1fdb3f", "tree": "d7f0d7cabaac5a7646c25bae584a82a9aa279cc0", "parents": [ "64bae9fb677aa0e2406d13ea9f8ebaa92e16f978" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 15 11:19:33 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 22 09:41:17 2017 -0800" }, "message": "Make CodeItem fields private\n\nMake code item fields private and use accessors. Added a hand full of\nfriend classes to reduce the size of the change.\n\nChanged default to be nullable and removed CreateNullable.\nCreateNullable was a bad API since it defaulted to the unsafe, may\nadd a CreateNonNullable if it\u0027s important for performance.\n\nMotivation:\nHave a different layout for code items in cdex.\n\nBug: 63756964\nTest: test-art-host-gtest\nTest: test/testrunner/testrunner.py --host\nTest: art/tools/run-jdwp-tests.sh \u0027--mode\u003dhost\u0027 \u0027--variant\u003dX32\u0027 --debug\n\nChange-Id: I42bc7435e20358682075cb6de52713b595f95bf9\n" }, { "commit": "f4886df5e72fa21eddfc4cc7860f4154929b3380", "tree": "897e74d9da4e92faae5c24350fa2f999cbfb2b1e", "parents": [ "7eb4333d0b60a13f9d3126eadef9eb16c399662a" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Dec 11 16:06:29 2017 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Dec 14 13:52:07 2017 +0000" }, "message": "Add mini-debug-info generation mode for JIT.\n\nThis excludes everything that is not needed for backtraces and\ncompresses the resulting ELF file (wrapped in another ELF file).\n\nThis approximately halves the size of the debug data for JIT.\nThe vast majority of the data is the overhead of ELF header.\nWe could amortize this by storing more methods per ELF file.\n\nIt also adds NOBITS .text section to all debug ELF files,\nas that seems necessary for gdb to find the symbols.\nOn the other hand, it removes .rodata from debug ELF files.\n\nTest: Manually tested that gdb can use this data to unwind.\nTest: m test-art-host-gtest\nTest: testrunner.py --optimizing --host\nTest: testrunner.py -t 137-cfi\n\nChange-Id: Ic0a2dfa953cb79973a7b2ae99d32018599e61171\n" } ], "next": "2196c651ecc77e49992c6c329dfce45f78ff46cb" }