)]}' { "log": [ { "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": "268764da8022cafa5661c5b514eaa343c5257e57", "tree": "7c4b46847a6dcfac7a1492edd79b434fb1a0d6ea", "parents": [ "f9decf915f6fa5b33fceba7c10c119e4cad481e5" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 13 12:09:38 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 13 15:18:29 2016 -0700" }, "message": "Make ScopedAssertNoThreadSuspension no overhead for non-debug\n\nPreviously it required Thread::Current() which may not be free.\nThe plan is to add a lot more ScopedAssertNoThreadSuspension in\nthe codebase.\n\nAlso cleaned up callers.\n\nBug: 31458474\nChange-Id: I5a1621a5435476504d22266cc01a9bf26aab7568\n" }, { "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": "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": "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": "103992b9171fac9eceb2fe4015ab04f544b9c559", "tree": "0e21aeb48e11130f9bfe1c1c42202ba715e30cb6", "parents": [ "376a6f3dbae7b71a6fc2c339ec416d3407277308" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jan 04 15:32:43 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jan 05 08:16:06 2016 -0800" }, "message": "ART: Move ThrowStackOverflowError to common_throws\n\nThis should really be in common_throws considering all the callers.\n\nChange-Id: I410c07cbf5659f216d44597530cb85f99740f320\n" }, { "commit": "e80673245c0433a71a4930e460be5dc0920885b2", "tree": "6c718161fd8861901a5cb67ee60e465010770863", "parents": [ "e6576390f957c82c2aede438834d028066757368" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 08 17:42:59 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Sep 09 10:13:56 2015 -0700" }, "message": "ART: Add missing GetInterfaceMethodIfProxy\n\nAdd missing uses of GetInterfaceMethodIfProxy in reflection code.\n\nAdd a test case for a JNI call to a proxy method.\n\nBug: https://code.google.com/p/android-developer-preview/issues/detail?id\u003d2973\nBug: 23886441\nChange-Id: I5b66b64b5561fcee15d0314707d67e8abc02ce5b\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": "3887c468d731420e929e6ad3acf190d5431e94fc", "tree": "67dacb849e722e33e118b97714a48e467c06cbd5", "parents": [ "6a5037eb3340e4c981fd7de3ff45167ee5b7fc82" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Aug 12 18:15:42 2015 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Wed Aug 12 18:15:42 2015 +0100" }, "message": "Remove unnecessary `explicit` qualifiers on constructors.\n\nChange-Id: Id12e392ad50f66a6e2251a68662b7959315dc567\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": "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": "83c81955bfbfe8fbe6daf215ca73e436747e3e60", "tree": "c4edffdab08d61120c7d6590fa46f9f054a50365", "parents": [ "b71c5878e6607fdc6a6f8624f4e9507d25116a53" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed May 27 19:29:29 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed May 27 19:46:50 2015 -0700" }, "message": "Fix updating of JNI references for String.\u003cinit\u003e.\n\nWas missing updates to globals and weak globals.\n\nBug: 21288130\nBug: 21440428\n\n(cherry picked from commit 1a302fb4f84525289c1cf7a437f5be1999a75251)\n\nChange-Id: I41b64e8aae906f5986de483db096d8191e45f260\n" }, { "commit": "39b6c24ce68cf05db0f82f454b4401cfb03d675f", "tree": "29e15d2597cd05419a1c0faedfe6977af9a7dfa3", "parents": [ "768b19b6bbddaa2262f6532e7fcb96fecbfee981" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue May 19 20:30:23 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed May 20 18:40:43 2015 -0700" }, "message": "Intercept JNI invocation of String.\u003cinit\u003e methods.\n\nlibmono uses JNI AllocObject and CallNonvirtualVoidMethod to create and\ninitialize a string instead of using the recommended NewObject. This\nchange adds an intercept to change the String.\u003cinit\u003e call to a\nStringFactory call instead. Then, it uses the object id of the original\nstring object referrer and maps it to the result of the StringFactory.\n\nBug: 21288130\n\n(cherry picked from commit 15e9ad1d028d7f12cb598b075453173532a00d91)\n\nChange-Id: I3421c43722c07397da4a398c2ca9110e1d40bcfa\n" }, { "commit": "54462546aad8adb552e74e422f5f2fb107b7ee1b", "tree": "6482480a84294f41769e3c8ba42d89c0c989ee00", "parents": [ "a6591ef95afa6bc9d89c0c6044635ec7a13d4c55" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu May 07 17:23:16 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu May 07 17:25:27 2015 +0200" }, "message": "Remove unused InvokeWithShadowFrame\n\nThis is an artifact of the Portable compiler, probably left after its\nremoval https://android-review.googlesource.com/119049.\n\nChange-Id: I7f6caa008e37e7a93792840c79713e90d0c7e1e1\n" }, { "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": "f36cb5f65cb150151aa40b23937e2b0ad75cc546", "tree": "25365c69a089bc491a4b070ee3403be964cfc606", "parents": [ "33984b4d6b79b2d65258e69506e5669be704db82" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 24 16:55:16 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Sat Apr 25 22:08:16 2015 +0000" }, "message": "Move Class.newInstance to native\n\nAvoids 1 allocation and several JNI transitions.\n\nBefore:\nClass_classNewInstance: 4462.39 ns; σ\u003d39.42 ns @ 3 trials\n\nAfter:\nClass_classNewInstance: 1073.39 ns; σ\u003d24.14 ns @ 10 trials\n\nBug: 20269715\nBug: 20566996\nChange-Id: Icd52155ce79a978a4d869855bfdfd7735abd8187\n" }, { "commit": "a61894d88fabe45677f491c9f6bde30059a49026", "tree": "b9d8b09e5f90792867b6720a1fb4ab5c76cdfa5f", "parents": [ "4ceed922d44b68c3fa7cbe670014c9e2e003b92b" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 23 16:32:54 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Apr 23 17:21:26 2015 -0700" }, "message": "Fix reflection handling and test flakiness\n\nFixed reflection invoke to handle exceptions which occur from\nFindClass or NewObject by throwing these instead of\nthe expected InvocationTargetException.\n\nAdded test case to 080 for this reflection invoke.\n\nFixed println throwing OOM in 104-growth-limit.\n\nChange-Id: I65766e7c3478e299da06fdc3a521fe3f3e8fdba9\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": "ca239af73e512df5eeb80fe6c09c2ca614649e06", "tree": "cee8641c42385f40b567c686cd5053fde152a088", "parents": [ "f381645a336f7092ab6f5900c0a2cf183a9dbdf7" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Mar 29 18:27:50 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun Mar 29 18:54:20 2015 -0700" }, "message": "Fix some reflection errors\n\nFixed incorrectly using 2nd frame instead of 1st in VerifyAccess.\nAdded regression test to ART. Fixed broken setShort,\ngetDeclaredFieldInternal.\n\nChange-Id: I4b21d52d998cb768fe9503b8bccec506b7b972e5\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": "0aa50ce2fb75bfc2e815a0c33adf9b049561923b", "tree": "9a3f9603ab30d5cbc7fc21aee0ceb48bbb0dd25a", "parents": [ "e8e42f3548fd894f860912bb1b71ce6fa2d7daf3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 10 11:03:29 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Mar 10 14:51:11 2015 +0000" }, "message": "Remove ThrowLocation.\n\nNote that this is a cleanup change, and has no functionality change.\nThe ThrowLocation had no use anymore.\n\nChange-Id: I3d2126af1dc673cec3a0453ff3d56a172663a5f6\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": "956af0f0cb05422e38c1d22cbef309d16b8a1a12", "tree": "b558c804d206dad8da648b815750f1b3c97610ae", "parents": [ "407d77f344cfbdbbfb50531c5f0766bc0892e2fe" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Dec 11 14:34:28 2014 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Dec 12 09:33:34 2014 -0800" }, "message": "Remove portable.\n\nChange-Id: I3bf3250fa866fd2265f1b115d52fa5dedc48a7fc\n" }, { "commit": "a0485607a4a4d8c683a9849f6f20902c4e1da7a4", "tree": "bbfa8e6182b702f161ea07efd53975fab1b6b736", "parents": [ "08f1f50d6c2e8f247b8f5f19711d75a792851c7a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 15:48:04 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 15:57:13 2014 -0800" }, "message": "Move GetClassFromTypeIdx to ArtMethod.\n\nMove GetClassFromTypeIdx out of MethodHelper into ArtMethod in\npreparation for the removal of MethodHelper.\n\nChange-Id: I9c03dd8c821944c606ea08cdf92afc80c4275247\n" }, { "commit": "e94652f1e321b2c8b71acbe5b07d2ebf69fbdb99", "tree": "b4c2a4435800222fa740a5ae57fa217b6aed0875", "parents": [ "f25c2ec6b63e116f24f359a10b59c78768fde67a" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 11:13:19 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 02 11:13:49 2014 -0800" }, "message": "Remove MethodHelper from the interpreter.\n\nUse ShadowFrame to get the executing method to avoid a handle for the current\nmethod.\nVarious associated bits of header file clean-up and removal of an unnecessary\nuse of MethodHelper in CompilerDriver.\n\nChange-Id: I3b6f4413701e8fc6b0c58b0041a0dd15472bedaa\n" }, { "commit": "6a3c1fcb4ba42ad4d5d142c17a3712a6ddd3866f", "tree": "9df58b57af13240a93a6da4eefcf03f70cce9ad9", "parents": [ "c6e0955737e15f7c0c3575d4e13789b3411f4993" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 31 00:33:20 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Nov 03 20:01:04 2014 -0800" }, "message": "Remove -Wno-unused-parameter and -Wno-sign-promo from base cflags.\n\nFix associated errors about unused paramenters and implict sign conversions.\nFor sign conversion this was largely in the area of enums, so add ostream\noperators for the effected enums and fix tools/generate-operator-out.py.\nTidy arena allocation code and arena allocated data types, rather than fixing\nnew and delete operators.\nRemove dead code.\n\nChange-Id: I5b433e722d2f75baacfacae4d32aef4a828bfe1b\n" }, { "commit": "ded66a01f81812e0129d17c3d08d5eda18433062", "tree": "74a3f7bad1c020b0136d6c173633a361a479f5cb", "parents": [ "e0205d519dd417f749243b42b8711dfa313e5390" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 28 18:12:55 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 28 18:36:32 2014 -0700" }, "message": "Move MethodHelper::GetReturnType to mirror::ArtMethod.\n\nAlso, fix missing handle in HasSameSignatureWithDifferentClassLoaders.\n\nChange-Id: I9e1ffd09be950ecc8346fc3c485760d82d9ecab3\n" }, { "commit": "6f3dbbadf4ce66982eb3d400e0a74cb73eb034f3", "tree": "f7a20779e4d665f948c5fbcd26dac0071dafb8d4", "parents": [ "2df6840f68dd18d7dd8dbf53f8b6181bbfdc4fc4" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Oct 14 17:41:57 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 16 19:27:28 2014 -0700" }, "message": "Make ART compile with GCC -O0 again.\n\nTidy up InstructionSetFeatures so that it has a type hierarchy dependent on\narchitecture.\nAdd to instruction_set_test to warn when InstructionSetFeatures don\u0027t agree\nwith ones from system properties, AT_HWCAP and /proc/cpuinfo.\nClean-up class linker entry point logic to not return entry points but to\ntest whether the passed code is the particular entrypoint. This works around\nimage trampolines that replicate entrypoints.\nBug: 17993736\n\nChange-Id: I5f4b49e88c3b02a79f9bee04f83395146ed7be23\n" }, { "commit": "c0d8229898c44c0f604f08a5df1de83ff56c18fd", "tree": "13b150e68e806204c547e04bc071246c06199c37", "parents": [ "c70535b4f9f1ff3e3da451734bb7d9601012ccc1" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 23 10:38:30 2014 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 29 11:40:16 2014 -0700" }, "message": "ART: Better IllegalAccessException message\n\nBug: 17618578\nBug: 17614623\nChange-Id: I0e3f15e676acd6ed5844fc86e136f75cc335372d\n" }, { "commit": "76433275dbe39e5ced1c223b006d1b900b1937f6", "tree": "a72c101defa86ceff8b633cf2eaa606c87f783eb", "parents": [ "88adf36315237947d3a5d10b28fdf8b9da7794a4" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 26 14:32:37 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 26 19:54:39 2014 -0700" }, "message": "Optimize get/set reflection performance\n\nSpeedups mostly from reducing how often access checks are needed,\nand adding more inlining, and adding templates.\n\nField_getInt from ~850ns -\u003e 350ns.\nField_setInt from ~900ns -\u003e 370ns.\n\nBug: 14063288\n\n(cherry picked from commit ffc788cb7b5b9f53734d7bb8af2d5e45d885546b)\n\nChange-Id: I2441581ff3478c6ae43b6aa49939ff3f07555ec8\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": "b74ff132f217776c1ab698712bc2dbc526297288", "tree": "86da3e98cbd942bb4c2fda6f0f50fd44c1c0a101", "parents": [ "826f1038187e00399a8c6e0044ad0ea29c0ba000", "648d7112609dd19c38131b3e71c37bcbbd19d11e" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Thu Aug 14 00:01:12 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Aug 12 19:23:43 2014 +0000" }, "message": "Merge \"Reduce stack usage for overflow checks\"" }, { "commit": "648d7112609dd19c38131b3e71c37bcbbd19d11e", "tree": "54062831327c660acb309e877e8d8df9ba0c2d5d", "parents": [ "99c251bbd225dd97d0deece29559a430b12a0b66" ], "author": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Fri Jul 25 16:15:27 2014 -0700" }, "committer": { "name": "Dave Allison", "email": "dallison@google.com", "time": "Wed Aug 13 09:01:41 2014 -0700" }, "message": "Reduce stack usage for overflow checks\n\nThis reduces the stack space reserved for overflow checks to 12K, split\ninto an 8K gap and a 4K protected region. GC needs over 8K when running\nin a stack overflow situation.\n\nAlso prevents signal runaway by detecting a signal inside code that\nresulted from a signal handler invokation. And adds a max signal count to\nthe SignalTest to prevent it running forever.\n\nAlso reduces the number of iterations for the InterfaceTest as this was\ntaking (almost) forever with the --trace option on run-test.\n\nBug: 15435566\n\nChange-Id: Id4fd46f22d52d42a9eb431ca07948673e8fda694\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": "68d8b42ddec39ec0174162d90d4abaa004d1983e", "tree": "0bb6ccf3b996bb8363c10d07aa39cde221980602", "parents": [ "9c522c2cbbf50dc687728747b37ad59985750b65" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 17 11:09:10 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Aug 08 08:29:53 2014 -0700" }, "message": "Wire up check JNI force copy mode.\n\nIncrease check JNI checks.\nBreak apart jni_internal.h in to jni_env_ext.h and java_vm_ext.h.\nFix the abuse of ScopedObjectAccess/annotalysis by ScopedCheck in the case\nof VM routines.\nMake class loader override and shared library class loader JNI global\nreferences rather than mirror pointers.\nClean-ups to native bridge.\n\nChange-Id: If7c6110b5aade7a402bfb67534af86a7b2cdeb55\n" }, { "commit": "e5877a12c30afe10a5c6a1afaff7a47ef44a2a5f", "tree": "161535abb39394fc3b1d96d31625ba7eb61c8497", "parents": [ "aab012d6196bd29b3167963ec8acb0b9780672b2" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jul 16 12:06:35 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jul 16 12:06:35 2014 -0700" }, "message": "Fix x86 build.\n\nAlso fix attributes/annotalysis on entrypoint_utils functions now we have\nclang that is smarter wrt warnings than GCC.\n\nChange-Id: I69257b4ad9a27d07acbc973d21a1cfa4260a8ed6\n" }, { "commit": "22d5e735f403c57525fe868304c7123f0ce66399", "tree": "2458684efa56f0b800dd75a9dedd0449f76f581f", "parents": [ "fbde4dd1cb6db729e3f3ee5bdae0cdd824d73054" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 22:23:51 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jul 16 06:13:46 2014 -0700" }, "message": "Remove object_utils.h.\n\nBreak into object_lock, field_helper and method_helper.\nClean up header files following this.\nAlso tidy some of the Handle code in response to compiler errors when resolving\nthe changes in this CL.\n\nChange-Id: I73e63015a0f02a754d0866bfaf58208aebcaa295\n" }, { "commit": "e63db27db913f1a88e2095a1ee8239b2bb9124e8", "tree": "893dee6783bca6717259321a6e4ba029c9c123e2", "parents": [ "07b8441303ea82fca3cb85d71ecf8752d73cedd7" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 15:36:11 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jul 15 17:07:49 2014 -0700" }, "message": "Break apart header files.\n\nCreate libart-gtest for common runtime and compiler gtest routines.\nRename CompilerCallbacksImpl that is quick compiler specific.\nRename trace clock source constants to not use the overloaded profiler term.\n\nChange-Id: I4aac4bdc7e7850c68335f81e59a390133b54e933\n" }, { "commit": "3bd7a6c1716935e758f230e2f199128cb3c28b42", "tree": "fda9d595f8ee976fcb309b94abebcd859db76d02", "parents": [ "dfc2091d2fb8a7694f69acf8bd39ce4953e026c2" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jun 12 15:22:31 2014 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Jun 12 18:08:44 2014 +0100" }, "message": "Fix reflection access check for attached native threads.\n\nBug: 15539150\nChange-Id: Ie580cad1c0169d550162bcc878b07643259323c4\n" }, { "commit": "bfd9a4378eacaf2dc2bbe05ad48c5164fc93c9fe", "tree": "3d3f667c8232a9c1bb6fe9daea0d364f9ae01d8c", "parents": [ "2e1ca953c7fb165da36cc26ea74d3045d7e272c8" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 21 17:43:44 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Jun 09 12:46:32 2014 -0700" }, "message": "Change MethodHelper to use a Handle.\n\nAdded ConstHandle to help prevent errors where you modify the value\nstored in the handle of the caller. Also fixed compaction bugs\nrelated to not knowing MethodHelper::GetReturnType can resolve types.\nThis bug was present in interpreter RETURN_OBJECT.\n\nBug: 13077697\n\nChange-Id: I71f964d4d810ab4debda1a09bc968af8f3c874a3\n" }, { "commit": "607fa7b07233a7233ebe21bba8f3e7c1925ae0f2", "tree": "a04f1074331ac82ad077044e3f373312f61e5901", "parents": [ "1dc387152a84193937bd1f88e9e10fff6449d695", "700a402244a1a423da4f3ba8032459f4b65fa18f" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue May 20 05:29:04 2014 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue May 20 05:29:04 2014 +0000" }, "message": "Merge \"Now we have a proper C++ library, use std::unique_ptr.\"" }, { "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": "2b7c4d196c8abe32f4ca633534917da9de53c359", "tree": "f14dc45f53d5681a5c0e57b8a8cc6b35eee6e896", "parents": [ "84e524207b23d58a1b1e5f4443000ccac97c4184" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon May 19 10:52:16 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon May 19 15:19:28 2014 -0700" }, "message": "Don\u0027t get and restore thread state for ScopedFastNativeObjectAccess.\n\nBefore we would ensure that we were runnable for fast native object\naccess. However, these are done when you are already runnable.\n\nChange-Id: Ia4c6e4c83d146fe2a988b37b3133ca46b0f0fa42\n" }, { "commit": "f832284dd847ff077577bb5712225430bbbb3b67", "tree": "44f6b91098639c6ebc438b4ec998d0dc128cef9a", "parents": [ "8f0776768712b2021aa8fb649b51017b9f0fc7a9" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 16 10:59:25 2014 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Sun May 18 12:50:33 2014 -0700" }, "message": "Delete ClassHelper and fix compaction bug in GetDirectInterface\n\nCleanup helps to prevent compaction bugs. Fixed a fairly serious\ncompaction error caused by calling ClassHelper::GetDirectInterface\nwithout handling the case where it causes thread suspension due to\nResolveType.\n\nBug: 8981901\n\nChange-Id: I82b3bb6dd48d21eb6ece7aae0733c4a23c2bc408\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": "925b687c29ddfe952c29e13e6c05c2491e3b37f7", "tree": "de09aba0c9f2419feb880f8842d59202a27f7c9f", "parents": [ "d265a8ab12e1b783af42bf07a264edbb86826d2c" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Apr 01 11:21:30 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Apr 01 18:28:13 2014 +0000" }, "message": "Fix reflection access checks for app compatibility.\n\nAlso updates field access test.\n\nBug: 13740995\n\nChange-Id: I224712cfcb0c05429b831c26980cabad7bdef49a\n" }, { "commit": "dd9699a536e5d0c1a10b4bdb37b88312930793e8", "tree": "39a7c5c3d6ed49561058826a4c52063090684c5a", "parents": [ "b59b2c8dbdea2a6a3086378f2cf65adc9bc282cd" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Apr 01 18:04:24 2014 +0000" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Apr 01 18:04:24 2014 +0000" }, "message": "Revert \"Fix access checks for app compatibility.\"\n\nThis reverts commit b59b2c8dbdea2a6a3086378f2cf65adc9bc282cd.\n\nChange-Id: I718658b8450a5ce6249df317b635f63356b1ce3c\n" }, { "commit": "b59b2c8dbdea2a6a3086378f2cf65adc9bc282cd", "tree": "18dcb6d665b27cd66e56db598da4a31f8b0d3459", "parents": [ "9f42a5b81d4e6ff20a588c70f17c21da04c29123" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Apr 01 10:58:29 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Apr 01 10:58:29 2014 -0700" }, "message": "Fix access checks for app compatibility.\n\nChange-Id: I3191e5e92d30e9bee24051fd305296ddcf143e18\n" }, { "commit": "cb4581aa13d6f43f705535818a4d0893d551be3a", "tree": "903cbcb3852650c7e3105c0c1819e6e480091cd4", "parents": [ "e759c9f01b558093466fc78f01900042120cd832" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 28 15:43:37 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 28 15:43:37 2014 -0700" }, "message": "Some fixes for comments and implied conversions.\n\nAddresses comments in:\nhttps://android-review.googlesource.com/#/c/89148/3\n\nChange-Id: If21cfaa541210c8702371efd1e6d4f071a7b9ec3\n" }, { "commit": "11d5d8fffe41cc7daadbfa2ca98ecb978f3029af", "tree": "fae2cf974b502420a942043d135d301aa75967e7", "parents": [ "a708e32a9f764a48175e705ec4bcd2201c84f492" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Mar 26 15:08:20 2014 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 28 14:27:55 2014 -0700" }, "message": "Add access checks to Method and Field reflection.\n\nArt side of this change. Has a corresponding libcore change.\n\nBug: 13620925\nChange-Id: Ie67f802a2a400e8212b489b9a261b7028422d8ba\n" }, { "commit": "84956ff6c0ff150d86a08157216ded79217d860a", "tree": "04177625872b89f27f00a53b574085416617ee0d", "parents": [ "636583c586b78e188168e510452cef0645578f8d" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Mar 26 23:52:41 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Mar 26 23:56:58 2014 -0700" }, "message": "Reflection tidying.\n\nMove out arguments to the right and make pointer not reference types.\nRemove unused unbox for argument routine.\nSimplify convert primitive routine for the case of identical types.\n\nChange-Id: I6456331b0f3f3e5f0b2c361a9f50b4ed1c9462a3\n" }, { "commit": "62f0512bf6d9bc6141358bf22e93afa70dc58b1a", "tree": "7fdc0b9be0df2da5ff2575862a24db5c10eae9dc", "parents": [ "215f3e60cd97471c0843eae22a041c335e89071c" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 21 11:21:29 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 21 11:25:56 2014 -0700" }, "message": "Improvements to Field.get/set.\n\nAvoid unnecessary repeated computation in Field.get/set.\nRefactor FromReflectedField and FromReflectedMethod into common helpers in\nmirror::ArtField and mirror::ArtMethod, and make use of thereby avoiding\ntransitions through JNI.\nAvoid JNI use from within FromReflectedField and FromReflectedMethod.\nTidy up Field.get/set wrt moving collector support.\nBug: 12189533\n\nChange-Id: I643ab3474bade4abac3a3ae2b6e373b2bb0891c8\n" }, { "commit": "e18fdd2f0e57873c22113975739139abe7418e09", "tree": "0bb8bbe557a8090b90ae9dc8fe20f83ddd68c5a8", "parents": [ "0735a7eb23d430b3f7276294a3a436eddcf07b98" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 14 13:29:43 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 14 13:29:43 2014 -0700" }, "message": "Work-around GCC polymorphism problems.\n\nChange-Id: Ica1d8e6840cdd9ba493405e66c1728b5da9540a2\n" }, { "commit": "11e4c038c5e319ee7dd423ff47ef9411920d198e", "tree": "b0de20323134798482e43955e961aa751bf1761b", "parents": [ "53b8b09fc80329539585dcf43657bc5f4ecefdff" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 14 12:00:39 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 14 12:00:39 2014 -0700" }, "message": "64bit build fix.\n\nChange-Id: I8c597217955d12b215bdcf1d92ca5fab9a7ed6e0\n" }, { "commit": "53b8b09fc80329539585dcf43657bc5f4ecefdff", "tree": "cac0f82fbb89bd907104e3fed6c36203e11a3de0", "parents": [ "0dea9872082bc3e576ed6cefed86b0d6c0c45ffd" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Mar 13 23:45:53 2014 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Mar 14 11:28:10 2014 -0700" }, "message": "Refactor reflective method invocation.\n\nMove invocation code out of JNI internal into reflection, including ArgArray\ncode. Make reflective invocation use the ArgArray to build arguments rather\nthan allocating a jvalue[] and unboxing arguments into that.\nMove reflection part of jni_internal_test into reflection_test.\nMake greater use of fast JNI.\n\nChange-Id: Ib381372df5f9a83679e30e7275de24fa0e6b1057\n" }, { "commit": "0177e53ea521ad58b70c305700dab32f1ac773b7", "tree": "db15627a19a04634cf84cecd15b813319d80d225", "parents": [ "abaf927f29f6feceb3df3e6ced7d01970ba0dbe9" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 11 16:30:46 2014 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 11 17:02:21 2014 -0800" }, "message": "Work in the direction of hard float quick ABIs.\n\nPass a shorty to ArtMethod::Invoke so that register setup can use it.\nDocument x86-64 ABI.\nAdd extra debug output for one JNI native method registration fails, namely a\ndump of the Class and its dex file\u0027s location.\nAdd hack to get testing of OatMethod\u0027s without GC maps working in 64bit.\n\nChange-Id: Ic06b68e18eac33637df2caf5e7e775ff95ae70f3\n" }, { "commit": "c528dba35b5faece51ca658fc008b688f8b690ad", "tree": "d95c113a639d07f7e3b88e8a6caa9b5f6e044801", "parents": [ "76f552307ee980221626d1dff0ada56c989d24ca" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Nov 26 12:00:11 2013 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Dec 10 16:00:58 2013 -0800" }, "message": "Enable moving classes.\n\nSlight reduction in Zygote size, memory savings are in the noise.\nBefore: Zygote size: 8739224\nAfter: Zygote size: 8733568\n\nFixed a bug where we didn\u0027t set the concurrent start bytes after\nswitching the allocator from bump pointer to ROSAlloc in the\nzygote. This caused excessive memory usage.\n\nAdded the method verifiers as roots to fix an issue caused by\nRegTypes holding a Class*.\n\nAdded logic to clear card table in the SemiSpace collector, this\nreduces DalvikOther from ~2400k -\u003e ~1760k when using the SemiSpace\ncollector.\n\nAdded a missing lock to the timing loggers which caused a rare\none time crash in std::set.\n\nBug: 11771255\nBug: 8499494\nBug: 10802951\n\nChange-Id: I99d2b528cd51c1c5ed7012e3220b3aefded680ae\n" }, { "commit": "dfb325e0ddd746cd8f7c2e3723b3a573eb7cc111", "tree": "0b63738b04404f42015da7164dfe5ec506452b02", "parents": [ "eda6df7374bf61c933d4a36635e2a9a30c9892a0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 30 01:00:44 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Oct 30 01:00:44 2013 -0700" }, "message": "Don\u0027t use UTF16 length as length for MUTF8.\n\nBug 11367555.\n\nChange-Id: Ia0b07072a1a49d435c3b71ed9a668b316b7ff5d8\n" }, { "commit": "fc0e94bed3f88ed7e50854fd8dfaf5dcb345250f", "tree": "5cfbe05084351576e9659cb8f7b66dcb6163a37b", "parents": [ "576fe9d4181c749aa510e32d2521ed4192bdfda0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Sep 23 23:51:32 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Sep 24 17:07:24 2013 -0700" }, "message": "StringPiece clean up.\n\nProfile guided clean up.\nTry to avoid creating StringPieces with the contents of a dex file where\nthe length is known.\nTry to avoid RegTypeCache::FromDescriptor when there\u0027s a class available.\nMake ConstantType::ConstantValue inlinable.\nSaving of about 50ms from a 2 threaded ThinkFree compile on host.\n\nChange-Id: I47a12c3c76f46e2c9805be1c3a3e3870fe1f5d85\n" }, { "commit": "2e450bf45e1bacc9c356f6ab239ccfb31bd8d7e4", "tree": "d695b5a0aa5049ed3936a65ab69b409e2eceb818", "parents": [ "dd3c27eeb8286ab53d8a2f1aec76a7a919ba353a" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Sep 06 15:39:46 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Sep 06 15:43:51 2013 -0700" }, "message": "Revert \"Remove bogus fastpath from String::Equals(const StringPiece\u0026)\"\n\nThis reverts commit 8438ed31e10f3881ed92f03877d5edaca7d5b48c.\n\nBug: 10614658\nChange-Id: I335f10a7140e1644957bc1cee21a9b310a558499\n" }, { "commit": "8438ed31e10f3881ed92f03877d5edaca7d5b48c", "tree": "a29524e6974a58d593362b49217649dc9a39f496", "parents": [ "51db7beb7faaea43d697321520732d1d71288f50" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Sep 04 17:17:19 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Sep 06 12:38:20 2013 -0700" }, "message": "Remove bogus fastpath from String::Equals(const StringPiece\u0026)\n\nBug: 10614658\nChange-Id: I907ec77a65c1ae29e800356abdf755a457620081\n" }, { "commit": "ea46f950e7a51585db293cd7f047de190a482414", "tree": "9dddc8073547a2dcb58a19e1728932a89cb149c3", "parents": [ "5e3572709b5a5d59957f835db4f73760ecef08da" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Jul 30 01:26:50 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 13 18:09:46 2013 -0700" }, "message": "Refactor java.lang.reflect implementation\n\nCherry-picked from commit ed41d5c44299ec5d44b8514f6e17f802f48094d1.\n\nMove to ArtMethod/Field instead of AbstractMethod/Field and have\njava.lang.reflect APIs delegate to ArtMethod/ArtField.\n\nBug: 10014286.\n\nChange-Id: Iafc1d8c5b62562c9af8fb9fd8c5e1d61270536e7\n" }, { "commit": "7934ac288acfb2552bb0b06ec1f61e5820d924a4", "tree": "43f3acd8af7fd34d4ae7b64f6e06bb8429d74bb8", "parents": [ "fb331d7ca004f39608fcfdae49d38df90c702ea9" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 26 10:54:15 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 26 11:55:10 2013 -0700" }, "message": "Fix cpplint whitespace/comments issues\n\nChange-Id: Iae286862c85fb8fd8901eae1204cd6d271d69496\n" }, { "commit": "df62950e7a32031b82360c407d46a37b94188fbb", "tree": "038bf95a2ce296ae3e0c30a131ac22c0986f0f52", "parents": [ "0cd7ec2dcd8d7ba30bf3ca420b40dac52849876c" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Wed Jul 17 22:39:56 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Thu Jul 18 00:13:18 2013 -0700" }, "message": "Fix cpplint whitespace/parens issues\n\nChange-Id: Ifc678d59a8bed24ffddde5a0e543620b17b0aba9\n" }, { "commit": "7940e44f4517de5e2634a7e07d58d0fb26160513", "tree": "ac90242d96229a6942f6e24ab137bc1f8f2e0025", "parents": [ "5cd9e3b122f276f610980cbaf0d2ad6ed4cd9088" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 13:46:57 2013 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Fri Jul 12 17:49:01 2013 -0700" }, "message": "Create separate Android.mk for main build targets\n\nThe runtime, compiler, dex2oat, and oatdump now are in seperate trees\nto prevent dependency creep. They can now be individually built\nwithout rebuilding the rest of the art projects. dalvikvm and jdwpspy\nwere already this way. Builds in the art directory should behave as\nbefore, building everything including tests.\n\nChange-Id: Ic6b1151e5ed0f823c3dd301afd2b13eb2d8feb81\n" }, { "commit": "62d6c772205b8859f0ebf7ad105402ec4c3e2e01", "tree": "e2f2ba6d71ed5a39c9f6909e3f7c08e998053315", "parents": [ "c9b17c7ee96cd04fac9048aab624ed554fe260bf" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Feb 27 08:32:07 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Apr 08 14:24:13 2013 -0700" }, "message": "Interpreter entries and instrumentation as a listener.\n\nMake the instrumentation responsible for whether we want method entry/exit\nstubs, and allow it to use interpreter entry stubs when instruction by\ninstruction instrumentation is required. Improve deoptimization so more JDWP\ntest cases are passing.\n\nRefactor exception debug posting, in particular improve reporting in the\ninterpreter. Improve class linker exception throwing so that broken dex files\nare more likely to be reported. Fixes the performance issue Bug: 8410519.\n\nFix some error reporting lock level errors for the large object space. Make\nfast object verification faster.\n\nAdd some debug mode robustness to finding dex PCs in GC maps.\n\nAdd printf attributes to JniAbortF and fix errors.\n\nExpand run-test 044 to test return behaviors and fix issues with not throwing\nappropriate exceptions for proxies.\n\nEnsure causes are reported with a class linker NoClassDefFoundError and JNI\nNoSuchFieldError.\n\nRemove unused debugMe and updateDebuggerFromCode.\n\nThere\u0027s a minor sizing tweak to the arg array builder, and an extra reference\narray check in the interpreter.\n\nSome clean-up of trace code.\n\nFix reg type cache destructor if it is called after the reg type cache is\nshutdown (as is the case in oatdump).\n\nChange-Id: I6519c7b35df77f978d011999354c864f4918e8ce\n" }, { "commit": "6474d190d5604898354ebf767f1944b6e3e9b445", "tree": "ac4be675c62fd8129fe85f740dcd2e3bfee424e2", "parents": [ "52003fe9e2020e9d68cee1300fdb4a3f55be85a5" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Mar 26 14:08:09 2013 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Mar 26 16:03:53 2013 -0700" }, "message": "Remove use of compiled invoke stubs from portable.\n\nNow the invoke stubs can be safely removed. Tested and working on ARM,\nbasic testing done on x86/MIPS, but portable is currently broken for them\neven without the change.\n\nChange-Id: Ib73f2b7aa9d81f5f0e1e817d16b9bec464c5a5aa\n" }, { "commit": "4f6ad8ab428038129b2d0d6c40b7fd625cca15e1", "tree": "d1025ed9203d0cdf7aa959d9cecd8dc1f0c0bfeb", "parents": [ "aabe8adacc7bf3a8ab6f6f3dd37fc839b6fd762f" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Mar 18 15:27:28 2013 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Mon Mar 18 15:29:59 2013 -0700" }, "message": "Various performance improvements.\n\nPerformance had regressed due to verify object and method invocation changes.\nAvoid trampolines for static calls in same class.\nVarious inlining changes.\nMake verify object something that\u0027s only compiled-in in debug builds.\n\nChange-Id: Ia261a52232c3b10667c668f8adfadc0da3048bc5\n" }, { "commit": "5d9173014c1ca09f7249a6b07629aa37778b5f8f", "tree": "80543dbe9af6795212373bb12532b638ff3b22f3", "parents": [ "aed0716b9592bb3095cdfc4b111011f9ed74877d" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Feb 27 17:57:33 2013 -0800" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Mar 15 14:18:28 2013 -0700" }, "message": "Changes to remove need for compiled invoke stubs for quick.\n\nARM, x86, and MIPS implementation complete, though MIPS is untested.\n\nThe ArgArray is changed to be a uint32_t array instead of a JValue array.\nAlso, a separate result for float/double was needed for x86/MIPS. The invoke\nstubs are currently still there, but only used for portable.\n\nChange-Id: I0647f8d5d420cea61370e662e85bdc0c13b5e378\n" }, { "commit": "c0fa3ad44a84b7f658d16a717027bf95abc85db6", "tree": "fd6b01ebb129a624da2cf73c1e27b5c488fccb47", "parents": [ "b6c31ea69973b41d57146ab0c319613213a9a85e" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 05 00:11:55 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Feb 05 10:56:54 2013 -0800" }, "message": "Improve performance of JNI field operations.\n\nChange improves performance of JniField Fadden test by around 25%, we\u0027re still\n2x slower than Dalvik.\nAggressively inline ScopedObjectAccess, Thread::SetState and field helpers.\nIf we\u0027re not contention logging don\u0027t call MilliTime (avoids a double register\nspill).\nRemove (broken?) thread checks in scoped object access, they are redundant with\nones being performed in check JNI.\n\nChange-Id: I128eed1e4205d4d540d5c6f430ef9e3853745585\n" }, { "commit": "2dd0e2cea360bc9206eb88ecc40d259e796c239d", "tree": "9d619dc9508cbe73e4793bf6f08cbc761abfb48a", "parents": [ "ac21b797b3a425975d656d6b84a7b24401d35f42" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jan 24 12:42:14 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 30 16:48:31 2013 -0800" }, "message": "Directory restructuring of object.h\n\nBreak object.h into constituent files.\nReduce number of #includes in other GC header files.\nIntroduce -inl.h files to avoid mirror files #include-ing each other.\nCheck invariants of verifier RegTypes for all constructors.\n\nChange-Id: Iecf1171c02910ac152d52947330ef456df4043bc\n" }, { "commit": "af6e67a4816d2593586115b89faa659225363246", "tree": "e3b330d6ed994f82972da4e5b8c3d06be2c66eae", "parents": [ "a6f3aaf8ca5385d870c35d1555b7787d56a70c95" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 16 08:38:37 2013 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Jan 16 15:37:56 2013 -0800" }, "message": "Further proxy refactorings.\n\nFactor the crawling of the quick stack arguments into a common visitor.\nFactor the proxy invocation dispatch into common runtime support code,\nfix numerous bugs relating to GC in the LLVM runtime support with this.\nClean up BoxPrimitive to not use an in argument as an out.\n\nChange-Id: I7b12c8d88d5083614e480b8fb1d2f2ef7c0a51b7\n" }, { "commit": "50b35e2fd1a68cd1240e4a9d9f363e11764957d1", "tree": "4f3c58b7b293380f19e4f33357cb352e3330772e", "parents": [ "8e9a1497f0d7da4d55f6e7ed8a7d96ba6db7222d" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Oct 04 10:09:15 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 05 11:07:39 2012 -0700" }, "message": "Explicitly pass Thread::Current to MutexLock and Alloc.\n\nChange-Id: I8b75bc0617915465f102815b32306aa7760dcae4\n" }, { "commit": "66f19258f9728d4ffe026074d8fd429d639802fa", "tree": "fd94009774c6cbbb1528ea096e606133bd35f104", "parents": [ "a5e1e3d153990845d80cb8d013157210f11a473c" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 18 08:57:04 2012 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Sep 21 17:32:56 2012 -0700" }, "message": "Change dex cache to be java object instead of array, add pointer to dex file in dex cache.\n\nGeneric clean up to facilitate having GDB macros for Pretty* helper functions.\n\nImproved cleanliness of DexCache since having it as an object array was not the best solution.\n\nFixed a bug in InOrderWalk caused by ResolveType sometimes allocating classes.\n\nRename C++ Method to AbstractMethod and add two new classes Constructor, Method which both inherit from AbstractMethod.\n\nRename done to have the C++ code be closer to the java code.\n\nChange-Id: I4995b4c5e47a3822192b08afa24a639d3b1f4da9\n" }, { "commit": "b726dcb581bf72da46527378ccb6889020f0e6e9", "tree": "c0383ed788652926e93147e50d659ed226519548", "parents": [ "fd678beb171a4686a4f2d53ca4188a4ade8fa54e" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 05 08:57:23 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Sep 05 09:06:19 2012 -0700" }, "message": "Rename GlobalSynchronization to Locks\n\nAlso address some review comments in common_throws relating to\nToStr\u003cInvokeType\u003e and exception detail messages.\n\nChange-Id: Ibf2c0f147689fa236d349bd7f01eed3c2522552b\n" }, { "commit": "00f7d0eaa6bd93d33bf0c1429bf4ba0b3f28abac", "tree": "6a2172ece15c4699e6c2a67ce76f019db0a9a21d", "parents": [ "634ea28f78c4a138e6a1de54eae8696095422415" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Thu Jul 19 15:28:27 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Aug 14 10:45:52 2012 -0700" }, "message": "Global lock levels.\n\nIntroduce the notion of the mutators/GC being a shared-exclusive (aka\nreader-writer) lock. Introduce globally ordered locks, analysable by\nannotalysis, statically at compile time. Add locking attributes to\nmethods.\n\nMore subtly, remove the heap_lock_ and split between various locks that\nare held for smaller periods (where work doesn\u0027t get blocked). Remove\nbuggy Dalvik style thread transitions. Make GC use CMS in all cases when\nconcurrent is enabled. Fix bug where suspend counts rather than debug\nsuspend counts were sent to JDWP. Move the PathClassLoader to\nWellKnownClasses. In debugger refactor calls to send request and\npossibly suspend. Break apart different VmWait thread states. Move\nidentity hash code to a shared method.\n\nChange-Id: Icdbfc3ce3fcccd14341860ac7305d8e97b51f5c6\n" }, { "commit": "365c10235438607541fa2259a5fec48061b90bd8", "tree": "26516a4985a8bbef4c8da36b99bf1e54af3ad24d", "parents": [ "147440427984816b6ed5235a9b1fb2cbf99496d0" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Jun 22 15:05:28 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Jun 26 15:21:01 2012 -0700" }, "message": "Increase use of ScopedJniThreadState.\n\nMove the routines for changing Object* to jobject and vice-versa\n(AddLocalReference and Decode) to ScopedJniThreadState to enforce use of\nObject*s in the Runnable thread state. In the Runnable thread state\nsuspension is necessary before GC can take place.\n\nReduce use of const ClassLoader* as the code bottoms out in FindClass\nand with a field assignment where the const is cast away (ie if we\u0027re\nnot going to enforce the const-ness we shouldn\u0027t pretend it is).\n\nRefactor the Thread::Attach API so that we\u0027re not handling raw Objects on\nunattached threads.\n\nRemove some unreachable code.\n\nChange-Id: I0fa969f49ee6a8f10752af74a6b0e04d46b4cd97\n" }, { "commit": "eac766769e3114a078c188ea26776a81f0edb3cf", "tree": "4b9b8dfc89725173431817baba4455c013e40577", "parents": [ "3f9ace8d90bfc48c8d7bf35af66de8ce8238de7f" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu May 24 21:56:51 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri May 25 12:04:43 2012 -0700" }, "message": "DO NOT MERGE Fix all unsafe caching to be like libcore.\n\nThis way, if a runtime is restarted within a process, we re-initialize all\nthe cached data.\n\nConflicts:\n\n\tsrc/native/java_lang_Runtime.cc -- nativeExit lost an argument in dalvik-dev\n\n(cherry picked from commit 7756d5473fa27ce7e6ac7c31770eef7030431da4)\n\nChange-Id: I6184fc20c2a9ec16c4b053584a4d1c3b64452d0f\n" }, { "commit": "37f7775b92398d87e48bc306f9d10a1b563dcdd5", "tree": "f2a949a1dc14c10c44b10333df7d8534da36df2a", "parents": [ "deafc8f6558e3ceae5963528f240a9f1545a6f32" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon May 21 15:12:47 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon May 21 15:12:47 2012 -0700" }, "message": "Slightly clearer reflection.\n\nChange-Id: I6529e50f230e60c7459150718f14fe443a1b42a0\n" }, { "commit": "84a5bb4556d8974d3aa1c436af36e48d622b4045", "tree": "0e0457b0c6489e30f50b06be4f6b21e11659d876", "parents": [ "aaa5edcf2deb1bddcbf5fb27820ad2240ac5b4f2" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed May 16 17:52:15 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed May 16 23:35:29 2012 -0700" }, "message": "Even better reflection IllegalArgumentException detail messages and less code.\n\nAlso ensure that the new 200-reflection-errors run-test test gets run\nby the \"test-art\" make target.\n\nBug: 6504175\nChange-Id: I5afee8845715151d25ed4751381b6ceb8cb4eca5\n" }, { "commit": "aaa5edcf2deb1bddcbf5fb27820ad2240ac5b4f2", "tree": "c2d0f408237ad5a30bfd67819e5c9a72cc3ac45e", "parents": [ "983f2e411aee6b1d09e6da30e059b782b2699909" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed May 16 15:54:30 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed May 16 15:54:30 2012 -0700" }, "message": "Improve reflection IllegalArgumentException detail messages.\n\nAlso add a missing InstanceOf check that was causing CheckJNI to kill us\nif someone tried to pass an inappropriate reference type through\nMethod.invoke. (Amusingly, CheckJNI produced pretty much the exact detail\nmessage that Method.invoke should have.)\n\nPlus a new test for this stuff.\n\nBug: 6504175\nChange-Id: Ice95eecbdba5a0927c6eaf68e56d6500dc52ad2e\n" }, { "commit": "f24d3cedd395690f6904aaac80f84a100420f7a3", "tree": "1867ff0b6f3bebb4f611eaf72b05b14843c55d93", "parents": [ "14fe6aea00598ba5d40d3e5bfa62f26a1c8cfdff" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Apr 11 17:43:37 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Apr 11 17:43:37 2012 -0700" }, "message": "Enforce the use of accessors on JValue so we can ensure sign extension.\n\nChange-Id: I34810af7661d8674c84707afe5cfdf9e2c233f8b\n" }, { "commit": "1d878f3e1c39162496d7c061f7f6710e414d182b", "tree": "203a26165bff88dd352580c088c0775b953d363a", "parents": [ "4b952e74b4b2efa15cecedfc643c212eaeac2cc7" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Apr 11 15:17:54 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Apr 11 15:17:54 2012 -0700" }, "message": "Add a default constructor to JValue.\n\nChange-Id: I7abe801b26f5003c480215c1f1ca64dfc07667ed\n" }, { "commit": "34e069606d6f1698cd3c33b39e72b79ae27e1c7b", "tree": "0feb7e9bde6cfe01c0df3ef2d8de3210570f7ce2", "parents": [ "e62934d85fbc2d935afdad57eeade39ecbd7440a" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Apr 09 13:55:55 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Mon Apr 09 13:55:55 2012 -0700" }, "message": "Remove the useless \"suspend count already zero\" message for new threads.\n\nWe can actually detect the expected case of this warning ourselves, and\nnot emit it. Then we can upgrade the WARNING to a FATAL.\n\nI also tripped over the fact that the operator\u003c\u003c for Thread::State was out\nof date, so I\u0027ve moved the Thread enums up to namespace scope so the script\ncan automatically generate correct operator\u003c\u003c implementations for us. (All\nthe high-numbered thread states have been off by one for a couple of weeks.)\n\nChange-Id: I5de573d33d641e5a3cba87b370e9620c8c66e633\n" }, { "commit": "0045a290e6b79a274250e3112880c04bde437d4a", "tree": "a125d556b6c817218b14f6f3f72079b83d0032c4", "parents": [ "d23f5206cf72ca3fb53ac3049649bb7d018232bf" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Mar 31 21:08:41 2012 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Sat Mar 31 22:38:22 2012 -0700" }, "message": "Avoid a case of eager initialization.\n\nDon\u0027t mark as initialized classes that contain static field\ninitialization.\n\nChange-Id: Iedcabbdf355e8861eb7731650eee1467f68ae0cd\n" }, { "commit": "dbac3090ace708eb27c707d4b72efc48c069129c", "tree": "2b811b40af24ce849d327643f05b4244ca2449fb", "parents": [ "5efa670d6a8cf6ad3771b9fb7d4c10598ad78f9d" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 16 18:00:30 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 16 18:00:30 2012 -0700" }, "message": "Ensure that BoxPrimitive only sees clean data...\n\n...rather than forcing it to zero irrelevant bits itself.\n\nChange-Id: Ibbd28db08b68fac84704a911a6f927f7d9343055\n" }, { "commit": "5efa670d6a8cf6ad3771b9fb7d4c10598ad78f9d", "tree": "0de2f261966ed1e86edd33b039cf9df990a6ffb6", "parents": [ "594e76f8d5fc7c8e419ead1185cfe91dd9036d0c" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 16 17:22:06 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 16 17:22:06 2012 -0700" }, "message": "Don\u0027t trust all the bits in a JValue when boxing.\n\nCallers can\u0027t be trusted to have zeroed the bits unused bits in the JValue;\nwe could chase them all down and fix them, but it\u0027s probably safer to go back\nto the old paranoia.\n\nChange-Id: Ic3967d5f58fc69b7dbbe00a8ea56baf6457631ee\n" }, { "commit": "907a0dbe5bbb536391158e8188b4c064c8898eff", "tree": "46269d78953f1529b79ccfda20df4e2178eafd4d", "parents": [ "1bac54ffa933fbe9b92b62437577f2f4583eff1a" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 16 13:11:09 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 16 13:11:09 2012 -0700" }, "message": "Fix boxing; my invoke stub changes broke this.\n\nChange-Id: If24ad7231de956d1f81bf65084508bcad97537db\n" }, { "commit": "1bac54ffa933fbe9b92b62437577f2f4583eff1a", "tree": "a27134343ed3cd45a286c3a68b9a4469e70f13e7", "parents": [ "96970cdab3e28511322ce37fa578f6eff8931d72" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 16 12:48:31 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Fri Mar 16 12:48:31 2012 -0700" }, "message": "Fix all our unused parameter warnings so we let GCC report them.\n\nThere were a couple of genuine bugs here (fixed), plus there\u0027s a missing\nfeature in trace.cc that I\u0027ve just added a TODO for.\n\nAlso note that I haven\u0027t touched the compilers; this warning is still\nexplicitly disabled for that code. I\u0027ll do that when there\u0027s less going\non in those directories.\n\nChange-Id: Ic3570bf82411a07c7530bfaf1995ac995b9fc00f\n" }, { "commit": "77405796564c6c1353807cda18b28678a719bd68", "tree": "948781c769760d71c7aff4e8db57aa5a9e311372", "parents": [ "f8bbb8448c733e9e3ad43aad69774c37888329b1" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Mar 15 15:22:12 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Thu Mar 15 15:46:59 2012 -0700" }, "message": "Rewrite the invoke stubs to use JValue[]s.\n\nThe tests were only testing the static stubs, so extend the tests to include\nnon-static stubs too.\n\nAlso add just enough of an ARM disassembler to disassemble the invoke stubs.\n\nChange-Id: If71dfb66b8b8188f9d871914f0eaf1013c9993b9\n" }, { "commit": "b600b3f8360f535913fedc91aef7c8e5abaca48a", "tree": "b8f4fdca44a70caec314b210372d678ad9501dbf", "parents": [ "002b915276064dbe2ce9caf7474d3032301d8d7c" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Mar 14 13:57:24 2012 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Wed Mar 14 13:57:24 2012 -0700" }, "message": "Fix/improve exception detail messages for unboxing.\n\nThis fixes libcore/luni/src/test/java/libcore/java/lang/reflect/MethodTest.java\nfailures.\n\nChange-Id: Idf6aae169bd72f0534d4094170536e6b2515ca07\n" }, { "commit": "955724179c6c739524f610023287f56b24dc31de", "tree": "64f751693e72173ae0d044d8c54989b01f1195af", "parents": [ "b87f73ea7524f6eb9da6da1af931a3d3fbb833ff" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Dec 13 18:14:20 2011 -0800" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Tue Dec 13 18:14:20 2011 -0800" }, "message": "Don\u0027t default-construct std::strings and then assign them.\n\nChange-Id: I8c994d1e6a8d2ebe52eaa4f0132e0deb2ecfa5f3\n" }, { "commit": "6d4d9fcb4f01e287ee29e81cd1c941ee5d11d379", "tree": "4c715bef86a3c5a47f9cea43c4a092cf4766ed4b", "parents": [ "86b0010c79ef95b5333cd540b7d3af34a9f1a643" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Wed Nov 30 16:24:48 2011 -0800" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Tue Dec 06 14:02:09 2011 -0800" }, "message": "Reduce meta-data object sizes, introduce meta-data helper classes.\n\nChange-Id: Id14ad218f1c74c659701352fdf1a45bf6444daa3\n" }, { "commit": "c129a6b44193ac3a872f852a69d336e0e7ed8640", "tree": "ff5b5503aff4e5173c570b2e471d2df0895b7b2b", "parents": [ "434424a3f60f2129aaa1780ac6a5c2618bc789e0" ], "author": { "name": "Jesse Wilson", "email": "jessewilson@google.com", "time": "Thu Nov 24 14:47:46 2011 -0500" }, "committer": { "name": "Jesse Wilson", "email": "jessewilson@google.com", "time": "Thu Nov 24 14:47:46 2011 -0500" }, "message": "Make methods and fields accessible by default in art.\n\nThis also fixes a corner-case bug where calling Field.setBoolean()\non a non-boolean field would crash the runtime.\n\nChange-Id: I2f6693e3c3fe3bf23b4761335300506b3fbd3008\n" }, { "commit": "9a6bae896a2f003d7216603bf29771d105c10ca4", "tree": "ccf8c190b8b4daf799e17ec8b04b77edfc5c72ff", "parents": [ "169c9a7f46776b235d0a37d5e0ff27682deffe06" ], "author": { "name": "Jesse Wilson", "email": "jessewilson@google.com", "time": "Mon Nov 14 14:57:30 2011 -0500" }, "committer": { "name": "Jesse Wilson", "email": "jessewilson@google.com", "time": "Mon Nov 14 17:46:40 2011 -0500" }, "message": "Don\u0027t run managed code until the runtime has started.\n\nPreviously we ended up running managed code because JNI was\ntriggering class initializers to run. This was triggered by\nboth Thread::CreatePeer() and InitBoxingMethods().\n\nWhen these initializers were prevented from running, other\ncode broke:\n - Creating the peer for the main thread was relying on\n ThreadGroup.\u003cclinit\u003e to assign the built-in thread groups.\n - Creating the boxed methods caused class initialization of\n the primitive wrapper classes; these need to be initialized\n before Thread.\u003cclinit\u003e is run to avoid a crash in its own\n initializer.\n\nThis change works around those breaks by splitting thread peer\ncreation into two parts (allocation and running \u003cinit\u003e) and\nby calling InitBoxingMethods() during runtime start.\n\nChange-Id: I44be7170ce08451adf876ee73cba0f1f66d5a59e\n" }, { "commit": "6b4ef025af12b158d117fc80fc79acf620f411a0", "tree": "223c580477cbb7bde240b599da4bbee793b8dcaf", "parents": [ "21b9f1355f0311b5c67bfdfa9dd63cacb1fba502" ], "author": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Sun Oct 23 14:59:04 2011 -0700" }, "committer": { "name": "Brian Carlstrom", "email": "bdc@google.com", "time": "Tue Oct 25 00:03:03 2011 -0700" }, "message": "Make ResolveField not rely on Field::GetType resolution\n\nChange-Id: I10f4a874809ac9db2cd54e200cf10eb7c8979fce\n" }, { "commit": "466bb25416b88fabd5d4387b7c7e5cc1ece78b8c", "tree": "8d0952cbd850f1d0ac2e43a8372ce571fb4982d1", "parents": [ "a10cd94bb77ed66fa0a8b66141c4504045c92d30" ], "author": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 14 03:29:56 2011 -0700" }, "committer": { "name": "Ian Rogers", "email": "irogers@google.com", "time": "Fri Oct 14 21:45:27 2011 -0700" }, "message": "Proxy implementation\n\nThis rounds out the proxy implementation by adding missing pieces to the\nclass linker, extending tests and fixing issues in the runtime support.\nThere are also some tweaks for performance and to clean up Method/Object\na little.\nA unit test of the functionality is \"art/test/run-test 044\"\n\nChange-Id: Id94102d10b81cd9b12b95ba8618f6187490204c4\n" }, { "commit": "5cb5ad27944efb08d4556b3c0d362302e37e832b", "tree": "63334f19072ca2ba7179973f0ce62101ee89605d", "parents": [ "d6fe38d96b6116bd53cf2cb14734af8d69e08661" ], "author": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Sun Oct 02 12:13:39 2011 -0700" }, "committer": { "name": "Elliott Hughes", "email": "enh@google.com", "time": "Sun Oct 02 12:13:39 2011 -0700" }, "message": "Fix exception throwing to support no detail message.\n\n(The empty string as a detail message is distinct from a NULL detail message,\nand is treated differently by Throwable.printStackTrace.)\n\nChange-Id: I8c65deac9f18c5782dcf6e72e4c37e6dd4174fe9\n" } ], "next": "2a20cfd0b7fc81099f5de0da782ebcc1cb262792" }