)]}' { "log": [ { "commit": "bc5a795c0d486c84913d987cad5846ded840cea6", "tree": "90db29a97a21fa15f4cbc5ffbc1f6dd191add49c", "parents": [ "38a4223fcc0493553d9ad324a1dc145869eb663a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 17 15:46:31 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 17 16:16:50 2016 -0700" }, "message": "Move art/native to ObjPtr\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I67eb89cf042c762c6dcd5eb8b008b9a28e9b3319\n" }, { "commit": "28bd2e4f151267b34b8e1eb19c489d8d547bbf5c", "tree": "d99ae3fe74ea63b83091898d830d3efe68cd479d", "parents": [ "6e5fa09510c7280168e040382d27dd8b55760d9a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 04 13:54:57 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 10:59:28 2016 -0700" }, "message": "Move mirror::Class to use ObjPtr\n\nLeave the return types as non ObjPtr for now. Fixed moving GC bugs\nin tests.\n\nTest: test-art-host\n\nBug: 31113334\n\nChange-Id: I5da1b5ac55dfbc5cc97a64be2c870ba9f512d9b0\n" }, { "commit": "c4f3925490a73da8dc74884a1deb965d4ecaf14e", "tree": "7fe566827f8ab903af1acb5697c86a6efe3187b3", "parents": [ "18f7de841fee3ca6f0c04e7caa57c3ce76b36231" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 18:32:08 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 06 13:55:58 2016 -0700" }, "message": "Move remaining jobject related functions to use ObjPtr\n\nAlso added ObjPtr::DownCast.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I59c253211dc435579ffdfd49f856861ab13d262c\n" }, { "commit": "f8ac97f94aad9ce2d324c2c7c21dde3abf13315b", "tree": "9359680f07a65531898113dd335020db33b9b6e6", "parents": [ "299958fc681f591fee7ff9ae0d81bd3adfebdd2b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 15:56:52 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Oct 05 15:59:57 2016 -0700" }, "message": "ObjPtr misc cleanup\n\nCheck for subtypes in constructor.\n\nRemove various calls to MakeObjPtr and ObjPtr::Ptr.\n\nBug: 31113334\n\nTest: test-art-host\nChange-Id: I8d680d514ea52cd64fc4f7c2e988926bc726174e\n" }, { "commit": "1cc62e4ea24828fdb3f3da0b8603f0b107d09a04", "tree": "718e322f3d05ac095770d4dc2a68b824bf3974a7", "parents": [ "82d4838d6bb3480cd25327cedc5179fb2d86881c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 03 18:01:28 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Oct 04 10:46:16 2016 -0700" }, "message": "Rename ObjPtr::Decode to ObjPtr::Ptr\n\nDone to prevent ambiguity with ScopedObjectAccess::Decode.\n\nBug: 31113334\n\nTest: test-art-host\nChange-Id: I07a2497cc9cf66386311798933547471987fc316\n" }, { "commit": "b890130a66e167404a9a60cf0893a015538778ca", "tree": "e50899e754132f3281c932c31203f8c34c8ca85f", "parents": [ "3398c7874e002beaa6c2b2fadf183e7d1ddad23a" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 30 10:27:43 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 03 16:00:55 2016 -0700" }, "message": "Use ObjPtr for ResolvedFieldAccessTest and ResolvedMethodAccessTest\n\nAlso added LookupResolvedType that is guaranteed to not do thread\nsuspension but deals with multidex since GetResolvedType will\nreturn null if the type was resolved in another dex file.\n\nAdded test.\n\nBug: 31113334\n\nTest: test-art-host CC baker\n\nChange-Id: I50493bca7d8ce9760546c3116b717484c62c47a4\n" }, { "commit": "3398c7874e002beaa6c2b2fadf183e7d1ddad23a", "tree": "7db754b5b0c3a6c68bc1f91d7cb2477f534d2ad3", "parents": [ "462e21cc8d28a4d48342cf0f634bd1ecc63647d7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 30 10:27:43 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 03 14:06:27 2016 -0700" }, "message": "Move ArtField to ObjPtr\n\nAdded EXPECT_OBJ_PTR_EQ and variants to gtests.\n\nFixed moving GC bugs in:\nClassLinker::CreatePathClassLoader\nClassLinkerTest: StaticFields\n\nObjPtr Decode call sites: 186 -\u003e 181.\n\nSome tests fail due to ResolvedFieldAccessTest, will fix in follow\nup CL.\n\nBug: 31113334\n\nTest: test-art-host CC baker\n\nChange-Id: I8b266ad00f3c20c8cbe7cfdf280d175083df0b88\n" }, { "commit": "42b3dd0f94d2da0ec5f42c4cb57eef71dde02c79", "tree": "9c39a95d1eeac50379821113888ea78541f0ebc6", "parents": [ "ebeee75e46d173d3803ed7740800c508c9c23580" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Oct 03 11:33:01 2016 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Oct 03 11:33:29 2016 +0100" }, "message": "Address review comments for change 25352fc06c84cdab8a2ab.\n\nTest: make test-art-host\nBug: 30550796\n\nChange-Id: Ic689e4c9bee691f03703288c3b12634a26841140\n" }, { "commit": "25352fc06c84cdab8a2ab6d173b0514066ade2b9", "tree": "bb8a6be0c9a144ccf33b07b3158409769a32daa2", "parents": [ "51cbe84ca33800c57fef9c96f1155f60bcc70bba" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Wed Aug 03 12:46:58 2016 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Oct 03 09:48:50 2016 +0100" }, "message": "class_linker: Add support for resolving method types.\n\n- Add a new fixed size dex cache array for resolved method types.\n The size of this array is set to 1024.\n- Also introduces a new runtime flag that controls this feature.\n\nTest: make test-art-host\nBug: 30550796\n\nChange-Id: I147b33398d71ee21f2e91b418d3700d4630801ff\n" }, { "commit": "0795f23920ee9aabf28e45c63cd592dcccf00216", "tree": "ff3f880c5e84f3316532b47d0e9a7729ade848ac", "parents": [ "d1224dce59eb0019507e41da5e10f12dda66bee4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 27 18:43:30 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Sep 29 17:31:09 2016 -0700" }, "message": "Clean up ScopedThreadStateChange to use ObjPtr\n\nAlso fixed inclusion of -inl.h files in .h files by adding\nscoped_object_access-inl.h and scoped_fast_natvie_object_access-inl.h\n\nChanged AddLocalReference / Decode to use ObjPtr.\n\nChanged libartbenchmark to be debug to avoid linkage errors.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I4d2e160483a29d21e1e0e440585ed328b9811483\n" }, { "commit": "afa48270228832ecd7df4506915354d1a41f374a", "tree": "7d63c5bde346eb129f3159b07b2a66dfa86aeb41", "parents": [ "72795d0a3368a7f921ce2f470917131cd946b717" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Wed Aug 03 12:46:58 2016 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Sep 29 13:15:32 2016 +0100" }, "message": "java.lang.invoke: Add mirror types for MethodHandle / MethodType.\n\nBug: 30550796\nTest: make test-art-host\nChange-Id: I096160464bc6e84f7e5ad021306a7e462cf3b0c5\n" }, { "commit": "ce092d0e374107fc7416416848d3f232233f84ee", "tree": "70a60a9c54cc8c4b9b0282f511910411be73efd4", "parents": [ "85bf9def7f1711fa402cd1e5bf02a6566870f9a7", "733ddb2af025aca54582fef95fb714e0df422c78" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Wed Sep 21 16:53:26 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Sep 21 16:53:26 2016 +0000" }, "message": "Merge \"Refactor Dex open methods\"" }, { "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": "0e8443995af602bcc2e70dcef1d20ee48acec934", "tree": "2d81b8db23bea139e40e33e4e9b8c69027fb1e56", "parents": [ "c4cb3d532db8a500c1a15e8a0052895960259209" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Thu Sep 08 13:43:31 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Wed Sep 21 11:42:21 2016 +0100" }, "message": "Collapse ART\u0027s AbstractMethod class into Executable\n\nAfter the OpenJDK 8 pull there is now a common base class\nfor Method and Constructor that can be used instead.\n\nMethods that are abstract on Executable but on Android\nwere implemented in AbstractMethod are now consistently\nrenamed to \"{method}Internal\", declared final and\ngiven package-protected access.\n\nFor consistency with all other private, natively\nimplemented methods, the method getDeclaredAnnotations()\nhas been renamed getDeclaredAnnotationsNative().\n\nBug: 28666126\nBug: 31052885\nTest: make test-art-host and cts run of CtsLibcoreTestCases\nChange-Id: Ibfe6af6a47fe0cfffd0859ffbe58980aeb84e053\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": "93bbee0c1c97bd9a5336ed0c9e41cd17613df585", "tree": "7f2fe8d9ec240b34e7ee153b95e69b821554ab7e", "parents": [ "1c29876b303ab392e615b2294db009a971613bea" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 31 09:38:40 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 31 18:32:33 2016 -0700" }, "message": "Faster allocation fast path\n\nAdded a new object size field to class, this field contains the\naligned object size if the object is not finalizable and is\ninitialized. If the object is finalizable or uninitialized the field\nis set to some large value that forces the ASM allocators to go slow\npath.\n\nOnly implemented for region/normal TLAB for now, will add the to\nRosAlloc stubs soon.\n\nCC N6P MemAllocTest: 1067 -\u003e 1039 (25 samples)\nCC N6P EAAC: 1281 -\u003e 1260 (25 samples)\n\nRAM overhead technically 0 since mirror::Class was not 8 byte aligned\npreviously. Since the allocators require 8 byte allignment, there\nwould have been 1 word of padding at the end of the class. If there\nwas actually 4 extra bytes per class, the system overhead would be\n36000 * 4 \u003d 120KB based on old N6P numbers for the number of loaded\nclasses after boot.\n\nBug: 9986565\n\nTest: test-art-host CC baker, N6P phone boot and EAAC runs.\n\nChange-Id: I119a87b8cc6c980bff980a0c62f42610dab5e531\n" }, { "commit": "bdf7f1c3ab65ccb70f62db5ab31dba060632d458", "tree": "25cc77adfeb05232d0ab00aa561a693f1d71745c", "parents": [ "d7eabc2cc1a88c1f7f927da61246ae65aab0626c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 30 16:38:47 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Aug 30 17:02:53 2016 -0700" }, "message": "ART: SHARED_REQUIRES to REQUIRES_SHARED\n\nThis coincides with the actual attribute name and upstream usage.\nPreparation for deferring to libbase.\n\nTest: m\nTest: m test-art-host\nChange-Id: Ia8986b5dfd926ba772bf00b0a35eaf83596d8518\n" }, { "commit": "16b21cd4874be8550f43987b0bd2e4ba8ba3ca42", "tree": "475aeec7476e039442009ca36c1ddde1c9cdbd47", "parents": [ "ca11dc008457e1596554eb9b1b77c823ae9dcf54" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Fri Aug 12 09:37:02 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Fri Aug 26 14:31:26 2016 +0100" }, "message": "Tracking java.lang.reflect.Executable changes\n\nAdded a method to Class: total vtable entries increases\nby one.\n\nHandling knock-ons of introducing Executable into the\nclass hierarchy above AbstractMethod.\n\nRearranging java_lang_reflect_*.cc code to track libcore\nchanges that reduce duplication between Constructor / Method\n/ Executable and AbstractMethod and track the OpenJDK 8\nmore closely.\n\nBug: 28666126\nChange-Id: I6b5c476a14b8ea25556c35448431de744519b6c2\n" }, { "commit": "3d0aa88808052436981f589b7af4f1706c365187", "tree": "6a6d232b20d5b275dac7ee849ba2fc81d9256afd", "parents": [ "95afcbcc25bb17c9c34e8b7708013b612a3bb581" ], "author": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Tue Aug 16 19:53:07 2016 +0100" }, "committer": { "name": "Neil Fuller", "email": "nfuller@google.com", "time": "Wed Aug 17 10:27:36 2016 +0100" }, "message": "Adjustments for changes made to java.lang.Class\n\nFix up the runtime after removal of the annotationType field\nfrom java.lang.Class.\n\nTest: make test-art-host / boot a device\nChange-Id: I5f37567d2b5ec381b35e66fef29287abe307b840\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": "a62cb9bb6cb2278cb41ab0664191623e178c6a4f", "tree": "62263dac644e2d80a34a04f2649e4741a1bed6f4", "parents": [ "bb8d501c9bb882a8927c6ceda07bf9577e06c3e1" ], "author": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Thu Jun 30 09:18:25 2016 +0000" }, "committer": { "name": "Artem Udovichenko", "email": "artem.u@samsung.com", "time": "Thu Jul 07 10:01:04 2016 +0000" }, "message": "Revert \"Revert \"Optimize IMT\"\"\n\nThis reverts commit 88f288e3564d79d87c0cd8bb831ec5a791ba4861.\n\nChange-Id: I49605d53692cbec1e2622e23ff2893fc51ed4115\n" }, { "commit": "88f288e3564d79d87c0cd8bb831ec5a791ba4861", "tree": "dd051a7b2985d1af3fea91ad6000de4bdc701a19", "parents": [ "e36389f20c83083c0aaba2dcac0888951e55cae1" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:17:52 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jun 29 08:41:25 2016 +0000" }, "message": "Revert \"Optimize IMT\"\n\nBug: 29188168 (for initial CL)\nBug: 29778499 (reason for revert)\n\nThis reverts commit badee9820fcf5dca5f8c46c3215ae1779ee7736e.\n\nChange-Id: I32b8463122c3521e233c34ca95c96a5078e88848\n" }, { "commit": "badee9820fcf5dca5f8c46c3215ae1779ee7736e", "tree": "982948a67a88a1f9a734c935f919f8d307969f48", "parents": [ "614968198625a6693666bdc1e5609e2f663f5638" ], "author": { "name": "Nelli Kim", "email": "nelli.kim@samsung.com", "time": "Fri May 13 13:08:53 2016 +0300" }, "committer": { "name": "Matthew Gharrity", "email": "gharrma@google.com", "time": "Tue Jun 14 10:05:47 2016 -0700" }, "message": "Optimize IMT\n\n* Remove IMT for classes which do not implement interfaces\n* Remove IMT for array classes\n* Share same IMT\n\nSaved memory (measured on hammerhead):\nboot.art:\nTotal number of classes: 3854\nNumber of affected classes: 1637\nSaved memory: 409kB\n\nChrome (excluding classes in boot.art):\nTotal number of classes: 2409\nNumber of affected classes: 1259\nSaved memory: 314kB\n\nGoogle Maps (excluding classes in boot.art):\nTotal number of classes: 6988\nNumber of affected classes: 2574\nSaved memory: 643kB\n\nPerformance regression on benchmarks/InvokeInterface.java benchmark\n(measured timeCall10Interface)\n1st launch: 9.6%\n2nd launch: 6.8%\n\nChange-Id: If07e45390014a6ee8f3c1c4ca095b43046f0871f\n" }, { "commit": "3612149aee482ab7a17da68b0ef5fef3879729a2", "tree": "42180a35266333cbf37163a12185cf8824255a71", "parents": [ "a5502e31cb558c6e257d1dd1889c073a793e3d9d" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Feb 22 13:43:29 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Feb 23 13:24:05 2016 -0800" }, "message": "Make JNI work correctly with default methods.\n\nAlso adds some tests for JNI and DefaultMethods.\n\nBug: 27259142\nBug: 24618811\n\nChange-Id: I31222e3e41059d803be1dbb0f40e1144ac4bf457\n" }, { "commit": "906ae5de68b5b98dc2d4def186df06b2f18a7b78", "tree": "78112454d1f10075925d57f4a68322bc68ba2197", "parents": [ "e33d20e321b630bcab83f6dfbfac2788d8145148" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Feb 22 13:30:35 2016 -0800" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Mon Feb 22 13:43:17 2016 -0800" }, "message": "Fix class_linker_test for valgrind.\n\nFix a memory leak.\n\nBug: 27156726\nChange-Id: I4d9a2dc9cff447b6f2fd02432e018ae49766563a\n" }, { "commit": "fcea56f9cc51957161fe7a6e35e895fd8c4c4a7f", "tree": "b43c197e9370eab38a58e1d54ec319ea45df613d", "parents": [ "aaf56c4c95331d4dd8ac298e6c234d4d58d28308" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Feb 17 11:59:05 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Feb 17 13:02:11 2016 -0800" }, "message": "Fix issue with copied methods not being checked.\n\nIn several places we were using IsMiranda to check if a method is\ncopied. This misses cases involving default methods.\n\nBug: 27216437\nChange-Id: I8c800e3e622a9c0ca0f8752c3d5202f433af9a1c\n" }, { "commit": "867d63b65f653d27dc7ea87e924f47148cec22a7", "tree": "2038bbb25fe121bdcf3653cc32211c6e2cb88936", "parents": [ "6006e2ce92fd86fdf028cd7b3afe972815b0e0f3", "df707e406877e9c0426dd051c00933ebb331673e" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Feb 04 00:34:43 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Feb 04 00:34:43 2016 +0000" }, "message": "Merge \"runtime: Don\u0027t skip verification for -Xverify:soft-fail\"" }, { "commit": "42d511da4bb2f918b4c5f4372f91771ab5b61c83", "tree": "a1eaac50bc6f117140388a53bf9f945434c67f0e", "parents": [ "287fd33a9f03f16ae52a3b290bcabb49bc65d97b" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Feb 03 13:45:41 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Feb 03 13:45:41 2016 -0800" }, "message": "Fix class_linker_test to take into account default methods.\n\nBug: 26963208\nChange-Id: Ie22ce4e7fae3b259b9a58356dbd9662fda6d49e3\n" }, { "commit": "df707e406877e9c0426dd051c00933ebb331673e", "tree": "db9c96c96d373935a83dda2fafbf3c7e1aef3377", "parents": [ "e3d4ff566f9545114bf9ed72421e49eaf9d37953" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Feb 02 16:56:50 2016 -0800" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Feb 03 13:39:19 2016 -0800" }, "message": "runtime: Don\u0027t skip verification for -Xverify:soft-fail\n\nWhen forcing the interpreter into access checks mode,\nmake sure that the regular verification is still run,\ngiving the verifier an opportunity to throw a VerifyError.\n\nIf verification would\u0027ve succeeded (without -Xverify:soft-fail flag),\noverride this and soft-fail, to force the interpreter-with-access-checks to be run\ninstead of the normal faster interpreter.\n\nThis fixes the following run-tests under the interpeter-access-checks:\n* 135\n* 412\n* 471\n* 506\n* 800\n\nBug: 22414682\nChange-Id: I5cb86a8bba71c7af9361a63c0802786c852b857b\n" }, { "commit": "76172164667d565c1d3316935a24b0d9712bb2f6", "tree": "ce6e40574623d297a13cfe92ade6cea31b27ebbc", "parents": [ "227e627c513ced3c6b515b6e5836c6e832c0eeaa" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 26 14:54:06 2016 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 26 17:19:37 2016 -0800" }, "message": "Convert to UTF8 before doing dex file location comparison\n\nNeed to have strings in the same format or else the lengths\nmight not match due to UTF8 conversion.\n\nAdded regression test.\n\nBug: 26799552\n\n(cherry picked from commit 3776db4e51ab61a6e535772417c2adf95920b569)\n\nChange-Id: Ifbc8c38418c9da307a0ff4446d18cea4fb5e71bc\n" }, { "commit": "7b4199a5fa9f151fbf3af2a34f26d04215a1016c", "tree": "111493b5fee26087dcc8abc1a1f56a140d549c8d", "parents": [ "15db4dcfcc17dfe6c41d3c7b26355ccfa2504f4e", "705ad49f353d3f90d8b63625aca2c2035bacdbef" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Jan 14 18:43:36 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jan 14 18:43:36 2016 +0000" }, "message": "Merge \"Support directly invoking interface default methods\"" }, { "commit": "705ad49f353d3f90d8b63625aca2c2035bacdbef", "tree": "ac70af53158a80bc35c057aefae11428281df9ac", "parents": [ "fae1db92d8433d0f75258c190bcf2c940731f036" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Sep 21 11:36:30 2015 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Jan 12 15:40:31 2016 -0800" }, "message": "Support directly invoking interface default methods\n\nWith the Java 8 Language one is allowed to directly call default\ninterface methods of interfaces one (directly) implements through the\nuse of the super keyword. We support this behavior through the\ninvoke-super opcode with the target being an interface.\n\nWe add 3 tests for this behavior.\n\nCurrently only supports slow-path interpreter.\n\nInvoke-super is currently extremely slow.\n\nBug: 24618811\n\nChange-Id: I7e06e17326f7dbae0116bd7dfefca151f0092bd2\n" }, { "commit": "6b1a50b561d3f1a282871f30e85e74433e2984af", "tree": "e592e69ddea871126d566fcaf5b3309a5c2e8643", "parents": [ "28ee8dbece62bf1ed9a7dfd3c581fd5860ed8867", "8a5e8d0118de17210316100e740d0417239256e9" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Dec 21 12:15:39 2015 -0800" }, "committer": { "name": "android-build-merger", "email": "android-build-merger@google.com", "time": "Mon Dec 21 12:15:39 2015 -0800" }, "message": "Merge \"Rename NullHandle to ScopedNullHandle\" am: 76b90e37df\nam: 8a5e8d0118\n\n* commit \u00278a5e8d0118de17210316100e740d0417239256e9\u0027:\n Rename NullHandle to ScopedNullHandle\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": "9c5572957bf357897f0969d002132fc647bfa649", "tree": "84e853d076d2a7449e0eba1710b2a4b3d0872fda", "parents": [ "90f5250d4ba7bcf6066edb022f8325df66d686f3", "9f784b26a13101222c5e5927cafd7f500d3cff67" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 17 15:48:01 2015 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 17 15:48:01 2015 -0800" }, "message": "resolve merge conflicts of 9f784b26a1 to master.\n\nChange-Id: I9ebcb6e3d51be922e08e5f867fb8dcaf5cb9788d\n" }, { "commit": "e64300b8488716056775ecbfa2915dd1b4ce7e08", "tree": "be23a7086c1e369527dda5450a9868f2d29c5669", "parents": [ "bc90a0538e56f98b8e138cb622e6b9d834244ad9" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Dec 15 15:02:47 2015 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Dec 16 14:16:31 2015 -0800" }, "message": "Revert \"Revert \"Combine direct_methods_ and virtual_methods_ fields of mirror::Class\"\"\n\nThis reverts commit ae358c1d5cef227b44d6f4971b79e1ab91aa26eb.\n\nBug: 24618811\n\nChange-Id: I8becf9bae3258450b90cfef5e79589db7c535a4d\n" }, { "commit": "2efb0aa57da168944f99a2d13aed2a426cfa76e7", "tree": "5ca742f9a2a89f9c3c588f6f1863ab55bbdb5c5c", "parents": [ "9539150b85142c18e9e8c2264b5b6100942667c3" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Dec 15 22:15:34 2015 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Dec 15 22:15:34 2015 +0000" }, "message": "Revert \"Combine direct_methods_ and virtual_methods_ fields of mirror::Class\"\n\nThis reverts commit 9539150b85142c18e9e8c2264b5b6100942667c3.\n\nChange-Id: I596876cd643ec0ad524a56621efb6b89e8886230\n" }, { "commit": "ae358c1d5cef227b44d6f4971b79e1ab91aa26eb", "tree": "95c22194af5ea38a4fa5fc8bbe07ba60b6d0a021", "parents": [ "6286a97bea0f584342803a215550038852b24776" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Dec 15 22:15:26 2015 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Dec 15 22:15:26 2015 +0000" }, "message": "Revert \"Combine direct_methods_ and virtual_methods_ fields of mirror::Class\"\n\nThis reverts commit 6286a97bea0f584342803a215550038852b24776.\n\nChange-Id: I5b00f6d1350e9c587acd4b185367dc815ea707de\n" }, { "commit": "9539150b85142c18e9e8c2264b5b6100942667c3", "tree": "733f2af56f94cbac19ac29f62d85850df2fd7b7d", "parents": [ "1997d7cef12ac9906c23b59fdaa9f7f68344b98d" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 03 17:38:56 2015 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Dec 14 14:54:01 2015 -0800" }, "message": "Combine direct_methods_ and virtual_methods_ fields of mirror::Class\n\nManual cherry-pick of 6286a97 to master\n\nThis makes several parts of the overall runtime simpler and reduces\nthe size of a class object by 32-bits.\n\nBug: 24618811\n\nChange-Id: I36129b52189e26898ea56fa2b7b45652e06af236\n" }, { "commit": "6286a97bea0f584342803a215550038852b24776", "tree": "3571735bdc691bf30e33abb65a95ada8c71a18b7", "parents": [ "377cb3ba2e1cfe51967e5292268a9aae71ceca8c" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Dec 03 17:38:56 2015 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Dec 14 14:27:32 2015 -0800" }, "message": "Combine direct_methods_ and virtual_methods_ fields of mirror::Class\n\nThis makes several parts of the overall runtime simpler and reduces\nthe size of a class object by 32-bits.\n\nBug: 24618811\n\nChange-Id: I36129b52189e26898ea56fa2b7b45652e06af236\n" }, { "commit": "a7a4759946d9f11c88dc108b2b6a9518ce9c1e18", "tree": "f5522eb632ff53b4c285ff76a3966d37a995777a", "parents": [ "099ab4d06857ccbe642d98ba25df1eb02aee99c7" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 24 09:17:30 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 24 10:54:16 2015 +0000" }, "message": "Revert \"lambda: Add support for invoke-interface for boxed innate lambdas\"\n\n955-lambda is flaky\n\nBug: 24618608\nBug: 25107649\n\nThis reverts commit 457e874459ae638145cab6d572e34d48480e39d2.\n\n(cherry picked from commit 3a0909248e04b22c3981cbf617bc2502ed5b6380)\n\nChange-Id: I24884344d21d7a4262e53e3f5dba57032687ddb7\n" }, { "commit": "3a0909248e04b22c3981cbf617bc2502ed5b6380", "tree": "b0b7013ea78b10f23fae75ef145e53a696ff797c", "parents": [ "457e874459ae638145cab6d572e34d48480e39d2" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 24 09:17:30 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Nov 24 09:17:30 2015 +0000" }, "message": "Revert \"lambda: Add support for invoke-interface for boxed innate lambdas\"\n\n955-lambda is flaky\n\nBug: 24618608\nBug: 25107649\n\nThis reverts commit 457e874459ae638145cab6d572e34d48480e39d2.\n\nChange-Id: I24884344d21d7a4262e53e3f5dba57032687ddb7\n" }, { "commit": "121b25e246b7012cfd895f205855c6102da48305", "tree": "8d7e4dae58b59e9a554db03a06c800f3caf5b6b9", "parents": [ "31f7afcd651ca5836892d1c24010933fa40c29e4", "13ca74fed068a6a49221b5213ce0b1bebeda3ed6" ], "author": { "name": "Przemyslaw Szczepaniak", "email": "pszczepaniak@google.com", "time": "Fri Nov 20 11:24:33 2015 +0000" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Sat Nov 21 16:28:15 2015 +0000" }, "message": "Merge mnc-dr-enso-dev into goog/master\n\nBug: 25758743\nChange-Id: I19d433934f01856e7b5a7392a58b759c3ac386e7\n" }, { "commit": "457e874459ae638145cab6d572e34d48480e39d2", "tree": "b192c4698c4889ae90db6abe2652199802213fe6", "parents": [ "3944f7175dcf60316ba58a42698ccf23c65ac57c" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Oct 22 17:37:50 2015 -0700" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Fri Nov 20 16:33:06 2015 -0800" }, "message": "lambda: Add support for invoke-interface for boxed innate lambdas\n\nLambda closures created with the \u0027create-lambda\u0027 instruction\n(termed \"innate lambdas\") can be turned into an object with \u0027box-lambda\u0027.\n\nThis CL enables support for those kinds of lambdas to work with\n\u0027invoke-interface\u0027 by generating a proxy class for the lambda.\n\nNote: MIPS32/64 support not included.\n\nBug: 24618608\nBug: 25107649\nChange-Id: Ic8f1bb66ebeaed4097e758a50becf1cff6ccaefb\n" }, { "commit": "d1ef4362bf799f9f5d50e5edef8433664b503051", "tree": "59aaa140925efa790f787d5e1caf6d0cafd6ae15", "parents": [ "e4cf5892c2a244034900d49499c071b255571bba", "f32e8327da5dd33abe18662fbca5e584cd047816" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Nov 12 11:49:06 2015 +0000" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Nov 12 11:52:42 2015 +0000" }, "message": "Merge lmp changes into mnc.\n\nMost merge conflicts are due to changes in the string representation.\nThey have been resolved in favour of \"mnc-dev\" since we\u0027ve changed\nthe string representation there. Other changes relate to the\nfact that there are now two jars in libcore (core-oj and core-libart).\n\nChange-Id: I1fcc6e5f8dab8d1954dcddca0493563e7677d433\n" }, { "commit": "3b73dd7f3a7a8465bb56ebd99d1d0803721255fe", "tree": "ed08b1cd825578b1413e3c1a8cbee747ab023aea", "parents": [ "5c0cf12a96c0ddc81a1aaa573e3497ef648e5d3a" ], "author": { "name": "Przemyslaw Szczepaniak", "email": "pszczepaniak@google.com", "time": "Wed Sep 30 14:55:02 2015 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Nov 12 10:15:59 2015 +0000" }, "message": "Fix class_linker_test\n\nBad Throwable field name in the test.\n\nChange-Id: I48f5b3f9aaaa073443058a0c0a7db2327e8a1445\n" }, { "commit": "90f397e9f1fd49a7fabe925729337e077bc5c5cb", "tree": "8d2fde994f2e4bba5ee44b52487e47d816702f95", "parents": [ "88f423f8ce96d521e8f54f84611520b67d320fdc" ], "author": { "name": "Piotr Jastrzebski", "email": "haaawk@google.com", "time": "Thu May 07 09:41:00 2015 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Nov 09 11:06:20 2015 +0000" }, "message": "Update class_linker_test to account for annotationType.\n\nChange-Id: If15b27381c0f3dd220d16e59bf93e6ba8e9f81a5\n" }, { "commit": "c16a50f4837d6204c0cf6c71129492cb4b86784d", "tree": "40cee618cb3e1b0190a0f19f584c78dd2fa86a4b", "parents": [ "87bd212e11d31ef62f1bb2377f921b09bc537b1f" ], "author": { "name": "Piotr Jastrzebski", "email": "haaawk@google.com", "time": "Thu May 07 09:41:00 2015 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Nov 09 10:35:43 2015 +0000" }, "message": "Track changes to the layout of java.lang.Throwable.\n\nThe stack trace is now stored in an object called |backtrace|.\n\nChange-Id: Icb3f43fed7bc13ecd9124b0e012fef33b7cdbe50\n" }, { "commit": "8a89433aafe6dbb88fa1659b4364d733e5af739b", "tree": "6b01c569b907ce6bf40df4812b3d9769c4871a01", "parents": [ "2c22f2e73e5b504bb748747912acc4d74671843e" ], "author": { "name": "Piotr Jastrzebski", "email": "haaawk@google.com", "time": "Thu May 07 09:41:00 2015 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Nov 09 10:34:58 2015 +0000" }, "message": "Update runtime to work with the OpenJdk String.\n\nThe key difference here is that the OpenJdk String class\ndoes not maintain an offset or a count field into its backing\nchar[] array. The offset is assumed to be zero, and the count\nis always array.length.\n\nChange-Id: I8b0518642f440cd38d24d9b90c575b7110233cc5\n" }, { "commit": "44095a15349445e87942a728a825bd32ea11be1d", "tree": "878ee5658278391c02c366fc8ab4ea668631363b", "parents": [ "8ca90774165b22fdbc87c89c8b55192e43fb2e43" ], "author": { "name": "Piotr Jastrzebski", "email": "haaawk@google.com", "time": "Mon Apr 27 11:49:00 2015 +0100" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Mon Nov 09 10:32:27 2015 +0000" }, "message": "String: account for the hash32 field.\n\nAlso temporarily disable intrinsics for now.\n\nChange-Id: I8158962efeba6e4ccb39cc147bff782aace3ffbc\n" }, { "commit": "99babb6add7db19ce7605f6d5e4aee79d52e386f", "tree": "116f63fc2f96ad103a5dc121667d4b3df9e6e2ff", "parents": [ "3f922d1f3ccd7a1341f887ac3e8176b5208ecf6d" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 02 16:20:00 2015 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Nov 04 12:16:16 2015 -0800" }, "message": "ART: Change behavior for rethrowing init failures\n\nAllow to store a Throwable instance or a throwable class. Handle\nrethrow accordingly.\n\nBug: 25444180\nChange-Id: I703c2c6eaf34ad0e3bc0f5a104d65f2ff1b212ca\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": "951ec2c93c79c5539cbcc669566f0808d4460338", "tree": "ce3505be2fc526afafaf6135e9d15d831860502d", "parents": [ "6a6916b378c413c2692782e901393607c47f0005" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 22 08:50:05 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 22 16:25:09 2015 +0000" }, "message": "Revert \"Revert \"Add one LinearAlloc per ClassLoader\"\"\n\nIssue was fixed by:\nhttps://android-review.googlesource.com/#/c/171945/\n\nBug: 22720414\n\nThis reverts commit 7de5dfe37f3cf24e1166412b589f6f67dcd1f1c0.\n" }, { "commit": "7de5dfe37f3cf24e1166412b589f6f67dcd1f1c0", "tree": "a87eeac15fd485bb14ea8ace2b2bc839c4c253be", "parents": [ "356412e2b7ba3fde164bc08a44fee0ddc19c54e1" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Sep 22 09:48:02 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Sep 22 09:48:02 2015 +0000" }, "message": "Revert \"Add one LinearAlloc per ClassLoader\"\n\nTimes out on 32bit target/host for 132-daemon-locks-shutdown test.\n\nBug: 22720414\n\nThis reverts commit 356412e2b7ba3fde164bc08a44fee0ddc19c54e1.\n\nChange-Id: I5ab3a09e88a5ad8c306a27d2606ecbecc80b9326\n" }, { "commit": "356412e2b7ba3fde164bc08a44fee0ddc19c54e1", "tree": "9e07c960e67b3627ced80fa9a0dcd0c9fd8b5951", "parents": [ "d0d11f20811f260453f6dfe2e26d7dbd6ed55f01" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Sep 21 09:07:37 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Sep 21 12:48:12 2015 -0700" }, "message": "Add one LinearAlloc per ClassLoader\n\nAlso added freeing linear alloc and class table when the\ncorresponding class loader is no longer reachable.\n\nBug: 22720414\n\nChange-Id: Icb32c3a4c865f240e147bc87ed080a6b1d8a5795\n" }, { "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": "673ed3d8aedc5462a47ded827c99f35d46525457", "tree": "83de4690228c9f7772c48c770c5e2e5ba2dac6fe", "parents": [ "ce209462cc1a7ce235e5ac0d0e6db6b402f73441" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 28 14:56:43 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Aug 31 10:58:31 2015 -0700" }, "message": "Revert \"Revert \"Change dex caches to be weak roots\"\"\n\nThis reverts commit 81a9087f0df0518c39405b7d18ba5858a6d8b77b.\n\nBoot.oat creation time goes from 4.7s to 4.9s on host due to extra\nlocking. Will try to improve this in another CL.\n\nBug: 23602225\nBug: 22720414\n\nChange-Id: I7e25b75cfb63faa196c7b0f60e46cce50bf12021\n" }, { "commit": "52a7f5caebdf359ab877f1928aad59f1e9ad29fa", "tree": "fe23aeb3682128a31c62324e427b906c9804afd3", "parents": [ "944973e56fd3c04c92d902b05d0148f77ed28a78" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Aug 18 18:35:52 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 21 10:25:36 2015 -0700" }, "message": "Add class flags to class to help GC scanning\n\nReduces GC time and pauses by reducing the number of loads required\nto scan an object.\n\nAverage total GC time before on EvaluateAndApplyChanges (EAAC): 7.452s\nAfter: 7.144s\n\nAverage GC pause times before on EAAC: 860.67us\nAfter: 722.75us\n\nAdding the class flags field cause a memory increase of ~24k system\nwide on low memory devices.\n\nChange-Id: I3f04212d5787bfbf5e55026584d149f55476105e\n" }, { "commit": "6b06953817bdf613b52389c457e2f4ed6cf46f5c", "tree": "fb4ba014d048eae6a28778568a9b7c30889a01b7", "parents": [ "7fbaa22e10979cf17b8c1c0c96bef4919e9d4163" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 05 15:08:12 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Aug 07 16:10:07 2015 -0700" }, "message": "Add class table field to class loader\n\nFixes bug with the class table where the comparator would cause read\nbarriers and break the map strict ordering properties.\n\nBug: 22957957\n\nChange-Id: I8dbc042db6e22e2172ab4ec58ddf1db0345dcaaa\n" }, { "commit": "54d220eb9cc51215d75b9e0fe921b94bebbb3fd6", "tree": "fe08749dea3ca6cad97bce48416beb37203efdf5", "parents": [ "275bc5e5f318ba6ea1bad6c89c2924d1248b7523" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 30 16:20:06 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Aug 05 18:15:22 2015 -0700" }, "message": "Move ArtFields and ArtMethods to be a length prefixed array\n\nFixes race conditions between changing method and fields arrays\nbeing seen in the wrong order by the GC.\n\nBug: 22832610\nChange-Id: Ia21d6698f73ba207a6392c3d6b9be2658933073f\n" }, { "commit": "90443477f9a0061581c420775ce3b7eeae7468bc", "tree": "8c74b81dfae162e0fd0ccf8d5ac50827ba815174", "parents": [ "6078aec213dfaf111c29969706e8e5967cfc9bea" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jul 16 20:32:27 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jul 22 15:13:56 2015 -0700" }, "message": "Move to newer clang annotations\n\nAlso enable -Wthread-safety-negative.\n\nChanges:\nSwitch to capabilities and negative capabilities.\n\nFuture work:\nUse capabilities to implement uninterruptible annotations to work\nwith AssertNoThreadSuspension.\n\nBug: 20072211\n\nChange-Id: I42fcbe0300d98a831c89d1eff3ecd5a7e99ebf33\n" }, { "commit": "3d21bdf8894e780d349c481e5c9e29fe1556051c", "tree": "61a5231f36c0dabd73457fec81df103462a05aff", "parents": [ "71f0a8a123fa27bdc857a98afebbaf0ed09dac15" ], "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": "Tue Jun 02 09:21:27 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\n\n(cherry picked from commit e401d146407d61eeb99f8d6176b2ac13c4df1e33)\n\nChange-Id: I622469a0cfa0e7082a2119f3d6a9491eb61e3f3d\n\nFix some ArtMethod related bugs\n\nAdded root visiting for runtime methods, not currently required\nsince the GcRoots in these methods are null.\n\nAdded missing GetInterfaceMethodIfProxy in GetMethodLine, fixes\n--trace run-tests 005, 044.\n\nFixed optimizing compiler bug where we used a normal stack location\ninstead of double on ARM64, this fixes the debuggable tests.\n\nTODO: Fix JDWP tests.\n\nBug: 19264997\n\nChange-Id: I7c55f69c61d1b45351fd0dc7185ffe5efad82bd3\n\nART: Fix casts for 64-bit pointers on 32-bit compiler.\n\nBug: 19264997\nChange-Id: Ief45cdd4bae5a43fc8bfdfa7cf744e2c57529457\n\nFix JDWP tests after ArtMethod change\n\nFixes Throwable::GetStackDepth for exception event detection after\ninternal stack trace representation change.\n\nAdds missing ArtMethod::GetInterfaceMethodIfProxy call in case of\nproxy method.\n\nBug: 19264997\nChange-Id: I363e293796848c3ec491c963813f62d868da44d2\n\nFix accidental IMT and root marking regression\n\nWas always using the conflict trampoline. Also included fix for\nregression in GC time caused by extra roots. Most of the regression\nwas IMT.\n\nFixed bug in DumpGcPerformanceInfo where we would get SIGABRT due to\ndetached thread.\n\nEvaluateAndApplyChanges:\nFrom ~2500 -\u003e ~1980\nGC time: 8.2s -\u003e 7.2s due to 1s less of MarkConcurrentRoots\n\nBug: 19264997\nChange-Id: I4333e80a8268c2ed1284f87f25b9f113d4f2c7e0\n\nFix bogus image test assert\n\nPreviously we were comparing the size of the non moving space to\nsize of the image file.\n\nNow we properly compare the size of the image space against the size\nof the image file.\n\nBug: 19264997\nChange-Id: I7359f1f73ae3df60c5147245935a24431c04808a\n\n[MIPS64] Fix art_quick_invoke_stub argument offsets.\n\nArtMethod reference\u0027s size got bigger, so we need to move other args\nand leave enough space for ArtMethod* and \u0027this\u0027 pointer.\n\nThis fixes mips64 boot.\n\nBug: 19264997\nChange-Id: I47198d5f39a4caab30b3b77479d5eedaad5006ab\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": "9d1e19035a59ff8bf05a09ad880ede99806ba73f", "tree": "7eb3c9eb01d4f581f3e354d947300547582fdd38", "parents": [ "3fd79afbb7490f52817f737ecceb694f6618dcf9", "848f70a3d73833fc1bf3032a9ff6812e429661d9" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Apr 28 02:04:09 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Apr 28 02:04:09 2015 +0000" }, "message": "Merge \"Replace String CharArray with internal uint16_t array.\"" }, { "commit": "848f70a3d73833fc1bf3032a9ff6812e429661d9", "tree": "b0349b3a40aab5a915af491b100659a5ca9fbbf6", "parents": [ "d14438f0c5071962be7fab572b54687d32d9d087" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Jan 15 13:49:50 2014 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Apr 27 18:54:52 2015 -0700" }, "message": "Replace String CharArray with internal uint16_t array.\n\nSummary of high level changes:\n - Adds compiler inliner support to identify string init methods\n - Adds compiler support (quick \u0026 optimizing) with new invoke code path\n that calls method off the thread pointer\n - Adds thread entrypoints for all string init methods\n - Adds map to verifier to log when receiver of string init has been\n copied to other registers. used by compiler and interpreter\n\nChange-Id: I797b992a8feb566f9ad73060011ab6f51eb7ce01\n" }, { "commit": "60f63f53c01cb38ca18a815603282e802a6cf918", "tree": "4f18427401ead0c790e926672957189a0c0a39eb", "parents": [ "633a37ece49c5afcf3fa9a89692f07d19c56229b" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Apr 23 16:12:40 2015 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Thu Apr 23 21:11:40 2015 -0700" }, "message": "Use the lock word bits for Baker-style read barrier.\n\nThis enables the standard object header to be used with the\nBaker-style read barrier.\n\nBug: 19355854\nBug: 12687968\n\nChange-Id: Ie552b6e1dfe30e96cb1d0895bd0dff25f9d7d015\n" }, { "commit": "fc58af45e342ba9e18bbdf597f205a58ec731658", "tree": "3f93906235cb4d2462d237223b72ccf4eedef645", "parents": [ "bbf02afc641a393d33342976e269218668c07386" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 16 18:00:39 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Apr 21 15:51:29 2015 -0700" }, "message": "Add AbstractMethod, Constructor, Method\n\nMoves functionality to ART from libcore. Precursor to moving\nArtMethods to native. Mostly performance improvements.\n\nN5 perf before (irrelevant results removed):\n Class_getConstructor 962.87 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Class_getDeclaredMethod 2394.37 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Class_getMethod 2509.20 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Class_newInstance 1999.81 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokeI 1439.02 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokePreBoxedI 1415.82 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokeStaticI 1456.24 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokeStaticPreBoxedI 1427.32 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokeStaticV 814.47 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokeV 816.56 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nAfter:\n benchmark ns linear runtime\n Class_getConstructor 1302.04 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Class_getDeclaredMethod 1459.01 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Class_getMethod 1560.40 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Class_newInstance 2029.94 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokeI 1312.89 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokePreBoxedI 1255.01 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokeStaticI 1289.13 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokeStaticPreBoxedI 1196.52 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokeStaticV 790.82 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Method_invokeV 791.73 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nPerformance improvements are more than just fixing regressions introduced\nin: http://android-review.googlesource.com/#/c/146069/\n\nBug: 19264997\n\nChange-Id: Ife79c469fdb09f30e3aefcfc3e0ce5ed32303fce\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": "bb87e0f1a52de656bc77cb01cb887e51a0e5198b", "tree": "113f014c6e20fab3e936a3ac05f9f738639541f6", "parents": [ "e57fc0f0260fcb1d08cbb720ec95c04c0f394b91" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 03 11:21:55 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Apr 06 10:44:37 2015 -0700" }, "message": "Refactor and improve GC root handling\n\nChanged GcRoot to use compressed references. Changed root visiting to\nuse virtual functions instead of function pointers. Changed root visting\ninterface to be an array of roots instead of a single root at a time.\nAdded buffered root marking helper to avoid dispatch overhead.\n\nRoot marking seems a bit faster on EvaluateAndApplyChanges due to batch\nmarking. Pause times unaffected.\n\nMips64 is untested but might work, maybe.\n\nBefore:\nMarkConcurrentRoots: Sum: 67.678ms 99% C.I. 2us-664.999us Avg: 161.138us Max: 671us\n\nAfter:\nMarkConcurrentRoots: Sum: 54.806ms 99% C.I. 2us-499.986us Avg: 136.333us Max: 602us\n\nBug: 19264997\n\nChange-Id: I0a71ebb5928f205b9b3f7945b25db6489d5657ca\n" }, { "commit": "daaf3265806eb2eadb2e03302bd68022fab5ca28", "tree": "aff5d6d53d6d2b65995aa204839f88ee66400989", "parents": [ "68e22f3b982ff9ccbdfb3b65b7cfc16fcae907ba" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Mar 24 13:30:28 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Mar 29 14:13:08 2015 -0700" }, "message": "Add AccessibleObject and Field to mirror\n\nMain motivation is to remove all the functionality / field access on\njava side to ArtField. Also comes with some reflection speedups /\nslowdowns.\n\nSummary results:\ngetDeclaredField/getField are slower mostly due to JNI overhead.\nHowever, there is a large speedup in getInt, setInt,\nGetInstanceField, and GetStaticField.\n\nBefore timings (N5 --compiler-filter\u003deverything):\n\n benchmark ns linear runtime\n Class_getDeclaredField 782.86 \u003d\u003d\u003d\n Class_getField 832.77 \u003d\u003d\u003d\n Field_getInt 160.17 \u003d\n Field_setInt 195.88 \u003d\n GetInstanceField 3214.38 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n GetStaticField 6809.49 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nAfter:\n Class_getDeclaredField 1068.15 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Class_getField 1180.00 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n Field_getInt 121.85 \u003d\n Field_setInt 139.98 \u003d\n GetInstanceField 1986.15 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n GetStaticField 2523.63 \u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nBug: 19264997\n\nChange-Id: Ic0d0fc1b56b95cd6d60f8e76f19caeaa23045c77\n" }, { "commit": "81c6f8db12b203878a7d72444ead2bc7cf5c47ad", "tree": "11caae30b75b700ac648356fd30759a6154be997", "parents": [ "cac51526bbd03947676a8d49700425b19a57e447" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Mar 25 17:19:53 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Mar 27 21:43:21 2015 -0700" }, "message": "ART: PathClassLoader for compiler\n\nUse an actual PathClassLoader when compiling apps, instead of a\nside structure and cutout.\n\nThis CL sets up a minimal object \u0027cluster\u0027 that recreates the Java\nside of a regular ClassLoader such that the Class-Linker will\nrecognize it and use the internal native fast-path.\n\nThis CL removes the now unnecessary compile-time-classpath and\nreplaces it with a single \u0027compiling-the-boot-image\u0027 flag in the\ncompiler callbacks.\n\nNote: This functionality is *only* intended for the compiler, as\nthe objects have not been completely initialized.\n\nBug: 19781184\n\nChange-Id: I7f36af12dd7852d21281110a25c119e8c0669c1d\n" }, { "commit": "14691c5e786e8c2c5734f687e4c96217340771be", "tree": "7c2156671b323c70ffdd1d48d5e2f1d1de79c5fc", "parents": [ "3d7d2af4c6502b771b032ee9bf3ab30e78f9c60d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Mar 05 10:40:17 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 09 10:09:33 2015 +0000" }, "message": "Compute the right catch location for the debugger.\n\nAlso remove tls ThrowLocation, it is not needed anymore.\n\nChange-Id: I78fddf09ce968ca475e39c17fa76d699c589c8d9\n" }, { "commit": "7f418db815f0eaef5b2f43e1f06fb8773a415494", "tree": "12a091013ad3d3a48fbf5a7af5b1499392b3c760", "parents": [ "f2df4941be3c48daee65f2ca1852994240872e4d" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Nov 26 22:11:27 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Sat Jan 17 15:28:49 2015 +0100" }, "message": "JDWP: fix breakpoint for method in the image\n\nWhen we set a breakpoint in a compiled method, we deoptimize it by\nchanging its entrypoint so it is executed with the interpreter.\nHowever, methods in the image can be called with their direct code\npointer, ignoring the updated entrypoint. In that case, the method\nis not executed with the interpreter and we miss the breakpoint.\n\nThis CL avoids that situation by forcing a full deoptimization so\neverything runs with the interpreter. However, if the image has been\ncompiled in PIC mode, we keep using selective deoptimization because\ndirect code pointer is not used in this mode.\n\nBug: 17965285\n\n(cherry picked from commit 6963e44331258b131bcc0599b868ba15902d6d22)\n\nChange-Id: I9bf738f89b9eb6d18733503216b376b8a1d181f5\n" }, { "commit": "e34fa1df67fbe0173b4ea9abddcc3ae3d0537037", "tree": "a5148f079b5671a95f60910c41981ebf91db3a02", "parents": [ "9f06b1946ae3ef1d2fd75bbf7f0a288bc611fe58" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jan 14 14:55:47 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 15 12:23:28 2015 -0800" }, "message": "Print more info in MarkSweep::VerifyRoot\n\nRefactored old root callback to use a new class called RootInfo.\nRootInfo contains all the relevant info related to the root\nassociated with the callback. The MarkSweep::VerifyRoot function\nnow uses this info to print the StackVisitor\u0027s described location\nif the GC root is of the type kRootJavaFrame.\n\nSome other cleanup.\n\nExample output:\nE/art (12167): Tried to mark 0x123 not contained by any spaces\nE/art (12167): Attempting see if it\u0027s a bad root\nE/art (12167): Found invalid root: 0x123 with type RootJavaFrame\nE/art (12167): Location\u003dVisiting method\n\u0027void java.lang.Runtime.gc()\u0027 at dex PC 0xffffffff (native PC 0x0)\nvreg\u003d0\n\n(cherry picked from commit 12f7423a2bb4bfab76700d84eb6d4338d211983a)\n\nBug: 18588862\nChange-Id: Ic5a2781f704e931265ffb3621c2eab4b2e25f60f\n" }, { "commit": "12f7423a2bb4bfab76700d84eb6d4338d211983a", "tree": "1c4b45aa3c80879a5fff124aa4ec1cb68133275c", "parents": [ "300dee9ff2c4ce59e821d0aeceb87fab13295f46" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jan 14 14:55:47 2015 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 15 10:19:29 2015 -0800" }, "message": "Print more info in MarkSweep::VerifyRoot\n\nRefactored old root callback to use a new class called RootInfo.\nRootInfo contains all the relevant info related to the root\nassociated with the callback. The MarkSweep::VerifyRoot function\nnow uses this info to print the StackVisitor\u0027s described location\nif the GC root is of the type kRootJavaFrame.\n\nSome other cleanup.\n\nExample output:\nE/art (12167): Tried to mark 0x123 not contained by any spaces\nE/art (12167): Attempting see if it\u0027s a bad root\nE/art (12167): Found invalid root: 0x123 with type RootJavaFrame\nE/art (12167): Location\u003dVisiting method\n\u0027void java.lang.Runtime.gc()\u0027 at dex PC 0xffffffff (native PC 0x0)\nvreg\u003d0\n\nBug: 18588862\nChange-Id: Ic5a2781f704e931265ffb3621c2eab4b2e25f60f\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": "baede348efa86600e64fb9db43cec1eef07c86d9", "tree": "7ab2c2a69c72b7cd2ce1b7bf1ed9d26b07e4fd7e", "parents": [ "989210eb453898bd94bf3527f18f6146b07aa4bb", "6963e44331258b131bcc0599b868ba15902d6d22" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Dec 04 10:21:02 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Dec 04 10:21:02 2014 +0000" }, "message": "Merge \"JDWP: fix breakpoint for method in the image\"" }, { "commit": "6963e44331258b131bcc0599b868ba15902d6d22", "tree": "24ef16e739e99d3e9d980f2acde8dd301c236c37", "parents": [ "220526b05d4365a1820a694c98527eda2d3dc980" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Wed Nov 26 22:11:27 2014 +0100" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Dec 04 10:20:24 2014 +0100" }, "message": "JDWP: fix breakpoint for method in the image\n\nWhen we set a breakpoint in a compiled method, we deoptimize it by\nchanging its entrypoint so it is executed with the interpreter.\nHowever, methods in the image can be called with their direct code\npointer, ignoring the updated entrypoint. In that case, the method\nis not executed with the interpreter and we miss the breakpoint.\n\nThis CL avoids that situation by forcing a full deoptimization so\neverything runs with the interpreter. However, if the image has been\ncompiled in PIC mode, we keep using selective deoptimization because\ndirect code pointer is not used in this mode.\n\nBug: 17965285\nChange-Id: Icaf8cbb7fe9ad01d36f7378c59d50d9ce42ae57f\n" }, { "commit": "08f1f50d6c2e8f247b8f5f19711d75a792851c7a", "tree": "dca490545d56168f7f9ae4a4616199b1b1c8ba0c", "parents": [ "8443637f71a777a13317fe7635028d758a0adf97" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 15:04:37 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 15:12:27 2014 -0800" }, "message": "Remove FieldHelper.\n\nChange-Id: I2d74e2d5b3c35a691c95339de0db9361847fca11\n" }, { "commit": "eace45873190a27302b3644c32ec82854b59d299", "tree": "73fbf327839263b6847bdc4359ac1dbea2b897e3", "parents": [ "8ac8d5556fae9c728bcebcc9036a1bbf40087c76" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 24 18:29:54 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 25 16:02:04 2014 -0800" }, "message": "Move dexCacheStrings from ArtMethod to Class\n\nAdds one load for const strings which are not direct.\n\nSaves \u003e\u003d 60KB of memory avg per app.\nImage size: -350KB.\n\nBug: 17643507\nChange-Id: I2d1a3253d9de09682be9bc6b420a29513d592cc8\n\n(cherry picked from commit f521f423b66e952f746885dd9f6cf8ef2788955d)\n" }, { "commit": "f521f423b66e952f746885dd9f6cf8ef2788955d", "tree": "97b0e77b3de86c5b405b7935addc01f8930b0ce5", "parents": [ "89ea4176ae89b81702e6a803828de5b95b161393" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 24 18:29:54 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 25 10:37:59 2014 -0800" }, "message": "Move dex cache strings from ArtMethod -\u003e Class\n\nAdds one load for const strings which are not direct.\n\nSaves \u003e\u003d 60KB of memory avg per app.\nImage size: -350KB.\n\nBug: 17643507\nChange-Id: I2d1a3253d9de09682be9bc6b420a29513d592cc8\n" }, { "commit": "2d7210188805292e463be4bcf7a133b654d7e0ea", "tree": "7705a3bf841ae44b2396728fa22ed0b5dcb44dbf", "parents": [ "e0491682d101c69bf88c3c24a965312129cbfa38" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 10 11:08:06 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 18 12:27:37 2014 -0800" }, "message": "Change 64 bit ArtMethod fields to be pointer sized\n\nChanged the 64 bit entrypoint and gc map fields in ArtMethod to be\npointer sized. This saves a large amount of memory on 32 bit systems.\nReduces ArtMethod size by 16 bytes on 32 bit.\n\nTotal number of ArtMethod on low memory mako: 169957\nImage size: 49203 methods -\u003e 787248 image size reduction.\nZygote space size: 1070 methods -\u003e 17120 size reduction.\nApp methods: ~120k -\u003e 2 MB savings.\n\nSavings per app on low memory mako: 125K+ per app\n(less active apps -\u003e more image methods per app).\n\nSavings depend on how often the shared methods are on dirty pages vs\nshared.\n\nTODO in another CL, delete gc map field from ArtMethod since we\nshould be able to get it from the Oat method header.\n\nBug: 17643507\n\nChange-Id: Ie9508f05907a9f693882d4d32a564460bf273ee8\n\n(cherry picked from commit e832e64a7e82d7f72aedbd7d798fb929d458ee8f)\n" }, { "commit": "e832e64a7e82d7f72aedbd7d798fb929d458ee8f", "tree": "307c7d13edf397b2b7b5491590d5404aa099be00", "parents": [ "a477bbb62ba197815d840d07708526a2bb9c8b4d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 10 11:08:06 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 17 17:43:40 2014 -0800" }, "message": "Change 64 bit ArtMethod fields to be pointer sized\n\nChanged the 64 bit entrypoint and gc map fields in ArtMethod to be\npointer sized. This saves a large amount of memory on 32 bit systems.\nReduces ArtMethod size by 16 bytes on 32 bit.\n\nTotal number of ArtMethod on low memory mako: 169957\nImage size: 49203 methods -\u003e 787248 image size reduction.\nZygote space size: 1070 methods -\u003e 17120 size reduction.\nApp methods: ~120k -\u003e 2 MB savings.\n\nSavings per app on low memory mako: 125K+ per app\n(less active apps -\u003e more image methods per app).\n\nSavings depend on how often the shared methods are on dirty pages vs\nshared.\n\nTODO in another CL, delete gc map field from ArtMethod since we\nshould be able to get it from the Oat method header.\n\nBug: 17643507\n\nChange-Id: Ie9508f05907a9f693882d4d32a564460bf273ee8\n" }, { "commit": "76649e8d775519fe19f2b14d18ac488c13296054", "tree": "a899ec37ce51672c9e14d2954b2a7af7397e63da", "parents": [ "d94a0a1d2868baaab49f4d2835bca086d98cf763" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Nov 10 18:32:59 2014 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Nov 14 11:33:53 2014 +0000" }, "message": "Keep original order of fields in Class.\n\nThe fields of a class are ordered alphabetically in the dex\nfile. Keep the same order in the field arrays so that we can\ndo binary search lookups by name. Those lookups will be\nimplemented in a subsequent change in libcore/.\n\nBug: 18211592\n\n(cherry picked from commit bfa3ed0ad988e1da13626ddbaf6dcae0c58ea79e)\n\nChange-Id: I8f979de62ffe37d1c7d5c721717d2f3501e7c9e6\n" }, { "commit": "bfa3ed0ad988e1da13626ddbaf6dcae0c58ea79e", "tree": "94a5efc09745b0dad86c9992869e84d48ff47a21", "parents": [ "7bcea103af7cd130254835a30b45d5277a478074" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Nov 10 18:32:59 2014 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 13 18:05:23 2014 +0000" }, "message": "Keep original order of fields in Class.\n\nThe fields of a class are ordered alphabetically in the dex\nfile. Keep the same order in the field arrays so that we can\ndo binary search lookups by name. Those lookups will be\nimplemented in a subsequent change in libcore/.\n\nBug: 18211592\nChange-Id: I8f979de62ffe37d1c7d5c721717d2f3501e7c9e6\n" }, { "commit": "5f9a95423dda3a2f945c81a04107fedf18e2c2c1", "tree": "27f55e828ea076674f9bc510523c675fe6caf3b5", "parents": [ "974bc2747b345667e07692109d63675ec50955a3" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Nov 07 18:38:47 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 10 10:07:57 2014 -0800" }, "message": "Delete unused imtable field\n\nBug: 17643507\n\nChange-Id: I1fc7ca2d3bdf1810bcc3b46f867b68a4a6d28ed3\n\n(cherry picked from commit f2d556401ec1d82fec31c0b29d712de18d838282\n" }, { "commit": "f2d556401ec1d82fec31c0b29d712de18d838282", "tree": "9b4e49d4c7a2be057c7eadae2adb274c861fcf8e", "parents": [ "90534189da43d03b45b3a214a86f8d6e54c6aec1" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Nov 07 18:38:47 2014 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Nov 10 09:48:19 2014 -0800" }, "message": "Delete unused imtable field\n\nBug: 17643507\n\nChange-Id: I1fc7ca2d3bdf1810bcc3b46f867b68a4a6d28ed3\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": "647b1a86f518d8db0331b3d52a96392b7a62504b", "tree": "7370f795ef3c7fbdd2695d23bc6f8171f40f43f1", "parents": [ "acfbbd4df2fc1c79a7102587bebf398f95b5e5de" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 10 11:02:11 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 10 12:26:02 2014 -0700" }, "message": "Fix 2 new sets of clang compiler warnings.\n\nFix issues that are flagged by -Wfloat-equal and -Wmissing-noreturn.\nIn the case of -Wfloat-equal the current cases in regular code are deliberate,\nso the change is to silence the warning. For gtest code the appropriate fix is\nto switch from EXPECT_EQ to EXPECT_(FLOAT|DOUBLE)_EQ.\nThe -Wmissing-noreturn warning isn\u0027t enabled due to a missing noreturn in\ngtest. This issue has been reported to gtest.\n\nChange-Id: Id84c70c21c542716c9ee0c41492e8ff8788c4ef8\n" }, { "commit": "cdc1aaffabbdf417d29b203b2cd2763ed2d623f8", "tree": "ff56b9bbaa3ce0305c0ef71c055d3664dddfe93a", "parents": [ "bbb1fc6e95f46470979936450b0d3f1e020c5904" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 13:21:38 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 09 14:31:29 2014 -0700" }, "message": "Simplify instance reference offsets.\n\nDon\u0027t encode Object\u0027s class. Use trailing rather than leading zeroes to give\noffset position.\n\nChange-Id: I1ae74e7a01f63696429644adf81cdf6ee58832fe\n" }, { "commit": "63bc11efaac0c041e849ab401f9fc368631a00f5", "tree": "b342dec2f39a0242a9f4e573eec06c77739a2643", "parents": [ "b87baed15b03e2c853e285253ac92391572502f8" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 18 08:56:45 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Sep 18 10:33:39 2014 -0700" }, "message": "DO NOT MERGE. Only have a portable entrypoint in portable builds.\n\nBug: 16214885\n\nChange-Id: Iff7b7415efdbdabd7e6020e221a540f6a774c852\n" }, { "commit": "5a4b8a236030460651a3136397d23ca6744e7eb7", "tree": "0e43891398e416d3fa77c7de391bf4db4408e8ee", "parents": [ "19f7c95491a053b818f914137fa73df0517b8792" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 11 08:30:08 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 15 19:50:12 2014 -0700" }, "message": "ART: Rename Handle hierarchy\n\nBring the names in line with normal OO principles: ConstHandle\nbecomes Handle, and Handle becomes MutableHandle.\n\nChange-Id: I0f018eb7ba28bc422e3a23dd73a6cbe6fc2d2044\n" }, { "commit": "7b078e8c04f3e1451dbdd18543c8b9692b5b067e", "tree": "414229c6b87eb20ea24c40780752da5a3999a49a", "parents": [ "f79ba17defbd9342e44ab9f3de0807054673d3c9" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 10 14:44:24 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Sep 12 14:57:53 2014 -0700" }, "message": "Compile time performance improvements focusing on interpret-only.\n\nReduce virtual method dispatch in the method verifier and make more code\ninline-able.\nAdd a StringPiece with const char* equality operator to avoid redundant\nStringPieces and strlens.\nRemove back link from register line to verifier and pass as argument to reduce\nsize of RegisterLine.\nRemove instruction length from instruction flags and compute from the\ninstruction, again to reduce size.\nAdd suspend checks to resolve and verify to allow for more easy monitor\ninflation and reduce contention on Locks::thread_list_suspend_thread_lock_.\nChange ThrowEarlierClassFailure to throw pre-allocated exception.\nAvoid calls to Thread::Current() by passing self.\nTemplate specialize IsValidClassName.\nMake ANR reporting with SIGQUIT run using checkpoints rather than suspending\nall threads. This makes the stack/lock analysis less lock error prone.\nExtra Barrier assertions and condition variable time out is now returned as a\nboolean both from Barrier and ConditionVariable::Wait.\n\n2 threaded host x86-64 interpret-only numbers from 341 samples:\nBefore change: Avg 176.137ms 99% CI 3.468ms to 1060.770ms\nAfter change: Avg 139.163% 99% CI 3.027ms to 838.257ms\nReduction in average compile time after change is 20.9%.\nSlow-down without change is 26.5%.\n\nBug: 17471626 - Fix bug where RegTypeCache::JavaLangObject/String/Class/Throwable\ncould return unresolved type when class loading is disabled.\nBug: 17398101\n\nChange-Id: Id59ce3cc520701c6ecf612f7152498107bc40684\n" }, { "commit": "48498591b90a8ff7b24b1ce05c220e3bc42013df", "tree": "39ca61335fafc57cbcf91c5087335f367ccbedc4", "parents": [ "b9620f305c79914f5159cf9279a7ccd173af1186" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Sep 10 19:48:05 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Sep 11 08:04:35 2014 -0700" }, "message": "ART: Correctly make methods preverified\n\nBug: 16828525\n\n(cherry picked from commit df1532b9ba0cda2d00b78fbdef461f8a6cf8a737)\n\nChange-Id: I66756348b2aa50e41dacca59769b6810a91c73b0\n" }, { "commit": "df1532b9ba0cda2d00b78fbdef461f8a6cf8a737", "tree": "9832d37264553bd5350e5d4e9f62a5edd81fdf8f", "parents": [ "827645f27237f3e77c87cb5e49e90389a22337a6" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Sep 10 19:48:05 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Sep 10 22:35:46 2014 -0700" }, "message": "ART: Correctly make methods preverified\n\nBug: 16828525\nChange-Id: I66756348b2aa50e41dacca59769b6810a91c73b0\n" }, { "commit": "faff0f05fef90577c9744505555675185832aacd", "tree": "40b5130526b80fe0a59f815f19dc2b87b0a9f879", "parents": [ "05382d22a2ebbc95142ec2fc46e2d56b23e699ee" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Sep 10 12:03:22 2014 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Wed Sep 10 14:40:37 2014 -0700" }, "message": "Remove reference_static_offsets used for iterating through class static fields.\n\nSince static fields are contiguous in class object and there is no need to\ntraverse super classes, it\u0027s not meaningful to use reference_static_offsets.\nAlso especially with embedded vtable/imt, static field offset can\u0027t be encoded\nwith an unsigned integer anyway.\n\nA corresponding change is made to Class.java to remove the member field.\n\nBug: 16236588\nChange-Id: I1fde3cd9efce884945876f0658c63d992164fd94\n" }, { "commit": "cb6b0f31ede2275e79e6199ec391147585a37a2a", "tree": "91eba661ab7dd329463104da33395d85df521399", "parents": [ "5f1dcee2b3cdaf0f745958ff366a84279940c65e" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 12 02:30:58 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Aug 13 10:19:45 2014 -0700" }, "message": "Avoid use of std::string where we have const char*.\n\nRemoving the ClassHelper caused std::string creation for all calls to\nClass::GetDescriptor and a significant performance regression. Make the\nstd::string an out argument so the caller can maintain it and its life time\nwhile allowing GetDescriptor to return the common const char* case.\n\nDon\u0027t generate GC maps when compilation is disabled.\n\nRemove other uses of std::string that are occuring on critical paths.\nUse the cheaper SkipClass in CompileMethod in CompilerDriver.\nSpecialize the utf8 as utf16 comparison code for the common shorter byte\nencoding.\nForce a bit of inlining, remove some UNLIKELYs (they are prone to pessimizing\ncode), add some LIKELYs.\n\nx86-64 host 1-thread interpret-only of 57 apks:\nBefore: 29.539s\nAfter: 23.467s\n\nRegular compile:\nBefore: 1m35.347s\nAfter: 1m20.056s\n\nBug: 16853450\nChange-Id: Ic705ea24784bee24ab80084d06174cbf87d557ad\n\nConflicts:\n\truntime/utils.cc\n" }, { "commit": "1ff3c98775a4577cf053dba9a0c2d5c21c07b298", "tree": "2d09c27c69678b53b4c9dc486024f3547efd4bca", "parents": [ "99c251bbd225dd97d0deece29559a430b12a0b66" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 12 02:30:58 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 12 16:08:05 2014 -0700" }, "message": "Avoid use of std::string where we have const char*.\n\nRemoving the ClassHelper caused std::string creation for all calls to\nClass::GetDescriptor and a significant performance regression. Make the\nstd::string an out argument so the caller can maintain it and its life time\nwhile allowing GetDescriptor to return the common const char* case.\n\nDon\u0027t generate GC maps when compilation is disabled.\n\nRemove other uses of std::string that are occuring on critical paths.\nUse the cheaper SkipClass in CompileMethod in CompilerDriver.\nSpecialize the utf8 as utf16 comparison code for the common shorter byte\nencoding.\nForce a bit of inlining, remove some UNLIKELYs (they are prone to pessimizing\ncode), add some LIKELYs.\n\nx86-64 host 1-thread interpret-only of 57 apks:\nBefore: 29.539s\nAfter: 23.467s\n\nRegular compile:\nBefore: 1m35.347s\nAfter: 1m20.056s\n\nBug: 16853450\nChange-Id: Ic705ea24784bee24ab80084d06174cbf87d557ad\n" }, { "commit": "4ef12f5b0e26c6016c87866f6a33da5ed8e98d74", "tree": "89ef8dfea38730e61dfed061a7ea71ca1a9b2a62", "parents": [ "47ba978e95896c932300fb3236ce5eb4b438d971" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 31 16:23:49 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Aug 08 14:36:23 2014 -0700" }, "message": "ART: Add guards to the dex cache and its shortcuts\n\nDo not return fields, methods or classes if the (declaring) class is\nerroneous.\n\nBug: 16692788\n\n(cherry picked from commit 58a5af8568d224ca7eccf2483396ff9862f8d1ee)\n\nChange-Id: I7d3e4cb8113e2e764ed7433eed25e1031e6a0f14\n" }, { "commit": "58a5af8568d224ca7eccf2483396ff9862f8d1ee", "tree": "4dfac17316970040e079244523b9b1e2ea163f05", "parents": [ "1b192268f167f603fc372f02f9067f3ce5d82daf" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jul 31 16:23:49 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Aug 06 15:57:05 2014 -0700" }, "message": "ART: Add guards to the dex cache and its shortcuts\n\nDo not return fields, methods or classes if the (declaring) class is\nerroneous.\n\nBug: 16692788\nChange-Id: If43c2414ad0eb22db5eba7cf66396c7f16c26597\n" } ], "next": "2cdbad7c62f126581ec5177104de961c4d71adaa" }