)]}' { "log": [ { "commit": "e512556ab85987c52e4be1fcd6d7a1d15a1b0fb6", "tree": "16e82b22829a269dcb7b8c305bb0a44c6f74c58e", "parents": [ "2f34e59c10c30ea24fa5f4bc4e3d5a9dc6469c34" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 06 17:38:26 2019 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 07 12:29:35 2019 +0000" }, "message": "Replace StringPiece with std::string_view in profman.\n\nAnd in dexoptanalyzer and hiddenapi, see below.\n\nThe parsing helpers in utils.h were mostly unused, only\nprofman was calling one of them, so move that helper to\nprofman and rewrite and fix it, remove the other helpers.\nThis exposed dependency of dexoptanalyzer and hiddenapi\non StringPiece indirectly included from utils.h, thus we\nreplace it with std::string_view also in those tools.\n\nDuring the rewrite, avoid using std::string_view::data()\nas a null terminated string, prefer to fall back to the\noriginal null terminated raw option instead.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 123750182\nChange-Id: Ibeec8069a5d82ea556e03e4577812f94dca6f387\n" }, { "commit": "9d9227ae2d8806446fa3e60870d0e8df5e5c9377", "tree": "adb174eac909e16705e95bb6e8b23759ebe96249", "parents": [ "163a37ad5f2d0b16bdd32fb8964580991900049a" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Dec 19 12:32:50 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Jan 07 21:41:30 2019 +0000" }, "message": "Build dexlist, dexlayout, and profman for Windows\n\nBug: 22322814\nTest: wine on linux on a large dex file.\nChange-Id: I93a89253799715de54e1c52213595b0749a5f47f\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": "3ee9cfd97cb3c7d6f8f47a166d2a001e12cc63ce", "tree": "4a4d7c05d235c84c697bf8d586176d6a8095e022", "parents": [ "c61b8c64ed2f3bd2e29f5bfa848e5b131d1c0f1e" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Tue Dec 11 13:38:35 2018 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Dec 20 05:11:02 2018 -0800" }, "message": "Add support for aggregation counters in the profile\n\nThe profile can be configured to keep track of aggregation counters for\nmethods and classes.\n\nOn device, this is particular useful for the boot image profile. When\naggregating multiple profiles into one and keep track of counters we can\ntell:\n1. how many times we aggregated a method or a class. This tells us how\npopular is a particular method or class amongst the apps.\n2. how many times we aggregated the profile. This tells how many apps\ncontributed to the profile (on a single device).\n\nNOTE: This expands the memory footprint and makes the aggregation slower.\nAs such it is disabled by default and not intended to use for the regular\nprofiling path of apps in prod.\n\nTest: profile_compilation_info_test, profile_assistant_test\nBug: 112617266\n\nChange-Id: I731abf31f65b12bf405c77fb7803bb0bda3b9908\n" }, { "commit": "2a487eb89b67d7ff030368e4e4a6ed0a3ac39003", "tree": "bedffc5f4ee16d07c957d06cdab5148c69bd6a56", "parents": [ "875b4f26517ce215342746f07e41cda4c1991237" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 19 11:41:22 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 20 13:04:36 2018 -0800" }, "message": "ART: Fix performance-inefficient-vector-operation\n\nBug: 32619234\nTest: m test-art-host\nChange-Id: I286e27424ac39c0fed308811dc3004e734c0560c\n" }, { "commit": "38a938e2e83f5baa3238ce3b28ba4c61a268dd3d", "tree": "8b29c26d90b998fc717181298349ac2348f29c5f", "parents": [ "a5de05da46e8a93436e68e7e0d242dce46269527" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Sep 21 10:55:51 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Nov 08 19:38:07 2018 +0000" }, "message": "Minimal Android Runtime APEX module.\n\nIntroduce build rules to generate com.android.runtime, a minimal\nAndroid Runtime APEX module containing just ART and its dependencies.\n\nWhen module com.android.runtime is built (`make com.android.runtime`),\nmake produces a `com.android.runtime.apex` package in the `apex`\ndirectory of the system partition\n(`$ANDROID_PRODUCT_OUT/system/apex/com.android.runtime.apex`).\n\nThis module is not built by default when generating a system image.\n\nTest: make com.android.runtime\nTest: art/build/apex/runtests.sh\nBug: 113373927\nChange-Id: I2019bd1934558feba6eccef13b887b2faa96caec\n" }, { "commit": "dfcd82c09e8ce4562ed39e006d4b1c8163b4e25e", "tree": "d9bfd3e043a95b3c8e302769cef23c13a89df3c5", "parents": [ "4f570a43146c1456071f2ffe6590ab96e723051a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 16 20:22:37 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 19 13:06:33 2018 -0700" }, "message": "ART: Fix android-cloexec warnings\n\nUse the DupCloexec helper wherever possible. Add O_CLOEXEC to open\nand fopen calls.\n\nBug: 32619234\nTest: WITH_TIDY\u003d1 mmma art\nChange-Id: I0afb1beea53ab8f68ab85d1762aff999903060fe\n" }, { "commit": "9b031f7e4834b263ae531409a9f6c82cfdc89477", "tree": "4b5fdfc3ecc12b5b7e32959bec0c26595b87d2ab", "parents": [ "c71f38bdf7dca39a070ba25d436edcd758551f42" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Oct 04 11:03:34 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 09 14:38:45 2018 -0700" }, "message": "ART: Refactor for bugprone-argument-comment\n\nHandles smaller subdirectories.\n\nBug: 116054210\nTest: WITH_TIDY\u003d1 mmma art\nChange-Id: I81c8f3396b9922684b68e9f3d8fccefe364f1279\n" }, { "commit": "0573f857041b7032837ee1686b1be7191e6353d2", "tree": "13ff3211c83059e844cc2ad56576e9521ab30cdf", "parents": [ "29391756f70489a5ab659988f058e359527a7af1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 01 13:52:12 2018 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 01 15:55:37 2018 -0700" }, "message": "Dump Dex file checksums for profman --dump-only\n\nMake it easier to diagnose mismatched dex checksums. Add checksum\nverification in ProfileCompilationInfo::DumpInfo. Some refactoring\nto remove unnecessary code.\n\nBug: 116509324\nTest: test-art-host-gtest\n\nChange-Id: I11fb8d1aa987c0f0a975ef6dcb811bf41d58fad6\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": "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": "bc802de2244ed9769d1ddf8841d7965a29493d9a", "tree": "acac22c89c8cdedb65d6dc0657755851834500f7", "parents": [ "35d4b9410c362685fdd8181483a6126dd3e20cdd" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 20 17:24:11 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 20 17:25:25 2018 -0700" }, "message": "Revert^2 \"ART: Fix some performance-X tidy\"\n\nThis reverts commit 6856214f2c849600bcb94cc6e84975ba8cb69920.\n\nFix lifetime issue in imgdiag.\n\nTest: mmma art\nTest: m test-art-target-gtest-imgdiag_test\nChange-Id: I919db6310c0713313340adfe66bcd75a1c5c236e\n" }, { "commit": "6856214f2c849600bcb94cc6e84975ba8cb69920", "tree": "a103f54ec49fac8d5cd15661f5c2d0cb5dab32e5", "parents": [ "89de444350fdbd7df4fb4d95bb71f00e6673b466" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 20 21:49:11 2018 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 20 21:49:11 2018 +0000" }, "message": "Revert \"ART: Fix some performance-X tidy\"\n\nThis reverts commit 89de444350fdbd7df4fb4d95bb71f00e6673b466.\n\nReason for revert: Seemingly breaks imgdiag_test on device.\n\nChange-Id: I46084ecb4205575381aafffd90a0f1a1126e8d2d\nTest: mmma art\n" }, { "commit": "89de444350fdbd7df4fb4d95bb71f00e6673b466", "tree": "57fa7214c4059afcfd11aa35a81999b8558b91b8", "parents": [ "e383d23918db4eede30c3d78589d4639de3ec446" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jun 18 13:33:55 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jun 19 17:11:21 2018 -0700" }, "message": "ART: Fix some performance-X tidy\n\nFix performance-for-range-copy, performance-unnecessary-copy-initialization\nand performance-unnecessary-value-param issues.\n\nTest: mmma art\nTest: m test-art-host\nChange-Id: I43d8736fc541030a3c61f66aeee0b9c2f1d295f7\n" }, { "commit": "18e2687c4126d66870d61d6548abc2c7661a66cc", "tree": "06eff605df9259dde8170c8efb48b11df4108eb3", "parents": [ "2607d6c47114a18ccf4a532b20e1099d1dd8fcca" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 04 17:19:02 2018 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 07 22:13:48 2018 -0700" }, "message": "Refactor ClassAccessor to use an index instead of ClassDef pointer\n\nRemoves a separate class_def_idx being required for getting the index\nof a ClassAccessor foreach loop.\n\nBug: 79758018\nTest: test-art-host-gtest\n\nChange-Id: Ie3010a17669f24cf492c678b55bdddba7ec62ea8\n" }, { "commit": "a8bba7d0853b372aea3ed3ea154fb2b2a23c2c9d", "tree": "973a89870e0cc8912401503ebab1b709a9df5ba0", "parents": [ "65e463cf7ea91003ef030c8a819ae4596c9b820f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 30 15:18:48 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 31 09:48:59 2018 +0100" }, "message": "ObjPtr\u003c\u003e-ify ClassLinker::FindClass(), fix 1 stale reference use.\n\nThread::CreateAnnotatedStackTrace() was using a stale\nreference `aste_array_class`.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: I191907c0053456bb57de425aa6ccd9668df818a2\n" }, { "commit": "0d896bd7da4f7b11559aa88aad82e9a9e170a4e4", "tree": "44a9278d98ace169e8296feba98038c173f29eaf", "parents": [ "267c83529850f51cd690b3e31882aaae98601afd" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 25 00:20:27 2018 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue May 29 10:20:59 2018 -0700" }, "message": "Add Method/Field iterator to ClassAccessor\n\nEnables ranged based for loops on fields and methods.\n\nFor visiting both fields and methods, VisitFieldsAndMethods will\nbe faster because of not needing to decode the fields twice for\nseeking purposes.\n\nAdded test.\n\nBug: 79758018\nBug: 77709234\nTest: test-art-host-gtest\n\nChange-Id: I593e23ccd138b87a27d8bab6927ff2b685c057f3\n" }, { "commit": "20f49922302d30c452eaf5588a6cf66946866922", "tree": "fddf03bbae499705c6a8c51fe1c6f4c63dc31191", "parents": [ "ebf5902272613d8bb563ae312ec5ce2580a59bb7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 24 16:04:17 2018 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 24 16:06:03 2018 -0700" }, "message": "Move profman/ to ClassAccessor\n\nBug: 77709234\nBug: 79758018\nTest: test-art-host\n\nChange-Id: I97111ef61a6735ef8719c1a6d7c80ad7c553af51\n" }, { "commit": "671af6c3f3858bb372fb4946b14c069c2d456a60", "tree": "41d0326309b5acb94566335c8825acdd70695427", "parents": [ "df810b9c8d5013cd6f5799a75b6aaf0f3991210c" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu May 17 11:00:35 2018 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu May 17 11:00:35 2018 -0700" }, "message": "Make dexlayout and profman independent of libart\n\nRemove the libart dependency from these two tools. Dexdiag remains\ndependent because vdex_file.* is in runtime. It could possibly be moved\nalso.\n\nBug: 78652467\nTest: make -j 40 checkbuild\n make -j 40 test-art-host-gtest\n\nChange-Id: I68a62f8b2a2730067aee5ff5e0cf81acdca0d703\n" }, { "commit": "1f010164bea4714e9f653c52e8948c5d5305040b", "tree": "e69b2c3f4d2645d4097655f8914325945fe5dbbd", "parents": [ "a7d37cf735f05cfc3a3585a5c44eb65d8a279874" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue May 15 08:59:32 2018 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed May 16 13:01:34 2018 -0700" }, "message": "Move ArtDexFileLoader to libdexfile\n\nAdds a dependency from libdexfile on libartbase, but allows the use of\nMemMap loaders, etc.\n\nBug: 78652467\nTest: make -j 40 checkbuild\nChange-Id: I15cf33893ca9192050762f8350a3cc1e39f88dc5\n" }, { "commit": "818cb80ff7503f4ce46f05386209d0070b6b70b0", "tree": "b0e1bf92b900a8fba3d85553f96849d5fe8abd1f", "parents": [ "2b80ed488c497393270c98f7a767d8495166db8e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 11 05:30:16 2018 +0000" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 11 05:30:16 2018 +0000" }, "message": "Revert \"Make dexlayout and profman build without libart\"\n\nThis reverts commit 2b80ed488c497393270c98f7a767d8495166db8e.\n\nBug: 78652467\n\nReason for revert: ASAN tests failing\n\nChange-Id: Id4bba2711d8b69c0a64e3e8eb335a18facab9fdd\n" }, { "commit": "2b80ed488c497393270c98f7a767d8495166db8e", "tree": "e1c249285b2e441b56ec7faf06757f3fe7affde5", "parents": [ "a6ce173839a1bca1811fca7d9a0a39efd720611d" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue May 08 08:58:15 2018 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu May 10 08:23:22 2018 -0700" }, "message": "Make dexlayout and profman build without libart\n\nUse libprofile and libartbase to remove the dependencies on libart for\ndexlayout and profman. dexdiag remains connected to libart because of\nvdex file APIs.\n\nBug: 78652467\nTest: make -j 40 test-art-host-gtest\nChange-Id: Ie4d58e7e75aa725a6d453a9d4c7fefd868aa7b2d\n" }, { "commit": "302dda50f7e3ff91f6afb2fca48c8ae26ce4f75c", "tree": "d74c89c28fe9249d7579f8ecd607c2fe2daffb47", "parents": [ "56189d23563610a63064b9ef97e5851f21bcadc8", "1bfe4bd4c444017cf40ccadcfbf20cb0ceb37880" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Mon Apr 30 18:29:20 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Apr 30 18:29:20 2018 +0000" }, "message": "Merge \"Do not verify apks when processing profiles\"" }, { "commit": "1bfe4bd4c444017cf40ccadcfbf20cb0ceb37880", "tree": "1b7ed77256b2debb0da609f8fddf64efe4fcc044", "parents": [ "f0cf86fd4085c725b667058b416e0f8a6a7dfcc2" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Apr 26 16:00:11 2018 -0700" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Apr 27 12:50:39 2018 -0700" }, "message": "Do not verify apks when processing profiles\n\nIt\u0027s a waste of time and it causes problems when taking snapshots for the\nboot image profile.\n\nBug: 73313191\nTest: profile_assistant_test\nChange-Id: I8e838af9515b41402eda455c23741a855c48ff98\n" }, { "commit": "82d046e7c9daad9b706cbec7df7025c5a4ef9163", "tree": "92b7deea052015dd64bea6fc72d76c67f69e0e5d", "parents": [ "5a87e19e4bf1b6719c2aad3effde1b38d2c3085c" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Apr 23 08:14:19 2018 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Apr 26 12:37:04 2018 -0700" }, "message": "Add a profiling library\n\nMove profile_compilation_info to a separate library. Another step\ntowards building many of our tools without libart[d].\n\nBug: 78459333\nTest: make -j 50 checkbuild\nChange-Id: Ib281d3d1fde6d06ebb429c5d39d62a7038af0f44\n" }, { "commit": "c24fa5d93102688097eaf5bebaf6c653ca5c168d", "tree": "ac2d1a74e5f91d125b8717983c459bc17da48dce", "parents": [ "a03627a94c0eead28e1c7d917218ad398d483495" ], "author": { "name": "Rico Wind", "email": "ricow@google.com", "time": "Wed Apr 25 12:44:58 2018 +0200" }, "committer": { "name": "Rico Wind", "email": "ricow@google.com", "time": "Wed Apr 25 12:46:50 2018 +0200" }, "message": "Allow INVOKE_VIRTUAL_RANGE as single invoke\n\nTest: new D8 dexer works\n\nChange-Id: I7480ca3d7a180f76d370de6ebf803ba07c76c5e0\n" }, { "commit": "f120ffc1b4864c28bc2aa249e714b51da11f3ea0", "tree": "793e21b1b398eee8d5240812d068f790e3c0f415", "parents": [ "597c5247e42964e85aef89c03d990df3bab29de1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Apr 23 11:27:31 2018 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Apr 23 13:19:50 2018 -0700" }, "message": "Dump profile checksums\n\nAlso dump profile checksums. This helps diagnose checksum mismatches.\n\nTest: manual\nBug: 70292748\nChange-Id: Iaab4b34378240a265d11d196dcad6db12dc3545d\n" }, { "commit": "f5d9f81542dcdf9f388d7daf4972123312fffa92", "tree": "f6e213a83b87cc48164752ff110c00aece01e9e1", "parents": [ "6648da11ae28d1bc8d288eba3c3131bade13e314", "daf374cc169ebc9beca38f1c6edf1883f061ae74" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Apr 13 21:12:30 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Apr 13 21:12:30 2018 +0000" }, "message": "Merge \"profman: removed unused header\"" }, { "commit": "79e2607ab50163bfdc283f4a49decec26a216df5", "tree": "0cdb2c65412b64a55ddf1a486ab3d961a0301a3e", "parents": [ "b4b91460b2d01d36ddcb7d55111ccd8faf390c2a" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Apr 06 17:58:50 2018 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Apr 13 08:29:30 2018 -0700" }, "message": "Move profile dependent modules to libartbase\n\nMove mem_map and zip_archive to libartbase. This should be the last two\nremaining modules that profile_compilation_info is dependent upon.\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make and boot a device\n\nChange-Id: I136ee23e426aa8ec7441e3d3f1978f1bebf4b562\n" }, { "commit": "daf374cc169ebc9beca38f1c6edf1883f061ae74", "tree": "11a71b327d8345014ce8b0cb55670aab20853660", "parents": [ "b8a5e8b0c4a03e5c2b401b968f3b25eac26ec460" ], "author": { "name": "Jeff Vander Stoep", "email": "jeffv@google.com", "time": "Thu Apr 12 06:51:01 2018 -0700" }, "committer": { "name": "Jeff Vander Stoep", "email": "jeffv@google.com", "time": "Thu Apr 12 06:54:50 2018 -0700" }, "message": "profman: removed unused header\n\nWhile root-causing a spurious getattr selinux denial, I noticed\nsys/stat.h is no longer used. Remove.\n\nTest: mmm art/profman\nChange-Id: I3e35946b80fe91cead268f2b89210ebd85e89892\n" }, { "commit": "312f3b2fd0094c028a7d243b116947a35a745806", "tree": "3d7ec049ded98c489098c87250c75e3f711f8290", "parents": [ "0a3d5eb2ff9e70fa5785638da938439835d0337e" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Mar 19 08:39:26 2018 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Mar 19 11:02:48 2018 -0700" }, "message": "Move some remaining dex utilities\n\nThere were several utilities related to building/walking/testing dex\nfiles that were not in libdexfile. This change consolidates these.\n\nBug: 22322814\nTest: make -j 50 test-art-host\nChange-Id: Id76e9179d03b8ec7d67f7e0f267121f54f0ec2e0\n" }, { "commit": "c431b9dc4b23cc950eb313695258df5d89f53b22", "tree": "422273559c3ae52caff0c6b1cf1a62a8312f0e26", "parents": [ "f46f46cf5bd32788d5252b7107628a66594a5e98" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Mar 02 12:01:51 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Mar 05 13:58:20 2018 -0800" }, "message": "Move most of runtime/base to libartbase/base\n\nEnforce the layering that code in runtime/base should not depend on\nruntime by separating it into libartbase. Some of the code in\nruntime/base depends on the Runtime class, so it cannot be moved yet.\nAlso, some of the tests depend on CommonRuntimeTest, which itself needs\nto be factored (in a subsequent CL).\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make -j 50 test-art-host\n\nChange-Id: I8b096c1e2542f829eb456b4b057c71421b77d7e2\n" }, { "commit": "b2ec9f5c128673c43f776cbe12c8eeb0a6884ebb", "tree": "724022162a4f3de021cebab32ee5ac67ec860f32", "parents": [ "f13343eb8b2738e2812fd7bd9d5592f529e93c21" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Feb 21 13:20:31 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Feb 21 13:51:15 2018 -0800" }, "message": "Remove duplication, split tests\n\nThe code move to libdexfile/dex/descriptors_names.cc apparently did not\nremove the original code from runtime/utils.cc. Fix that duplication\nand all the header mentions needed. Also, split the test files to go\nalong with the new locations for the code to be tested.\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make -j 50 test-art-host-gtest\n flash \u0026 boot marlin\n\nChange-Id: Ie734672c4bca2c647d8016291f910b5608674545\n" }, { "commit": "29d4cc0097b64ead74a570f4fb80949c24786700", "tree": "6e114191c07994266e38f0c15e3af3e3dd54b2c2", "parents": [ "a502c7202f56b8f5f2c5de567359fa7f6bab406f", "02c08794435da77724486a5170d1733f760bc8af" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Feb 16 22:49:08 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Feb 16 22:49:08 2018 +0000" }, "message": "Merge \"Actually use --copy-and-update-profile-key argument in profman...\"" }, { "commit": "02c08794435da77724486a5170d1733f760bc8af", "tree": "3c27cc9dc503c7f603a370563df0e1272ebad8d2", "parents": [ "ee9cb41260bc76cdb8572b10e99e6da866d9a8c8" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Feb 15 19:40:48 2018 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Feb 16 12:11:48 2018 -0800" }, "message": "Actually use --copy-and-update-profile-key argument in profman...\n\nThe argument was not parsed and thus always false. This went under the\nradar because the tests were verifying the profile assistant code\ndirectly, bypassing profman cmd line invocation.\n\nTest: profile_assistant_test\nBug: 30934496\nChange-Id: I8f0b4f63759db39a6948d92a701b0e2794f253b1\n" }, { "commit": "f1c29aa501eeb38b6cc6a142887b2ea4e4fd5db1", "tree": "44fd8e0de329a5890605b5b09406a906af072c9c", "parents": [ "3e55ea3118c97f0ffecc24939a193a9d40dc315d", "ee9cb41260bc76cdb8572b10e99e6da866d9a8c8" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Feb 16 01:43:15 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Feb 16 01:43:15 2018 +0000" }, "message": "Merge \"Ensure that we always set the method hotness in the profile\"" }, { "commit": "ee9cb41260bc76cdb8572b10e99e6da866d9a8c8", "tree": "c5192809ccfe6042800b5710fa22ec600060a81d", "parents": [ "b40fa7c33075292beeb6840ac679ffd08fd1f719" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Tue Feb 13 20:32:35 2018 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Feb 15 15:31:19 2018 -0800" }, "message": "Ensure that we always set the method hotness in the profile\n\nThe method hotness were not recorded for methods extracted from the JIT\ncode cache.\n\nTest: gtest \u0026 run-test\nBug: 71588770\n\nChange-Id: Ifdf6340caa9faf5adb6f3b3b5b4046f31f34189c\n" }, { "commit": "fcbe15ce26930c0c4bb33998fc2abbe4f8770dbe", "tree": "6513a2eaf1251e93578916b5171dc42984de7da3", "parents": [ "b40fa7c33075292beeb6840ac679ffd08fd1f719" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Feb 15 09:41:13 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Feb 15 15:15:13 2018 -0800" }, "message": "Build debug library libdexfiled\n\nAdd a debug version of libdexfile for use when CHECKs, etc., are\nenabled.\n\nBug: 22322814\nTest: make -j 50 checkbuild\nChange-Id: I263138611bcdde9dbc8f619bcb2a0a6d3a09fbf6\n" }, { "commit": "e10c1e23223c50daca784e6208c1153935100e31", "tree": "14c8461faf8c9998354ab4b64f7a89a6cd9f37e2", "parents": [ "2827ff64c7a385cfb9d6e01e6385708461eb65fc" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Jan 26 20:10:15 2018 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Feb 02 06:47:09 2018 -0800" }, "message": "Enable profile data filtering in profman\n\nUpdate profile merging to accept a set of apks (passes with --apk) which\nwill dictate what data should be processed.\n\nWhen profman is invoked with a list of --apk files, only profile data\nbelonging to that apks will be in the output reference profile.\n\nIf no --dex-location is specified then the locations is inferred from\nreding /proc/self/fd/apk_fd link.\n\nTest: profile_assistant_test\nBug: 30934496\nChange-Id: I44698c6db545ecf91454db1387c3d0e47fe5b9b3\n" }, { "commit": "0225f8e2939a9340cb7dcebfcfe7996a2bd9bce9", "tree": "fd32362dff4ecdc8ff8b0d800a1b46fb0d9d0104", "parents": [ "c7e546ff3963a1d51b1f100d308db735bd19f736" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Jan 31 08:52:24 2018 +0000" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Jan 31 01:24:53 2018 -0800" }, "message": "Revert \"Revert \"Make libdexfile build independent of runtime dir\"\"\n\nThis reverts commit 787784f9effb126b5d0d3dc97d544c4a477b5daf.\n\nReason for revert: Bot configuration issue.\n\nChange-Id: I6a10bb4a9571f89c7e4dd095f9157e830a44e2de\nBug: 22322814\nTest: make -j 50 checkbuild\n" }, { "commit": "85f890b792d1547182d195db3f43240180ab880b", "tree": "1f30b196288ddd8fdc09614145d3c17acc92af94", "parents": [ "d2b5229453af661572f0c76ac08dfd3f52727d5b", "787784f9effb126b5d0d3dc97d544c4a477b5daf" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 18:56:50 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Jan 30 18:56:50 2018 +0000" }, "message": "Merge \"Revert \"Make libdexfile build independent of runtime dir\"\"" }, { "commit": "787784f9effb126b5d0d3dc97d544c4a477b5daf", "tree": "1ff6d9fd284a4dd2a337165dff20e71137997c9d", "parents": [ "b40b7e73469339a6b667b4a2e2b8690112a74dc9" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 18:48:51 2018 +0000" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 18:48:51 2018 +0000" }, "message": "Revert \"Make libdexfile build independent of runtime dir\"\n\nThis reverts commit b40b7e73469339a6b667b4a2e2b8690112a74dc9.\n\nReason for revert: on device libdexfile.so missing\n\nChange-Id: I9bd61a98bef870400580e8c991cb061d3f57fa72\n" }, { "commit": "d2b5229453af661572f0c76ac08dfd3f52727d5b", "tree": "61e45029e86bc81a23eeac0842bcb2738c35f0c5", "parents": [ "47ec7ad0f819c171e41d323aff71f5ac425b1ccd", "f13ac7ce507c759f34e9ff4f4116c956f166c4c6" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Jan 30 18:08:24 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Jan 30 18:08:24 2018 +0000" }, "message": "Merge \"Profman: Skip dex file verification for boot jars\"" }, { "commit": "f13ac7ce507c759f34e9ff4f4116c956f166c4c6", "tree": "0c0d77e9992fd34d9673dd3057d8a896919a5fd9", "parents": [ "9690ad794b324ba54e936608881ac0f62538b97a" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Jan 30 10:09:08 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Jan 30 16:22:29 2018 +0000" }, "message": "Profman: Skip dex file verification for boot jars\n\nThe introduction of new access flags into boot jars causes them to\nfail dex file verification. The files are verified in `hiddenapi`\nand verification should be skipped in subsequent attempts to open\nthem. This patch adds a new flag \u0027--skip-apk-verification\u0027 which\ninstructs `profman` to skip dex file verification.\n\nBug: 64382372\nTest: make test-art-host-gtest-profile_assistant_test\nChange-Id: Iad614e47feada293218c11cff985a8012ca49a0f\n" }, { "commit": "b40b7e73469339a6b667b4a2e2b8690112a74dc9", "tree": "7e8a34d9c7894c4c946f674f19f0a0a512b95184", "parents": [ "9690ad794b324ba54e936608881ac0f62538b97a" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Jan 25 17:11:07 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 01:17:58 2018 -0800" }, "message": "Make libdexfile build independent of runtime dir\n\nRemove libdexfile\u0027s dependency on utils.cc and move utf.cc into\n/dex. Remove libdexfile\u0027s constituent sources from libart and\nuse libdexfile wherever libart is. Also remove some ART-specific\ninterfaces. Libdexfile\u0027s tests remain to be converted, plus\nmoving the files to a new directory peer to runtime/.\n\nBug: 22322814\nTest: make -j 50 test-art-host\nChange-Id: Ifaf695216e4a0e43d3aa377984d933f7a2a243c2\n" }, { "commit": "2dba0ab8498e5ff3ba5be2e37e26310a661cf259", "tree": "4dcc90f214855c7521817f9809712726132d4a1e", "parents": [ "49971eb50c0213b7506bdd29954a6a4423e6426a" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Mon Jan 22 19:22:24 2018 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Jan 24 12:14:47 2018 -0800" }, "message": "Enable profile key updates via profman\n\nAllow profman to update the profile key in an existing profile based on\nnew dex location.\n\nThis is needed in order to support profiles for dex files that might move\nfrom their original profiling location (e.g. during install)\n\nThe matching [profile key \u003c-\u003e dex_file] is done based on the dex checksum\nand the number of methods ids. If neither is a match then the profile key\nis not updated. If the new profile key would collide with an existing key\n(for a different dex) then the update will fail.\n\nTest: profile_compilation_info_test\nBug: 30934496\n\nChange-Id: Ic696b3f6fe9da2007421bf044d58a21c90fd9ee7\n" }, { "commit": "3913e488afb884ccee76d620d4357b6308e55010", "tree": "7b20f16dc542a082983d9ba390a1a9ee8948e8b3", "parents": [ "b6d077aceb38922891a6b9e79714bad3072f9c15" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jan 22 18:58:01 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jan 22 18:58:01 2018 -0800" }, "message": "ART: Refactor profile_compilation_info.h includes\n\nUse forward declarations.\n\nTest: m\nChange-Id: Ife25be9e2eb5ba9224f082aaa935f821a3597963\n" }, { "commit": "013fd8073f3ece22b0bba1853d3f3430c8a9e4bd", "tree": "d1fb1ce709d4927a8b5f5d7fc673b6932ca4977a", "parents": [ "a7e4a15ba7b40fbe1ecd76ce0d99de90aa42201f" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Jan 11 22:55:24 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Jan 12 12:47:13 2018 -0800" }, "message": "Create an ART-independent DexFileLoader\n\nOpening DEX files should not rely on instantiating a runtime or having a\nlarge number of dependencies on runtime components. This CL makes\nDexFileLoader a stub class that is independent of ART, and introduces a\nsubclass ArtDexFileLoader that contains the current implementations.\n\nBug: 22322814\nTest: make -j 50 test-art-host\nChange-Id: Ia6e92ae93c347057ea0c10455525239cbbe42c03\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": "73f21d45a41aaad1a02eecdf3bbdbf78ef599d5e", "tree": "88d52c615cd8ec5bd8d2fad44fddeef0911e2458", "parents": [ "d096b3a09f0e7129813fb663fec5b5f131565d71" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 02 14:26:50 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 02 17:53:21 2018 -0800" }, "message": "Remove CodeItem accessor functions\n\nThese are replaced by the accessor helpers.\n\nBug: 63756964\nTest: test-art-host\nTest: test/testrunner/testrunner.py --host -j30\n\nChange-Id: Ic93d60b68b684eeb5f69be286b4e15b8f8f97542\n" }, { "commit": "57943810cfc789da890d73621741729da5feaaf8", "tree": "367677a982a45af98ffe3e79543615875e8550b4", "parents": [ "d5153627778e71ef68b510ce03c77467fa4d85bd" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Dec 06 21:39:13 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 07 16:26:11 2017 -0800" }, "message": "ART: Replace base/logging with android-base/logging\n\nReplace wherever possible. ART\u0027s base/logging is now mainly VLOG\nand initialization code that is unnecessary to pull in and makes\nchanges to verbose logging more painful than they have to be.\n\nTest: m test-art-host\nChange-Id: I3e3a4672ba5b621e57590a526c7d1c8b749e4f6e\n" }, { "commit": "f3c52b42a035902245d00a619fed0275afb063d2", "tree": "c46dab07826be55e9ca92ab301eed586c2f307ca", "parents": [ "b360bff818ad0bf59668cd2bebaaeeaa8a3b5dfe" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Nov 17 17:32:12 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Nov 21 17:09:44 2017 +0000" }, "message": "Fill Class and String .bss slots in runtime.\n\nShift the responsibility for filling Class and String .bss\nslots from compiled code to runtime. This reduces the size\nof the compiled code.\n\nMake oatdump list .bss slot mappings (ArtMethod, Class and\nString) for each dex file.\n\naosp_taimen-userdebug boot image size:\n - before:\n arm boot*.oat: 36534524\n arm64 boot*.oat: 42723256\n - after:\n arm boot*.oat: 36431448 (-101KiB, -0.3%)\n arm64 boot*.oat: 42645016 (-76KiB, -0.2%)\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nTest: testrunner.py --target --optimizing\nTest: m dump-oat, manually inspect output.\nBug: 65737953\nChange-Id: I1330d070307410107e12c309d4c7f8121baba83c\n" }, { "commit": "b7c273cb44fcbdab3c17ec69124fe4bbea2696b1", "tree": "11242e0bd24427183dd630780825214737ccbda1", "parents": [ "27f5fefeb4a7e87e9537fbdd0ed392d5c1de59df" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Nov 10 18:07:56 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Nov 10 18:07:56 2017 -0800" }, "message": "Add ClassDataItemIterator::HasNextMethod\n\nReturns true if there are either static of virtual methods remaining,\nchanged most places to use this where possible.\n\nSlight behavioral change for duplicate method checking, we not\npersist the method index across the static method / virtual method\nboundary.\n\nMotivation: Generic cleanup to remove copy paste.\n\nTest: test-art-host\nChange-Id: I7a1b507e681b2c40452f8a9913b53a96b181e171\n" }, { "commit": "e983b1658589e2a3d12620846b7e2351af26afa6", "tree": "4a5abce1e079b54262248fd809f02e97d120740a", "parents": [ "77526ffcc5f25ef9c4aa4e87d416c22f2586cf17", "b0a6aeee250945b1d156ebab94053380f2e5a3c5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Nov 10 10:02:44 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Nov 10 10:02:44 2017 +0000" }, "message": "Merge \"Record @{Fast,Critical}Native in method\u0027s access flags.\"" }, { "commit": "b0a6aeee250945b1d156ebab94053380f2e5a3c5", "tree": "7ddb52bb20997b8f1c81be3df2250949a088e697", "parents": [ "0db16e00e3927445585a588499731c58c1ae1bef" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Oct 27 10:34:04 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Nov 10 08:30:36 2017 +0000" }, "message": "Record @{Fast,Critical}Native in method\u0027s access flags.\n\nRepurpose the old kAccFastNative flag (which wasn\u0027t actually\nused for some time) and define a new kAccCriticalNative flag\nto record the native method\u0027s annotation-based kind. This\navoids repeated determination of the kind from GenericJNI.\nAnd making two transitions to runnable and back (using the\nScopedObjectAccess) from GenericJniMethodEnd() for normal\nnative methods just to determine that we need to transition\nto runnable was really weird.\n\nSince the IsFastNative() function now records the presence\nof the @FastNative annotation, synchronized @FastNative\nmethod calls now avoid thread state transitions.\n\nWhen initializing the Runtime without a boot image, the\nWellKnowClasses may not yet be initialized, so relax the\nDCheckNativeAnnotation() to take that into account.\n\nAlso revert\n https://android-review.googlesource.com/509715\nas the annotation checks are now much faster.\n\nBug: 65574695\nBug: 35644369\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nChange-Id: I2fc5ba192b9ce710a0e9202977b4f9543e387efe\n" }, { "commit": "bda66c79fa705a0f74d2233fb75bdde4d8dc5e5f", "tree": "f715660e6b0ff00bce89510aa84664493e6547ea", "parents": [ "b2269c1657c78bdeb23e5dd72708d6c0e216b89b", "2ffb703bf431d74326c88266b4ddaf225eb3c6ad" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Nov 09 01:31:19 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Nov 09 01:31:19 2017 +0000" }, "message": "Merge \"cpplint: Cleanup errors\"" }, { "commit": "2ffb703bf431d74326c88266b4ddaf225eb3c6ad", "tree": "0552c3c76a42b18f9e7460d501fb71a6dc2e7f33", "parents": [ "c4b6f3116f15c8e4fdf2e4f604ababdee12d8923" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Nov 08 13:35:21 2017 -0800" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Nov 08 15:15:52 2017 -0800" }, "message": "cpplint: Cleanup errors\n\nCleanup errors from upstream cpplint in preparation\nfor moving art\u0027s cpplint fork to upstream tip-of-tree cpplint.\n\nTest: cd art \u0026\u0026 mm\nBug: 68951293\nChange-Id: I15faed4594cbcb8399850f8bdee39d42c0c5b956\n" }, { "commit": "0021feb36ca7021e2c255e6eaf16b967180290c6", "tree": "43f1ce8f6d91630a831d03b51a465d57c2a47119", "parents": [ "550c0aee52bf8981dfe1d22d3b661d61e80cc7ee" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 07 00:08:52 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 07 13:15:41 2017 -0800" }, "message": "Delete CodeItemIterator\n\nReplace uses with DexInstructionIterator.\n\nBug: 63756964\nTest: test-art-host-gtest\n\nChange-Id: I28c839c372edcb60583867355d46b14f8752d41b\n" }, { "commit": "095c6c96236476b605b3ac672f6d2b8c151e9479", "tree": "fe04b7fe93e65efadd3f019dd1ecba4c2c40df18", "parents": [ "b277aa1385f7f4593c9978d8106669142d158f4f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Oct 19 13:59:55 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Oct 20 14:22:52 2017 +0100" }, "message": "Pass the verify flag to DexFileLoader utilities.\n\nRefactoring-only change.\n\nAlso remove unused DexFilerLoader::OpenMemory.\n\nbug: 30972906\nbug: 63920015\n\nTest: build.\n\nChange-Id: I7cd4b5787565ab1a3457ce2be6bb14657229c550\n" }, { "commit": "0284f43d625f0776ee0586a7cc321e11f5405e8c", "tree": "983c86ff79f2a5a089bfa44ec27a5f2c5d6206b6", "parents": [ "128acd4b5b34cdd51328de03df085deaa040b864", "52d52f5dc3e005829926e68c656fb27e8b008ae9" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Oct 12 09:05:08 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Oct 12 09:05:08 2017 +0000" }, "message": "Merge changes I4bbb21bf,Ie79b46cd,Ia50aafc8\n\n* changes:\n Use ScopedArenaAllocator in GVN.\n Use ScopedArenaAllocator for Phi elimination pass.\n Use ScopedArenaAllocator for building HGraph.\n" }, { "commit": "79c87da9d4698ec58ece65af0065eebd55a1cfe0", "tree": "399797f37712da9d20829d5b5059b6c84943cb02", "parents": [ "656e97f949df4081fba908113406b32915174502" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 10 11:54:29 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 11 14:47:06 2017 -0700" }, "message": "Add DexFileLoader class\n\nAdded DexFileLoader class, moved functionality from DexFile there:\n- Multidex loading logic\n- DexFile opening logic for Zip and etc\n- Some other helpers\n\nBug: 63756964\nTest: test-art-host\n\nChange-Id: Ic3dfa458947d4b69912dea5cdd836e7e8f55061c\n" }, { "commit": "69d310e0317e2fce97bf8c9c133c5c2c0332e61d", "tree": "fba05a1530e6fc4a2e6950303c1f7c6b0ffbb936", "parents": [ "e764d2e50c544c2cb98ee61a15d613161ac6bd17" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Oct 09 14:12:23 2017 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Oct 11 09:44:26 2017 +0100" }, "message": "Use ScopedArenaAllocator for building HGraph.\n\nMemory needed to compile the two most expensive methods for\naosp_angler-userdebug boot image:\n BatteryStats.dumpCheckinLocked() : 21.1MiB -\u003e 20.2MiB\n BatteryStats.dumpLocked(): 42.0MiB -\u003e 40.3MiB\nThis is because all the memory previously used by the graph\nbuilder is reused by later passes.\n\nAnd finish the \"arena\"-\u003e\"allocator\" renaming; make renamed\nallocator pointers that are members of classes const when\nappropriate (and make a few more members around them const).\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host\nBug: 64312607\nChange-Id: Ia50aafc80c05941ae5b96984ba4f31ed4c78255e\n" }, { "commit": "9ab6e1dd23781e5c0a4526622d9dbb7f120d3530", "tree": "2e4d3f4e76743df99b2944340726d29190880e92", "parents": [ "208dd6076bdddcb04afb0397e4d4708f44a5dd60" ], "author": { "name": "Shubham Ajmera", "email": "shubhamajmera@google.com", "time": "Mon Sep 25 18:40:54 2017 -0700" }, "committer": { "name": "Shubham Ajmera", "email": "shubhamajmera@google.com", "time": "Wed Sep 27 16:03:35 2017 -0700" }, "message": "Revert^2 \"Increase threshold for profile compilation\"\n\nIncrease threshold for profile compilation\n\nThreshold to trigger compilation based on profiles has been increase\nto avoid re-compilation too frequenctly.\nNow compilation will take place if methods/classes in the new profile exceeds\nby maximum of the following:\n- 2% methods/classes in the existing profile.\n- 100 methods or 50 classes.\n\nContext for above numbers:\nI analyzed profiles on my local device. Average number of methods and classes\nin the profiles were 8000 and 2500 respectively.\n\nAlso added tests for the same.\n\nBug: 66732454\nTest: test-art-[host|target]-gtest-profile_assistant_test\nThis reverts commit a660171d8fbf907def7720b2af5e045081f11094.\n\nChange-Id: I5c5f8d76a32900c42cda21052636a8588d28e521\n" }, { "commit": "dbeacd6f1687439bd391bdd6a66a4b48d7d45ebf", "tree": "97ab216c77bd586e93beb5074c558b4729a97cb5", "parents": [ "da6f3eee076aced30fdfdc75604509cde516c307", "fc8b422c286501346b5b797420fb616aaa5e952a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Sep 18 20:03:45 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Sep 18 20:03:45 2017 +0000" }, "message": "Merge \"Clean up AtomicDexRefMap\"" }, { "commit": "e2abbc604ce003c776c00ecf1293796bb4c4ac5a", "tree": "f7d124d1861cad2162c30dfe932bb4e1beaf41ef", "parents": [ "7090dfe84f78b1928fcbdfd664d0dd9ea52633ff" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Sep 15 11:59:26 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 18 11:36:24 2017 -0700" }, "message": "ART: Move kDexNoIndex to dex_file_types.h\n\nDefine the constant with the types to allow lowering the dependency\non DexFile.\n\nTest: m\nChange-Id: I3c61421db45be96d2057e01b1a7825883d8bd178\n" }, { "commit": "fc8b422c286501346b5b797420fb616aaa5e952a", "tree": "61c857a895cdad9ce387a899f92824701259df32", "parents": [ "7090dfe84f78b1928fcbdfd664d0dd9ea52633ff" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Sep 17 13:44:24 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Sep 18 10:57:06 2017 -0700" }, "message": "Clean up AtomicDexRefMap\n\nMake ClassReference, TypeReference, and MethodReference extend\nDexFileReference. This enables using all of these types as the key\nfor AtomicDexRefMap.\n\nTest: test-art-host\nBug: 63851220\nBug: 63756964\n\nChange-Id: Ida3c94cadb53272cb5057e5cebc5971c1ab4d366\n" }, { "commit": "d0b08252a508ad86780e494462cbf005718f1c2b", "tree": "287a875ee7b67dad162c67db51b97ba1fb8de6b3", "parents": [ "cdf659e4b282b78f7f497545a7f16fcfa5c89f28", "d704f0b4414fc1bd4f0be4ac615386ae9ed8bfd7" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Aug 30 19:58:56 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Aug 30 19:58:56 2017 +0000" }, "message": "Merge \"Profman: Use method_percentage and class_percentage values passed for generating profiles\"" }, { "commit": "641a473912b3bcaaff2c71070611490b7c547cfc", "tree": "67bf5b2eaaeaf19c6cebac29984b49c19f172f9b", "parents": [ "5555dd1df3c7f3aaea548c3b657b9325538e9780" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Aug 24 13:21:35 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Aug 25 08:59:29 2017 -0700" }, "message": "ART: Fix reference-related warnings\n\nMake some parameters and locals references to avoid unnecessary\ncopies.\n\nBug: 32619234\nTest: m test-art-host\nChange-Id: Idfed600c7c5492a80f72f51424253bcbea824a94\n" }, { "commit": "3157fc26d96951ff4acfb4feb4597e74a92da2a7", "tree": "93c3a03a8eae4fc9f430531f25d8be3d4d8b5af7", "parents": [ "27bd4dd81962b0e59df9318ae8efc83df99ebe0a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Aug 23 09:43:46 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Aug 23 16:06:00 2017 -0700" }, "message": "ART: Move cmdline parser to header library\n\nExpress the cmdline parser as a header library.\n\nTODO: Express the libart dependency correctly.\n\nTest: m\nChange-Id: I9e90029d047ab0d75a455a2361d5213a8fa6f361\n" }, { "commit": "d704f0b4414fc1bd4f0be4ac615386ae9ed8bfd7", "tree": "59ba287fdfa887c8b540be18242a4de02b8f8489", "parents": [ "c1bb1cd339b2ebea9c4770fb4d61bacd7d77746f" ], "author": { "name": "Shubham Ajmera", "email": "shubhamajmera@google.com", "time": "Wed Jul 26 16:33:35 2017 -0700" }, "committer": { "name": "Shubham Ajmera", "email": "shubhamajmera@google.com", "time": "Mon Aug 14 11:11:27 2017 -0700" }, "message": "Profman: Use method_percentage and class_percentage values passed for generating profiles\n\nThe percentage was only used for profiles without dex files, however, this\ncould also be used when the dex files are provided.\n\nTest: manual\nChange-Id: Ia8eff4231951b6531ce887938ae2eb10ea502089\n" }, { "commit": "8cf9cb386cd9286d67e879f1ee501ec00d72a4e1", "tree": "88e86e214b425e444760fe4e0ffeee677e1558a2", "parents": [ "914b7b6a6c9f399b26b41e9160e9871ef749e0db" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jul 19 09:28:38 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 24 16:07:10 2017 -0700" }, "message": "ART: Include cleanup\n\nLet clang-format reorder the header includes.\n\nDerived with:\n\n* .clang-format:\n BasedOnStyle: Google\n IncludeIsMainRegex: \u0027(_test|-inl)?$\u0027\n\n* Steps:\n find . -name \u0027*.cc\u0027 -o -name \u0027*.h\u0027 | xargs sed -i.bak -e \u0027s/^#include/ #include/\u0027 ; git commit -a -m \u0027ART: Include cleanup\u0027\n git-clang-format -style\u003dfile HEAD^\n manual inspection\n git commit -a --amend\n\nTest: mmma art\nChange-Id: Ia963a8ce3ce5f96b5e78acd587e26908c7a70d02\n" }, { "commit": "8eecddf0ccb18bd3b4d32a48c3b57852e03c12b4", "tree": "a3398ee0cd07d8ad8a8b89144c019c881a6e4388", "parents": [ "c9267c48979698a9217760c914aba13ea20b5990" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 12 16:05:54 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 12 16:44:11 2017 -0700" }, "message": "Add missing profman arg for compiled_method_threshold\n\nThe arg is --boot-image-sampled-method-threshold, this value\nspecifies how many profiles a non-hot method should be before it gets\nmarked as hot in the boot profile. This is useful if we want to do\nthings like compiling all executed methods for the boot image.\n\nBug: 37966211\nTest: test-art-host-gtest-profile_assistant_test\n\nChange-Id: Ib882df1856d555048a4a19343c233232ee848e8a\n" }, { "commit": "9c9b9364ad24b4e20b2d88e50f4e013b1f3fa093", "tree": "11a60dbff92e963f67824efbb92ce064611a98ec", "parents": [ "f45cf4bfb341051d9d0959f2e2619eaec71fa732", "247fc334ca91c22b19ac57b9c8bbbd15a1735aa1" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Jun 23 15:03:31 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Jun 23 15:03:33 2017 +0000" }, "message": "Merge changes I8d333ea5,Ie4efc8bb\n\n* changes:\n ART: Make gAborting an atomic\n ART: Refactor abort code\n" }, { "commit": "2f7945540cf511148c73672f2ce80ae859b95bed", "tree": "ae78e1978b99b991a51a73281378ab70b2629e8a", "parents": [ "a08c9bbe02073d34358c15e3339fa47ed49c39fc" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 19 10:58:08 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jun 23 00:00:34 2017 +0000" }, "message": "Add support for generating boot image profile\n\nAdded three options:\n--generate-boot-image-profile:\nIf this option is passed in, profman creates a boot image profile.\n\n--boot-image-clean-class-threshold\u003d\u003cvalue\u003e\nSpecifies how many occurrences of a likely clean class are required\nbefore a class is added to the profile.\n\n--boot-image-class-threshold\u003d\u003cvalue\u003e\nSpecify how many occurrences of a possibly dirty class are required\nbefore a class is added to the profile.\n\nAdded unit test.\n\nTest: test-art-host\n\nBug: 37966211\n\nChange-Id: I8e12b0ec34dfa1d1bed0b51f342fffde09815348\n" }, { "commit": "51d80ccca5eb2ea0eef0de836dcc03e0545f63db", "tree": "14ff2bf5b9a9ca86d8a8d6f736cf27d8a1d47554", "parents": [ "19df565ffcb30cb7b5217f074771078f0bb6cc1c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 21 21:05:13 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 22 16:04:07 2017 -0700" }, "message": "ART: Refactor abort code\n\nRemove Runtime::Aborter and let Runtime::Abort set the abort message\non target.\n\nThis works around a missing tail-call optimization that leads to a\nsuperfluous frame on the call-stack when aborting.\n\nPartially reverts commit 3fec9ac0d5af1358d216eb2fdc2000ec0205f3f0.\n\nBug: 62810360\nTest: m test-art-host\nTest: Manual inspection of abort dump\nChange-Id: Ie4efc8bbdc8b665b23081b37a11921fe26b182b4\n" }, { "commit": "f70fe3d0b0b29c955a4f7310beae1cfb926ca5c7", "tree": "d69a44ce49c1745c2c2e08680915c7a1ce587693", "parents": [ "7888b59b4542e15dd061047e4cb34f795cc2354c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jun 21 15:24:02 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 22 11:11:50 2017 -0700" }, "message": "Add dex2oat support for profile based image creation\n\nAllow creating a boot image with only a profile without requiring\nany of --compiled-classes, --compiled-methods, or --image-classes.\n\nTo do so, you need to pass --profile-file and\n--compiler-filter\u003dspeed-profile.\n\nAdded a test dex2oat_image_test to verify behavior. This test covers:\n--compiled-classes\n--compiled-methods\n--image-classes\n--profile-file\n\nTest: test-art-host-gtest-dex2oat_image_test\n\nBug: 37966211\n\nChange-Id: I36d41b1c6e6560e6b3494b1df7024a4450ed0c0e\n" }, { "commit": "e46f3a8399fec0f2cbb6d730f4f8316c37d7f3c5", "tree": "b23f2da67063bee35ae64032f4d0b389a22e7adf", "parents": [ "e882b6dc06804c08206a9fd027a7f67705941f48" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 19 19:54:12 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jun 20 16:21:31 2017 -0700" }, "message": "Address some review comments for aog/415919\n\nTest: test-art-host\nBug: 62040831\nChange-Id: I9d8c359ab3a1ff090fe664bbb798a215a3d9eb52\n" }, { "commit": "e17cf243978ca7f5356e746ee460f0fcf04e04a6", "tree": "e8ad423bf1dee5756fec451ba22db749ed4f5472", "parents": [ "2bc159ebd1115f71d80639e47fc38663aa84ba4a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 19 11:05:51 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 19 11:11:12 2017 -0700" }, "message": "Add field/method skipping helper functions to DexFile\n\nSpecifically added SkipStaticFields, SkipInstanceFields,\nSkipAllFields, SkipStaticMethods, and SkipVirtualMethods.\n\nAlso applied the helpers to the code base.\n\nTest: test-art-host\nChange-Id: Idda77f9a6e2564c2e588d2bbe11cc320063fdb32\n" }, { "commit": "a5dcdf346158d47bfe80d8adb29b1b020cede286", "tree": "50e56b3445dc8e4724d6707ffecd23f9dae96e31", "parents": [ "942c597b004aa328cf291a78392b405ef5d5f040", "bbe3a5efcdfb2fb15a4e5f724cc323ae0ada5111" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 15 20:21:23 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jun 15 20:21:24 2017 +0000" }, "message": "Merge \"Refactor some profiling info functions\"" }, { "commit": "bbe3a5efcdfb2fb15a4e5f724cc323ae0ada5111", "tree": "65fd149e1159674681d866e47e485d85b8464334", "parents": [ "d58932f98a673c9d687863a7efafb2da168063a2" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jun 13 16:36:17 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 15 11:45:06 2017 -0700" }, "message": "Refactor some profiling info functions\n\nDeleted profile_compilation_info-inl.h\n\nAdded a new helper class MethodHotness to reduce the number of\nrequired getters for reading method hotness, startup, and post\nstartup booleans.\n\nTest: test-art-host\nBug: 62040831\n\nChange-Id: I799a38e2bea6177ba5816c54524f4ccacedba772\n" }, { "commit": "a3d27ebd2802fd6031cc928a67752541775bf52b", "tree": "96c4a460fbf81fd00b8c71063f5ef9ae3bc56eda", "parents": [ "d38b67c9892b603f57a6b51a1b9d7f4e610158aa" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu May 11 13:50:59 2017 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Wed Jun 14 10:29:55 2017 +0100" }, "message": "ScopedFlock: Refactor it to be a subclass of FdFile.\n\nMakes callers cleaner, since they only have to worry about\ndealing with regular File objects that they know will be locked\nfor the duration of their existence. Prevents issues and other\nclunky code relating to acquire\n\nTest: scoped_flock_test, test_art_host\nBug: 36369345\n\nChange-Id: I2c9644e448acde6ddac472d88108c7d9a4e1a892\n" }, { "commit": "28b5c582070a14979dc8cbcddba136ac334ed52a", "tree": "9ae2955e5d1c2ac832c86a8b0e71f0f6226007e8", "parents": [ "6ecff4d2127e70738aa2493d6deceb946c204eff" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jun 06 14:12:50 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jun 06 14:15:42 2017 -0700" }, "message": "Dump startup and post startup methods for --dump-only\n\nThe format is:\nhot methods: ...\nstartup methods: id0, id1, ...\npost startup methods: id0, id1, ...\nclasses:\n\nAlso added test.\n\nTest: mm test-art-host-gtest-profile_assistant_test -j32\nBug: 62040831\n\nChange-Id: I2ffbe17e888d4adae348bb14e0b92c284ce1081c\n" }, { "commit": "ea650f3bc4f54eb2c647cf0f7134398ef5038792", "tree": "cdddde74424cf9fbeb2ef62e6bebca619c5ed989", "parents": [ "6559ba1cfb9d24fdca4b1b1ec9e0b8dc31679982" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 24 12:04:13 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jun 02 18:25:09 2017 +0000" }, "message": "Move profiles to use bitmaps for startup methods\n\nThis CL enables changes like compiling only hot methods while still\ndoing layout for hot and startup methods. The bitmaps are also a\nbit smaller for post-launch use cases.\n\nNo change in compilation strategy yet.\n\nFixed some bugs in dexlayout test like using a profile with the wrong\ndex location. This meant the second invocation of dexlayout didn\u0027t\nhave any profile data.\n\nAdded profman support for dump-classes-and-methods,\ncreate-profile-from, and related test.\n\nProfile sizes (bytes) post launch:\nGmail: 7290 -\u003e 6136\nMaps: 22896 -\u003e 18984\nMusic: 8582 -\u003e 7050\nYouTube: 16733 -\u003e 14592\n\nTest: test-art-host\n\nBug: 62040831\n\nChange-Id: I9915b81a2ff2c47464acbbdeb55ce30a33d5483f\n" }, { "commit": "08556886a16ff2bb9fc3f184ac699de21c0369cd", "tree": "c90ddce83ac98252ced0a4181b0c4b1e0e34c1f8", "parents": [ "854461a4cd5e4a38debe3616e12b52fe7f160782" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri May 26 16:40:45 2017 -0700" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Jun 01 06:54:11 2017 -0700" }, "message": "Handle gracefully profiles with invalid classes or methods\n\nBug: 38410980\nTest: m test-art-host-run-test-707\nChange-Id: I8c1b0a00c113c0faf0cc5d141e67e4183322520f\n" }, { "commit": "dbddc22f5dc2d1ff4d4783fbd66c27812f4980d1", "tree": "2a0a8efa1c2630e57ab48ab2de171f2847ff282f", "parents": [ "a559fa1b0d6c276dde2cdc707de1acd4950f7190" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 24 12:04:13 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 25 16:59:44 2017 -0700" }, "message": "Refactor profiles to use TypeReference instead of ClassReference\n\nRefactor type reference into runtime and use it for profiles.\nClassReference was just duplicated code since it wasn\u0027t even using\nthe class def indexes.\n\nTest: test-art-host\n\nBug: 62040831\nChange-Id: Ia92f21c0e85c00321f52c97bb7a90158d882849b\n" }, { "commit": "a559fa1b0d6c276dde2cdc707de1acd4950f7190", "tree": "8314a003ff09cee50acd15929c1edbb8a448eef8", "parents": [ "673253e1628ca9483ea121571bb9a96f90a1c944", "e6f87cc95e5351f3872a4c50e18915a282578ff8" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu May 25 23:57:29 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu May 25 23:57:32 2017 +0000" }, "message": "Merge \"Fix compiler crash due to inline caches and improve docs\"" }, { "commit": "e6f87cc95e5351f3872a4c50e18915a282578ff8", "tree": "94af0c8fb8c8926f2bf3fe20d8c3ad6d7dcc4efb", "parents": [ "798ba16bda747d74a27b7a748d3bb5deb2ccbf68" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed May 24 17:41:05 2017 -0700" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu May 25 15:09:31 2017 -0700" }, "message": "Fix compiler crash due to inline caches and improve docs\n\nFix a potential crash when extracting the inline caches from the method.\nThe way we copied the inline cache into a new map was not correct. In\nProfileCompilationInfo::GetMethod() we reused the same profile arena\nfor allocation which is not thread safe. When compiling with multiple\nthreads the profile arena could become corrupted due to races.\n\nAddress all the comments from the late reviews on the CL which migrates\nthe profiles to arena storage.\n\nTest: m test-art-host\n compile with speed-profile apps on device\nBug: 37711886\nBug: 62062532\nChange-Id: I61af5175bc68b2c7dba77afb3cdff221989cc387\n" }, { "commit": "5115efb3369d522ad34b919114de2797bcd81642", "tree": "2cba2a6038f13e52f8d416a91ef0021d83c03017", "parents": [ "7dd1e683ea61341323ba8b138684f4c4fbddb3ab" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 24 16:55:54 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 24 16:55:54 2017 -0700" }, "message": "ART: Fix soong defaults order\n\nDefaults are prepended in order of their appearance. Move debug\ndefaults first.\n\nBug: 31098551\nTest: m test-art-host\nChange-Id: I9a64db34d0e398d336dac080c7234cad77b719ee\n" }, { "commit": "cc3171ab3af35b4e4ad5671d7425aa8a6bfcc5f3", "tree": "ec2c8c37b4d4cb1c213aafd2e5ab41fb115fb241", "parents": [ "80dd30abe3fac52f6de4aec0543918d360f7d08a" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri May 19 16:47:53 2017 -0700" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Mon May 22 19:00:04 2017 -0700" }, "message": "Use arena allocation for profiles\n\nBy using our arena allocator we can madvise the memory used during\nprofile processing right way. jemalloc may defer the release based on\nunpredictable native allocation.\n\nThe other advantage of arenas is a much simpler way to measure the\nmemory needed by profiles.\n\nTest: m test-art-host\nTest: manual inspection with meminfo and heaptrack\nBug: 37711886\n\nChange-Id: Ib656a8ac63600477ff553831087a83dc40d9c537\n" }, { "commit": "66aae3b2191fd8cbb4b932bf517f3cb94b7c8b1e", "tree": "f59a312872a9e24cc8fd71c216b0013ab8d82885", "parents": [ "ff120c2ad8862d595908907fb8f83d024ca45957" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 18 10:38:28 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu May 18 10:38:56 2017 -0700" }, "message": "Move some profman messages from LOG(ERROR) to LOG(WARNING)\n\nReduces build spam since we filter out warnings during the build.\n\nBug: 38411714\nTest: m\nChange-Id: I109e261d319e6afda099b4c0c887e150bdc36325\n" }, { "commit": "34067269b9493239e1fdbdd562eda1b0d2c9ec39", "tree": "bb0f37b56e62167896c60a1346d7a91d4c6c7aa2", "parents": [ "8309d9b173b205bb9530dbd94e48731c20b8e4f4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 06 13:55:46 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 07 16:23:43 2017 -0700" }, "message": "Change dump-classes profman option to dump-classes-and-methods\n\nThe new option is called \"--dump-classes-and-methods\". This option\ndumps methods and classes in the profile as a format that is valid\ninput for the \"--create-profile-from\" option.\n\nThe goal is to be able to store text based profiles that don\u0027t\ndepend on dex indices or checksums.\n\nTest: mm test-art-host-gtest-profile_assistant_test -j40\n\nBug: 36457259\nBug: 34927277\n\n(cherry picked from commit 09f6c36c0c5ac055bbb5e36991ab60156be1fb46)\n\nChange-Id: I79c4bd8da5c0b4dc9c4c6daed538845792f52299\n" }, { "commit": "cea9e9d26c32372ec8c75e6c89b1bfc08c5e1911", "tree": "5132e19736f2afdd2b10f3243297e667b2ab0a21", "parents": [ "f8d00ff206e33ff66fcabcf75d6b5bbe30cddb98" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Mar 23 19:04:59 2017 -0700" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Tue Mar 28 16:51:02 2017 -0700" }, "message": "Fix profile merges in profman\n\nprofman used ProfileCompilationInfo::Load() which was not preserving the\ncorrect order of the dex files (in a multidex profile).\n\nThe CL fixes profman to use ProfileCompilationInfo::MergeWith which\nguarantees the right dex order and redesigns profile storage to avoid\nsuch mistakes in the future. Instead of keeping data in a map indexed by\nthe profile key, store it in a vector whose index match profile_index.\nThis way, any iteration over profile info becomes deterministic with\nrespect to the profile index of the dex files.\n\nTest: m test-art-host-gtest-profile_assistant_test\n m test-art-host-gtest-profile_compilation_info_test\n profile YouTube.apk and compile it based on the reference profile\n(failing before)\n\nBug: 36371709\nChange-Id: Ideda1336e5aff59a7c5560429da645fe02c804c9\n" }, { "commit": "f0a31f865c3b3e5843ddd52f015c70293b54b4e4", "tree": "c6881a22ddb9523220861c070991a83e1bc1f97d", "parents": [ "1082e0e46ddfae2ed9cc3d0a3d6025e7335ac239" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Mar 27 15:50:37 2017 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Mar 28 10:57:11 2017 -0700" }, "message": "Add profman option to generate a random profile based on dex files.\n\nExtends --generate-test-profile to make a randomized profile based on a\nset of dex files. Will randomly incorporate classes and methods from all\nthe dex files given. Can be given a random seed for reproducibility.\n\nBug: 36107940\nTest: mm test-art-host-gtest-profile_assistant_test\nChange-Id: Ib0e54bde5317439516a39afa41c3c9980834fa54\n" }, { "commit": "37c5846be19f28bd629ffe7d6cfde779abbccb89", "tree": "794faf53dceaff722889a94c01122cc2ef6b632a", "parents": [ "46bfb7c047a590ac5c24b658f31c170631556bb6" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Mar 27 15:14:27 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Mar 27 15:14:27 2017 -0700" }, "message": "ART: Fix some tidy warnings\n\nFix some tidy warnings for macros and references.\n\nTest: m test-art-host\nChange-Id: I2f8ed2174634263d4784c08e637f60ed70977b73\n" }, { "commit": "d808e8bb070b853639e41245d846f2fd9354b92f", "tree": "211dc821337006864fd4de739e0c24886f389894", "parents": [ "f9806373ca50896ac54cb9da49a030c13ec14769" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 21 13:37:41 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Mar 22 13:10:32 2017 -0700" }, "message": "Add syntax for adding all class methods to profman\n\nUsed to generate methods from the class list. The logic adds all\nthe methods if the method string is \"*\". The class is also added to\nthe profile.\n\nBug: 36457259\n\nTest: mm test-art-host-gtest-profile_assistant_test -j32\n\nChange-Id: Ibee856646b94eccc48d9ef6566475be74e77a0cd\n" }, { "commit": "a6fa68f22965ae065f06bebd1388a8ebfdeedca6", "tree": "817791f8a5cbba017e2e2efbc156c6932440a3b1", "parents": [ "7733305ab5716d1b1399bd7e261bfa6396eabba3", "589e71eb71477603c88b14118b2d3a33508c3c79" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Tue Mar 07 23:27:22 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Mar 07 23:27:23 2017 +0000" }, "message": "Merge \"Encode inline caches with missing types in the profile\"" }, { "commit": "589e71eb71477603c88b14118b2d3a33508c3c79", "tree": "95d88c7c8ba3a9b2009361d61164cef6a6339308", "parents": [ "d9f647ac9a5cb0c6f112332d584756251c2cfb9f" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Mar 03 16:05:05 2017 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Mon Mar 06 12:57:30 2017 -0800" }, "message": "Encode inline caches with missing types in the profile\n\nNot all runtime types can be encoded in the profile. For example if the\nreceiver type is in a dex file which is not tracked for profiling its\ntype cannot be encoded.\n\nPreviously we would just skip over these types but that can lead to\nencode a polymorphic inline cache when in fact it should be megamorphic.\n\nWith this CL, inline caches for which types are missing are marked in\nthe profile with a special bit, kIsMissingTypesEncoding.\n\nAlso, extend profman to understand text lines which specify an inline\ncache with missing types.\n\nTest: test-art-host\n\nBug: 35927981\nBug: 32434870\nChange-Id: I34528a39c227f3133771fd4454701c1ddc234f40\n" }, { "commit": "3f1213445ebfa212d6a03bb381f30333e02dca48", "tree": "5ebea247a4c996d495fc814922d62f435a886f0e", "parents": [ "5c90d0b0f332436f7ca5c028256bf5e91e9023d4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Mar 06 11:16:20 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Mar 06 11:16:31 2017 -0800" }, "message": "Change \"could not find class\" to warning\n\nOtherwise its too spammy during build.\n\nBug: 34929159\nBug: 34927277\nTest: test-art-host\n\nChange-Id: I94ab0ad4e3d4b8f4817c6ddadafa2576b415c746\n" }, { "commit": "e0ac1151b360be7147fa20320c0b427688b1424f", "tree": "c33419b00c47b731075bcfd7b75b81f366beda91", "parents": [ "425b5d23e2c60d295471817a75b1b554481c5334" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Mon Feb 13 19:03:47 2017 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Mar 03 13:12:31 2017 -0800" }, "message": "Extend profman to generate profiles with inline caches\n\nExtend profman logic to generate profiles based on a simple textual\nrespresentation. This will help writing tests for profile guided\ncompilation.\n\nBefore this CL, profman was able to generate profiles based on a list of\nclasses like:\njava.lang.Comparable\njava.lang.Math\njava.lang.Object\n\nThis CL, enables profman to understand methods and classes alike. The\nnew format is:\n\n# Classes\nLjava/lang/Comparable;\nLjava/lang/Math;\n# Methods with inline caches\nLTestInline;-\u003einlinePolymorhic(LSuper;)I+LSubA;,LSubB;,LSubC;\nLTestInline;-\u003enoInlineCache(LSuper;)I\n\n\"LTestInline;-\u003einlinePolymorhic(LSuper;)I+LSubA;,LSubB;,LSubC;\"\nmeans that method `int inlineMonomorphicSubA(Super)` from class Main\nwill be added to the profile with the inline cache (SubA,SubB) for its\none and only invoke virtual.\n\n@Main#noInlineCache:(LSuper;)I+;\nmeaning that method `int noInlineCache\u0027 from class Main will be added\nto the profile with no inline cache.\n\nNote that the methods are allowed to have a single invoke virtual in\ntheir dex bytecode. That is to keep the parsing the file format\nsimple and easy to use.\n\nAlso, add a few more tests for profiles and fix an issue caused by\nwriting the dex files in a possibly wrong order.\n\nTest: m run-test-host-gtest-profile_assistant_test\nBug: 32434870\nChange-Id: I6b7340cf613007117d9818be206ccb3a27b815bf\n" } ], "next": "940eb0c00ef531dd9a0a68dbd61e377832e81eb4" }