)]}' { "log": [ { "commit": "41b2f537aca2c9779e39651fb222a1132a7eff56", "tree": "8a916e481bf5a66dd0b14d84509d5d97c3de6f2b", "parents": [ "f040eca7329382d48dc2b8bfc228ee3f27c9f13d" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Mar 02 16:36:31 2017 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Mar 02 19:51:11 2017 -0800" }, "message": "Add nullptr and size check for opening a mem mapped dex file.\n\nThe bug has an apk with 0 size classes.dex. When dex2oat tries to open\nthe file for layout, it crashes. This adds checks to ensure that the mem\nmap isn\u0027t null and has a sane size before opening it as a dex file.\n\nBug: 35892406\nTest: mm test-art-host\nChange-Id: I5f6b5a1e7bbccf4fe3483b68023d51436eb71805\n" }, { "commit": "f39d42255214cf5e7297889e237a1917f2f5a963", "tree": "0a7fb86b1e2216b8607dfdb7f10600394fe080fb", "parents": [ "b3f2284d10450c8cabd1733fb7e83d4e031a35f6", "c9229220cc334529bbd4bebe06a3495cecc23c88" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Feb 15 05:39:20 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Feb 15 05:39:23 2017 +0000" }, "message": "Merge \"Fix DEX zip open error message printing\"" }, { "commit": "271a0f806d945592c6c0ac348004acb8ca3cba8b", "tree": "17b48ea21425bb8d6013b7f486c2f86ddc4d734f", "parents": [ "a6dfa573fa951747a09a5fb8b5e1f276e569e934" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Feb 14 21:14:17 2017 +0000" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Feb 14 13:44:51 2017 -0800" }, "message": "Revert \"Revert \"runtime: Mmap uncompressed dex files (in zip) as clean memory\"\"\n\nThis reverts commit 960b2af8a05f0844e78004e2d0d3ae6ab058d430.\n\nFix failing 071-dex-file-map-clean on target\nwhich was due to an IO race in the run-test script.\n\nTest: make test-art-target\nBug: 27650033\nOriginal-Change-Id: I18efbd392c5980ffe0d983833b6cc581e0237b92\nChange-Id: I6f4ff1e85f8326916c2ae0842a32f53fb7901639\n" }, { "commit": "c9229220cc334529bbd4bebe06a3495cecc23c88", "tree": "a60bc8c613617d04d2c6ad6223490ea70b828a9a", "parents": [ "7d863344e987bde473ae4f6b607e1a49efa1ea02" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Feb 14 10:57:47 2017 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Feb 14 10:57:47 2017 -0800" }, "message": "Fix DEX zip open error message printing\n\nWe were printing the pointer to the error message string rather than the\nstring itself.\n\nBug: none\nTest: test-art-host\nChange-Id: I06bd9afd100c425039847c52df1f4912966053b8\n" }, { "commit": "84f50aecdeed2d9ed3206681820e41ad6af73a9c", "tree": "8b2b6de40365dcfc41d9139c9c5d73b95a541b67", "parents": [ "69bcf2c940d26d65767bea7417317cf9a0431e0c" ], "author": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Mon Feb 06 15:12:45 2017 +0000" }, "committer": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Tue Feb 14 09:14:20 2017 +0000" }, "message": "Remove remaining uses of DexFile::GetChecksum.\n\nRemove or replace with DexFile::GetMultiDexChecksums as appropriate.\n\nBug: 34604632\nTest: m test-art-host, added image_space_test to test ValidateOatFile.\nChange-Id: I1042e87f29a242046d055f0be374ad9fbce4671a\n" }, { "commit": "69bcf2c940d26d65767bea7417317cf9a0431e0c", "tree": "7458a29402ff70a49b6d1cd6ca28d1eebb162063", "parents": [ "3f38398380b80d1ded078ebed1211b7e4f51460f" ], "author": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Tue Jan 24 10:25:21 2017 +0000" }, "committer": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Tue Feb 14 09:11:30 2017 +0000" }, "message": "Improve checking of multidex dex checksums.\n\n* Fall back to odex file for all multidex entries if the apk is stripped,\n not just for the main multidex entry.\n* Verify the number of multidex entries has not changed.\n* Improve performance by getting all checksums from the apk in one go\n and cache the results instead of repeatedly opening the apk.\n* Stop referring to non-main multidex entries as \"secondary\" dex files.\n\nBug: 34604632\nTest: added tests to dex_file_test and oat_file_assistant_test\nTest: m test-art-host\n\nChange-Id: I58b17ecfbc9165a5bfeffd5281ee21d108f64479\n" }, { "commit": "12f4ff43bcd21ba82e7aca7b6229160f28b5ff04", "tree": "35b70dcb29b325535c6de00b6e242543ccc30869", "parents": [ "ab58a629d6cfe75f0ed34326c2ca6364b2203885" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Jan 13 16:43:12 2017 +0000" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Mon Feb 13 14:53:05 2017 +0000" }, "message": "ART: Add DEX support for MethodHandle and CallSite info.\n\nAdds new DEX file map items for CallSiteIds and MethodHandles.\n\nInitializes CallSiteIds and MethodHandles from the DEX file map_list.\n\nBug: 33191717,30550796\nTest: m test-art-host-gtest-dex_file_verifier_test\nChange-Id: I3ad9c7342b661c3f6a8264709412650eee6bde01\n" }, { "commit": "960b2af8a05f0844e78004e2d0d3ae6ab058d430", "tree": "89515676fc9882d04e317a1c7cf8f719a16804dc", "parents": [ "19e5f834501c5e69fbd731038b88c10332cc6bc7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sun Feb 12 15:48:07 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sun Feb 12 15:48:07 2017 +0000" }, "message": "Revert \"runtime: Mmap uncompressed dex files (in zip) as clean memory\"\n\nGetting on the bots:\n+ERROR: Memory mapping for /data/run-test/test-6248/071-dexfile-map-clean-ex.jar is unexpectedly dirty\n+Private_Dirty: 4 kB\n\n\nBug: 27650033\n\nThis reverts commit 19e5f834501c5e69fbd731038b88c10332cc6bc7.\n\nChange-Id: I833ef95401187b764d336ab4558936c63678061c\n" }, { "commit": "19e5f834501c5e69fbd731038b88c10332cc6bc7", "tree": "439e382f697ed114e117d7ae21dc818f17ec8187", "parents": [ "e4530079672682ff3900d84c08b666a6a899d973" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Feb 07 15:20:54 2017 -0800" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Feb 10 10:33:12 2017 -0800" }, "message": "runtime: Mmap uncompressed dex files (in zip) as clean memory\n\nSaves 3MB of RAM (system-wide) in AGSA (search toolbar).\nAlso makes 3MB of RAM clean (previously dirty).\n\n----\n\nclasses.dex from different processes will now have\ntheir memory shared (as clean) globally.\n\nTo take advantage of this, the app/service just\nneeds to build their .JARs zipaligned and with\nclasses.dex being stored uncompressed.\n\n-----\n\nclasses[N].dex within ZIP files will now get memory-mapped\nas file-backed, clean memory when opening a DexFile.\n\n(The zip entry must be at least 4-byte aligned).\n\nCompressed DEX files still get extracted first to dirty memory.\n\nTest: art/tools/run-test 071-dexfile-map-clean\nBug: 27650033\nChange-Id: I18efbd392c5980ffe0d983833b6cc581e0237b92\n" }, { "commit": "6c4921b3a13ab645045dd20e5386daf552a31f41", "tree": "46ff2d2d320a250a1ce6e5094a67632293093a4b", "parents": [ "a6040d4cca09c1030bbfcfdc131e38c7d6da8888" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Sep 21 15:41:06 2016 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Thu Jan 19 09:01:02 2017 +0000" }, "message": "ART: Improve class initializer and constructor verification.\n\nDEX file verifier checks additional properties of class initializers\nand constructors:\n\n(i) Names match expected \u003cclinit\u003e / \u003cinit\u003e.\n(ii) The method descriptor for \u003cclinit\u003e is ()V.\n(iii) The return type of \u003cinit\u003e is V.\n(iV) No other names start with \u0027\u003c\u0027.\n\nBug: 31313719\nChange-Id: I60bffa6561e1bae353f97c42377ea556bfa790af\nTest: m test-art-host-gtest-dex_file_verifier_test\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": "8a0128a5ca0784f6d2b4ca27907e8967a74bc4c5", "tree": "0dec75200282ae5e49785395e97bd4e6459f1c09", "parents": [ "60438b46090d22bb9b978196f5aa53fab3b89759" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 28 07:38:35 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 29 11:11:46 2016 -0800" }, "message": "ART: Add dex::StringIndex\n\nAdd abstraction for uint32_t string index.\n\nTest: m test-art-host\nChange-Id: I917c2881702fe3df112c713f06980f2278ced7ed\n" }, { "commit": "a5b09a67034e57a6e10231dd4bd92f4cb50b824c", "tree": "304be738f4fa528b7ad2676103eecc84c79eaeeb", "parents": [ "dac7ad17c78387d15d7aefae0f852dddf5f37e34" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Nov 17 15:21:22 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 21 10:57:00 2016 -0800" }, "message": "ART: Add dex::TypeIndex\n\nAdd abstraction for uint16_t type index.\n\nTest: m test-art-host\nChange-Id: I47708741c7c579cbbe59ab723c1e31c5fe71f83a\n" }, { "commit": "7f019717cc9632c15cba6e993fc0c3d32e487075", "tree": "78f70767190c3f4c22cf717c8627055445281c0d", "parents": [ "a70a2e18ee0beaecb2c17a6340ce81821db36f5d" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Oct 26 16:09:13 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Oct 26 16:09:13 2016 -0700" }, "message": "Remove some unneeded header includes\n\nBug: none\nTest: Builds still work\nChange-Id: I6c83326cc12aa0e7825312b238f5e0ae694ccffc\n" }, { "commit": "709b070044354d9f47641f273edacaeeb0240ab7", "tree": "3a8ac051d7c35076303984d0d892cdd396b60427", "parents": [ "1a4de6a2453a3ad0310aca1a44e7e2d3b6f53bc1" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Oct 13 09:12:37 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Oct 18 14:10:04 2016 -0700" }, "message": "Remove mirror:: and ArtMethod deps in utils.{h,cc}\n\nThe latest chapter in the ongoing saga of attempting to dump a DEX\nfile without having to start a whole runtime instance. This episode\nfinds us removing references to ArtMethod/ArtField/mirror.\n\nOne aspect of this change that I would like to call out specfically\nis that the utils versions of the \"Pretty*\" functions all were written\nto accept nullptr as an argument. I have split these functions up as\nfollows:\n1) an instance method, such as PrettyClass that obviously requires\nthis !\u003d nullptr.\n2) a static method, that behaves the same way as the util method, but\ncalls the instance method if p !\u003d nullptr.\nThis requires using a full class qualifier for the static methods,\nwhich isn\u0027t exactly beautiful. I have tried to remove as many cases\nas possible where it was clear p !\u003d nullptr.\n\nBug: 22322814\nTest: test-art-host\nChange-Id: I21adee3614aa697aa580cd1b86b72d9206e1cb24\n" }, { "commit": "9fddd3657ba500fdd761ec13ffe4f0c3cbab9097", "tree": "c1678ed0cd5e23f6663661cb3a1f6739ac0cf376", "parents": [ "304533ed8475e9438e6ca7ef5fa644fd5dab5e97" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Sep 22 14:05:37 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Sep 23 11:04:06 2016 -0700" }, "message": "Fix CTS crash from DexFile refactoring\n\nAvoid path to nullptr access introduced by the previous refactoring.\n\nBug: 22322814\nChange-Id: Id8874b26c072a11b0494e8126f8b0602a7c5b9e8\nTest: m test-art-host, cts-tradefed run cts --skip-preconditions --skip-device-info --module vm-tests-tf\n" }, { "commit": "9c20a14103f87a8a7ad53f7d3e863887ea4e800a", "tree": "1fd4b123bae008f670f806024c88d3dc404da24b", "parents": [ "a51a135f114f6f0dbf7c4afd336f68b4a0d1bb7d" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Aug 23 15:05:12 2016 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Sep 21 15:47:06 2016 -0700" }, "message": "Get a basic modification of dex file working\n\nThis allows the modification of a single classes methods through\ntransformation. One must ensure that the provided dex file only\ncontains one function and does not add or remove any methods or fields\nand does not change the inheritance hierarchy in any way. The provided\ndex file must verify and there must be no frames of the old code\npresent on any thread. These constraints are not checked or verified.\nBreaking them might cause undefined behavior in all parts of the\nruntime. Code that has been inlined in any way might not be replaced.\nThis feature is extremely experimental.\n\nBug: 31455788\nTest: ./test/run-test --host 902-hello-transformation\n\nChange-Id: I35133d24f6cdafdd2af9dc9863e15ba8493fc50e\n" }, { "commit": "733ddb2af025aca54582fef95fb714e0df422c78", "tree": "e728e115a21e395b8d1d68f95f2552897e85e1f5", "parents": [ "c4cb3d532db8a500c1a15e8a0052895960259209" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Mon Sep 19 15:02:18 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Sep 21 08:15:26 2016 -0700" }, "message": "Refactor Dex open methods\n\nConsolidate dex opening around a simpler single interface in DexFile.\nMaking this change prior to moving all but the simplest interface out of\ndex_file.{cc,h}.\n\nBug: 22322814\nChange-Id: I4c55019912a19a1c90f9022c20f5b8b954fc7a47\nTest: test-art-{host,target}\n" }, { "commit": "9aa352e92b6ca0f2250cb7f54dfbf4b1be714c19", "tree": "647b9274daa64b3497c202da4327be26fe4e494b", "parents": [ "47c22ff721503c0eb325508450663adacf0bd377" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Sep 15 18:13:52 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Sep 16 16:31:19 2016 -0700" }, "message": "Remove TypeLookupTable from DexFile.\n\nOne more step towards removing runtime dependencies from the DexFile\nAPI. This severs the ties to OatFile. Work remains to move MemMap out\nof DexFile.\n\nBug: 22322814\nChange-Id: I29e7ad8fd292c7919ed2689dc754b958b88d6819\nTest: test-art-host\n" }, { "commit": "9323e6e44060545d6c4f925139572868f65b7fe3", "tree": "65f3f380605d75f34d49aec49fca494bec26ef5b", "parents": [ "58735fbfbb2da0c0a19b3239da07c2353c40767d" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Sep 13 08:58:35 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Sep 14 08:01:39 2016 -0700" }, "message": "Separate annotations from dexfile reading.\n\nBug: 22322814\nChange-Id: I867d66da407dd80394a10d19903ebbc1ec3986ff\nTest: test-art\n" }, { "commit": "60458a072c4f0b24038420b0f1bed0fe94c72965", "tree": "e1e10a29c99f99538695c12da1af914d84ddd4c3", "parents": [ "9d185da3bef8caf015d3dbf4ad79c520af7ce3b1" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Thu Sep 01 15:32:44 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Wed Sep 07 11:12:02 2016 +0100" }, "message": "Track libcore Executable.getParameter() changes\n\nExecutable.getParameter() has been added which added\ntwo new fields to java.lang.reflect.Executable,\nimpacting class arrangements and padding.\n\nThe java.lang.reflect.Parameter class has been added\nwhich has some associated native code and support\nmethods in DexFile.\n\nTest: make test-art-host\nBug: 28666126\nChange-Id: I4ff6469388879651a6c9e568eca0b1cd716f0c0d\n" }, { "commit": "ebb7b22f4e227176eca40a554990a8e89e283fcf", "tree": "574152c57f5dc44f5f877d994d34b541afeb67b9", "parents": [ "38c8fcd78dcb6707e30b489b57d61c1232ee5757", "3a21e386fa55a8b86dffa5250985b263fcfd2155" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Mon Sep 05 09:18:53 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Sep 05 09:18:54 2016 +0000" }, "message": "Merge \"Clean up some includes.\"" }, { "commit": "3a21e386fa55a8b86dffa5250985b263fcfd2155", "tree": "ce6fecaae067f86e094ae7844ee5fa4ae8ee63e8", "parents": [ "f94a4cf18946bda4a4f19378436d2bf131a492ca" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 02 12:38:38 2016 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 02 15:15:28 2016 +0100" }, "message": "Clean up some includes.\n\nRemove some unnecessary includes from header files, replace\nothers with forward references and add includes to source\nfiles as needed. Reduce dependency on stack.h by pulling\nStackReference\u003c\u003e out to its own file.\n\nTest: m test-art-host\nChange-Id: I0fb182145e328870cbd918b0ef6ae2950223c1b2\n" }, { "commit": "a4c2a054ca946eb14496fe8e9fd6057a59ccf692", "tree": "e6ba8afbff39b30775c24d81a05ee255fcc9457b", "parents": [ "79c3e4e558215ffae1601fc6d5f1626e063a4b22" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Aug 17 10:51:42 2016 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Fri Sep 02 11:40:21 2016 +0100" }, "message": "Support for loading classes from DEX files stored in a ByteBuffer.\n\nBug: http://b/26878838\nTest: test-art-target-gtest-dex_file_test\n\nChange-Id: I68845c35784386bde6270102f0079d4f07bc4d03\n" }, { "commit": "9d4b6da934934c322536ee3309b63ce402740f49", "tree": "9e7ee5023d6036b98e0560411bb0527efdedca01", "parents": [ "2af1aa066e3d20edd8fea5d5b6dbbbad73102d52" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Jul 29 09:51:58 2016 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Aug 16 20:19:36 2016 +0000" }, "message": "jni: Fast path for @FastNative annotated java methods\n\nAdds a faster path for java methods annotated with\ndalvik.annotation.optimization.FastNative .\n\nIntended to replace usage of fast JNI (registering with \"!(FOO)BAR\" descriptors).\n\nPerformance Microbenchmark Results (Angler):\n* Regular JNI cost in nanoseconds: 115\n* Fast JNI cost in nanoseconds: 60\n* @FastNative cost in nanoseconds: 36\n\nSummary: Up to 67% faster (vs fast jni) JNI transition cost\n\nChange-Id: Ic23823ae0f232270c068ec999fd89aa993894b0e\n" }, { "commit": "d81c0f8a7c8d6853edda01bceeae85ef831118fb", "tree": "d207638c2c8580c9d2781bbbfadceca8b55b240d", "parents": [ "53d9a24c77b0f00fd1936d957cebb6792fa17bdf" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Aug 03 09:05:20 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Aug 03 16:15:19 2016 +0000" }, "message": "Fix constant type\n\nRemove the unnecessary cast to long for DCHECK, a nit that was omitted\nfrom the last CL, https://android-review.googlesource.com/#/c/248243/.\n\nBug: b/28856653\nTest: test-art-host-gtest-dex_file_test\nChange-Id: I8ca18d19cbac61e40ddfbe32729aef5c20347174\n" }, { "commit": "cd16d0aa7e30115323caf040a11b4605f1acfea3", "tree": "a9a2495f5a50a1a0a514420fdfceb6ab5c6e47fc", "parents": [ "c984725d0035e1925371757c38fed339b409e525" ], "author": { "name": "ganxiaolin", "email": "ganxiaolin@xiaomi.com", "time": "Mon Jul 18 11:21:44 2016 +0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Aug 02 14:46:45 2016 -0700" }, "message": "Return error directly when uncompressed length is 0\n\nIf uncompressed length is 0, in function MemMap::MapAnonymous, it will generate a MemMap instance whose base_begin is nullptr.\nThen, when OpenMemory, it will create a DexFile instance.\nAt DexFile::DexFile constructor, header_ is asigned to 0, then header_-\u003estring_ids_off_ will cause crash.\n\nBug: b/28856653\nTest: test-art-host-gtest-dex_file_test\nSigned-off-by: ganxiaolin \u003cganxiaolin@xiaomi.com\u003e\nChange-Id: Id37f7629f4646cbc385ef054cb83b15be4c59b00\n" }, { "commit": "52e665084f46b5ba77a2a14d41110807539c41bc", "tree": "f6b323cd02eb5fee25db71ebb3b3b9f31136620e", "parents": [ "3d6094531e3790b25c43e59fd6cd0b6b99d3447c" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Aug 01 14:20:31 2016 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Tue Aug 02 08:31:04 2016 +0000" }, "message": "DexFile: Add basic support for v38 dex files.\n\nMakes sure they aren\u0027t rejected, essentially. Also adds positive\nand negative test cases in dex_file_test.\n\nbug: 30550796\nChange-Id: I8c95055a22f8a1435868c8bf3497f9641b7bb900\ntest: make test-art-host\n" }, { "commit": "542451cc546779f5c67840e105c51205a1b0a8fd", "tree": "11e09bb5abaee12dddffefbe7e425291076dfa7a", "parents": [ "85c4a4b8c9eabfe16e4e49f9b4aa78c1bf4be023" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 26 09:02:02 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Aug 01 18:54:48 2016 -0700" }, "message": "ART: Convert pointer size to enum\n\nMove away from size_t to dedicated enum (class).\n\nBug: 30373134\nBug: 30419309\nTest: m test-art-host\nChange-Id: Id453c330f1065012e7d4f9fc24ac477cc9bb9269\n" }, { "commit": "e01e3644205c3644546889237c20185391a0092e", "tree": "413fcf279b415a305605a601297e3856f13a2948", "parents": [ "de4cf16f467b531373560ca6eb785f0f36606aae" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 25 13:06:04 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 25 13:09:41 2016 -0700" }, "message": "ART: Add pointer-size template to some reflection functions\n\nThe unstarted runtime may run code for a different pointer size,\neven when no transaction is active (e.g., during startup). To\nretain performance when the runtime is up and executing under\nnormal conditions, add a template parameter and use sizeof(void*)\nin places where it is adequate.\n\nFor maintainability, it is necessary to drop the default for\nthe transaction template parameter. Implicit conversions from\nbool to size_t may lead to incorrect code and hard to diagnose\nproblems. So instead ensure that all callers must give all\ntemplate parameter values.\n\nTest: m test-art-host\nChange-Id: I3076883422c8553ede4de5642409c5684a5a9aa8\n" }, { "commit": "43e10b031e3bb42df54adf8f0525a29d2b308a4e", "tree": "ea914f7c0061817f51449d15b900e6c1878e5cc8", "parents": [ "edec0eb18d9a45d994acec9e8e509a1dc05bd5b3" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jul 15 17:17:34 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Jul 15 20:54:09 2016 -0700" }, "message": "ART: Replace ScopedFd with FdFile\n\nFdFile can now be used like ScopedFd. Remove ScopedFd.\n\nBug: 21192156\nTest: m test-art-host\nTest: m test-art-target (shamu)\nChange-Id: I32115fa8b2b8bb5aa5d1886eae63522f80ce836b\n" }, { "commit": "37d6a3bbf5826abb73f872ed0bc4c6ae9553e274", "tree": "0fbe5fda78e251597f909f654014f2c25efa62e5", "parents": [ "739dc72773c5dee583a1d322f91b5abd61f9889d" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Tue Jun 21 18:30:10 2016 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Jun 22 15:39:07 2016 -0700" }, "message": "Support the -i flag in the new dexdump.\n\nRationale:\nThe old dexdump supported the -i feature (ignore checksum failures).\nThis CL adds the same functionality to the new dexdump. Note that\nthis feature is, for example, useful to inspect DEX files that are\nextracted from images (with quickened instructions), since the\nchecksum is not always valid in those cases.\n\nBUG\u003d29548017\n\nChange-Id: I6017b2f3d789f450560b568297f43c62bf9453b1\n" }, { "commit": "3d0808655cd8cfbc867bf330bcc6f954c59beb9d", "tree": "e7e6e6c62562b6f19e0dbf7b2249a900a1938afa", "parents": [ "d355b75ca09d83c47e596daeb1a434942de83bee" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu May 26 18:39:17 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri May 27 17:27:28 2016 -0700" }, "message": "Relax annotation visibility so runtime includes build.\n\nPrevious behavior in M and earlier would allow annotations marked\nVISIBILITY_BUILD to be visible to the runtime when they should not\nhave been. When targeting older sdks, revert to this behavior.\n\nBug: 28826610\n\n(cherry-picked from commit 7e50a7a5a1ad947e84c425efb2e97c442f91b155)\n\nChange-Id: I43d2328be41ec9f4e679b3151f586c0b87b38c7c\n" }, { "commit": "ab880f4371e1ae5239c1b887911bd88f0c216a3d", "tree": "2e13093b2255369257844865db39c56816dfb054", "parents": [ "2273f678ea3a55ec0f6b3b8c29c1e5100d6fc846" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu May 12 16:24:36 2016 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Mon May 16 15:56:23 2016 +0100" }, "message": "Typos and stylistic fixes.\n\nChange-Id: I1c88af9bd1d2cac6c803c8c61088c569a3847e01\n" }, { "commit": "1133db79350060158f99210c56f111c6dad43563", "tree": "c84b4ae2df13a32539645c0904f08c874c20ae31", "parents": [ "a07be75830af60ce50aba357e8de066e849aa21c" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Apr 04 19:50:14 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Apr 04 19:50:14 2016 -0700" }, "message": "Remove AnnotationAccess and its remaining uses.\n\nArt side of this change. Adds a test to ensure annotations not marked\nfor runtime retention can\u0027t be seen at runtime.\n\nBug: 27912552\nChange-Id: I078069b7b3cb72bfe7d0b9ea61e527fee04d56a3\n" }, { "commit": "76ed99d5ec208d5adcd63b41c2c290194ee0ecf7", "tree": "acfc8f526561079c8389572af97c96fda7475450", "parents": [ "8a31b7d44a13b1d7e71852dc3c992598698d351d" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Mar 28 18:31:29 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Mar 29 17:46:17 2016 -0700" }, "message": "ART: Postpone interface-related dex failure to version 37\n\nFor app compat, at least for now make the check for public-final-static\nof interface members not fail on dex file versions less than 37. This\nmay be changed in future releases.\n\nBug: 27831184\nChange-Id: If8ee50321298b951d4a78062c8eb583fec27394f\n" }, { "commit": "c49618160d5fa64ca4883d2e09fa34b83491c8ea", "tree": "50065864dd6131a19c119eef877e936a44ec5778", "parents": [ "7ee5bc54fbe1bf5218696e63a80e76cd0c8ea387" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Mar 23 10:20:41 2016 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Mar 24 15:17:16 2016 -0700" }, "message": "Add support for Dex version 37 in Runtime.\n\nWe are skipping version 36 of the dex file format due to a bug in\nDalvik dating back to ICS where dex files marked version 036 would\nerroneously be accepted.\n\nBug: 27538761\nBug: 27809626\n\nChange-Id: Ic053f7e25f5a8c3df83ff34b6656528824b2df12\n" }, { "commit": "32ce2adefb8a3d0eda59a29f5e87c1eb43eef796", "tree": "50aaa40b31ede355359765259edc7989485cc7d5", "parents": [ "c71dc3852edfa8e0cd2083d12845ff8607e189ff" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 04 14:58:03 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 04 18:37:19 2016 -0800" }, "message": "Add more systracing everywhere\n\nAdded to:\nJIT\nDexFile functions\nOat file manager\n\nAdded helper ScopedTrace to prevent errors and reduce excess code.\n\nBug: 27502458\n\n(cherry picked from commit dabdc0fe183d4684f3cf4d70cb09d318cff81b42)\n\nChange-Id: Ifaeff8913d79eefc797380987d13cc00456266f8\n" }, { "commit": "3a2bd29d274f60fdcfabebb052078edef0190164", "tree": "1e98b0b4ccdfc60c2a3f13abd4b0141b0c8be75f", "parents": [ "6a507f3bef2c970b29bada165269d2764d260fc8" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 26 17:23:47 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 27 11:46:00 2016 -0800" }, "message": "ART: Make sure dex files are verified in the compiler\n\nFollow-up to 9bdf108885a27ba05fae8501725649574d7c491b. Make sure\nthat dex files from the oat writer are verified with the dex file\nverifier.\n\nBug: 26793137\nBug: 26808512\nChange-Id: I1a5f51751491eead21d8f9f1b31e37c7374c72a5\n" }, { "commit": "9bdf108885a27ba05fae8501725649574d7c491b", "tree": "a4ddf98b6cf10b343f15164f18e7089bc54c29ca", "parents": [ "a92ee11b9b0ed4033efc5982269e3c0a075315e0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jan 21 12:15:52 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jan 21 12:19:08 2016 +0000" }, "message": "Revert \"Revert \"Write dex files to oat file early.\"\"\n\nThis reverts commit 919f5536182890d2e03f59b961acf8f7c836ff61.\n\nFix linker error (Mac build):\nReplace inline definition of art::ZipArchive::~ZipArchive()\nwith an out-of-line definition in zip_archive.cc to avoid\ndirect reference to CloseArchive() from libart-compiler due\nto inlining. Note that libart is linked against -lziparchive\nbut libart-compiler is not.\n\nChange-Id: I92620ea0200282ca7ba9b7f61a592cb6468d90d8\n" }, { "commit": "919f5536182890d2e03f59b961acf8f7c836ff61", "tree": "8e8b9c330ae9c886611c6e5ae598e21d9b3ea17a", "parents": [ "625a64aad13905d8a2454bf3cc0e874487b110d5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jan 20 19:13:01 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jan 20 19:13:01 2016 +0000" }, "message": "Revert \"Write dex files to oat file early.\"\n\nThis reverts commit 625a64aad13905d8a2454bf3cc0e874487b110d5.\n\nBreaks the Mac build:\n\nUndefined symbols for architecture i386:\n \"_CloseArchive\", referenced from:\n ... in oat_writer.o\nld: symbol(s) not found for architecture i386\n\nChange-Id: I21608bc51437834e1e6abde9bcbe5e7d9998197e\n" }, { "commit": "625a64aad13905d8a2454bf3cc0e874487b110d5", "tree": "49035d031166ce9fe0c5dd2f4b34157100e1703f", "parents": [ "e1b0f475e851072d0083faf6e07d274e9f1fe6a5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 26 14:44:16 2015 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jan 15 12:28:06 2016 +0000" }, "message": "Write dex files to oat file early.\n\nWrite dex files to oat file before we actually open and\nverify them. Instead, open and verify the copies from the\noat file and use these. This way, in the most common case\nof zipped dex files, we have mmapped dex files instead of\ninflated dex files. That reduces the number of dirty pages\nused by dex2oat.\n\nReading /proc/self/statm after we write the oat file for\na compilation of a certain large app on Nexus 5 AOSP build\nwith -j1, three attempts before and after this CL gave\n before: 346061 189462 6269 26 0 140723 0\n 346189 189450 6269 26 0 140851 0\n 346061 189463 6269 26 0 140723 0\n after: 346186 185808 23040 27 0 140468 0\n 346186 185819 23040 27 0 140468 0\n 346186 185822 23040 27 0 140468 0\nThese values are in pages (4KiB), so while the \"size\"\n(\u003dVmSize) is essentially unchanged, the \"resident\" (\u003dVmRSS)\nis over 14MiB less and the \"shared\" (i.e. backed by a file)\nis 65.5MiB more. That is, the amount of dirty non-pageable\nmemory used is reduced by about 80MiB.\n\nThe oat file format has changed slightly, the class offset\ntable has been moved from the OatDexFile to its own section.\nThis actually fixes the alignment of these offsets as they\ncould have been unaligned previously, yet accessed as normal\nwith significant performance impact if the kernel has to\nemulate the unaligned access (say, mips).\n\nChange-Id: I0f4799bb1f1ca28e3533156a3494f55345c3e10a\n" }, { "commit": "9865bde5d822f56c4732214c2005dfcaa41f94cf", "tree": "cd2eae058f4f4f13b5a82ff557b7eaaf13a1ecfb", "parents": [ "115a02b737dd5f4d485b2f6c359e02988df66b83" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Dec 21 09:58:16 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Dec 21 09:58:16 2015 -0800" }, "message": "Rename NullHandle to ScopedNullHandle\n\nThis makes it clearer that is invalid to do things like:\nHandle\u003cT\u003e h \u003d ScopedNullHandle\u003cT\u003e();\n\nBug: 26233305\nChange-Id: I6d8f54eae01ec2e901cb7043afa853ea77db79fe\n" }, { "commit": "96c9de9510b33e2c8def669c4dfcb65db729e7d2", "tree": "7949b6c0289b63ab48004569489a3d0fe102b7be", "parents": [ "74768fb83073a2ae84c9173d4fc53654e3092b24", "b06e28e5b9fbabe3e69b18f31bf353eaff5d0c1f" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Dec 15 15:29:17 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Dec 15 15:29:17 2015 +0000" }, "message": "Merge \"Refactor DexFile::DecodeDebugInfo.\"" }, { "commit": "b06e28e5b9fbabe3e69b18f31bf353eaff5d0c1f", "tree": "18ff419c6a3b80ecff3125f13b72a8851e848ab0", "parents": [ "f71b3ade9c99ce2fec2f5049ce9c5968721e1b81" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Dec 10 13:15:00 2015 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Dec 15 15:11:50 2015 +0000" }, "message": "Refactor DexFile::DecodeDebugInfo.\n\nSplit the method into two - one for locals and one for positions.\nAll uses of the method request only one of the two and it makes the\ncode slightly cleaner. The position variant requires fewer parameters.\n\nExpose additional line table information which was previously ignored\nby the decode method (prologue, epilogue, source file).\n\nChange-Id: Idf8ba98fa58ea0d2103932b5cc0af81365885107\n" }, { "commit": "50a2f8deb8982c11966764ada4c01df95d8310c5", "tree": "a69035464f2106e2033dceeb6425865fd78d26f8", "parents": [ "748047de833061466e230baf374480a147568f73" ], "author": { "name": "Shinichiro Hamaji", "email": "hamaji@google.com", "time": "Fri Dec 11 09:45:28 2015 +0900" }, "committer": { "name": "Shinichiro Hamaji", "email": "hamaji@google.com", "time": "Mon Dec 14 15:43:49 2015 +0900" }, "message": "Get DCHECK back to EncodedStaticFieldValueIterator\n\nThis is a follow-up of\nhttps://android-review.googlesource.com/#/c/185000/\n\nChange-Id: Ia7311ab948712324f92814e4d415a0a78d16bb84\n" }, { "commit": "1a74dc42099bd1ea4583edabef42b5a4d6f70389", "tree": "43701750599f099159e5abcdadab226846fb9b7c", "parents": [ "170e01a6b59b3242a5afc76c3a03c00ce288150f", "82863f0ce9fa45f6b14d12c35a6a50e2772ab26a" ], "author": { "name": "Shinichiro Hamaji", "email": "hamaji@google.com", "time": "Thu Dec 10 23:45:02 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Dec 10 23:45:02 2015 +0000" }, "message": "Merge \"Output static field values in dexdump\"" }, { "commit": "82863f0ce9fa45f6b14d12c35a6a50e2772ab26a", "tree": "16863b18db5dff2d3d125c956910d45048f24341", "parents": [ "e8be90a673feb573d53277f42282955cbf7f58e9" ], "author": { "name": "Shinichiro Hamaji", "email": "hamaji@google.com", "time": "Thu Nov 05 16:51:33 2015 +0900" }, "committer": { "name": "Shinichiro Hamaji", "email": "hamaji@google.com", "time": "Thu Dec 10 13:16:59 2015 +0900" }, "message": "Output static field values in dexdump\n\nThis is necessary to generate table-of-contents of .dex files\nto prevent unnecessary rebuilds for implementation-only\nchanges because these values can be used while compiling other\nmodules.\n\nAlso modify EncodedStaticFieldValueIterator so it can be used\nwithout ClassLoader/Linker.\n\nBug: 24597504\nChange-Id: Ida0c839f9dd6961e1c1b3a380e2092042fad03bb\n" }, { "commit": "5096e66d07db8041589518f8c5b0281d859d0817", "tree": "52a1d28a6286d9054d2559fea8168541612093bd", "parents": [ "88b2b80aed15bb1f931cddd40e44ca525ef10018" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Dec 08 19:25:49 2015 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Dec 09 15:40:31 2015 +0000" }, "message": "ART: Add FdFile::Copy() to copy data from another file.\n\nAlso move utilities for inspecting file magic numbers to\nbase/file_magic.{h,cc} and drop the unused IsOatMagic().\n\nChange-Id: I2cc4dd18a5e8b9738fb386c8057faad3722bdd68\n" }, { "commit": "d297b554b673bbc627c8080d177defe1b4fd8c58", "tree": "1c3fec42597ca96ba88a84c3c247e5566339254c", "parents": [ "2f49b0a90720fe9f4cab040c376aab5f387967e3" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Nov 20 14:56:09 2015 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Nov 20 14:56:09 2015 -0800" }, "message": "Fix null pointer in processing of enum annotations.\n\nBug: 25802263\nChange-Id: Ib20b7049fd3824a5eb3e396d34ef32574771d074\n" }, { "commit": "42bddcec51e71d206f6d3b30a881ee6c1d50a63c", "tree": "2035b4dd7dcbefbe83c04d41987e520eeb8798f0", "parents": [ "78232f2482e71fef255796633da68bb769baa93a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 09 15:16:56 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 09 17:18:58 2015 -0800" }, "message": "Add low_4gb support to MapFile and MapFileAtAddress\n\nMotivation is to use this for loading app images in low 4GB at a\nnon fixed address.\n\nAdded test.\n\nBug: 22858531\nChange-Id: I0f79a4a7bfbfbdfc112e41b25c8682b1fb932ab7\n" }, { "commit": "9507fa2c9545156747a5eb248bc1af3159bfd8c8", "tree": "f8facbd68dabc781cc4f8e76353a418ef633896e", "parents": [ "c65d1f3d26a47e2fce3c3f8872e7942fec23f9b9" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 29 15:08:57 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 29 15:22:00 2015 -0700" }, "message": "Change a few places to use string version of GetTypeId\n\nGetTypeId is about 20% faster than GetStringId + integer GetTypeID\nsince it does less binary searches.\n\nChange-Id: I876c4ac89ab206acca217b2287b0197ef2e408c2\n" }, { "commit": "d9786b0e5be23ea0258405165098b4216579209c", "tree": "88641c107d29d0848ce8219652b8d0b1cdd3755c", "parents": [ "57bee8dc2206404164606d4f5cabde272505f5fa" ], "author": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Wed Oct 14 16:36:55 2015 +0300" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 29 09:38:06 2015 -0700" }, "message": "Implementation of fast lookup table to search class_def by descriptor\n\nLookup table is a hash table which built at compile time and stored\ninto oat file. At runtime the table is restored and used in the\nmethod DexFile::FindClassDef(const char*) to perform fast search of\nthe class_def_idx by class descriptor. Advantages of the lookup table\nover the HashSet (runtime/base/hash_set.h) are:\n1. Lookup table is built at compile time and uses read-only memory at\nruntime\n2. Lookup table uses less memory then DexFile::Index (less by 80% for\n/system/framework/framework.jar on Nexus5)\n3. Lookup table does less string comparisons compared with HashSet\n(less by 70% for zygote process on Nexus5)\nThe disadvantage of the lookup table is it increased boot.oat size by\n0.2% on Nexus5 and application .oat file by 0.3% in average on Nexus5.\n\nmathieuc changes:\nCreate lookup table in dex2oat to speed up compilation. Clean up code\nto follow style guide and use less static functions. Added\nperformance measurements.\n\nCompile ~100 APKs 5 times with filter interpret-only:\nBefore:\nreal 1m8.989s\nuser 0m59.318s\nsys 0m7.773s\n\nAfter:\nreal 1m1.493s\nuser 0m52.055s\nsys 0m7.581s\n\nApp launch (AOSP N5 maps, average of 45 runs):\nBefore: 966.84ms\nAfter: 923.733ms\nLaunch speedup is 4.7%\n\nMemory usage compared to HashSet index on 50 various APK:\n32 bit: HashSet ~625694b vs TypeLookupTable ~404268b\n64 bit: HashSet ~1251390b vs TypeLookupTable ~404268b\n\nBug: 10921004\nBug: 20269715\n\nChange-Id: I7246c1d9ad9fe81fe5c5907a4bf70396d8f9242a\n" }, { "commit": "35717f8cfb6c1de5c89b2f9b990a050b2cbab66f", "tree": "468d9275ebb774de9c373aea6387e5c55c4336e6", "parents": [ "b6f7dd330f115fc977d1d5a10122c41c9dd7c210", "637ee0b9c10ab7732a7ee7b8335f3fff4ac1549c" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Sep 14 10:42:01 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Sep 14 10:42:01 2015 +0000" }, "message": "Merge \"ART: Add some utilities for working with containers.\"" }, { "commit": "637ee0b9c10ab7732a7ee7b8335f3fff4ac1549c", "tree": "21cf63333abcc07ed1e3138ec3c9e690e0abc227", "parents": [ "68ffda887e35f35e978f2f607b7a91e44a5e1969" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 04 12:47:41 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Sep 08 18:09:20 2015 +0100" }, "message": "ART: Add some utilities for working with containers.\n\nAdd utility functions for searching, removing and replacing\nexisting values in a container, to be used with std::vector\n(including the ArenaVector alias) and other containers.\n\nAlso move UniqueCPtr\u003c\u003e and MakeUnique() to base/stl_utils.h\nand clean up related includes.\n\nChange-Id: I1e61762df91c046866591bda167d42bf8b67a692\n" }, { "commit": "957fb8930766ae422568e7b1b816159a9e9bc18c", "tree": "849a611f978656f1c85182402e1ef2c8ca50c5e6", "parents": [ "68ffda887e35f35e978f2f607b7a91e44a5e1969", "05792b98980741111b4d0a24d68cff2a8e070a3a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 04 10:34:04 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Sep 04 10:34:04 2015 +0000" }, "message": "Merge \"ART: Move DexCache arrays to native.\"" }, { "commit": "05792b98980741111b4d0a24d68cff2a8e070a3a", "tree": "bad79a387bcbdaefc87c07b388099960ca9caff3", "parents": [ "c26b4512a01d46756683a4f5e186a0b7f397f251" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Aug 03 11:56:49 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Sep 03 17:30:57 2015 +0100" }, "message": "ART: Move DexCache arrays to native.\n\nThis CL has a companion CL in libcore/\n https://android-review.googlesource.com/162985\n\nChange-Id: Icbc9e20ad1b565e603195b12714762bb446515fa\n" }, { "commit": "fc8d247275d06047c652abbdd368fe784bf85e67", "tree": "8b71dd7c2a5c2f95c8f2c158016995718c023d93", "parents": [ "fe3879e6011f629d0dd6b04fab00b9496bd4ea08" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Sep 02 13:52:20 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Sep 02 14:56:43 2015 -0700" }, "message": "Fix native annotations returning TypeNotPresentException.\n\nAlso adds test cases for inner classes and TypeNotPresentException.\n\nChange-Id: I28041af455f09b43fcf0b07b79b5a21d4741079b\n" }, { "commit": "2a5892f9ebcd4b7bd7343db50e578bd24b8e55a6", "tree": "ef5ef732bab16392bc3a67b3d797797a2292cd66", "parents": [ "51995e135794e01a7aecccfed81dc00a737e24bd" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Aug 31 15:00:40 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Sep 01 15:40:48 2015 -0700" }, "message": "Move more Class annotations to native.\n\nArt side of this change. There is also a corresponding Libcore change.\n\nSeeing speedup in AnnotatedElementBenchmark.\nGetDeclaredClasses\t 21.61x\nGetDeclaringClass\t108.73x\nGetEnclosingClass\t 87.81x\nGetEnclosingConstructor\t 23.35x\nGetEnclosingMethod\t 22.71x\nGetModifiers\t\t152.47x\nGetSimpleName\t\t106.11x\nIsAnonymousClass\t 91.28x\nIsLocalClass\t\t 51.95x\n\nChange-Id: I2b7f7eb7785fc20671fd7c338ffa9c7048a44a48\n" }, { "commit": "13e748b28c5f2bd1e83674d2ca899ff61ae5c0a1", "tree": "387724eebfe7cd0f7be368e64b22af3894eea4c7", "parents": [ "dda43e7e9680e6eead02876e67718d01837d89e0" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Aug 25 20:44:19 2015 +0000" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Aug 25 14:20:09 2015 -0700" }, "message": "Revert \"Revert \"Move annotations to native.\"\"\n\nThis reverts commit 7db6dd79a24570448ae737ee1946b00396696cac.\n\nAdds check if field\u0027s declaring class is proxy.\nBug: 23508574\n\nChange-Id: Ie829f1526e74427711e818b56d1588d92946cbf6\n" }, { "commit": "7db6dd79a24570448ae737ee1946b00396696cac", "tree": "490c05f1488141adf5668b34cbec28830fe8295c", "parents": [ "0042c6d49b8488c78f0b937063e316e8d6244439" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 25 10:53:19 2015 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 25 10:53:19 2015 +0000" }, "message": "Revert \"Move annotations to native.\"\n\nThis reverts commit 0042c6d49b8488c78f0b937063e316e8d6244439.\n\nReverting this change (as well as the companion CL\nhttps://android-review.googlesource.com/#/c/167510/ in\nplatform/libcore) as they make libcore test\nlibcore.java.lang.reflect.ProxyTest#test24846 fail.\n\nChange-Id: Ie0676cabb128277c7df5dab7bde17aefd3b2c09c\n" }, { "commit": "0042c6d49b8488c78f0b937063e316e8d6244439", "tree": "170286a01d08badebacaebed1db325bc889e259e", "parents": [ "300432d27ba9dd6961458fb1e1c845766f25c50a" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Jul 29 20:14:10 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Aug 21 17:05:27 2015 -0700" }, "message": "Move annotations to native.\n\nArt side of this change. There is also a corresponding Libcore change.\n\nSeeing ~2-3x speedup over dalvik KK MR1 in AnnotatedElementBenchmark.\nBenchmark\t\t\t Speedup of Art AOSP to Dalvik KK MR1\nGetAllReturnsLargeAnnotation\t 2.99\nGetAllReturnsMarkerAnnotation\t 2.20\nGetAllReturnsNoAnnotation\t 2.43\nGetAllReturnsSmallAnnotation\t 2.52\nGetAllReturnsThreeAnnotations\t 2.87\nGetAnnotationsOnSubclass\t 2.42\nGetDeclaredAnnotationsOnSubclass 2.49\nGetFieldAnnotation\t\t 2.68\nGetFieldAnnotations\t\t 2.60\nGetMethodAnnotation\t\t 2.66\nGetMethodAnnotations\t\t 2.61\nGetParameterAnnotations\t\t 2.52\nGetTypeAnnotation\t\t 2.56\nGetTypeAnnotations\t\t 2.17\nIsFieldAnnotationPresent\t 3.26\nIsMethodAnnotationPresent\t 4.99\nIsTypeAnnotationPresent\t\t 1.34\n\nChange-Id: Ibdbb6d23b17eaab6e83c8774b1bb9401e8227941\n" }, { "commit": "2a5c4681ba19411c1cb22e9a7ab446dab910af1c", "tree": "883ea0c07aad9efdb7c86960056cbefd7992b2bc", "parents": [ "228b3973b2b24783c727a55fda2b4b80375f7207" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Aug 14 08:22:54 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Aug 17 15:42:25 2015 -0700" }, "message": "ART: Some header cleaning around bit-utils\n\nTry to remove dependencies where they are not necessary.\n\nChange-Id: I5ff35cb17aea369bed3725b1610b50d7eb05b81e\n" }, { "commit": "23682bf61e91a4c34373d55b085ab7cbade3ed1b", "tree": "c3291a501b1419a030c0619db9c7d0439805a3b6", "parents": [ "4a08e17a9db0f68b9623849bc288c31a47868fbc" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Jun 24 14:28:03 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Jun 26 15:26:30 2015 +0100" }, "message": "ART: Ignore repeated field indexes when loading a class.\n\nThis provides a deterministic behavior for classes with\nduplicate field indexes in class_data_item and avoids\nfailures when verifying the field ordering in debug build.\n\nRegression test not feasible without hand-edited dex file.\n(Smali deduplicates field definitions.)\n\nBug: 21868015\nChange-Id: I4f97ba005e063686f8f44248ed7f1286d987888a\n" }, { "commit": "4cc6073c0c225ce6e886d4d29757408b09249d8f", "tree": "e6eb38f3d6dd11514e39a51810ac3701f24b8274", "parents": [ "1f1d53ebfc7f955c28f8d5659b7811ad6cef8f19" ], "author": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Wed Jun 24 16:33:32 2015 -0700" }, "committer": { "name": "Aart Bik", "email": "ajcbik@google.com", "time": "Thu Jun 25 10:33:42 2015 -0700" }, "message": "Fixed bug in debugging information in dex file\n\nRationale:\nNot resetting signature field may cause \"leaking\" old extended\nvalue into new non-extended value (exposed while implementing\nan Art-based dexdump; it affects all debug iterators though)\n\nBug: 17442393\nChange-Id: I4d1e2357ab7e0d77dbcf09786f24ac5cc54160b9\n" }, { "commit": "e401d146407d61eeb99f8d6176b2ac13c4df1e33", "tree": "17927f9bfe7d2041b5942c89832d55f9dedb24c5", "parents": [ "2006b7b9b8e32722bd0d640c62549d8a0ac624b6" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 13:56:20 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 29 18:45:49 2015 -0700" }, "message": "Move mirror::ArtMethod to native\n\nOptimizing + quick tests are passing, devices boot.\n\nTODO: Test and fix bugs in mips64.\n\nSaves 16 bytes per most ArtMethod, 7.5MB reduction in system PSS.\nSome of the savings are from removal of virtual methods and direct\nmethods object arrays.\n\nBug: 19264997\nChange-Id: I622469a0cfa0e7082a2119f3d6a9491eb61e3f3d\n" }, { "commit": "0cba004b97245300d7f39318d5921ee8edbef1ac", "tree": "edef4ed18df47c3c033c33b14bdd2aa6652f8696", "parents": [ "eff3229ff3c8faead4b6d400e273ee27b7004dab" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Apr 29 20:47:16 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Apr 29 20:47:16 2015 -0700" }, "message": "ART: Allow oat files with duplicates classes in corner case\n\nWhen the oat file is actually an odex file, that is, a preopted\n/system app, then it is impossible to fall back to the original\nAPK, as that has been stripped.\n\nWhen it looks like it will be impossible to successfully open the\noriginal dex location, grudgingly allow to open the found oat file,\neven if it has duplicate classes, but warn accordingly.\n\nBug: 20697582\nChange-Id: I1dd459563d977a2e77806eacd03e49334d5b1f14\n" }, { "commit": "90e34043ff7bf8926e8fe6a6d5011a9daa68d320", "tree": "ab9d0eeee40981d436b3c09081ccc736133b3b9b", "parents": [ "13d38aabf01f8391946730b7c2e8d077b1ec7d13" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 27 20:01:52 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Apr 28 22:08:35 2015 -0700" }, "message": "ART: Remove multidex limit\n\nRemove the arbitrary multidex limit. If users want to use many\nfiles, allow them, but print a warning after a considerable amount.\n\nBug: 20071800\n\n(cherry picked from commit 32c26b8f9b995250479c185172f4ffd881a59996)\n\nChange-Id: Ic51c96b84042f769a7d33ec53fe587b68cd69df4\n" }, { "commit": "2cebb24bfc3247d3e9be138a3350106737455918", "tree": "d04d27d21b3c7733d784e303f01f873bb99e7770", "parents": [ "1f02f1a7b3073b8fef07770a67fbf94afad317f0" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Apr 21 16:50:40 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 12:44:27 2015 -0700" }, "message": "Replace NULL with nullptr\n\nAlso fixed some lines that were too long, and a few other minor\ndetails.\n\nChange-Id: I6efba5fb6e03eb5d0a300fddb2a75bf8e2f175cb\n" }, { "commit": "c785344b87221f5e4e6473e5b762e4e61fe65dcf", "tree": "cd32ad2c2604596a18926f04d4c313dab255ecfd", "parents": [ "a29d93b380c9aeb8270e281aefbdd0c77a430d43" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Mar 27 14:35:38 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 10 12:57:27 2015 -0700" }, "message": "Move ArtField to native\n\nAdd linear alloc. Moved ArtField to be native object. Changed image\nwriter to put ArtFields after the mirror section.\n\nSavings:\n2MB on low ram devices\n4MB on normal devices\n\nTotal PSS measurements before (normal N5, 95s after shell start):\nImage size: 7729152 bytes\n23112 kB: .NonMoving\n23212 kB: .NonMoving\n22868 kB: .NonMoving\n23072 kB: .NonMoving\n22836 kB: .NonMoving\n19618 kB: .Zygote\n19850 kB: .Zygote\n19623 kB: .Zygote\n19924 kB: .Zygote\n19612 kB: .Zygote\nAvg: 42745.4 kB\n\nAfter:\nImage size: 7462912 bytes\n17440 kB: .NonMoving\n16776 kB: .NonMoving\n16804 kB: .NonMoving\n17812 kB: .NonMoving\n16820 kB: .NonMoving\n18788 kB: .Zygote\n18856 kB: .Zygote\n19064 kB: .Zygote\n18841 kB: .Zygote\n18629 kB: .Zygote\n3499 kB: .LinearAlloc\n3408 kB: .LinearAlloc\n3424 kB: .LinearAlloc\n3600 kB: .LinearAlloc\n3436 kB: .LinearAlloc\nAvg: 39439.4 kB\n\nNo reflection performance changes.\n\nBug: 19264997\nBug: 17643507\n\nChange-Id: I10c73a37913332080aeb978c7c94713bdfe4fe1c\n" }, { "commit": "07b3c2351bb527ea91c084dc19434600af9ae66b", "tree": "b286122b660ce16abae1e56c57b957b8c69c9531", "parents": [ "dcff612c3a6e1427749771c4559f198fa480f709" ], "author": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Tue Mar 31 15:57:54 2015 -0700" }, "committer": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Thu Apr 02 09:47:03 2015 -0700" }, "message": "Store OatDexFile instead of OatFile in DexFile.\n\nThis requires moving OatDexFile out of the OatFile class so that\na forward class declaration can be used for OatDexFile.\n\nBug: 19071355\nChange-Id: Ibda85b78d0577e9e81073090616fc0f2fa526be3\n" }, { "commit": "0b3ed3db963b80603c4e3d8e5df1f72b9327b24d", "tree": "04177d83f2dcf2ae9e84b33f66b536bac036f200", "parents": [ "18cd8711d0059415dc8a99470fc105279a9ce0c6" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Mar 04 15:38:51 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Mar 04 15:38:51 2015 -0800" }, "message": "ART: Report zip opening error\n\nDo not swallow the error message from ZipArchive::OpenFromFd.\n\nBug: 19574093\nChange-Id: I14ea771e63b7bc3e8051012a841e66b9e894aa3a\n" }, { "commit": "ab1eb0d1d047e3478ebb891e5259d2f1d1dd78bd", "tree": "a2d211ec81294adab2981d0179c8f04be3e8c8c4", "parents": [ "6e27f82193a8f54cd8ecdc8fb2c4c1adadafbaf4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Feb 13 19:23:55 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Feb 18 16:50:22 2015 -0800" }, "message": "ART: Templatize IsInt \u0026 IsUint\n\nEnsure that things are used correctly.\n\nChange-Id: I76f082b32dcee28bbfb4c519daa401ac595873b3\n" }, { "commit": "5a3399deaf448c8434d9ba0916ff799b1b791d95", "tree": "b6075870aaee70635b3d7e3714072808e042bad7", "parents": [ "aa874e0bbfb21aec0661b93cb1c2ce065bc20302", "a48aef4234768ed37828df613919391c21f561a7" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 11 10:06:21 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Feb 11 10:06:21 2015 +0000" }, "message": "Merge \"Fix ImageWriter::ComputeEagerResolvedStringsCallback().\"" }, { "commit": "a48aef4234768ed37828df613919391c21f561a7", "tree": "eb77858bc679512245336ec3ca96c8be59f6f34b", "parents": [ "ccce1c02b162b667139eaef563460b1e1f330891" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Dec 03 17:53:53 2014 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Feb 11 10:02:55 2015 +0000" }, "message": "Fix ImageWriter::ComputeEagerResolvedStringsCallback().\n\nChange-Id: I1a2abd6d78dd7067d9bdbadbd81dd2fd7711fbc5\n" }, { "commit": "fd9eb3923dcf417afcf5ed4ebb13867fd10f2de3", "tree": "8b44d56da6cd06372d962396fa905d3dedaf3907", "parents": [ "4db3aeb88001367a032df33e5801c9add6a14b06" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Nov 06 16:52:58 2014 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jan 14 16:07:43 2015 -0800" }, "message": "ART: Simple structural class check\n\nAdds a simple check to class-loading when the embedded dex file in\nan oat file and the dex file on the class path where we found the\nclass do not match.\n\nWe require that the number of methods and fields do not change, as\nthat will almost certainly mean that quickened and other compiled\noffsets are wrong now. This is a reasonably lightweight change, but\nwe should investigate a full comparison including name and type of\nmembers.\n\nBug: 17937814\nBug: 18708951\n\n(cherry picked from commit 15a33b3f88546bce85dcb9d28caf200da51154d7)\n\nChange-Id: Icb9638bebd369ab23822817f4a97c8dd8625fea5\n" }, { "commit": "fbef44de596d298dc6430f482dffc933a046dd28", "tree": "57345e86b7dda80b82a263069230b7e312db5ef2", "parents": [ "603104b5b5c3759b0bc2733bda2f972686a775a3" ], "author": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Tue Dec 23 09:48:51 2014 -0800" }, "committer": { "name": "Richard Uhler", "email": "ruhler@google.com", "time": "Tue Jan 13 16:32:34 2015 -0800" }, "message": "Use unique_ptr to track ownership of dex files.\n\nBug: 18809837\nChange-Id: Ie571eae8fc19ee9207390cff5c7e2a38071b126a\n" }, { "commit": "e7c9a8c2b8481aafbc6af4ce6229bd361ba24742", "tree": "f6d8fe8fd7aeae117a6547dc4f012cd4085cb4e8", "parents": [ "00b2da5c02339c36ffa4006f731f55203b09265d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Nov 06 16:35:45 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Nov 07 11:45:06 2014 -0800" }, "message": "Add hash map, reduce excessive hashing\n\nChanged the class def index to use a HashMap instead of unordered_map\nso that we can use FindWithHash to reduce how often we need to compute\nhashes.\n\nFixed a bug in ClassLinker::UpdateClass where we didn\u0027t properly\nhandle classes with the same descriptor but different class loaders.\nIntroduced by previous CL.\n\nBefore (fb launch):\n1.74% art::ComputeModifiedUtf8Hash(char const*)\n\nAfter:\n0.95% art::ComputeModifiedUtf8Hash(char const*)\n\nBug: 18054905\nBug: 16828525\n\nChange-Id: Iba2ee37c9837289e0ea187800ba4af322225a994\n\n(cherry picked from commit 564ff985184737977aa26c485d0c1a413e530705)\n" }, { "commit": "277ccbd200ea43590dfc06a93ae184a765327ad0", "tree": "d89712e93da5fb2748989353c9ee071102cf3f33", "parents": [ "ad17d41841ba1fb177fb0bf175ec0e9f5e1412b3" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 03 21:36:10 2014 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Nov 04 18:40:08 2014 -0800" }, "message": "ART: More warnings\n\nEnable -Wno-conversion-null, -Wredundant-decls and -Wshadow in general,\nand -Wunused-but-set-parameter for GCC builds.\n\nChange-Id: I81bbdd762213444673c65d85edae594a523836e5\n" }, { "commit": "2c4257be8191c5eefde744e8965fcefc80a0a97d", "tree": "9db3e1f1c60f2df29638ba3ce9d5d5bb8b26ca2c", "parents": [ "98c271d517bc4d25fc6879b4b8e35ea93885d9e2" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 24 14:20:06 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 24 14:23:42 2014 -0700" }, "message": "Tidy logging code not using UNIMPLEMENTED.\n\nChange-Id: I7a79c1671a6ff8b2040887133b3e0925ef9a3cfe\n" }, { "commit": "c7dd295a4e0cc1d15c0c96088e55a85389bade74", "tree": "0c08a2236bc9ba5d9a4dc75d4dd0ed2d76f8f1c6", "parents": [ "94e5af8602150efa95bde35cc9be9891ddf30135" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 21 23:31:19 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 22 12:06:23 2014 -0700" }, "message": "Tidy up logging.\n\nMove gVerboseMethods to CompilerOptions. Now \"--verbose-methods\u003d\" option to\ndex2oat rather than runtime argument \"-verbose-methods:\".\nMove ToStr and Dumpable out of logging.h, move LogMessageData into logging.cc\nexcept for a forward declaration.\nRemove ConstDumpable as Dump methods are all const (and make this so if not\ncurrently true).\nMake LogSeverity an enum and improve compile time assertions and type checking.\nRemove log_severity.h that\u0027s only used in logging.h.\nWith system headers gone from logging.h, go add to .cc files missing system\nheader includes.\nAlso, make operator new in ValueObject private for compile time instantiation\nchecking.\n\nChange-Id: I3228f614500ccc9b14b49c72b9821c8b0db3d641\n" }, { "commit": "13735955f39b3b304c37d2b2840663c131262c18", "tree": "0a731ac42b8230f9929172fa3e3d8051874e2b18", "parents": [ "25b18bbdaa36ff936eb44f228f0518d4223e9d52" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 08 12:43:28 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 16:05:58 2014 -0700" }, "message": "stdint types all the way!\n\nChange-Id: I4e4ef3a2002fc59ebd9097087f150eaf3f2a7e08\n" }, { "commit": "ecaebd37bc9af576ddfe1be0bd4f3889e88d6f23", "tree": "f461acc73a65791f5a1a5c0d2b7d42b35add9c5b", "parents": [ "9b43d91b21c83cb5fb3f618e3f8053fd7c39c906" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Sep 12 23:10:21 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Sep 12 23:10:21 2014 -0700" }, "message": "Remove dex file index building mutex.\n\nFirst thread to get to max misses builds the index and sets an atomic, other\nthreads continue and return null. Avoids lock contention and removes a member\nvariable.\n\nChange-Id: Ia91e4a8fd915941aea849f019c85b67894ec6e71\n" }, { "commit": "b9620f305c79914f5159cf9279a7ccd173af1186", "tree": "edfb15d759f16808d575cb849302fc2fd22d6709", "parents": [ "575a5649715ee50e0de8a107e8a5379d4c465382", "928f72bd75c385ba2708c58521171a77264d4486" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Sep 10 23:43:32 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Sep 10 23:43:32 2014 +0000" }, "message": "Merge \"ART: Fix things for valgrind\"" }, { "commit": "928f72bd75c385ba2708c58521171a77264d4486", "tree": "86f7fa7a21e3f6d21c9cab2d4fffe4aaa42dc458", "parents": [ "dab9ed52f2df7189b81ccf3237b030ff638a492a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 09 19:53:48 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Sep 10 15:50:42 2014 -0700" }, "message": "ART: Fix things for valgrind\n\nWire up valgrind gtests. Add valgrind-test-art-host, currently\nonly depending on valgrind-test-art-host-gtest32.\n\nFix an Alloc setting to allow running valgrind.\n\nRefactor the fault handler to manage (and correctly release) the\nhandlers.\n\nFix minor failure-case leaks exposed by tests.\n\nFailing tests:\n\nThe optimizing compiler is leaking non-arena-ed structures\n(e.g., assembler buffers), as code generators are not destroyed.\nThe solution has been moved to a follow-up CL.\n\nNote: All 64b tests are failing as we cannot allocate a heap.\n\nChange-Id: I7f854cfd098d9f68107ce492363e7dba9a82b9fa\n" }, { "commit": "aa4497db59f1eeec954f2ba5da6d458fcdf9b3a4", "tree": "1efe5c6c5fe93ae73b6ad02db1a4ac368c71c00e", "parents": [ "6724a9531c92368491dd17937d0087f73a7c0642" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 05 14:01:17 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Sep 10 16:02:14 2014 +0100" }, "message": "Improve dex location canonicalization-related performance.\n\nEagerly add canonical dex file locations to the OatFile\u0027s\nprimary lookup map in Setup(). This moves the boot.oat work\nfrom every app startup to the zygote initialization. Since\nwe always ended up initializing the canonical location map\nanyway due to the way that we\u0027re loading dex files, the lazy\ninitialization didn\u0027t save anything.\n\nClean up dex file name canonicalization to make sure we\nfree() the memory returned by realpath() rather than using\nstd::unique_ptr\u003c\u003e with the default deleter.\n\nAvoid some unnecessary duplicate OatDexFile lookups.\n\nBug: 16828525\nBug: 17346103\nChange-Id: Id8fbc8992f62996138eb2006a0046c6529747c09\n" }, { "commit": "68b56858367e29461ae290fd797443a1ef6d8005", "tree": "c086a68bcbbb50d7c3b829bcc7d4231ff815db4a", "parents": [ "7c7686e62d262f1823876cebb3700e20f967dd56" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Aug 29 20:19:11 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Sep 02 08:01:04 2014 -0700" }, "message": "Reduce and speed-up class def searches.\n\nUse the class linker for descriptor lookups from the compile driver so that\ndex caches are populated.\nReduce the scope of functions for scanning class paths to just the class\nlinker where they are performed.\nIf we see more than a threshold number of find class def misses on a dex file\nlazily compute an index, so that future lookups are constant time (part of the\ncollection code is taken from\nhttps://android-review.googlesource.com/#/c/103865/3). Note that we take a lazy\napproach so that we don\u0027t serialize on loading dex files, this avoids the\nreason the index was removed in 8b2c0b9abc3f520495f4387ea040132ba85cae69.\nRemove an implicit and unnecessary std::string creation for PrintableString.\n\nSingle threaded interpret-only dex2oat performance is improved by roughly 10%.\n\nBug: 16853450\n\nChange-Id: Icf72df76b0a4328f2a24075e81f4ff267b9401f4\n" }, { "commit": "67ef46adfb2c4990832e23aebeb9c0582d8519c4", "tree": "07f95a8a9c022c547842d43fedd155cbaff4df33", "parents": [ "ed9fe4ffd9516cfb01b53a3a2c81d3e223ee352c" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Aug 21 15:59:43 2014 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Aug 21 16:00:27 2014 -0700" }, "message": "Avoid handle-less fields in ClassLinker::InitializeClass()\n\nThere were some handle-less fields in a SafeMap across GC points.\n\nBug: 12687968\nChange-Id: Ib8c6527d4e23031f1d0074fa11d8f85499b68340\n" }, { "commit": "c991107a90698012c2b0babc030b8ab85491d1e3", "tree": "a66f5809f26ff571336c5de64da4a50555d09ac6", "parents": [ "7493a5be0b05a27de80974a8a0bbaff07bf9909f" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Aug 08 21:29:02 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Aug 08 22:09:28 2014 -0700" }, "message": "Remove clang detected warning on tautological comparison\n\nBug: 16903117\nChange-Id: I5847ef5a6091c01e14b9ffcd8f6d12cabfa8b63b\n" }, { "commit": "4e1d579d6401fef2dd57b16f8d406e33221a69d9", "tree": "2abc27cbd45cc8a4775df928280c63c1606cff75", "parents": [ "89c210bf418a152ccabfbbf853ddcce33aea450d" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Tue Jul 15 23:56:47 2014 +0100" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Mon Aug 04 17:52:05 2014 +0100" }, "message": "Use canonical paths when searching for dex files\n\nApps which use the DexPathClassLoader directly may\npass symlinks when trying to load dex files. This\nwill not work as we use string comparision to find\nthe dex in an oat file. The CL fixes this issue by\nusing using dex conical paths for comparisons.\n\nBug: 15313272\n\nChange-Id: Ic314374b17612c3afbcadec93a88b2515a0aca5e\n" }, { "commit": "bacce5c9324497c77f8196069e692077a2e57591", "tree": "897fb2610d6cccaf71027e54bd72dba715f481c0", "parents": [ "b1cbf58b4d0bfe58d1ac8716c1d5c4c646983478" ], "author": { "name": "Alexander Ivchenko", "email": "alexander.ivchenko@intel.com", "time": "Thu Jun 26 16:32:11 2014 +0400" }, "committer": { "name": "Alexander Ivchenko", "email": "alexander.ivchenko@intel.com", "time": "Thu Jun 26 16:34:45 2014 +0400" }, "message": "Return \"false\" instead of \"nullptr\" in bool DexFile::Open.\n\nOtherwise on 4.10 the following error appears:\n\"error: converting to \u0027bool\u0027 from \u0027std::nullptr_t\u0027 requires direct-initialization [-fpermissive]\"\nHere is the detailed description with links:\nhttps://gcc.gnu.org/bugzilla/show_bug.cgi?id\u003d52174\n\nChange-Id: I77c6571b557dde55f4e19a79db7bf997303ede3b\nSigned-off-by: Alexander Ivchenko \u003calexander.ivchenko@intel.com\u003e\n" }, { "commit": "833a48501d560c9fa7fc78ef619888138c2d374f", "tree": "add308298a5486d44caddea120cc9200dd70c38a", "parents": [ "b849f6dd638fd1246724160cd5c01ab1a5ff33bd" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed May 21 18:46:59 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 25 19:34:58 2014 -0700" }, "message": "ART: Native support for multidex\n\nNative support for zip files with multiple classesX.dex.\n\nWorks by explicitly looking for those files in ascending order. As\nthese files have no file system representation for themselves,\nintroduce synthetic dex locations: the name of the originating file\nplus a colon plus the name of the dex file, e.g., test.jar:classes2.dex.\n\nOpening a zip dex file will return all dex files in this way. This\nkeeps the changes to dex2oat minimal.\n\nTo hide multidex/synthetic names from the Java layer, let the handle\nof dalvik.system.DexFile refer to a vector of DexFile objects. When\nopening a location, test possible synthetic names and add them to the\nvector. Thus, the original multidex jar in the classpath will be\nassociated with all embedded dex files.\n\nChange-Id: I0de107e1369cbc94416c544aca3b17525c9eac8b\n" }, { "commit": "4fdbba09cdb16840c675608730bcd056ef27cfd6", "tree": "9a7b09d270fe118b96821f7e2b872cf42d317c55", "parents": [ "a81fe3d512ef846aad3398222afd1f0588bbaf7c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 19 20:24:22 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 19 20:24:22 2014 -0700" }, "message": "ART: Do not try to pretty-print method in dex file\n\nThe code is called in verification when it is not clear yet whether\nstructures are valid. Simplify warning message.\n\nBug: 15754233\nChange-Id: I85858a71143f41444ce9f00536d901c268afe547\n" }, { "commit": "d6cec905b62dff38285b350e67d5e223346e6d66", "tree": "96e32830904eae3e64e6ad2798732f7870653180", "parents": [ "30395fc865599619f22df7c99d7553b18304622c" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun May 25 16:08:51 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun May 25 22:25:22 2014 -0700" }, "message": "Make DexFile content read only before running DexFileVerifier\n\n(cherry picked from commit 98725444129efe2b1b51c87740860caadaef2bf7)\n\nChange-Id: I18a932b2f19d44b820232f4d4e9c358df20ab364\n" }, { "commit": "0aa504b5bb19f0944d50941b20c8eeaca4165328", "tree": "8ec437c95a023c649dc96d9e88e2403d93c77392", "parents": [ "52925af1e3136bcf68bc5754fbfa407950235217" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri May 23 02:47:28 2014 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri May 23 13:52:01 2014 -0700" }, "message": "Fix ExtractToMemMap to show original zip file name in ashmem\n\n(cherry picked from commit 1fca8e91f32dc8b13d3129b7ef4a0194839736e6)\n\nChange-Id: Id7d22600496b090ac32150c8c6424da89964b6be\n" }, { "commit": "700a402244a1a423da4f3ba8032459f4b65fa18f", "tree": "4c22fcda04d271bd55a37aff30650214af17a90c", "parents": [ "047c11adcbcbc0bcf210defdfcbada763961ffee" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon May 19 16:49:03 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon May 19 22:27:39 2014 -0700" }, "message": "Now we have a proper C++ library, use std::unique_ptr.\n\nAlso remove the Android.libcxx.mk and other bits of stlport compatibility\nmechanics.\n\nChange-Id: Icdf7188ba3c79cdf5617672c1cfd0a68ae596a61\n" }, { "commit": "507dfdd147c97bfbadebfd63584d094b6a4e7b47", "tree": "cce43931b6dcd088cb2932c2491f86116353a27f", "parents": [ "922ddb30982d2597eab634d8b8598bec0eb7d3b7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 15 16:42:40 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu May 15 16:50:51 2014 -0700" }, "message": "Compatibility layer to transition from UniquePtr to std::unique_ptr.\n\nUse ART_WITH_STLPORT (enabled for the target) to cause the use of UniquePtr,\nfor the host switch to std::unique_ptr. For now the type remains called\nUniquePtr.\nMake dalvik compile with clang on the host, move its build to C++11.\n\nChange-Id: I5ba8d2757904bc089ed62047ea03de3c0853fb12\n" }, { "commit": "eb8167a4f4d27fce0530f6724ab8032610cd146b", "tree": "bcfeaf13ad78f2dd68466bbd0e20c71944f7e854", "parents": [ "6fb66a2bc4e1c0b7931101153e58714991237af7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 07 15:43:14 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue May 13 14:45:54 2014 -0700" }, "message": "Add Handle/HandleScope and delete SirtRef.\n\nDelete SirtRef and replaced it with Handle. Handles are value types\nwhich wrap around StackReference*.\n\nRenamed StackIndirectReferenceTable to HandleScope.\n\nAdded a scoped handle wrapper which wraps around an Object** and\nrestores it in its destructor.\n\nRenamed Handle::get -\u003e Get.\n\nBug: 8473721\n\nChange-Id: Idbfebd4f35af629f0f43931b7c5184b334822c7a\n" }, { "commit": "b0fa5dc7769c1e054032f39de0a3f6d6dd06f8cf", "tree": "839d13ebfa7170967dd9b4abd434b7abda53da99", "parents": [ "948740c1938860df055ddc801f20fd1707331e38" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Apr 28 16:47:08 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Apr 29 14:36:28 2014 -0700" }, "message": "Force inlining on trivial accessors.\n\nMake volatility for GetFieldObject a template parameter.\nMove some trivial mirror::String routines to a -inl.h.\n\nBug: 14285442\n\nChange-Id: Ie23b11d4f18cb15a62c3bbb42837a8aaf6b68f92\n" }, { "commit": "cb19ebf7609f74b223bd86c94f721498795f9bba", "tree": "0db31e4f83046b3d8159d4ba28d9298ff94e06c3", "parents": [ "f17ce4c5e3b6882aa8849d1ed82df4238c436da2" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Mar 11 15:26:35 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Mar 17 09:15:31 2014 +0100" }, "message": "Fix debugger crash in native method frames.\n\nThe main crash happens when we try to read (StackFrame::GetValues) or write\n(StackFrame::SetValues) values in native frames. We use the method\u0027s vmap to\nknow where Dalvik registers live but native methods don\u0027t have vmap. The fix\nis to reply with the OPAQUE_FRAME error which indicates local values are not\naccessible in the frame.\n\nWe prevent from dereferencing null code item which causes some crashes too.\nThis happens when we compute the line table (Method::LineTable) and variable\ntable (Method::VariableTable) of methods without code: native, proxy and\nabstract methods. We do not expect to encounter abstract methods though. We\ntake care of these kinds of method when mangling/demangling local value slots.\n\nWe also fix the location\u0027s pc of native and proxy frames where it must be -1\n(as 8-byte value). We\u0027ll use this property to detect such frames in the JDWP\ntests.\n\nBug: 13366758\nChange-Id: I78e3263fbf2681b5573571c846390d52b9193849\n" } ], "next": "f79fccbf8b5e59a0f48bec754cb7a53877b3c90f" }