)]}' { "log": [ { "commit": "90faceb71e25748172ba6369209f8a2a66735394", "tree": "64707bbd718b29de9a19c01b12b0cc872685bb13", "parents": [ "62a4bcf3fe11e6800f5d451b41003c135358ed6a" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri Dec 14 14:36:15 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Jan 21 09:48:59 2019 +0000" }, "message": "hiddenapi: Support \u0027core-platform-api\u0027 flag\n\nAdd support for parsing @CorePlatformApi stubs and encoding it in\nhiddenapi dex flags of the corresponding fields/methods.\n\n(1) The CL refactors hiddenapi::ApiList class to store a second value:\na bit vector of \"domain API\" flags. These are intended for encoding\nmembership in a set of API stubs only available to certain callers,\ne.g. @CorePlatformApi when platform code calls core platform or\n@TestApi for CTS tests.\n\n(2) Parse @CorePlatformApi stubs and set domain flags for its members.\n\n(3) Parse the flags at runtime and set kAccCorePlatformApi access flag\non the corresponding ArtField/ArtMethod objects.\n\nBug: 119068555\nTest: m appcompat\nTest: dexlayout -b \u003ccore-oj jar\u003e | grep \u0027CORE-PLATFORM-API\u0027\nChange-Id: Idbfa6d3af7459258a5a0b6da7c03c037a577eb75\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": "dcfa89bfc06a6c211bbb64fa81313eaf6454ab67", "tree": "4aaf6da88861029e387525957bf68796cf95648c", "parents": [ "fd667d4165aefe51010e4fe3bd81b820ed9884ef" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Oct 31 11:04:10 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Nov 22 12:49:26 2018 +0000" }, "message": "Rename and reorganize hiddenapi::ApiList\n\nChange values of ApiList flags to make them easier to extend in the\nfuture and unify naming across all components. Light greylist is now\njust \"Greylist\", dark greylist becomes \"GreylistMaxO\".\n\nNote that the version code in \"GreylistMaxO\" must also include any\nmaintenance releases, i.e. entries on \"GreylistMaxO\" are accessible\nto apps with targetSdkVersion\u003c\u003d27 (O MR1).\n\nTest: m, phone boots\nTest: m test-art\nChange-Id: I9622e0646eb265008a8bb2652270876ae95dac84\n" }, { "commit": "20c765f645fa9be77e045463c5064d41211a2815", "tree": "dcb9dfb6bf5363d1e933589f22339dc999b96703", "parents": [ "ce2a00daa92670a4fc01ef59fdbc3769a846f69c" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Sat Oct 27 21:45:15 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Sun Oct 28 20:00:03 2018 +0000" }, "message": "Revert^4: Add dex item for hiddenapi flags\n\nMove hiddenapi access flags to own data section so as to:\n(a) increase amount of information stored per method/field\n(b) use encoding which can be supported long-term.\n\nThe dex item is implemented as:\n - array of offsets indexed by class def index\n - streams of uleb-128 encoded flags.\nOffsets in array point to the beginning of the flags stream\nfor the given class def. Flags are encoded in the same order\nas fields and methods are encoded in class data. Zero offset\nmeans that the class either does not have class data, or all\nof its flags are zero.\n\nThe patch updates:\n(a) libdexfile with data structure declarations and accessor\n methods,\n(b) hiddenapi tool to create the new item from hiddenapi lists\n and insert it into the given dex file,\n(c) dexlayout to copy the flags into compact dex,\n(d) dex file verifier to verify the item.\nIt also removes skipping of verification for boot class path\ndex files as those now pass DexFileVerifier, and removes the\nneed for removing the flags for JVMTI.\n\nThe size increase is 450 KB.\n\nThis reverts commit d33d318685ec4a1c9e7995c914c104ab6487513b.\n\nChange-Id: Id00e0efb38ee1eab8d7ed5c645a7778b6b94b849\nTest: phone boots\nTest: m test-art\n" }, { "commit": "d33d318685ec4a1c9e7995c914c104ab6487513b", "tree": "3bb8a93f499ac537d445e90533598571f3d2d120", "parents": [ "fe3a979f6214e2ea94a9aed4c573c6a6f64c7096" ], "author": { "name": "Luman Qu", "email": "quluman@google.com", "time": "Fri Oct 26 16:53:25 2018 +0000" }, "committer": { "name": "Luman Qu", "email": "quluman@google.com", "time": "Fri Oct 26 16:53:25 2018 +0000" }, "message": "Revert \"Revert^2: Add dex item for hiddenapi flags\"\n\nThis reverts commit fe3a979f6214e2ea94a9aed4c573c6a6f64c7096.\n\nReason for revert: Caught by droidcop https://b.corp.google.com/issues/118481351\n\nChange-Id: I17cc3586c9ae3b49839633d9543edce2937c388d\n" }, { "commit": "fe3a979f6214e2ea94a9aed4c573c6a6f64c7096", "tree": "b31c1748dcd30a5bf76c2186b4c0124fb18b9dd6", "parents": [ "d9a0437b0d6cb1438bf21284a2502cbfe293db06" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Oct 25 17:24:38 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Oct 25 17:25:17 2018 +0000" }, "message": "Revert^2: Add dex item for hiddenapi flags\n\nMove hiddenapi access flags to own data section so as to:\n(a) increase amount of information stored per method/field\n(b) use encoding which can be supported long-term.\n\nThe dex item is implemented as:\n - array of offsets indexed by class def index\n - streams of uleb-128 encoded flags.\nOffsets in array point to the beginning of the flags stream\nfor the given class def. Flags are encoded in the same order\nas fields and methods are encoded in class data. Zero offset\nmeans that the class either does not have class data, or all\nof its flags are zero.\n\nThe patch updates:\n(a) libdexfile with data structure declarations and accessor\n methods,\n(b) hiddenapi tool to create the new item from hiddenapi lists\n and insert it into the given dex file,\n(c) dexlayout to copy the flags into compact dex,\n(d) dex file verifier to verify the item.\nIt also removes skipping of verification for boot class path\ndex files as those now pass DexFileVerifier, and removes the\nneed for removing the flags for JVMTI.\n\nThe size increase is 450 KB.\n\nThis reverts commit d9a0437b0d6cb1438bf21284a2502cbfe293db06.\n\nTest: phone boots\nTest: m test-art\nChange-Id: I224bf1b587529f36ea00a666680c4e4c8ca16a42\n" }, { "commit": "d9a0437b0d6cb1438bf21284a2502cbfe293db06", "tree": "3bb8a93f499ac537d445e90533598571f3d2d120", "parents": [ "fd2aa2bf72563870c9261f248f992a20ac4837a8" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Oct 24 12:34:06 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Oct 24 12:37:49 2018 +0000" }, "message": "Revert \"Add dex item for hiddenapi flags\"\n\nThis reverts commit 0dbc363f32d075017e1c4fb5e17715e3f12d0157.\n\nReason for revert: marlin running out of space\n\nChange-Id: I80218af6408812b639b7c807bf8cb2a1d5239d94\n" }, { "commit": "0dbc363f32d075017e1c4fb5e17715e3f12d0157", "tree": "950dfed748f63863db6b95d7b854940171b544c6", "parents": [ "62ada4cd2c9a7ef5d1ab4c538d660f8a6a7f7571" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Sep 10 09:14:30 2018 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Oct 23 14:51:01 2018 +0100" }, "message": "Add dex item for hiddenapi flags\n\nMove hiddenapi access flags to own data section so as to:\n(a) increase amount of information stored per method/field\n(b) use encoding which can be supported long-term.\n\nThe dex item is implemented as:\n - array of offsets indexed by class def index\n - streams of uleb-128 encoded flags.\nOffsets in array point to the beginning of the flags stream\nfor the given class def. Flags are encoded in the same order\nas fields and methods are encoded in class data. Zero offset\nmeans that the class either does not have class data, or all\nof its flags are zero.\n\nThe patch updates:\n(a) libdexfile with data structure declarations and accessor\n methods,\n(b) hiddenapi tool to create the new item from hiddenapi lists\n and insert it into the given dex file,\n(c) dexlayout to copy the flags into compact dex,\n(d) dex file verifier to verify the item.\nIt also removes skipping of verification for boot class path\ndex files as those now pass DexFileVerifier, and removes the\nneed for removing the flags for JVMTI.\n\nThe size increase is 450 KB.\n\nTest: phone boots\nTest: m test-art\nChange-Id: Idec301db540cf164fccc97136d1df4abb8f758bd\n" }, { "commit": "afaf7f8198fe5ffc054278da6800f81dd83f272c", "tree": "3ebdec9fca813351d178709743121fa66815a7a0", "parents": [ "6ca8ec7809f87ccac8d9d66d267f2379bdfdfe66" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 16 11:32:38 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Oct 18 08:20:09 2018 -0700" }, "message": "ART: Enable bugprone-unused-return-value\n\nEnable bugprone-unused-return-value as an error. This is on top of\nthe compiler warning for attribute((warn_unused)).\n\nMark the current occurrences (all unique_ptr.release()) with NOLINT\nto signal that we know what we\u0027re doing.\n\nBug: 32619234\nBug: 117926937\nTest: WITH_TIDY\u003d1 mmma art\nChange-Id: I36659722335eef36acfa5845289104257a393874\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": "c74d9cbd4c7b1b5b1a4c5a6bb34e9426336bbecc", "tree": "d1030d4bc5fb0986ef9f6e80d81b8a204c011ced", "parents": [ "5ec6f36d4688e70c652882bff8c6a3c060ccc83e" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 20 13:44:44 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 20 20:31:46 2018 -0700" }, "message": "ART: Modernize counting-down loops\n\nPreparation for readability-implicit-bool-conversion.\n\nBug: 32619234\nTest: WITH_TIDY\u003d1 mmma art\nChange-Id: I7a6f9636d5f4537020f99e8cf8560f7f6b6b7ba0\n" }, { "commit": "e5afbf335a8bc110ff7d4f86585a284c98a25a13", "tree": "7596a7b52e5f9a63b409969aba2805aaf012f199", "parents": [ "7dca45b9677c16a54347cdc0d08bfa2bdd94b464" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 12 17:51:54 2018 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 13 14:44:52 2018 -0700" }, "message": "Refactor DecodeDebugLocalInfo\n\nRefactor the function to use a visitor instead of callback.\n\nTest: test-art-ghost\nBug: 112311591\nBug: 77709234\n\nChange-Id: I55385994952373b3fec4f9122400cf3c9d43da3b\n" }, { "commit": "3e2e123239952c80e1b37431bf2efbbe07a41940", "tree": "153be0dafdbce55b5e5432cb10025b802a637402", "parents": [ "68851b53e4152d3edaea977573af8bdc0bd23313" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 11 12:35:30 2018 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 12 11:32:40 2018 -0700" }, "message": "Refactor debug info position visiting\n\nMove logic to CodeItemDebugInfoAccessor, makes it easier to modify\nthe debug info format.\n\nTest: test-art-host\nBug: 112311591\nBug: 77709234\nChange-Id: Ice56a88951efa54dc07d88f2c228c01be4670e73\n" }, { "commit": "4146840f4a88d7c4f44300758c942fd4d95db17a", "tree": "0fb437912c47ab5b5b9e9f9f099102129ba6a409", "parents": [ "ecbb5701276dd5965d68fe825da31637ca62d8c1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 29 11:39:00 2018 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 29 14:48:55 2018 -0700" }, "message": "Change dex naming for output directory\n\nWrite to Dex file names to match storage inside APK for -w option.\ndir/classes.dex\ndir/classes2.dex\n...\ndir/classesn.dex\n\nThis makes it easier to overwrite the Dex files in the APK.\n\nBug: 113059564\nTest: test-art-host\n\nChange-Id: Iadf1621402315887165813da4abf00d466f0f623\n" }, { "commit": "fe42d218cf8a3dcaa832651ea62e49ce7384c867", "tree": "cca4e8ed50410e965363ff7e776de7eda4ebea1a", "parents": [ "d3678dc2531f95ced2d015b800ecd9018ce96c73" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Aug 24 14:01:14 2018 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Aug 24 15:50:47 2018 +0100" }, "message": "dexdump2: Remove conversion from dollar sign to dot\n\nThis was presumably added to render nested classes similar to how they\nare referred to in source, but since a dollar sign (\u0027$\u0027) is a valid\ncharacter in a class name and its use is growing thanks to annotation\nprocessors and D8/R8 it only resulted in very wrong names. If you are\nat the level where you can understand dexdump\u0027s output, you can handle\ndollar signs being present in nested class names.\n\nBefore:\n\n [000234] -..Lambda.Lambda.fcyZxanqBZSHC_nf-noKh-e3bnY.\u003cclinit\u003e:()V\n 0000: new-instance v0,\n L-$$Lambda$Lambda$fcyZxanqBZSHC_nf-noKh-e3bnY; // type@0000\n 0002: invoke-direct {v0},\n L-$$Lambda$Lambda$fcyZxanqBZSHC_nf-noKh-e3bnY;.\u003cinit\u003e:()V // method@0001\n 0005: sput-object v0, L-$$Lambda$Lambda$fcyZxanqBZSHC_nf-noKh-e3bnY;.INSTANCE:L-$$Lambda$Lambda$fcyZxanqBZSHC_nf-noKh-e3bnY; // field@0000\n 0007: return-void\n\nAfter:\n\n [000234] -$$Lambda$Lambda$fcyZxanqBZSHC_nf-noKh-e3bnY.\u003cclinit\u003e:()V\n 0000: new-instance v0, L-$$Lambda$Lambda$fcyZxanqBZSHC_nf-noKh-e3bnY; // type@0000\n 0002: invoke-direct {v0}, L-$$Lambda$Lambda$fcyZxanqBZSHC_nf-noKh-e3bnY;.\u003cinit\u003e:()V // method@0001\n 0005: sput-object v0, L-$$Lambda$Lambda$fcyZxanqBZSHC_nf-noKh-e3bnY;.INSTANCE:L-$$Lambda$Lambda$fcyZxanqBZSHC_nf-noKh-e3bnY; // field@0000\n 0007: return-void\n\nBug: 113152880\nTest: art/test/dexdump/run-all-tests\nTest: dalvik/dx/tests/run-all-tests\nChange-Id: I22a1d3db5b7e0fe6b6c77b5cf8f37e7254bd40f4\n" }, { "commit": "2b5a38f8fbdecf9c82f06ec108c7ee7463bd8093", "tree": "51e3648106f2f74196f9476209a7a649e7885dcb", "parents": [ "5f7c6a1fc35e04b75e1154620c5fd0ab8758e62c" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Jun 14 15:13:04 2018 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Jun 15 10:55:56 2018 -0700" }, "message": "Refactor ownership in dex_ir\n\nRemove some unnecessary generality, consolidating and simplifying the\ninterface to Header and moving all the construction code out of dex_ir.\nThis makes item ownership exclusively controlled by the collections\nthemselves, preparing for in-place construction.\n\nBug: 33017139\nTest: make -j 40 test-art-host-gtest\nChange-Id: Ice461ae89ef9f8bed3350780e8dd6283bc6eca1b\n" }, { "commit": "d83437cba0eaac9797def3c09f7812e25fdcd6ac", "tree": "e29156febe965efb57099b232d9b85342db39a41", "parents": [ "a316f9129a698ccb995b035f8781f93c58981893" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Jun 11 14:06:23 2018 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Jun 13 11:08:05 2018 -0700" }, "message": "Use factory pattern for dex_ir objects\n\nIn preparation for memory improvements, make dex_ir API more friendly to\npre-allocated or in-place type allocation.\n\nBug: 33017139\nTest: make -j 50 test-art-host\nChange-Id: I09b67e279a04535b175433287ce98ecbdc37f7a3\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": "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": "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": "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": "05f90d133d288b524eaa9deacdc56e7d29ea9f99", "tree": "89153d3bccc0aba04423f5712fee09e768d5aa2c", "parents": [ "0efe3243d4f1878e8220ff8bc469beee9ac90518" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 07 13:47:17 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 07 14:56:50 2018 -0800" }, "message": "Avoid generating cdex for dex files with duplicate methods\n\nFor the cases where two different method have the same code item or\ndebug info, avoid generating cdex and write the input dex into the\noat file isntead.\n\nTest: test-art-host-gtest\nBug: 72997076\nBug: 72100771\nBug: 72100668\nBug: 63756964\n\nChange-Id: I5dfe2b47b2c908921c8421367536c2d950186ac6\n" }, { "commit": "26f927518fda8e334e2af2003056b44a9265d5a0", "tree": "c4cb024261ef4d6da7c12c88cfe8e3ab8a16ef38", "parents": [ "eaf09d10411c8b56f2b4b0c882eb63ef7d01ec0c", "221d9810aa8af1ceab4626620a81baf0fd8377d7" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Sat Jan 27 03:31:17 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Sat Jan 27 03:31:17 2018 +0000" }, "message": "Merge \"ART: Use libbase for logging in command-line tools\"" }, { "commit": "7517555480074dcbd5c6d9472d5a394ce0e88023", "tree": "8b9f74989f3e94d810ed99590e8c207a5dc551c4", "parents": [ "bf9eedaac5a9723f022150d0f5fd58cb91779497" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 25 11:23:01 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jan 26 14:33:40 2018 -0800" }, "message": "Add dexlayout class filtering\n\nFor fixed_up_dexfile, the returned dex only needs to contain a single\nclass. This CL adds filtering during IR creation to only contain that\nclass. This speeds up wifi tests from 25s to 15s on marlin.\n\nBug: 63756964\nBug: 72064989\nTest: mm test-art-host-gtest-dexlayout_test -j64\nChange-Id: I3648ac886b377d95e33e5b40a63a34c2a9dbde58\n" }, { "commit": "c3a22aa19bbe35ff8447460b29e07d42937a39de", "tree": "9f795deb249eebcf573dfe07a98d70c1d52ce898", "parents": [ "d3233abdf14f173bb99e3905e8543ffff845230e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jan 19 18:58:34 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jan 24 19:53:20 2018 -0800" }, "message": "Add shared separate data section for compact dex\n\nAdded a shared dex data buffer for compact dex files, this buffer\nis referenced by all compact dex files in a vdex file. Repurposed\nthe existing data_off / data_size fields in the header.\n\nAfter the shared buffer is filled up, it is placed after the dex\nfiles in the oat writer and the dex file headers are fixed up to have\nthe correct offsets / sizes to the shared buffer.\n\nMotivation:\nMake it easy to deduplicate data across dexes.\n\nBug: 63756964\nTest: test-art-host\nChange-Id: I17855a0c78b20be3d323d12dedb9c695962be3ed\n" }, { "commit": "e6b6ff89e3cac4e443e731fbcdd957e32c7bc189", "tree": "64a48abd2a97b1ece83b014fa02cd5c40b580eb4", "parents": [ "49971eb50c0213b7506bdd29954a6a4423e6426a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jan 19 18:58:34 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 23 12:08:13 2018 -0800" }, "message": "Revamp dexlayout writing functions\n\nAdded some stream abstractions to make it easier to split apart\nthe main part of the dex file compared to the data one.\n\nMoved to using a vector based stream instead of a mem_map one.\n\nMotivations:\nMake it easy to separate data into a separate stream for multidex data\ndeduping.\nRemove upper bounds on dex expansion that would SIGSEGV if the writer\nwent over the limit.\n\nBug: 72051652\nBug: 63756964\nTest: test-art-host\n\nChange-Id: Ic622a4142b161566d149166a1767434ff4cc7fec\n" }, { "commit": "221d9810aa8af1ceab4626620a81baf0fd8377d7", "tree": "ae8e10655b727bb7a79607d9778dab6aee29c2ab", "parents": [ "8105dad6cb540f6521d25507ae3e70718bc7a264" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jan 22 17:48:56 2018 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jan 22 17:48:56 2018 -0800" }, "message": "ART: Use libbase for logging in command-line tools\n\nUse LOG for logging, and StderrLogger to redirect all logging to\nthe terminal.\n\nThis only applies to tools used only on the command-line.\n\nTest: m test-art-host\nChange-Id: Ia3a6363a06c6a849eb1068213962d686c4495e29\n" }, { "commit": "8740c66cbfbeb3b7f306279c843650c3b18f2dca", "tree": "f6c095c184fdf3e5fb2f243acdc818ce20e928a5", "parents": [ "dcbe96b23b8be48d4cce799caf5a9a207b137f06" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 11 14:50:02 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 16 11:24:02 2018 -0800" }, "message": "Use efficient encoding for compact dex code item fields\n\nUse a compact format for compact dex code items. In the common case,\nthere are only 4 bytes worth of fields.\n\nThis is accomplished by shrining the fields and adding an optional\npreheader before each code item that describes fields that don\u0027t fit\nin the fast path case. The preheader may use up to 12 extra bytes.\nExperiments showed that the preheader is only non empty for ~1% of\ncode items.\n\nCompactDex code item:\n0-12 bytes: Optional preheader\n\u003cCode item pointer\u003e\n4 bits: register size\n4 bits: ins size\n4 bits: outs size\n4 bits: tries size\n11 bits: num code item units\n5 bits: preheader flags\n[Dex Instructions]\n\nStandard dex code item:\nSee: https://source.android.com/devices/tech/dalvik/dex-format\n\u003cCode item pointer\u003e\n16 bits: register size\n16 bits: ins size\n16 bits: outs size\n16 bits: tries size\n32 bits: num code item units\n[Dex Instructions]\n\nAlso change code item alignment to be 2 byte aligned for code items\nthat don\u0027t have payload instructions that require 4 byte alignments.\n\nResults (on golem).\nThe CL reduces vdex size by an average of 4.3%. Interpreter performance\nis 0.3-1% slower, but may be noise.\n\nTest: test-art-host\n\nBug: 63756964\n\nChange-Id: I836fd098063dd1573a3dfb6ac465a6d33d2cb2c9\n" }, { "commit": "1557778b03b02b589283100ee8de821692b88813", "tree": "701ee8ebc00d919693cd29b913315076623195a9", "parents": [ "6d4c343ee5db18f039aeb3e07ff8d3c1fd37c3a0" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 16 09:43:24 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 16 09:49:47 2018 -0800" }, "message": "Add more extra space for dexlayout writing\n\nTemporary fix to prevent aborts that can happen when converting very\nsmall dex files to compact dex. Fixes a broken build.\n\nTest: make\nBug: 63756964\n\nChange-Id: I8fce5ebd6d6659d740b100890c825a1899575c31\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": "21cf2581844c478db3627d3e0259c205bce76ae7", "tree": "f74e21fe818dfe4fe40db8b774751b1a8aeaf628", "parents": [ "680e88ba30d1c599c5eaab4a207db3e39bf2d57f" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jan 08 17:09:48 2018 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 09 09:26:09 2018 -0800" }, "message": "Convert to StandardDex in fixup_dex_dex_file\n\nIn fixup_dex_file, run dexlayout to convert compact dex back to\nstandard dex since the output is supposed to be standard dex.\n\nFixed a bug in dexlayout where conversion from compact dex -\u003e\nstandard dex was not supported becuase the dex header was always\npreserved.\n\nTest: test/testrunner/testrunner.py --host ---redefine-stress --compact-dex-level fast\nBug: 63756964\n\nChange-Id: Id13e8593458213e7040d053c35e8914404ae5381\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": "2c4b084bf93ddfea6b60d05ff82c44dab9de9f28", "tree": "e1dc28ba175bfa67def5db8b22847b15f87bdb93", "parents": [ "ba3a790338725a37ecd4cb314c4a6147e29aef38" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Dec 13 11:49:51 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 15 08:20:08 2017 -0800" }, "message": "Fix cdex bugs to enable ART_DEFAULT_COMPACT_DEX_LEVEL\u003dfast tests passing\n\nCompute dex checksum for compact dex generation. Handle input vdex\nby not aborting in oat_writer, instead just avoid generating compact\ndex for the input vdex case.\n\nRe-enabled some compact dex tests.\n\nBug: 63756964\nTest: ART_DEFAULT_COMPACT_DEX_LEVEL\u003dfast test-art-host\nChange-Id: Ic9b4e4e59e6cd22b66ee2fc0d32c9b4a15f13497\n" }, { "commit": "d13126d1ba8f65e483cd2e9dd18e60c9a0992c6e", "tree": "4f1189a912bb56bb1adb01219cf8e27b5442ef83", "parents": [ "7f3b60825f6cfe20359b389708b9e98e6deb9924", "219cb9021fa74af7773066ffb8fc77ac85f3d0de" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Dec 11 10:20:50 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Dec 11 10:20:50 2017 +0000" }, "message": "Merge \"Reduce memory allocations in dexlayout\u0027s CodeFixups.\"" }, { "commit": "219cb9021fa74af7773066ffb8fc77ac85f3d0de", "tree": "2ecee11a4ad831c0f9f407e9a1240c5534b888e1", "parents": [ "4388fb213ec746ee18a6bea38ee894f8c19990b9" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Dec 07 16:20:39 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Dec 08 11:08:54 2017 +0000" }, "message": "Reduce memory allocations in dexlayout\u0027s CodeFixups.\n\nUsing heaptrack for the services.odex compilation of\naosp_taimen-userdebug:\n - before: calls to allocation functions: 5032880\n - after: calls to allocation functions: 4645378\n\nTest: Rely on TreeHugger\nBug: 70331349\nChange-Id: I477d5a84e0bf22994ce3de9fe56525de0b498801\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": "3e0c5170c87b39c88c7f3b5fbd7b07e60547bfd7", "tree": "cbb52ff88b9a643d0a08a2c890a15f3442db13d5", "parents": [ "cb90e3a29041ea55c4b5eb3d1477aa360381a1f1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Nov 12 12:58:40 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 20 13:34:53 2017 -0800" }, "message": "Add logic for calculating offsets during writing\n\nDelete all the offset related layout logic, move the logic to\nwriting. Layout is now controlled by changing the order of the data\nvectors.\n\nCalculate all offsets during writing instead of reading. This allows\nmore flexible layout optimizations without needing to be as careful\nabout offsets in other sections changing.\n\nMaintained logic for using existing offsets in the case where\nchanging the layout is not required.\n\nAreas to improve:\nHow to size the memmap, do we want 2 passes? Currently the algorithm\nreserves 10% extra space in case required. This is sufficient for top\n1000 apps in the play store. Will consider a two pass approach later.\n\nBug: 63756964\nBug: 68948395\nBug: 68867570\nBug: 68864106\nBug: 68864168\n\nTest: test/testrunner/testrunner.py --host -j40\nTest: test-art-host-gtest\nTest: Dexlayout speed profile top 1000 apps\n\nChange-Id: I7dee869da3a010559547f8cfdf93e9aa4c3f47ff\n" }, { "commit": "2b2bef245d5b2c6faa2d6f36da14866b2d8f5e4f", "tree": "7948ee1d32e211198a595bb7ed35558d22838da8", "parents": [ "f199f1d3a9816edf9282766a8874d894d7bf87c3" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 26 17:10:19 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Nov 01 15:36:21 2017 -0700" }, "message": "Refactor DexInstructionIterator\n\nAdd a way to get the dex PC for the \"for each\" use case.\n\nBug: 67104794\nTest: test-art-host\nChange-Id: I144c459c9a2a03ec8d56842280338d1f7ce1caf0\n" }, { "commit": "2ef3b88203961af0022540ea5893bfe91cab14b7", "tree": "da815b15e6cc68ba0b3a79f9b5cceffe8ee891a4", "parents": [ "0aff0fffa828a1a1920a3db10599642b45b2c68e" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 20 19:50:39 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 26 11:03:41 2017 -0700" }, "message": "Re-generate dex IR to verify dex file\n\nInstead of using the existing IR, re-generate it to catch possible\nbugs writing the output dex.\n\nTest: test-art-host\nBug: 63756964\nChange-Id: Ib63f3137f5807ee5de1b05f8278137a3b3a3aee0\n" }, { "commit": "24066ec509949f777a4c6e61ef631de33bfdc28a", "tree": "5620a9f0ef3f476ada8b1763a1854877d136f022", "parents": [ "31275df5ea1b91ed668618d6ec64a76d67cee73c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sat Oct 21 16:01:08 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 26 09:35:02 2017 -0700" }, "message": "Re-enable dex verifier after dexlayout\n\nDisabled class def ordering since it violates the spec regarding\nsuperclasses occurring before subclasses. This fixes a dex verifier\nfailure.\n\nAdjust the data section size based on string data and code item\ndiff. This fixes dex2oat_image_test failing dex file verification.\n\nFixed handling of unreference annotations. Previously, these would\nnot get written back out. This resulted the dex file verifier\nprematurely ending for annotations during CheckIntraSectionIterate\nand then complaining about non zero padding (that was actually an\nannotation).\n\nTest: test-art-host\nBug: 63756964\nBug: 68208404\n\nChange-Id: Id2b9b360640c360ac562826e9193971f7bb30eea\n" }, { "commit": "6291ac84088ba32888d6b7d714f9965eeb82d345", "tree": "af7a80adcced7432125398be36657a5e8e651ee5", "parents": [ "358347ae8fcf662def560367fe35a9ed9ee3b187", "b4156286f889c7dea1fc3e3db85ecfcc53880ac3" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Oct 24 13:25:55 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Oct 24 13:25:55 2017 +0000" }, "message": "Merge \"Handle de-duped CodeItems in dexlayout\"" }, { "commit": "b4156286f889c7dea1fc3e3db85ecfcc53880ac3", "tree": "f05aac1dd50512abe6afd265b7d8425d3ca15320", "parents": [ "716f34dd5517d348fe7ce6626d1250c8853083c6" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Oct 19 17:57:02 2017 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Oct 23 20:52:56 2017 -0700" }, "message": "Handle de-duped CodeItems in dexlayout\n\nDe-duplicated (shared) CodeItems were added to the layout structures\nmultiple times, resulting in multiple update of offsets and\nuninitialized data in the resulting dex files.\n\nBug: 67024225\nTest: make test-art-host\nChange-Id: I76ffd2cb52fe72eab3fb0dbf6ad8607951cee595\n" }, { "commit": "9e842d3e7d6102d964178e36e5d596ca91895147", "tree": "0903b495d4742e17bd6f7d0b89dbf5bd67e8a93d", "parents": [ "e4df8d98afd512e8ed0b7664b31856ab49edb13a", "603ccabb82511824a2a9d9531397005009a0e28a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 23 16:27:01 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Oct 23 16:27:01 2017 +0000" }, "message": "Merge \"Add basic support for writing cdex container in dexlayout\"" }, { "commit": "dfeec3a983c81e62fbfeb4eb81d4f58b65effd80", "tree": "fe80df785c35671790d13728d1f6d114278c43c2", "parents": [ "59c5dfe792d3288c5df75a035f6614cb228d7352", "095c6c96236476b605b3ac672f6d2b8c151e9479" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Oct 23 08:29:27 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Oct 23 08:29:27 2017 +0000" }, "message": "Merge \"Pass the verify flag to DexFileLoader utilities.\"" }, { "commit": "5929beb4994f55da3afdb58298225094f959976e", "tree": "c350c564fa39436c68f887cd741c75316af8bbea", "parents": [ "6479e488e627c0e470e2068422316f6966d254f5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 20 23:24:10 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 20 23:34:00 2017 -0700" }, "message": "Make libdexlayoutd actually have art debug flags\n\nPreviously it didn\u0027t, this caused DCHECKs to not happen. Disabled\ndex verifier for dexlayout since it fails for dex2oat_image_test\nnow that DCHECKs are actually run.\n\nThe failure is:\nCheck failed: output_dex_file !\u003d nullptr Failed to re-open output\nfile:Failure to verify dex file \u0027memory mapped file for\n.../core-oj-hostdex.jar\u0027: Out-of-bounds end of data subsection:\n4c56d1\n\nTest: test-art-host\nBug: 63756964\nChange-Id: I05a6617ffdf24fd45024a13576c2a5ff1fcd84a8\n" }, { "commit": "603ccabb82511824a2a9d9531397005009a0e28a", "tree": "2c83d0e217b87f9ddb1d2e16085abb437a6d389d", "parents": [ "6479e488e627c0e470e2068422316f6966d254f5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 20 14:34:28 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Oct 20 18:03:48 2017 -0700" }, "message": "Add basic support for writing cdex container in dexlayout\n\nPass compact dex level option to dexlayout and generate a file with\na compact dex header if it is enabled.\n\nAdded test in dex2oat_test.\n\nAdded logic to check for CompactDexFile header in\nDexFileLoader::OpenCommon.\n\nBug: 63756964\nTest: test-art-host-gtest-dex2oat_test\n\nChange-Id: Icfec03cdb64f71de900b92754ab4be3789c71cc3\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": "aa6abb081cbe14ccd693459bf016b14627f58616", "tree": "3f7c8d392ef39c12e5ab902de3922115f312695e", "parents": [ "776f0b98414fe0d996928e7dce982c93195fc113" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Oct 12 08:25:11 2017 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Oct 16 16:57:40 2017 +0000" }, "message": "Fix dexlayout debug info printing\n\nParsing local info and position info made shared debug info print\nincorrectly. Remove the dexlayout structs that represent the pre-parsed\nversions and print from the byte buffer.\n\nBug: 67664147\nTest: make test-art-host\nChange-Id: Iae33ae8ff486914d8d7a5973f81faaf3c95615a6\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": "1d2d4ff8570bb88d9d2d4633706fd7f6fb18d75e", "tree": "cbe67e8e9214828656314117121e8ce906a762ab", "parents": [ "e5b35ed787fbfb388d162361310bae5b0e7682a7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sat Sep 23 16:11:06 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 27 11:51:53 2017 -0700" }, "message": "Add DexInstructionIterator and use it a few places\n\nMotivation:\nWant to start abstracting away dex specific functionality to enable\nCompactDex. Adding an iterator will enable CompactDex iteration to\nwork differently than normal dex iteration.\n\nWill eventually replace CodeItemIterator.\n\nBug: 63756964\nTest: test-art-host\n\nChange-Id: I90e67c1a994b7698aaac0523a82816b0a003fbdc\n" }, { "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": "120aa286ab6adf3e76a31bc61fb4e583e5158d71", "tree": "07f1546ce7a090af8c88110598920769bfb5e669", "parents": [ "5bfead584f56b2a1cfb69f78c385965ec57f7e8b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sat Aug 05 16:03:03 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Aug 14 17:47:25 2017 -0700" }, "message": "Store layout info in dex files\n\nStore layout info for code sections inside of the oat file. This will\nbe used to advise the kernel when dex files are loaded in\na follow up CL.\n\nAdded unit test in dex2oat_test.\n\nBug: 63178181\nTest: test-art-host\n\n(cherry-picked from commit 75c5ed6e75f70002db5fa7c609137c04dd2bdf40)\n\nChange-Id: I4777506886bde42ff0affdac412a8395e8013a40\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": "acc83d77cc99e6280391604ebd3e2e51d9a60b99", "tree": "1cc7bd5b7d33d9eb0bba0f4d7b0dd48c95b6e02e", "parents": [ "6d2e76b421181ebb3a419b51bfc2d3243572a677" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Jul 06 17:51:01 2017 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Jul 06 17:51:01 2017 -0700" }, "message": "Improve string layout in dexlayout.\n\nAdds more strings to what dexlayout considers hot:\n - A profiled class\u0027s superclass and interface class names\n - The class names of accessed fields\n - The class name, name, and shorty of methods accessed by clinits\n\nTODO: Gather memory numbers\n\nBug: 35800981\nTest: mm test-art-host\nChange-Id: I1a7de5bd5b1b73559624af939de05a2111f78195\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": "7c1be8ba84b81ad2edfb6461bb4e2f3073bbc553", "tree": "3da1fa92b59386134d0dae307ba21a21ee2f530d", "parents": [ "2bc159ebd1115f71d80639e47fc38663aa84ba4a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 15 13:56:05 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jun 16 14:02:04 2017 -0700" }, "message": "Layout code items of post startup methods\n\nLayout code items in the following manner:\n\u003chot\u003e\n\u003cstartup only\u003e\n\u003cother clinit\u003e\n\u003cexecuted\u003e\n\u003cnot executed\u003e\n\nFor basic use cases (write email, start navigation)\nMaps vdex PSS: 17464k -\u003e 16856k\nGmail vdex PSS: 4004k -\u003e 3820k\nNumbers are from marlin\n\nTest: test-art-host\nBug: 36457259\n\nChange-Id: Id7ae676b45d5ba4f5b4dabfdc954875756fd2bcb\n" }, { "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": "206cbaabf42430347f8d83e7b9e8fd9f0d29d010", "tree": "169a4a136973067b2901290d70b66aa7516e2942", "parents": [ "4d3df9131c4098828f889b9470c82880efdc91be" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Jun 07 19:11:01 2017 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Jun 07 19:11:01 2017 -0700" }, "message": "Layout clinit code items of profiled methods together.\n\nMaps startup dex file pss drops from 9.88MB -\u003e 9.12MB.\n\nBug: 35800981\nTest: mm test-art-host\nChange-Id: I5c0167c553cd21c23bed05e824f14ab19f505d59\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": "a07394a0f356513a08266e2e198a0f4064f6e694", "tree": "1bbbd89b3772c852e5cbdd49d19b3fa3966cf910", "parents": [ "aad63d06328e745a7f13e812bf41676274aa8f49" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed May 17 10:29:56 2017 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed May 17 10:29:56 2017 -0700" }, "message": "Enable dex diff check on debug builds.\n\nDex diff checks that the input and output dex files are equivalent at\nthe IR level. Normally enabled by passing -v to dexlayout, dex diff is\nnow enabled for debug builds as well.\n\nBug: 36107940\nTest: mm test-art-host\nChange-Id: I8c5ebb6a7cc0f699299a7200e8bc019762601feb\n" }, { "commit": "3a2380da2496423daff1eedee2a1b0882dbd3154", "tree": "3fbf84a3f16a35ed531f190d1a54b7ffcebf90aa", "parents": [ "0d2ea2f09937e55600f8a1eda6b102fef1ab9c0b", "36a282b8714071d9f16c98c316efa4304fd8d36a" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Tue Apr 25 18:42:26 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Apr 25 18:42:28 2017 +0000" }, "message": "Merge \"Layout codeitems based on profile data\"" }, { "commit": "36a282b8714071d9f16c98c316efa4304fd8d36a", "tree": "85c41bcb9d70778be8afe273eb57ef469f7b11d4", "parents": [ "4e34bd190287f10214e8ea11a61025f18d2429b8" ], "author": { "name": "Shubham Ajmera", "email": "shubhamajmera@google.com", "time": "Mon Apr 03 10:04:28 2017 -0700" }, "committer": { "name": "Shubham Ajmera", "email": "shubhamajmera@google.com", "time": "Mon Apr 24 14:54:23 2017 -0700" }, "message": "Layout codeitems based on profile data\n\nTest: ./art/test/testrunner/testrunner.py --host -t 638-c\n\nChange-Id: I15cef5d8bd25cbb74f844bd4c8241700b0f7dc13\n" }, { "commit": "0dfc315a096bf5678cb5182771e3fdcca8ea4433", "tree": "ac04fef45e7d002734d3d79b9ab9a128cb5dc263", "parents": [ "291ce17ada5a126be99f0fc069a028d2100bcf9e" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 24 07:58:06 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 24 14:18:45 2017 -0700" }, "message": "ART: More header cleanup - mem_map.h\n\nMove MemMap maps_ into cc file and move allocator include. Remove\nother unused includes. Force clients to import sys/mman.h if they\nneed it. Try to lower dependencies on ART by using libbase helpers\nmore.\n\nTest: mmma art\nChange-Id: I8f31e5f371789b3d72e12af377181fee2ac9a611\n" }, { "commit": "7639cdcf4dbbbb4ef63cc8c3ae2f2b4e170a5c83", "tree": "4b9e74bdf3fd4ef9ab7d29bca1e6234ac02c53f1", "parents": [ "39d562895ab3ff835003fe3df6c13a93db492aa8" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Sat Apr 15 10:06:21 2017 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Sat Apr 15 10:06:21 2017 -0700" }, "message": "Add art_defaults to dexlayout\n\nThe build rule defaults for dexlayout targets didn\u0027t include defaults\nfrom the ART project. Because of this ART_TARGET_* weren\u0027t defined,\namong other problems\n\nBug: 35800981\nTest: make -j 40 test-art-host\nChange-Id: Ic528bfc6b622fa9b4f5c0ddf883f15912641b720\n" }, { "commit": "3ba51e854d6ee3287641d37ca5e108dd78de697b", "tree": "3f0939d171df1fd70e14118aa4494254c1b25f58", "parents": [ "ce121a5fe3e537872af077c801443be9a525859b" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Apr 12 16:14:54 2017 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Apr 13 10:16:18 2017 -0700" }, "message": "Add check that dexlayout can open output file.\n\nTest: mm test-art-host\nChange-Id: I49b37b3e3696c8c002e22533c2e86e47a0e13e51\n" }, { "commit": "fa0aa0953875427d7381ac44a6427bd5b1d5020a", "tree": "ffffbc915c05b8a5d0a830618364261dedfa3948", "parents": [ "8b71f086ade1f48f3780cc81706669abbfe9c184" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Mar 27 15:43:54 2017 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Mar 29 10:42:47 2017 -0700" }, "message": "Layout string data\n\nInspect dex code in class initializers and hot methods to find out\nwhat const strings are likely to be resolved. Along with these,\nlook at static field accesses and group the field name and type with\nother hot strings.\n\nGroup method shorties together.\n\nMaps vdex PSS: 7478k -\u003e 6900k\nNo change in odex PSS.\n\nChanged dexlayout_test to use -v instead of comparing to a hex\noutput.\n\nBug: 36457259\nBug: 35800981\n\nTest: test-art-host\n\nChange-Id: Iaa5de649c5b2b1d9f178525d2f42fe00e3fb879f\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": "cc8295946caf3b965bff778fd7ae3adbcba47de3", "tree": "d7c9b77d14217d049911ded65f346271cf5c7849", "parents": [ "fcfaa8d823017eb209fd6e18a6fe496f4c979026" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Mar 14 16:13:39 2017 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Mar 15 19:25:02 2017 -0700" }, "message": "Finish adding checks to dex ir verification.\n\nAdds missing checks for class defs, annotations, class data, code, etc.\n\nBug: 36107940\nTest: mm test-art-host\nChange-Id: Ifaac07299cf237827160c29b4230b0dc0845dc41\n" }, { "commit": "ec7f1a96683836d9375adc48c6bf802c0ce2160e", "tree": "f07435d668c04c239dca0036b6b072611d45b046", "parents": [ "13ef0447b43ecbf18d3c4e4c72d66bbbc1815f0c" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Mar 13 16:24:24 2017 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Mar 13 16:59:46 2017 -0700" }, "message": "Added new -v option to dexlayout to verify output dex file.\n\nPassing -v will cause dexlayout to do an IR level comparison of the\noutput with the original input dex file. This checks that the data in\nthe dex files are the same, but allows for different offsets since the\noutput dex file may have a different layout.\n\nTest: mm test-art-host\nBug: 36107940\nChange-Id: If75a93973ffdd2d91111727f089713c800d8cee8\n" }, { "commit": "8f48f4cb1760cd5d1b193e87ba6b56043bb08f35", "tree": "467c2da76191237292be4b0cbe3f36d9315c3a0a", "parents": [ "5442ff6bad80f910f84ddf265393796d5fa9d977", "4a436ac4b4daa4e322d2a7cc2395462fbb16cd54" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Sat Mar 11 02:29:10 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Sat Mar 11 02:29:10 2017 +0000" }, "message": "Merge \"Fix dex file verification check for dexlayout output file.\"" }, { "commit": "4a436ac4b4daa4e322d2a7cc2395462fbb16cd54", "tree": "243bc4a490fc31203f7fcb0e8747ac3eedc57477", "parents": [ "1352e02ceeb30a3191809df4a7ee9fd711962f30" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 10 17:05:01 2017 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 10 17:05:01 2017 -0800" }, "message": "Fix dex file verification check for dexlayout output file.\n\nFor debug builds, we want to run the output dex file through the dex\nfile verifier as a sanity check. The initial check was actually checking\nthe input dex file again...\n\nBug: 36107940\nTest: mm test-art-host\nChange-Id: Ibd434d67f661c325a443d4ca9cced481e7a9b02c\n" }, { "commit": "bd40af72aac39c53fadca6fe2e8d90f2ba2c4502", "tree": "37aa77fec55f4fa97190efbe49278dde27a13cf5", "parents": [ "1352e02ceeb30a3191809df4a7ee9fd711962f30", "9335749d25f8826c72ea7ea4d470ee8bc171fa87" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Mar 10 18:08:50 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Mar 10 18:08:51 2017 +0000" }, "message": "Merge \"Add section size statistics\"" }, { "commit": "9335749d25f8826c72ea7ea4d470ee8bc171fa87", "tree": "9f6fffab6b8c44d1d38434e66a8f345cb5eeceeb", "parents": [ "74f94dd8edb4a6070f6f7bfef6d28a8b4a21fc30" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Mar 09 08:02:44 2017 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Mar 10 08:02:08 2017 -0800" }, "message": "Add section size statistics\n\nAdd an option to display the size of dex file sections for easier\nanalysis.\n\nBug: none\nTest: test-art-host\n\nChange-Id: I779f776657ad79e7a50c18c2b9486e6778c70f13\n" }, { "commit": "b756815e581e2bcc3db7ee0cb3ae6c031fb22ae8", "tree": "c49999dc1d8683d14d628f1e595f86900368bc93", "parents": [ "74f94dd8edb4a6070f6f7bfef6d28a8b4a21fc30" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Mar 09 18:14:48 2017 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Mar 09 18:14:48 2017 -0800" }, "message": "Add dex file verification check for dexlayout output file.\n\nFor debug builds, run the output dex file of dexlayout through the dex\nfile verifier as a sanity check.\n\nBug: 36107940\nTest: mm test-art-host\nChange-Id: I2b556db32cfd360cc4fe8d1d080508deebd27aa0\n" }, { "commit": "863f1d79a70403443c089a91ca893db6cbadd3d8", "tree": "383e99645839dc26735369273d665cd59ee7bbb6", "parents": [ "fddc19338d9fdee24c4e10b758db1a6997004e2e" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Mar 01 12:18:19 2017 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Mar 01 12:31:59 2017 -0800" }, "message": "Prevent dexlayout from moving code items if preceded by class data.\n\nClass data has uleb encoded offsets to the code item, and moving code\nitems will change the size of the class data, which would in turn affect\nthe code item offsets if they precede the code items.\n\nFor now, make it so that dexlayout does not move the code items at all\nin this case. A better fix would be to swap the order of the sections in\ndexlayout, but that can be done in a future CL.\n\nUnit test to follow.\n\nBug: 35855748\nTest: mm test-art-host\nChange-Id: Ica6da1e5e951cf0003fc9793f13ad10b74004eb9\n" }, { "commit": "e17f589d8fa46d0d0179499354b7967801f9582c", "tree": "ab0d1d2c46c8016c0f13a7469333d895ad3fab0d", "parents": [ "06f658ab309a5f1930b8a6bc306ef79d0ef0616e" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Feb 23 16:14:04 2017 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Feb 24 10:47:43 2017 -0800" }, "message": "Dexlayout fix for duplicate class data and preserving last code item.\n\nThis change fixes a dexlayout issue when multiple class defs reference\nthe same class data. A set is used to tell what class data has already\nbeen visited.\n\nThis change also makes dexlayout preserve the offset of the last code\nitem in the dex file if the following section is not 4-byte aligned.\nDue to limitations in dexlayout, it is difficult to adjust the offsets\nof sections if more space is needed. An overhaul of dexlayout should\nbe done to properly fix this.\n\nBug: 35451910\nTest: mm test-art-host-gtest-dexlayout_test\nChange-Id: I8d066c2151a1a57c382ce35f12bf53a519da89f6\n" }, { "commit": "33083d626dbf2c8b06badfd73f50e98114483059", "tree": "f6f5db21855a6182b591b57e6a34e5c1fc41b7d0", "parents": [ "fd5f52b60cfa217fb0c4d7dcabdd0336b55990d7" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Jan 18 15:29:12 2017 -0800" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Jan 18 19:00:21 2017 -0800" }, "message": "Rename offline_profiling_info to profile_compilation_info\n\nThis solves a long standing TODO to align the names of the files and the\nclass.\n\nBug: 32434870\nTest: test-art-host\n\nChange-Id: I2707da8fef8736a7223189d894fc00ff11bfab12\n" }, { "commit": "46ee31b67d7ee1bd085fbc240502053caa3cf8fa", "tree": "83e870f293cd60d533207e389d7b094acd87b31f", "parents": [ "9538f9e2a5d03f1b1bc07ebfbd93b61dcf8ad604" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Dec 14 10:11:49 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 15 09:50:39 2016 -0800" }, "message": "ART: Move to libbase StringPrintf\n\nRemove ART\u0027s StringPrintf implementation. Fix up clients. Add\nmissing includes where necessary.\n\nTest: m test-art-host\nChange-Id: I564038d5868595ac3bb88d641af1000cea940e5a\n" }, { "commit": "042e898f18767ed326980e3b07d39adbf98f6222", "tree": "e2ea81bc5ddbab51cb3d5c0d5b7812cf9f85562e", "parents": [ "608f2ce4b9870354079b9d63d40363914889f01a" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Oct 19 11:17:11 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Nov 22 13:33:09 2016 -0800" }, "message": "Add layout scheme to dexlayout for dex file reordering.\n\nCurrently takes startup classes from profile and groups them together\nwith the lowest class indexes. Moves the ClassDef, ClassData, and\nCodeItem sections.\n\nResults gathered from 5 second systrace of app startup (no other input):\nMaps\nbase.vdex - 24.70MB -\u003e 19.35MB (30.50MB file size)\nbase.odex - 13.22MB -\u003e 12.82MB (13.22MB file size)\nRunning time of dex2oat: 27.5s -\u003e 37.1s\n\nPhotos\nbase.vdex - 12.98MB -\u003e 9.77MB (15.10MB file size)\nbase.odex - 6.84MB -\u003e 6.24MB ( 6.84MB file size)\nRunning time of dex2oat: 15.6s -\u003e 19.9s\n\nPlus\nbase.vdex - 14.09MB -\u003e 13.29MB (16.72MB file size)\nbase.odex - 2.47MB -\u003e 2.47MB ( 2.47MB file size)\nRunning time of dex2oat: 14.1s -\u003e 18.5s\n\nPerformance and memory tracking bug: b/33017139\n\nTest: mm test-art-host-gtest-dexlayout_test\nBug: 29921113\nChange-Id: Ib500ed353d71a606e2db0dd80378750c5f7b8e33\n" }, { "commit": "ea7c62983beec6a5a2a6676cc910a436b20ae92c", "tree": "cc41732efa47b2301f463f2f9fcabe957ed89c0c", "parents": [ "b46782abf6752ccf688b0d9f5b4bb83e5d33793b" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Nov 14 18:10:16 2016 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Nov 22 09:12:14 2016 -0800" }, "message": "Dexlayout cleanup and refactoring.\n\nCreated option to output to a mem map in preparation of hooking\ndexlayout into dex2oat.\n\nTest: mm test-art-host-gtest-dexlayout_test\nBug: 29921113\nChange-Id: Id42ef15cb8f83cc8d05b025b7647a4338e9b96b0\n" }, { "commit": "ca620d7bc03b23a0bcf0ef58df58603ee000dca0", "tree": "07cb026075b70a958d14ae84b4e213178a6ba0b4", "parents": [ "b02b8d7df48ea3314cfcb3c08d84ac9556363833" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 08 08:09:33 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 08 15:04:48 2016 -0800" }, "message": "ART: Fix tidy warnings\n\nSwitch to char versions of find variants.\n\nAdd \"explicit\" constructor variants or refactor and\nremove defaults.\n\nUse const references.\n\nBug: 32619234\nTest: m test-art-host\nChange-Id: I970cc2f47d6cf8f0c74104b994b075b2fafb3d45\n" }, { "commit": "caacd11864383aac65e61be837fb1bb5f91e3878", "tree": "0fc2836395d93349aa2e745f31a3cb248e1fdacf", "parents": [ "3667e26de4856cccf24bcbab54ad3349a05267c0" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Oct 20 16:27:02 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Oct 24 08:55:22 2016 -0700" }, "message": "Move dex CFG dumping out of utils.cc\n\nMove CFG dumping to dexdump, the only client.\n\nBug: 22322814\nTest: test-art-host\nChange-Id: I0f39f1d5dfc446419d26d709b78d04e45616f42c\n" }, { "commit": "b34bb193dd2d45af196592bb4bae26f30d529b13", "tree": "e0c38aef953ba388b7e0fb0ef41ec78ab95d4044", "parents": [ "3da7190bae7c1d75edbc87f01343652c246639f2" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Tue Oct 18 17:02:58 2016 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Oct 20 19:04:07 2016 +0100" }, "message": "Dex disassembly suppport for invoke-polymorphic.\n\nBug: 30550796\nTest: art/test/dexdump/run-all-tests\nChange-Id: I013ce2ebbcf9555e01170dc47fc38036c276b1b4\n" }, { "commit": "a86210038524cecd0d96d6ba6f8f116da348a25c", "tree": "3c5c243a9530f3cd64ca12b94ea2e2a4ada34260", "parents": [ "b4f0076e7faca1382e0b2f68557b71b11ac34a9d" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Oct 04 18:13:44 2016 +0000" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Oct 04 14:21:13 2016 -0700" }, "message": "Revert \"Revert \"Add dex file writer to dexlayout tool.\"\"\n\nThis reverts commit fd1a6c2a08ca3e2476b7424b9b0fa58e73b29e87.\n\nFixed output being clobbered during DexLayoutTest.DexFileOutput.\nOption added to put dex output file in scratch directory.\n\nBug: 29921113\nTest: mm test-art-host-gtest-dexlayout_test\n\nChange-Id: I9e6b139cf06aaa39c83ad1e74329db266464a8e4\n" }, { "commit": "cdcfde74ccd72c60dc9456013e32d8b00fefdee6", "tree": "1c29e986405c23285771a31250c5f26857f41046", "parents": [ "82d4838d6bb3480cd25327cedc5179fb2d86881c" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Sep 26 07:44:04 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Oct 04 10:47:24 2016 -0700" }, "message": "Option to gnuplot dex layout\n\nDisplay the portions of the dex file that are accessed by the classes in\nthe dex file. Optionally limit the display to only those classes that\nare present in a runtime profile.\n\nBug: 29921113\nChange-Id: I2a998fba448fec6c23941d3b4358531b1336e1b3\nTest: dexlayout test\n" }, { "commit": "fd1a6c2a08ca3e2476b7424b9b0fa58e73b29e87", "tree": "1f3e846a8cf6eef30683dcad63c4dc3ce39c7058", "parents": [ "69b58cf399c3507ca3debf9ec9b2db3c5a0fe7f2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 04 11:01:17 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 04 11:01:17 2016 +0000" }, "message": "Revert \"Add dex file writer to dexlayout tool.\"\n\nBug: 29921113\n\nThis reverts commit 69b58cf399c3507ca3debf9ec9b2db3c5a0fe7f2.\n\nChange-Id: I8bf9b1222d99489059e45f47d220396a853170a4\n" }, { "commit": "69b58cf399c3507ca3debf9ec9b2db3c5a0fe7f2", "tree": "eeea96181b252426bc7d9363c62cf22c9aac100a", "parents": [ "d295618ac764b334424b0bc7a592337edef19783" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Sep 22 18:02:49 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Oct 03 14:33:25 2016 -0700" }, "message": "Add dex file writer to dexlayout tool.\n\nAll sections are being written out properly.\nThere is one small difference:\n - unindexed string data is not written out\n\nBug: 29921113\nTest: mm test-art-host-gtest-dexlayout_test\nChange-Id: I3872327526a350f33f0f851516a389622c1a54ae\n" }, { "commit": "3ab96b460684781af58b3f9c5b9fb0e370de6b50", "tree": "8165154e9ec5d6977ec3b2405f6b4ba0a6731b7e", "parents": [ "6a40fc9bbb9421f78ffd7a9cce7f9acff3aa7d0d" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Sep 09 18:35:01 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Sep 19 18:42:34 2016 -0700" }, "message": "Dexlayout changes to support offset/size output of items.\n\n- Created offset \u0026 size fields for items, initially populated with\n values from original dex file.\n- Created index field for indexed items in header.\n- Created Item objects for StringData and TypeList.\n- Created Collections container to hold unique_ptrs for all objects\n- Reorganized EncodedArrayItems to make it easier to collect them\n- Added relevant annotations-related items to Collections\n- Some size calculations and section offsets left as TODO\n\nBug: 29921113\nTest: mm test-art-host-gtest-dexlayout_test\nChange-Id: Id812f104c1930a3ee381e885f353a9a49f29ac9b\n" }, { "commit": "72359225691975281f0bd5a0a19b07a0b2e6b498", "tree": "228df961b259fa72be55950a52ad7b357546e369", "parents": [ "ef858c25d043074fe572c5d578e08af12e055733" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Sep 07 13:04:01 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Sep 07 15:56:40 2016 -0700" }, "message": "Fix some memory leaks found by valgrind.\n\nBug: 29921113\nChange-Id: If70d475a3317751d206658c5794a32d78bc33e47\nTest: valgrind-test-art-host-gtest (with --trace-children\u003dyes)\n" }, { "commit": "853a8e10844b9b9e791c6d40e5d332cd38af97d6", "tree": "7339753d96f5e346c28a5c57a561a9464d83e719", "parents": [ "3e65825978056adf7cbad646d4ab92c161a60e09" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Sep 01 13:03:50 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Sep 06 14:06:23 2016 -0700" }, "message": "Separate DexIr building from constructors.\n\nMove all the construction from DexFile out of the constructors of the\nbasic IR.\n\nBug: 29921113\nChange-Id: I3f79c104ce7183ddde73f143c047061416009a54\nTest: test-art-host-gtest\n" }, { "commit": "c3acfc5bed5d0cb22bee05e72cdbdec49a182224", "tree": "3f9e89e770efbea5dac6bd601d29b7707c609922", "parents": [ "7629f60ffa7b8533ea0b50263d52cc1df72b2904" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Aug 29 14:18:26 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Aug 31 11:09:41 2016 -0700" }, "message": "Finish dexlayout dumping of dex files and added test.\n\nThe test runs dexdump and dexlayout over the first core library jar file\nand diffs the output, checking that they\u0027re the same. More dex files\nwill be added later for better coverage.\n\nBug: 29921113\nTest: mm test-art-host-gtest-dexlayout_test\n(cherry-picked from commit a378e88db648cf19e771714dd1059cf3d41d4753)\n\nChange-Id: Idfd8ac9f069885b675a4a2f0c70aef2cbef5a9b8\n" }, { "commit": "7629f60ffa7b8533ea0b50263d52cc1df72b2904", "tree": "ce4d4f25c48dddf5930ff0d55c9d071f041a27fd", "parents": [ "081e7a16c4fcbdb014441a236e12f58eb89ff99a" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Sun Aug 07 16:01:51 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Aug 31 10:15:50 2016 -0700" }, "message": "Add dex file intermediate representation\n\nAdd an intermediate representation, constructed from the DexFile\nutilities, that can be used to optimize the layout and re-emit dex\nfiles.\n\nBUG: 29921113\nTEST: TODO: Add dex files to compare output with dexdump\n\n(cherry-picked from commit af14d9f551e6169c201dc7981d9cfbcb53c7d2c7)\n\nChange-Id: I3dacb78fad5228e43d77f20b110bd40e0e116cdb\n" } ] }