)]}' { "log": [ { "commit": "a8bba7d0853b372aea3ed3ea154fb2b2a23c2c9d", "tree": "973a89870e0cc8912401503ebab1b709a9df5ba0", "parents": [ "65e463cf7ea91003ef030c8a819ae4596c9b820f" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 30 15:18:48 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 31 09:48:59 2018 +0100" }, "message": "ObjPtr\u003c\u003e-ify ClassLinker::FindClass(), fix 1 stale reference use.\n\nThread::CreateAnnotatedStackTrace() was using a stale\nreference `aste_array_class`.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: I191907c0053456bb57de425aa6ccd9668df818a2\n" }, { "commit": "679730e6c4be680b3e72b55a4200de9590641009", "tree": "ca7d0719aed00457a2c898cf2dc56300956ea875", "parents": [ "6834d3414308e3d536bc685dbb3d60fe70186f23" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 25 15:06:48 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 25 15:06:48 2018 +0100" }, "message": "Remove static GcRoot\u003c\u003es from Field, Method, Constructor.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: I648b88339995761fb81180286ef48a42bbd2f83d\n" }, { "commit": "c7aa87e1666ac48ddf9149cfdfd64b026b3969e5", "tree": "32d5d74718cc558e13642873e55724782ac9df22", "parents": [ "0278be74269fcfe4f2517d449f2bd53472f9b2f9" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu May 24 15:19:52 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 25 11:37:45 2018 +0100" }, "message": "Remove static_class_ from Method/VarHandle and CallSite.\n\nAnd add MethodHandle to the class roots to avoid extra\nindirection through MethodHandleImpl.\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 31113334\nChange-Id: Iaf172f3732677f2b4509e8297e6e9af5fb81a89f\n" }, { "commit": "2d3065e6ca0bd707bc998b7d260bb8e8ec07cf87", "tree": "2b1361fb8c0d860e735858d41a7be6b7699a576a", "parents": [ "3eb68cf6c05e8c015f0da2c50e7e720976702e63" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 22 13:56:09 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 22 14:54:45 2018 +0100" }, "message": "ObjPtr\u003c\u003e-ify annotation processing.\n\nTest: Rely on TreeHugger.\nBug: 31113334\nChange-Id: Ifd69c15c0df1530d8860cf50e06bde0d356b0c23\n" }, { "commit": "59a49a6fe102b26aacda906b6a2c466d84819c73", "tree": "f33b21de298b1943ae3483ba877cdbf50313e049", "parents": [ "48af3b39cd7c8abaa7f9671191ec8260630239b2" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed May 09 10:58:27 2018 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu May 10 12:38:11 2018 +0100" }, "message": "Implement VMDebug API to mark a class \"platform\"\n\nAdd a native implementation of VMDebug.allowHiddenApiReflectionFrom\nwhich takes a Class object and sets a new access flag\nkAccSkipAccessChecks on the corresponding mirror::Class instance.\n\nThe function will throw a SecurityException if the process has not\nbeen forked as Java debuggable.\n\nhiddenapi::IsCallerInPlatformDex is extended to take an optional\nargument of the mirror::Class of the caller. If it is set and\nthe class has kAccSkipAccessChecks, the caller is allowed to\nacccess non-SDK APIs.\n\nNote that the mirror::Class of the caller is only provided for\nreflection and JNI. The access flag is ignored for other means of\naccess.\n\nBug: 64382372\nTest: N/A\nChange-Id: I2bf0ca7dcb45c17fe91eb2d421c947b892bd6fec\n" }, { "commit": "a3ad0cdd711857f04f477e2cdc5b56a2c74a3018", "tree": "7e0e4c7888c2497d86ce96c9574d7c9886721ccf", "parents": [ "abd9e1515bc6be88372c61071971c5432a51553d" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri May 04 10:06:38 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 09 11:11:09 2018 +0100" }, "message": "ART: Move JNI files to runtime/jni/ .\n\nTest: Rely on TreeHugger.\nChange-Id: I9cae11191ef1567ae9453be498882a7767285140\n" }, { "commit": "9a13d423c71517bc9cb3cf40e97dd0f6f5d5c72c", "tree": "db25de5c310f8c3b14dbaad91bfe3b02a8a4c5b3", "parents": [ "9a81945ca49de3c5b3969cc85278ecbadf238c84" ], "author": { "name": "Mathew Inwood", "email": "mathewi@google.com", "time": "Mon Apr 09 12:24:55 2018 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue May 08 10:31:05 2018 +0100" }, "message": "Consider whitelist when listing class members.\n\nPreviously, only the enforcement policy was considered when getting\ndeclared fields or members, meaning whitelisted APIs would still not be\ndiscoverable. Fix this by calling hiddenapi::GetMemberAction from within\nIsDiscoverable.\n\nBug: 77787686\nBug: 64382372\nTest: cts/tests/signature/runSignatureTests.sh (with ag/3863796)\nTest: art/test.py --host -t 674-hiddenapi\nMerged-In: I234d274f47f377e3e105c81aae2d49072287992a\nChange-Id: I234d274f47f377e3e105c81aae2d49072287992a\n(cherry picked from commit 64ee8aeaeb70aa2d5d1c3ff57a682a5001869653)\n" }, { "commit": "c0d988a9b5f2c34072fff100af9bd12464a6c55b", "tree": "1bd0cf8026f3f98c04f9a8445ee034762a1dbe23", "parents": [ "9574f493cb072466b7b491de97c381e6c2b8b639" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Tue Apr 10 14:32:35 2018 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Apr 11 11:47:16 2018 +0100" }, "message": "ART: Stop Hidden API stackwalk for class initializers in j.l.i.\n\nClass initializers in java.lang.invoke retrieve field offsets. The\nhiddenapi stack walk does not need to progress past these initializers\nas they are permitted irrespective of how their invocation is\ntriggered.\n\nBug: 77631986\nTest: art/test/run-test --host --64 674-hiddenapi\n\n(cherry picked from commit 29e64cfc32e17c9111a5ed2a6b141bebf891cbe3)\n\nChange-Id: I77a29999177850bd50ca4d043b0cd40c40692fe4\nMerged-In: Iaabedc9c016d546e10072107d79c7b6701582c83\n" }, { "commit": "9574f493cb072466b7b491de97c381e6c2b8b639", "tree": "b7f8990b96dc29df2e38bfc480bd8ac62bcdf59b", "parents": [ "b143d262c523f022499b211d5c2b1fbd9f8130cc" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Tue Apr 10 10:20:32 2018 +0100" }, "committer": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Wed Apr 11 11:43:35 2018 +0100" }, "message": "ART: Walk past j.l.i in stackwalk for Hidden API\n\nBug: 77631986\nTest: art/test.py --host 674-hidden-api\n\n(cherry picked from change 198a27e92a529115a2db2a17ec11d8b52bd6a315)\n\nChange-Id: I2364036a6b7bd8bb06d2e456fb22f4db9c6cce21\nMerged-In: I2e3e0399765da7f554259fe14247b45e42110ef4\n" }, { "commit": "6097672947fe8b2632cfd10e3d8e44792199a346", "tree": "40f63402acbbafdab2f4b0afd3f7efd620308282", "parents": [ "1451f86cce740befd52cb3665827aafef1497f54" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Apr 06 15:29:38 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Apr 09 11:40:55 2018 +0100" }, "message": "Remove some unnecessary items from WellKnownClasses.\n\nReplace a few uses with alternatives, namely\nmirror::{Constructor,Method,Field}::StaticClass().\nAlternatively, we could use class roots in the ClassLinker.\n\nAnd clear dalvik_system_VMRuntime_nonSdkApiUsageConsumer\nin WellKnownClasses::Clear().\n\nTest: testrunner.py --host\nBug: 74943277\nChange-Id: I7561db76cd62d376c22efd4386dffd83ec74aa66\n" }, { "commit": "f5f1f80aa6c1c10c61b6723bbc52d5aec2eba2b9", "tree": "6a83730a41748fcfd6e42be0f31ec7243d631495", "parents": [ "8c5de0f16444441c23a5ae807e4dd5cc0dd586a3" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Tue Apr 03 15:23:46 2018 +0100" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 06 10:08:56 2018 -0700" }, "message": "Revert^2 \"hidden_api: Call back into libcore on hidden api detection\"\"\n\nThis reverts commit bbe60d58496991c16e2943e174e26ab8a096b3d0.\n\nThis CL deviates from the approach of the original change. Instead of\ncalling back every time ShouldBlock.. was called, we explicitly call\nback in cases where it\u0027s safe to do so.\n\nNote that we only call back on reflective accesses for now, and not\nlink time accesses. Coverage for the latter will be added in a follow up\nchange.\n\nBug: 73896556\nTest: test-art-host\nTest: art/test.py --host -t test-art-host-run-test-debug-prebuild-\\\n interpreter-no-relocate-ntrace-gcstress-checkjni-picimage-pictest-\\\n ndebuggable-no-jvmti-cdex-fast-674-hiddenapi64\n\n(cherry picked from commit e453a8dd87731f4b37b86a1284f7655d86c2a809)\n\nMerged-In: Ie99ac268a083af167accbdf955639da068bea950\nChange-Id: I76860519d40b87032dbb8db38b04fcf79ef09723\n" }, { "commit": "8c5de0f16444441c23a5ae807e4dd5cc0dd586a3", "tree": "a0753accc18ccc000280e18327c57f7352e2ace5", "parents": [ "a9660f1dc13b4d595b3f89b06dd5b70eeee18c43" ], "author": { "name": "Orion Hodson", "email": "oth@google.com", "time": "Tue Apr 03 14:13:13 2018 +0000" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 06 10:08:21 2018 -0700" }, "message": "Revert \"hidden_api: Call back into libcore on hidden api detection\"\n\nThis reverts commit 757a9d0a2e97d43bafeb8a95cc3c51102be99586.\n\nReason for revert: Test failures with \"art/test.py --host -t test-art-host-run-test-debug-prebuild-interpreter-no-relocate-ntrace-gcstress-checkjni-picimage-pictest-ndebuggable-no-jvmti-cdex-fast-674-hiddenapi64\"\nBug: 73896556\nTest: art/test.py --host -t test-art-host-run-test-debug-prebuild-interpreter-no-relocate-ntrace-gcstress-checkjni-picimage-pictest-ndebuggable-no-jvmti-cdex-fast-674-hiddenapi64\n\n(cherry picked from commit 9e68ade384abdb15714054feaed06cb38eb5432f)\n\nMerged-In: Ib2ad89c16ad797c37f6212bc7e5c0b6b92ce56b5\nChange-Id: I11fa9b76da07162fde8773eb05cfc6a6514e0ca1\n" }, { "commit": "a9660f1dc13b4d595b3f89b06dd5b70eeee18c43", "tree": "fba8c4742d8f866b8342794542436ab3cd3f32a8", "parents": [ "9c85c79af0e5fe181df97dfc2801f4999aef77eb" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Mar 29 10:21:47 2018 +0100" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Apr 06 10:07:49 2018 -0700" }, "message": "hidden_api: Call back into libcore on hidden api detection\n\nThis change also removes some unnecessary RI specific logic for\nbuilding src-ex since it isn\u0027t required.\n\nBug: 73896556\nTest: run-test --host 674-hiddenapi\nTest: StrictModeTest\n\nCo-Authored-By: Andreas Gampe \u003cagampe@google.com\u003e\n\n(cherry picked from commit 757a9d0a2e97d43bafeb8a95cc3c51102be99586)\n\nMerged-In: Ib2b4dfad55c5d829630bfe2adb4a468124bea61c\nChange-Id: Ida0943990aa1b3bad0c674bc31ff46766ae493a6\n" }, { "commit": "8e1a7cb303d7c8f763dfb99ae311b820996b1ab4", "tree": "ea952245e2b907c44d5c0fcadf90f00a1fe31913", "parents": [ "10c4390a7d63775b1c7e6f6a291e9aade388d733" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Mar 27 08:14:25 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Mar 27 18:15:21 2018 +0100" }, "message": "Revert \"Revert \"Allow hidden API access from system libraries\"\"\n\nLibraries like RemoteDisplay provide an APK that an app loads into\nits process and which accesses internal APIs on the app\u0027s behalf,\nwithout exposing the internals to the app. These libraries are\nconsidered part of the platform, but were not exempt from hidden API\nchecks because they are not loaded with the boot strap class loader.\n\nThis patch adds a new flag to DexFile class which the constructor\nsets to true of the canonical location of the newly loaded dex file\nstarts with \"${ANDROID_ROOT}/framework/\". Hidden API enforcement\nthen checks this flag when determining whether the caller of\na hidden class member is allowed to access it or not.\n\nThis reverts commit 0127b71a2588efcd1a53c192c5c267157878b010.\n\nPrevious CL saw two issues:\n- buildbots would set non-existent ANDROID_ROOT for host-side builds\n- calling realpath on unquickened dex files would overflow the stack\n\nBug: 64382372\nBug: 76138670\nBug: 76165623\nBug: 76112393\nBug: 76452688\nBug: 76429651\nTest: art/test.py --target -r -b -t 674-hiddenapi\nTest: SystemUI APCT test\nChange-Id: Ie07a088509002593353965d3d24bf7362b643f40\n" }, { "commit": "0127b71a2588efcd1a53c192c5c267157878b010", "tree": "cfe89ca742bed927e67d9cf926937fe5e6f89347", "parents": [ "eefabd28c21e6ae02ca05e97bd22f9e23cbf98a5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 26 19:31:41 2018 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 26 19:31:41 2018 +0000" }, "message": "Revert \"Allow hidden API access from system libraries\"\n\nBug: 64382372\nBug: 76138670\nBug: 76165623\nBug: 76112393\n\nSeems to cause run-test issues.\n\nThis reverts commit eefabd28c21e6ae02ca05e97bd22f9e23cbf98a5.\n\nChange-Id: Ia9b3d946742dbfde4fb56cf14413fde0600c8252\n" }, { "commit": "eefabd28c21e6ae02ca05e97bd22f9e23cbf98a5", "tree": "58e4c2635e0d5682c3385bed639664ccfdd44f6a", "parents": [ "e8a4e378c5a928d5de07bee6db99150a57dabcd8" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Mar 22 16:27:18 2018 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Mar 26 08:05:33 2018 +0000" }, "message": "Allow hidden API access from system libraries\n\nLibraries like RemoteDisplay provide an APK that an app loads into\nits process and which accesses internal APIs on the app\u0027s behalf,\nwithout exposing the internals to the app. These libraries are\nconsidered part of the platform, but were not exempt from hidden API\nchecks because they are not loaded with the boot strap class loader.\n\nThis patch adds a new flag to DexFile class which the constructor\nsets to true of the canonical location of the newly loaded dex file\nstarts with \"${ANDROID_ROOT}/framework/\". Hidden API enforcement\nthen checks this flag when determining whether the caller of\na hidden class member is allowed to access it or not.\n\nBug: 64382372\nBug: 76138670\nBug: 76165623\nBug: 76112393\nTest: art_dex_file_loader_test gtest\nTest: art/test.py --gtest\nChange-Id: If062bd668d7ba494bbb7b828e40932748d173b9a\n" }, { "commit": "597d7f650b0656fcb3985b01f53284717b41e5cc", "tree": "59ba9853104427b19f2449ce547301764d22756b", "parents": [ "753ce1bcf458ad6c6fbb41689901943d44e7738e" ], "author": { "name": "Mathew Inwood", "email": "mathewi@google.com", "time": "Thu Mar 22 11:36:47 2018 +0000" }, "committer": { "name": "Mathew Inwood", "email": "mathewi@google.com", "time": "Fri Mar 23 11:43:35 2018 +0000" }, "message": "More flexible API enforcement policy support.\n\nThis CL adds the ability to configure which banned API lists to enforce,\ndefined by new enum hiddenapi::ApiEnforcementPolicy. Currently, the policy\ncan be set at zygote fork time, but not at dex optimization time where\nblacklist enforcement is still assumed. As such, making the policy more\nstrict will not work as expected yet. This will be improved in a follow up\nCL.\n\nTest: art tests pass\nTest: Device boots\n\nBUG: 73337509\n\n(cherry-picked from commit 159f596eec01adbb5a1c9654402c137cdb943131)\n\nChange-Id: I6c319bb8a3000cb1d3c4693b4fb196e749c36d96\nMerged-In: I33f9afce628a86727e400052f4d5979d3536da8c\n" }, { "commit": "e42d71719442d40baf07a306a02a5653f444c10d", "tree": "b9dfc3999609d1d2e84e636e9e12097b785b9465", "parents": [ "c8483841a764dff86f72813381d51df00809ef68", "06537f7a6f0772cbef08bdded933828378b2d32a" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Fri Feb 23 01:35:17 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Feb 23 01:35:17 2018 +0000" }, "message": "Merge \"native: Cleanup jni usage code\"" }, { "commit": "06537f7a6f0772cbef08bdded933828378b2d32a", "tree": "e1505a3dea19d6585973d2db0d9dd6988bf0f03d", "parents": [ "a502c7202f56b8f5f2c5de567359fa7f6bab406f" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Feb 22 15:03:05 2018 -0800" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Feb 22 15:28:50 2018 -0800" }, "message": "native: Cleanup jni usage code\n\nMake the C++ signature consistent with the JNI descriptor by using the\nsame type as the descriptor whenever possible.\n\ne.g. \"()Ljava/lang/String;\" should actually be \"jstring fn(JNIEnv*,jobject)\"\ninstead of \"jobject fn(JNIEnv*,jobject)\".\n\nBug: 35325126\nChange-Id: I72318525fc3b18b013b8d6fa604d8dd6b5dd4400\nTest: make -j32 test-art-host # and also manually that it boots\n" }, { "commit": "b2ec9f5c128673c43f776cbe12c8eeb0a6884ebb", "tree": "724022162a4f3de021cebab32ee5ac67ec860f32", "parents": [ "f13343eb8b2738e2812fd7bd9d5592f529e93c21" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Feb 21 13:20:31 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Feb 21 13:51:15 2018 -0800" }, "message": "Remove duplication, split tests\n\nThe code move to libdexfile/dex/descriptors_names.cc apparently did not\nremove the original code from runtime/utils.cc. Fix that duplication\nand all the header mentions needed. Also, split the test files to go\nalong with the new locations for the code to be tested.\n\nBug: 22322814\nTest: make -j 50 checkbuild\n make -j 50 test-art-host-gtest\n flash \u0026 boot marlin\n\nChange-Id: Ie734672c4bca2c647d8016291f910b5608674545\n" }, { "commit": "7287c4d75e38e7a01c7ea3fdebb47d9cfe128a56", "tree": "2b43d1b334e05e1c5719fee3f2a16bebaff059d9", "parents": [ "1efa7cb8c415f4df8a248242cd773a1fc671878e" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 15 10:41:07 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Feb 15 15:36:21 2018 +0000" }, "message": "Add Class.getPrimitiveClass().\n\nImplement the native part in ART and remove the native parts\nof the previously needed workaround for java.lang.Void.TYPE.\n\nCompanion libcore/ change:\n https://android-review.googlesource.com/619326\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nTest: Pixel 2 XL boots.\nBug: 73346078\nChange-Id: Icfb201a1f08d8ff2a65c58f7c0ec25aa2296db29\n" }, { "commit": "068d68dabfc37d9a5a7d7bf834b27793b99303fe", "tree": "02f2cb087f682222de4d93238a6c8753a8d07cef", "parents": [ "0dc1b92c76bd8a85ae4f5dad1dea61a11660d9cc" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Feb 12 13:04:17 2018 -0800" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Feb 13 12:47:36 2018 -0800" }, "message": "Print more information about hidden API accesses\n\nThis patch changes when we print hidden API access warnings and the\nmessage that is printed:\n\n(a) prints message even if reflection is denied\n(b) prints if the access was due to reflection or JNI\n\nBug: 64382372\nTest: manual\nChange-Id: I1e59982516c12580a207fd294aa8aab851b1ad46\n" }, { "commit": "70e88d6df6cc97163fa8b2bdd756a6e910bc9aa5", "tree": "3801c976e92f02247042e4c124f6b7dc00d4e435", "parents": [ "cf026fec63bea5056988452a6c014df75bf13e19", "a02cb113cd0dd023eec61716d3d1bd0dc1c1453a" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Feb 01 15:54:46 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Feb 01 15:54:46 2018 +0000" }, "message": "Merge \"Move hidden API warnings into resolvers\"" }, { "commit": "a02cb113cd0dd023eec61716d3d1bd0dc1c1453a", "tree": "8553be51d5f6c2f0ebce20d7e364085ccf0a3206", "parents": [ "c7e546ff3963a1d51b1f100d308db735bd19f736" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Jan 31 11:36:39 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Feb 01 13:47:33 2018 +0000" }, "message": "Move hidden API warnings into resolvers\n\nChecks inserted to maybe warn about accessing a hidden API member\nare proving too expensive. Move them into the resolvers.\n\nBug: 64382372\nBug: 72649186\nTest: art/test.py --host -b -r -t 674-hiddenapi -v\nChange-Id: I11bceccb8fe647e829c491be38ba0af72f1b3996\n" }, { "commit": "0225f8e2939a9340cb7dcebfcfe7996a2bd9bce9", "tree": "fd32362dff4ecdc8ff8b0d800a1b46fb0d9d0104", "parents": [ "c7e546ff3963a1d51b1f100d308db735bd19f736" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Jan 31 08:52:24 2018 +0000" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Jan 31 01:24:53 2018 -0800" }, "message": "Revert \"Revert \"Make libdexfile build independent of runtime dir\"\"\n\nThis reverts commit 787784f9effb126b5d0d3dc97d544c4a477b5daf.\n\nReason for revert: Bot configuration issue.\n\nChange-Id: I6a10bb4a9571f89c7e4dd095f9157e830a44e2de\nBug: 22322814\nTest: make -j 50 checkbuild\n" }, { "commit": "787784f9effb126b5d0d3dc97d544c4a477b5daf", "tree": "1ff6d9fd284a4dd2a337165dff20e71137997c9d", "parents": [ "b40b7e73469339a6b667b4a2e2b8690112a74dc9" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 18:48:51 2018 +0000" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 18:48:51 2018 +0000" }, "message": "Revert \"Make libdexfile build independent of runtime dir\"\n\nThis reverts commit b40b7e73469339a6b667b4a2e2b8690112a74dc9.\n\nReason for revert: on device libdexfile.so missing\n\nChange-Id: I9bd61a98bef870400580e8c991cb061d3f57fa72\n" }, { "commit": "b40b7e73469339a6b667b4a2e2b8690112a74dc9", "tree": "7e8a34d9c7894c4c946f674f19f0a0a512b95184", "parents": [ "9690ad794b324ba54e936608881ac0f62538b97a" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Jan 25 17:11:07 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Jan 30 01:17:58 2018 -0800" }, "message": "Make libdexfile build independent of runtime dir\n\nRemove libdexfile\u0027s dependency on utils.cc and move utf.cc into\n/dex. Remove libdexfile\u0027s constituent sources from libart and\nuse libdexfile wherever libart is. Also remove some ART-specific\ninterfaces. Libdexfile\u0027s tests remain to be converted, plus\nmoving the files to a new directory peer to runtime/.\n\nBug: 22322814\nTest: make -j 50 test-art-host\nChange-Id: Ifaf695216e4a0e43d3aa377984d933f7a2a243c2\n" }, { "commit": "ee7d2fd16d47b54d7fb5b9d5ec772fbc315faf4b", "tree": "844ed2998f78723aa7909ce0a7b19995c34f0bd9", "parents": [ "5a61bb7969347ffe8e0bf4f4dff841cc6c21ed85" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Sat Jan 20 17:25:23 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jan 25 10:40:56 2018 +0000" }, "message": "Start warning on hidden API greylist\n\nInsert checks into reflection, JNI and the verifier to print\na warning when greylisted methods are invoked and fields accessed.\nWe do this on actual access, because reflection allows to list\nall methods/fields and simply listing a greylisted member would\nprint too many false positives.\n\nIssuing a warning also sets a boolean flag in Runtime. This will\nbe made accessible through VMRuntime to the framework which will\nissue a Toast on Activity start.\n\nThe change was tested with internal microbenchmarks of reflection\nand those flagged one issue. Microbenchmark invoking a field getter\nhas regressed by 35%. We will profile this benchmark in detail and\nconsider options for improvement. Bug b/72482474 was created to track\nprogress.\n\nTest: art/test.py -b -r -t 674-hiddenapi\nBug: 64382372\nBug: 72482474\nChange-Id: I323244935e9091a2f8d012385cefaac6b1fe3777\n" }, { "commit": "5a61bb7969347ffe8e0bf4f4dff841cc6c21ed85", "tree": "44c266c97576dcbdc931cf94d81774614a2c762b", "parents": [ "d02784e17c20c91a50dcb10522351f4937e250ff" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Fri Jan 19 16:59:46 2018 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Jan 25 10:11:11 2018 +0000" }, "message": "Start enforcing hidden API blacklist\n\nInsert checks into reflection, JNI and Class::CanAccessMember to\nmake blacklisted hidden APIs undiscoverable.\n\nThe change was tested with internal microbenchmarks of reflection\nand those showed no measurable performance impact.\n\nTest: art/test.py -b -r -t 674-hiddenapi\nBug: 64382372\nChange-Id: I9e39804b837ae9ffeba926f2a5b1e8e9986c472b\n" }, { "commit": "9e734c7ab4599d7747a05db0dc73c7b668cb6683", "tree": "dce1d1993734a947fb2e6f626eb1b425cb72143b", "parents": [ "b496af808eaf3af5ebac50aef4fbec33323b5016" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Jan 04 17:56:19 2018 -0800" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Fri Jan 05 11:07:19 2018 -0800" }, "message": "Create dex subdirectory\n\nMove all the DexFile related source to a common subdirectory dex/ of\nruntime.\n\nBug: 71361973\nTest: make -j 50 test-art-host\nChange-Id: I59e984ed660b93e0776556308be3d653722f5223\n" }, { "commit": "666ee3d7c6039c80e75287e311895bd6a9b01e9f", "tree": "5e65f39ff7cc83af4be3b691c247d7348052f854", "parents": [ "890111968fbd3f5ae528d97e42984c12a3dd27bd" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Dec 11 18:37:36 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Dec 12 13:48:02 2017 +0000" }, "message": "Do not pass DexFile to ClassLinker::Lookup/ResolveType().\n\nThe DexFile can be easily retrieved from the DexCache,\nso reduce the number of arguments that need to be passed.\n\nAlso refactor the code to avoid doing the DexCache lookup\ntwice and avoid unnecessary read barriers in the initial\nDexCache lookup (also for Lookup/ResolveField()).\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nChange-Id: Idea9aa42b6a5bade947e93e330b1abdb9d11b2da\n" }, { "commit": "373a9b5c718a45ac484afcf4fe6ce84f4bb562b3", "tree": "52a3e78fa8bc94a76674caaae837c728332d05dc", "parents": [ "d0332e01ac26c2fd0a900de000a55ed5768ff936" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 18 09:01:57 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Oct 25 10:13:59 2017 -0700" }, "message": "ART: Depend on libnativehelper headers only\n\nDepend on header-only versions where possible. Move projects\nexcluding libart to libnativehelper_header_only.\n\nBug: 65522645\nTest: mmma art\nChange-Id: I53dd3b2a97e94ee685f72de007ed3858f7f5c6b6\n" }, { "commit": "e431e2758d62cf56f7f347f5a8c9d79e41b6dcd7", "tree": "b8733533e2478069aabbdc3a34d0f40aff33e1b1", "parents": [ "b98d384eb424ad7e1dc9816392ded33dce0e0372" ], "author": { "name": "Steven Moreland", "email": "smoreland@google.com", "time": "Tue Jul 18 16:53:49 2017 -0700" }, "committer": { "name": "Steven Moreland", "email": "smoreland@google.com", "time": "Tue Jul 18 18:25:55 2017 -0700" }, "message": "art: use proper nativehelper headers\n\nlibnativeheader exports headers under nativeheader. These were\navailable before incorrectly as global headers in order to give\naccess to jni.h.\n\nTest: modules using art find headers\nBug: 63762847\nChange-Id: I5c820d677e94e07b2859e78610bc997fe51b41dc\n" }, { "commit": "87583b3de6e811e141f8c97c1f796eb516e8a751", "tree": "5b4688f306a2fda2f29bf319a8e1ff21309f93a3", "parents": [ "854461a4cd5e4a38debe3616e12b52fe7f160782" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu May 25 11:22:18 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Jun 01 08:24:12 2017 -0700" }, "message": "ART: Move RegisterNatives helper\n\nMove the helper closer to the users.\n\nTest: m test-art-host\nChange-Id: Id05ce5f60af54d0c5aef0c7d8932706c4602fc94\n" }, { "commit": "a14100ccf51cc63a5c472188d1e2d337627e49eb", "tree": "d659a038e35fdd790b642f51e3eddbc9b51d003a", "parents": [ "1171deea4cb2f2db67a310ea8797d06206b2bdea" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 24 15:09:56 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue May 30 16:10:27 2017 -0700" }, "message": "ART: More header cleanup - jni_internal.h\n\nMove jni_macros.h to the clients (users of RegisterNativeMethods).\n\nTest: mmma art\nChange-Id: I052ac9f703ec69f0a2df1efcaf9f4a450eb7be1c\n" }, { "commit": "c6ea7d00ad069a2736f603daa3d8eaa9a1f8ea11", "tree": "a6d3332a8592fb806841314d55b206b573d37d86", "parents": [ "d68677c5fde1ace16ea58d65733776c954e7acb4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Feb 01 16:46:28 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Apr 21 08:40:33 2017 -0700" }, "message": "ART: Clean up art_method.h\n\nClean up the header. Fix up other headers including the -inl file,\nin an effort to prune the include graph. Fix broken transitive\nincludes by making includes explicit. Introduce new -inl files\nfor method handles and reference visiting.\n\nTest: source build/envsetup.sh \u0026\u0026 lunch aosp_angler-userdebug \u0026\u0026 mmma art\nTest: source build/envsetup.sh \u0026\u0026 lunch aosp_mips64-userdebug \u0026\u0026 mmma art\nChange-Id: I8f60f1160c2a702fdf3598149dae38f6fa6bc851\n" }, { "commit": "6b2dc3156a2140a5bfd9cbbf5d7dad332ab5f5bd", "tree": "a335cae93c0eabae63e39b8bc13ef3e15f4f7e50", "parents": [ "f83f3f6ecb1153d96cc8007e8a0d1e35af4d3f38" ], "author": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Tue Mar 14 13:26:12 2017 +0000" }, "committer": { "name": "Narayan Kamath", "email": "narayan@google.com", "time": "Thu Mar 16 17:56:04 2017 +0000" }, "message": "ART: Get rid of most of java.lang.DexCache.\n\nAll remaining functionality is moved over to native. The DexCache\nobject itself is allocated in the Java heap, even though there\u0027s no\nlonger much of a reason to do so. It can be changed in a future change\nif needed.\n\nThis also renames mirror::Class:GetInterfaces to GetProxyInterfaces\nsince it\u0027s supposed to be called only for proxies.\n\nTest: test-art-host, cts -m CtsLibcoreTestCases\n\nChange-Id: Ie261f22a9f80c929f01d2b456f170c7a464ba21c\n" }, { "commit": "3b6f440dbd066f03a737da6d292074f47b3fbc29", "tree": "af0da4bdea7ff689929c8e23e1eaf6361dfc3ee8", "parents": [ "5c19b373ae45ec6d67b41917af63cac8c3ebbe89" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Feb 16 16:13:17 2017 -0800" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Feb 21 11:40:51 2017 -0800" }, "message": "jni: Switch to @FastNative for all JNI functions. Deprecate !bang JNI.\n\nSwitches all (248) methods that previously used !bang JNI in art/libcore\nto all use @FastNative.\n\nAlso deprecate !bang JNI since nothing in Android seems to (or should\nbe) using it anymore.\n\nThis measures to be a 3% startup time improvement in system_server.\n\nTest: make test-art-host\nBug: 34955272\nChange-Id: I0881f401c7660c79f275235362777bfa58241deb\n" }, { "commit": "fa4333dcb481e564f54726b4e6f8153612df835e", "tree": "ae597c7587dc214434a180962c4373d3748f51ab", "parents": [ "2d98ba68f13dc219c088a12f369c5778bf398f14" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Feb 14 11:10:34 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Feb 14 11:44:48 2017 -0800" }, "message": "ART: Add operator \u003d\u003d and !\u003d with nullptr to Handle\n\nGet it in line with ObjPtr and prettify our code.\n\nTest: m\nChange-Id: I1322e2a9bc7a85d7f2441034a19bf4d807b81a0e\n" }, { "commit": "a35510de5aa1ec5d39c6870da556e2d60666cdc3", "tree": "65e2c9f63cbe409b2fe7a48194a69ac4ecca6dde", "parents": [ "0d478f289f0e33f19693d135f1d562b57427ed32" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 22:42:09 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 16 23:03:24 2017 +0000" }, "message": "Fix missing OOME check in reflection.\n\nTest: 080-oom-throw\nChange-Id: I5f18ea756e815978bf8682555470505959a8fad5\n" }, { "commit": "19a4d374738da4dc668a078f92dbe887ff9f00d9", "tree": "a210638e93d80cf0a297de07f4c9cdc60784ce50", "parents": [ "aea9ffece7eb32f3884a4ad0553e1df4d90fd9e4" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Dec 08 14:41:46 2016 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Dec 09 13:30:00 2016 +0000" }, "message": "Clean up Class::GetDirectInterface().\n\nOnce the `klass` has been resolved, this function should not\nneed to resolve interface types anymore. Drop the type\nresolution from this function and pass the Class as ObjPtr\u003c\u003e\ninstead of Handle\u003c\u003e. Make callers that expect non-null\nresult DCHECK() that assumption. For the callers that may\nactually need to resolve these interface classes, introduce\na new function ResolveDirectInterface().\n\nAlso improve ObjPtr\u003c\u003e constructors and assignment operator\nto improve overload resolution. For example, if we have\nfoo(ObjPtr\u003cT1\u003e) and foo(ObjPtr\u003cT2\u003e), calling foo(.) with\na T1* would have previously been ambiguous even if T1 is\nnot a base of T2.\n\nTest: m test-art-host\nChange-Id: Iff44a7285f68f0af4507fc9ba37343865422fe84\n" }, { "commit": "709b070044354d9f47641f273edacaeeb0240ab7", "tree": "3a8ac051d7c35076303984d0d892cdd396b60427", "parents": [ "1a4de6a2453a3ad0310aca1a44e7e2d3b6f53bc1" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Thu Oct 13 09:12:37 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Oct 18 14:10:04 2016 -0700" }, "message": "Remove mirror:: and ArtMethod deps in utils.{h,cc}\n\nThe latest chapter in the ongoing saga of attempting to dump a DEX\nfile without having to start a whole runtime instance. This episode\nfinds us removing references to ArtMethod/ArtField/mirror.\n\nOne aspect of this change that I would like to call out specfically\nis that the utils versions of the \"Pretty*\" functions all were written\nto accept nullptr as an argument. I have split these functions up as\nfollows:\n1) an instance method, such as PrettyClass that obviously requires\nthis !\u003d nullptr.\n2) a static method, that behaves the same way as the util method, but\ncalls the instance method if p !\u003d nullptr.\nThis requires using a full class qualifier for the static methods,\nwhich isn\u0027t exactly beautiful. I have tried to remove as many cases\nas possible where it was clear p !\u003d nullptr.\n\nBug: 22322814\nTest: test-art-host\nChange-Id: I21adee3614aa697aa580cd1b86b72d9206e1cb24\n" }, { "commit": "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": "1a5337fff2cc6cb9d563c8b32aca75f485d23373", "tree": "03616e90a91c5c4c1bd7ff03c3d193c3b24bf0db", "parents": [ "c8b722b3cebff245d614e95d1849de45ef7bdf06" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 13:48:23 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Oct 13 17:37:56 2016 -0700" }, "message": "Move mirror::ObjectArray to ObjPtr\n\nAlso reduce calls to ObjPtr::Ptr.\n\nBug: 31113334\n\nTest: test-art-host -j32\n\nChange-Id: I73e1b356972bb079b66332f00f7f07451601044e\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": "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": "9d07414a255cd53f9582497cbc918b7df698c1e4", "tree": "757ebcdd5420f564ab51b0369d0f17292efbb84c", "parents": [ "74a69eda053486e2d563f888a689586c5cfa7217" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 30 09:51:32 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Sep 30 10:09:08 2016 +0100" }, "message": "Move two dangling mirror objects into handles.\n\ntest: libcore.java.lang.reflect.MissingClassesTest#testGetFieldFails\ntest: ./art/tools/run-libcore-tests.sh --mode\u003dhost --variant\u003dX64\nChange-Id: Ib708e914cac6abe4d4fc7cb88e258da83a8219ab\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": "a59d9b228b1eda3bf71a81b6201ec64e26086c23", "tree": "91adf9f0e282eec50ccb28aa2dbc432a5e28d13f", "parents": [ "05ce52b406dcf6b9a24287d14f0e611211e0db3d" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Sep 26 18:13:17 2016 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Sep 27 16:10:35 2016 -0700" }, "message": "Use ObjPtr for reflection.cc/h/inl\n\nChanged Pretty helpers to use this to reduce usage of Decode. The\neventual goal is not have almost any calls to ObjPtr::Decode.\n\nMoved ObjPtr out of mirror namespace for convenience. Added more\nPoisonObjectPointers calls in class linker, thread suspension.\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I44d08db5143d95ed1b65e2f00f9749ef5cf379f7\n" }, { "commit": "26ead4975e1752e8ae2f5ed6fda73876c4f9ff59", "tree": "2ce1b047f54bfb4287483de8eaf79cd12f6058b6", "parents": [ "79f6d706185714dccf80aca20d9f3261337473ae", "268764da8022cafa5661c5b514eaa343c5257e57" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Sep 14 16:55:56 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed Sep 14 16:55:56 2016 +0000" }, "message": "Merge \"Make ScopedAssertNoThreadSuspension no overhead for non-debug\"" }, { "commit": "9323e6e44060545d6c4f925139572868f65b7fe3", "tree": "65f3f380605d75f34d49aec49fca494bec26ef5b", "parents": [ "58735fbfbb2da0c0a19b3239da07c2353c40767d" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Sep 13 08:58:35 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Sep 14 08:01:39 2016 -0700" }, "message": "Separate annotations from dexfile reading.\n\nBug: 22322814\nChange-Id: I867d66da407dd80394a10d19903ebbc1ec3986ff\nTest: test-art\n" }, { "commit": "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": "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": "3aaa37bba53d6df0265793de48b4b0b57327e57a", "tree": "ce795803f0f10003eb3d96d9348da620937675b3", "parents": [ "792c98bb773c8c2390f9cbf774f85be9d9a75332" ], "author": { "name": "jessicahandojo", "email": "jessicahandojo@google.com", "time": "Fri Jul 29 14:46:37 2016 -0700" }, "committer": { "name": "jessicahandojo", "email": "jessicahandojo@google.com", "time": "Tue Aug 23 15:12:26 2016 -0700" }, "message": "creating workflow for mirror::String compression\n\nAll-ASCII String characters are stored in 8-bit blocks\ninstead of 16-bit. The compression has not taken place, but all\nworkflow are in the code already (changing kUseStringCompression in\nstring.h file to TRUE will enable the feature)\n\nNotes: Feature works on interpreter only without optimizing\n\nTest art: m ART_TEST_INTERPRETER\u003dtrue ART_TEST_OPTIMIZING\u003dfalse\n test-art-host\n\nAlso tested with String tests from libcore/:\n1. libcore.java.lang.StringTest\n2. libcore.java.lang.StringBufferTest\n3. libcore.java.lang.StringBuilderTest\n4. libcore.java.lang.OldStringTest\n5. libcore.java.lang.OldStringBufferTest\n\nMemory improvement is 33% (from 6.03% to 4.03%, total String memory\nfrom all apps per total memory of all apps) measured on Angler\nwith Hprof tools\n\nBug: 31040547\nChange-Id: I9cc92c265ebf1305fc06b5fc33efd83797660cce\n" }, { "commit": "542451cc546779f5c67840e105c51205a1b0a8fd", "tree": "11e09bb5abaee12dddffefbe7e425291076dfa7a", "parents": [ "85c4a4b8c9eabfe16e4e49f9b4aa78c1bf4be023" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Jul 26 09:02:02 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Aug 01 18:54:48 2016 -0700" }, "message": "ART: Convert pointer size to enum\n\nMove away from size_t to dedicated enum (class).\n\nBug: 30373134\nBug: 30419309\nTest: m test-art-host\nChange-Id: Id453c330f1065012e7d4f9fc24ac477cc9bb9269\n" }, { "commit": "e01e3644205c3644546889237c20185391a0092e", "tree": "413fcf279b415a305605a601297e3856f13a2948", "parents": [ "de4cf16f467b531373560ca6eb785f0f36606aae" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 25 13:06:04 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Jul 25 13:09:41 2016 -0700" }, "message": "ART: Add pointer-size template to some reflection functions\n\nThe unstarted runtime may run code for a different pointer size,\neven when no transaction is active (e.g., during startup). To\nretain performance when the runtime is up and executing under\nnormal conditions, add a template parameter and use sizeof(void*)\nin places where it is adequate.\n\nFor maintainability, it is necessary to drop the default for\nthe transaction template parameter. Implicit conversions from\nbool to size_t may lead to incorrect code and hard to diagnose\nproblems. So instead ensure that all callers must give all\ntemplate parameter values.\n\nTest: m test-art-host\nChange-Id: I3076883422c8553ede4de5642409c5684a5a9aa8\n" }, { "commit": "a96f316f6f0526c91c6c102a267df6b1e4df39da", "tree": "308f1886e026df0ba6aa816b107172637f11984a", "parents": [ "a6513729ae2ad0d2f9dd21d77ea6cf9ed9fa6eef" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue May 03 14:04:02 2016 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue May 03 14:04:02 2016 +0100" }, "message": "Put a handle on the String object.\n\nOtherwise, a GC could occur in the loop below, and the String\nbe moved (note that there was already one for the Class object to\nprotect for that same problem).\n\nbug:27561834\nChange-Id: I5ad0e0cac148bc6e028d05eff67f18932ec46ea8\n" }, { "commit": "715fdc2c99f688f97c684978632f311a284ba43c", "tree": "a45838f4788a04ec18332bf386e8eaf2daa07632", "parents": [ "a584db5460a31198bf621cdacf5bc304a984efa4" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 18 17:07:30 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Apr 20 10:20:45 2016 -0700" }, "message": "ART: Add some reflection cutouts\n\nAdd Class.getInnerClassFlags and Method.invoke to unstarted runtime.\n\nAllows to compile-time initialize:\n* android.net.NetworkInfo\n* android.net.wifi.WifiInfo\n\nBug: 27265238\n\n(cherry picked from commit b3ffbe32b78c18739736fc998d65430b46c510e5)\n\nChange-Id: I3e267519acf14b08c687f1e831e2027d37158767\n" }, { "commit": "2c6760a0eb1e05d3a89a6cca9d2746da3d547e9e", "tree": "f82d6e7594a8b688c3db6494822a907a1f1ef488", "parents": [ "7fa1f92cbf98c40baa00c6fb9331ec7ac72abfa8", "1133db79350060158f99210c56f111c6dad43563" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Thu Apr 07 18:06:13 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Apr 07 18:06:14 2016 +0000" }, "message": "Merge \"Remove AnnotationAccess and its remaining uses.\"" }, { "commit": "6039e5662507e42896084e561fa546b432698ee6", "tree": "d8a4b6260cc134fcdf7f4a24d89d062dae7f7a61", "parents": [ "dc148d24cc11845036063a2876c9bfef03d00ce5" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Apr 05 18:18:43 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Apr 06 13:43:54 2016 -0700" }, "message": "ART: Add getDeclaredConstructor cutout for unstarted runtime\n\nIn the vein of getDeclaredMethod and getDeclaredField. Refactor\ncode from java_lang_Class to share code.\n\nAllows to compile-time initialize:\n* sun.security.x509.AVAKeyword\n* sun.security.x509.X500Name\n\nBug: 27265238\n\n(cherry picked from commit f72f19fac0016499d33fd75b0463d0128a58a9f4)\n\nChange-Id: I075c1b11f3408f5b31e4f84140a24fd8d3eaa17e\n" }, { "commit": "1133db79350060158f99210c56f111c6dad43563", "tree": "c84b4ae2df13a32539645c0904f08c874c20ae31", "parents": [ "a07be75830af60ce50aba357e8de066e849aa21c" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Apr 04 19:50:14 2016 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Apr 04 19:50:14 2016 -0700" }, "message": "Remove AnnotationAccess and its remaining uses.\n\nArt side of this change. Adds a test to ensure annotations not marked\nfor runtime retention can\u0027t be seen at runtime.\n\nBug: 27912552\nChange-Id: I078069b7b3cb72bfe7d0b9ea61e527fee04d56a3\n" }, { "commit": "bc4d218ce2ceef0c4f308d4ff42f7ec1ec43c40e", "tree": "a583db9be377183508653607a724dcb8742f4ca2", "parents": [ "01c30e8dbc45bdc5d922cef6e5a404be7bed0e8c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Feb 22 10:03:12 2016 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Feb 26 15:43:14 2016 -0800" }, "message": "ART: Add unstarted-runtime functions\n\nAdd more functions to allow compile-time initialization of\ncode.\n\nBug: 27248115\n\n(cherry picked from commit 0866f4ed6338faa4a193b7e819fc7cd72bd7b0ae)\n\nChange-Id: Iaf8d92deb73547ccd31c0d6dde68da3bc14c3985\n" }, { "commit": "64fdc804f6b4d8a6f7d86bd9672e3bb1cde7056f", "tree": "4cb67ba411e3845054347745657bdc06ca8472ab", "parents": [ "27ec302da670ffddaf1278b5b7155b7742aa37fd" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Feb 08 10:30:04 2016 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Mon Feb 08 10:34:36 2016 -0800" }, "message": "Default methods should not be found by getDeclaredMethod\n\nWe were incorrectly returning default methods from getDeclaredMethod\ncalls on an implementing class that uses the default implementation.\n\nBug: 27060609\n\nChange-Id: I2e07023a11585e5f7cd92c4c0e1263f54aea1f5f\n" }, { "commit": "df187e4cc8b1b2dc6cb0043eb91d1dfd72cb490b", "tree": "a2fbb7ec726c57311238eae04a395936b8709cbd", "parents": [ "9f95ba750583735cff1f2921b8014099dd801734" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Mon Feb 01 11:43:26 2016 -0800" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Tue Feb 02 10:56:01 2016 -0800" }, "message": "Revert \"Revert \"reflection: Add new 1.8 AnnotatedElement methods and tests\"\"\n\nThis reverts commit 28a2a186537db9fc5a8492e36d9603b48854c04f.\n\nRuntest failure has been fixed in I28c3707e38c1f69ea9d3660f68136f688122ac4e\n\nChange-Id: I1749dc89b790d44b5e40bc8b20aa62885bda792b\n" }, { "commit": "28a2a186537db9fc5a8492e36d9603b48854c04f", "tree": "31fc65e44fac0d24b53c61df322624fe381a75d8", "parents": [ "c167ee9b65f05f7c6f007d587fd1655388edaee9" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 29 10:25:01 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 29 10:25:01 2016 +0000" }, "message": "Revert \"reflection: Add new 1.8 AnnotatedElement methods and tests\"\n\nBreaks interpreter access checks:\n\n+Exception in thread \"main\" java.lang.InternalError: java.lang.IllegalAccessError: Method \u0027void java.lang.reflect.Proxy.doNewInstanceCheck()\u0027 is inaccessible to class \u0027$Proxy0\u0027 (declaration of \u0027$Proxy0\u0027 appears in generated class)\n\n\nThis reverts commit c167ee9b65f05f7c6f007d587fd1655388edaee9.\n\nChange-Id: I6c9429cb6f298e89a2da22f7ded0728251321446\n" }, { "commit": "c167ee9b65f05f7c6f007d587fd1655388edaee9", "tree": "57e846f6aaad969020972f55a082e3e22884592e", "parents": [ "2aaf58e90c9229610b2a16644e9866b6741ce9ca" ], "author": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Thu Jan 14 16:05:47 2016 -0800" }, "committer": { "name": "Igor Murashkin", "email": "iam@google.com", "time": "Wed Jan 27 13:14:10 2016 -0800" }, "message": "reflection: Add new 1.8 AnnotatedElement methods and tests\n\nAlso:\n* Bump the size of the Class vtable size since 3 new virtuals were\nadded to java.lang.Class\n* Refactor Method#isDefault test into a separate file within 048 test.\n\nChange-Id: I8420a4d208bb60874a9cf996766313c7d5058c45\n" }, { "commit": "0e7fa6b06e37e7c95509cd234571f469bf5821c2", "tree": "887228f1be1a8837d059af5fc83c27a75b281dd9", "parents": [ "d6ff932575b99d51e3c37e77d696160acbf53e21" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 13 15:28:58 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Jan 13 15:42:35 2016 +0000" }, "message": "Also print to error stream.\n\nSo we can test between all kinds of configuration (for example\nhost no-dex2oat removes all error logs).\n\nChange-Id: I5936fdd963e3805c49258ee6ed50d8def23d1aca\n" }, { "commit": "7ea4c09989b37351e6c17a10d35934812053a746", "tree": "2ccc1f223830e30e4c2ca27310764957125476f2", "parents": [ "a90d8bc3a3772877f4490d0509d53053069e5b55" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 07 10:55:14 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 07 12:08:22 2016 +0000" }, "message": "Emit an error message when looking for String#offset.\n\nDebugging two different apps relying on it and swallowing\nthe getDeclaredField exception took too much of my lifetime.\nAdding a LOG(ERROR) can help diagnose quicker.\n\nAlso remove unused method getDeclaredFieldInternal.\n\nChange-Id: I0923556a4fdbad2b06c1811ed741d23dbd4aa0a0\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": "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": "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": "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": "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": "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": "35831e8bfa1c0944d4c978d99c4c5b9577945170", "tree": "a347ecaf4265a1713c86d528392e162d5edfaebc", "parents": [ "171b12e8060f63cf900ffc43d6db35125f6b8c83" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Sep 11 11:59:18 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Oct 27 15:13:37 2015 +0000" }, "message": "Reduce memory used by CompiledMethods.\n\nUse LengthPrefixedArray\u003c\u003es instead of SwapVector\u003c\u003es to store\nCompiledMethod data and get rid of the unnecessary members\nof CompiledMethod to reduce dex2oat memory usage. Refactor\nthe deduplication from CompilerDriver to a new class.\n\nUse HashSet\u003c\u003e instead of std::set\u003c\u003e for the DedupeSet\u003c\u003e to\nfurther decrease the memory usage and improve performance.\n\nThis reduces the dex2oat memory usage when compiling boot\nimage on Nexus 5 (with Optimizing, -j1) by ~6.75MiB (5%).\nThis also reduces the compile time by ~2.2% (~1.6% dex2oat\ntime; with Optimizing, without -j).\n\nChange-Id: I974f1f5e58350de2bf487a2bca3907fa05fb80ea\n" }, { "commit": "6cfc2c086c47342fd8f5cb09f565979333066473", "tree": "c30f4bae6bbd0d50151d3a7f2f1679d39b62de70", "parents": [ "16065ce56394c73c87dcb78ead4164ddc80fddb3" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 12 15:06:16 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Oct 12 15:16:05 2015 -0700" }, "message": "Avoid visiting find array class cache as roots\n\nIf we visit the find array class cache as roots it will prevent\nunloading for any array classes in the cache. This is not ideal\nsince it may take a long time for the entries to get replaced.\n\nAlso added a missed exception check in getDeclaredClasses.\n\nBug: 22720414\n\nChange-Id: Id34557fd034b3e3967ef629301ee251931937849\n" }, { "commit": "72f9075e92cfc4632d6c50f282a0f6319b718882", "tree": "59da1347488e2298af46017992d0ab9f44c5966a", "parents": [ "9b29f3ef7df18c84d922634709e38a1913709ad7" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Oct 01 13:35:09 2015 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Oct 01 16:58:42 2015 -0700" }, "message": "Move getPublicFieldRecursive to native JNI\n\nBenchmark: libcore/benchmarks/src/benchmarks/regression/ReflectionBenchmark.java\n\nPrevious benchmarks:\n Class_getField 814.26 ns; σ\u003d6.44 ns @ 3 trials\n GetInterfaceStaticField 1552.28 ns; σ\u003d38.22 ns @ 10 trials\n GetSuperClassField 939.85 ns; σ\u003d2.94 ns @ 3 trials\n\nNew benchmarks:\n Class_getField 735.97 ns; σ\u003d0.98 ns @ 3 trials\n GetInterfaceStaticField 1178.98 ns; σ\u003d6.14 ns @ 3 trials\n GetSuperClassField 853.76 ns; σ\u003d16.86 ns @ 10 trials\n\nBug: 24209213\n\nChange-Id: I08073cb36b007ac33af010dab917a6f1e51107c0\n" }, { "commit": "2a5892f9ebcd4b7bd7343db50e578bd24b8e55a6", "tree": "ef5ef732bab16392bc3a67b3d797797a2292cd66", "parents": [ "51995e135794e01a7aecccfed81dc00a737e24bd" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Aug 31 15:00:40 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Sep 01 15:40:48 2015 -0700" }, "message": "Move more Class annotations to native.\n\nArt side of this change. There is also a corresponding Libcore change.\n\nSeeing speedup in AnnotatedElementBenchmark.\nGetDeclaredClasses\t 21.61x\nGetDeclaringClass\t108.73x\nGetEnclosingClass\t 87.81x\nGetEnclosingConstructor\t 23.35x\nGetEnclosingMethod\t 22.71x\nGetModifiers\t\t152.47x\nGetSimpleName\t\t106.11x\nIsAnonymousClass\t 91.28x\nIsLocalClass\t\t 51.95x\n\nChange-Id: I2b7f7eb7785fc20671fd7c338ffa9c7048a44a48\n" }, { "commit": "13e748b28c5f2bd1e83674d2ca899ff61ae5c0a1", "tree": "387724eebfe7cd0f7be368e64b22af3894eea4c7", "parents": [ "dda43e7e9680e6eead02876e67718d01837d89e0" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Aug 25 20:44:19 2015 +0000" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Tue Aug 25 14:20:09 2015 -0700" }, "message": "Revert \"Revert \"Move annotations to native.\"\"\n\nThis reverts commit 7db6dd79a24570448ae737ee1946b00396696cac.\n\nAdds check if field\u0027s declaring class is proxy.\nBug: 23508574\n\nChange-Id: Ie829f1526e74427711e818b56d1588d92946cbf6\n" }, { "commit": "7db6dd79a24570448ae737ee1946b00396696cac", "tree": "490c05f1488141adf5668b34cbec28830fe8295c", "parents": [ "0042c6d49b8488c78f0b937063e316e8d6244439" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 25 10:53:19 2015 +0000" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 25 10:53:19 2015 +0000" }, "message": "Revert \"Move annotations to native.\"\n\nThis reverts commit 0042c6d49b8488c78f0b937063e316e8d6244439.\n\nReverting this change (as well as the companion CL\nhttps://android-review.googlesource.com/#/c/167510/ in\nplatform/libcore) as they make libcore test\nlibcore.java.lang.reflect.ProxyTest#test24846 fail.\n\nChange-Id: Ie0676cabb128277c7df5dab7bde17aefd3b2c09c\n" }, { "commit": "0042c6d49b8488c78f0b937063e316e8d6244439", "tree": "170286a01d08badebacaebed1db325bc889e259e", "parents": [ "300432d27ba9dd6961458fb1e1c845766f25c50a" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Wed Jul 29 20:14:10 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Fri Aug 21 17:05:27 2015 -0700" }, "message": "Move annotations to native.\n\nArt side of this change. There is also a corresponding Libcore change.\n\nSeeing ~2-3x speedup over dalvik KK MR1 in AnnotatedElementBenchmark.\nBenchmark\t\t\t Speedup of Art AOSP to Dalvik KK MR1\nGetAllReturnsLargeAnnotation\t 2.99\nGetAllReturnsMarkerAnnotation\t 2.20\nGetAllReturnsNoAnnotation\t 2.43\nGetAllReturnsSmallAnnotation\t 2.52\nGetAllReturnsThreeAnnotations\t 2.87\nGetAnnotationsOnSubclass\t 2.42\nGetDeclaredAnnotationsOnSubclass 2.49\nGetFieldAnnotation\t\t 2.68\nGetFieldAnnotations\t\t 2.60\nGetMethodAnnotation\t\t 2.66\nGetMethodAnnotations\t\t 2.61\nGetParameterAnnotations\t\t 2.52\nGetTypeAnnotation\t\t 2.56\nGetTypeAnnotations\t\t 2.17\nIsFieldAnnotationPresent\t 3.26\nIsMethodAnnotationPresent\t 4.99\nIsTypeAnnotationPresent\t\t 1.34\n\nChange-Id: Ibdbb6d23b17eaab6e83c8774b1bb9401e8227941\n" }, { "commit": "cf36d493124d8048efa0bd6f67d817ce3cd6b725", "tree": "fac1336f54ea477ce1afe2e99a04b68db38c7ac8", "parents": [ "1cad8c7c63b600a3da83bf05fb645e08ac8fafc0" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Aug 12 19:27:26 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Aug 17 10:04:42 2015 +0100" }, "message": "ART: Compress LengthPrefixedArray on 32-bit targets.\n\nPreviously, the LengthPrefixedArray\u003cArtMethod\u003e on 32-bit\ntargets contained a 64-bit length field followed by the\nArtMethod elements with size only a multiple of 4, not 8.\nConsequently, an odd-length array broke the alignment for\nthe following array which would have the 64-bit length\nplaced at an unaligned address.\n\nTo fix that, we make the length field 32-bit and explicitly\npass the alignment information to the LengthPrefixedArray.\nThis also makes the 32-bit boot image a bit smaller.\nOn Nexus 5, AOSP, ToT, the field section is 11528B smaller\nand the method section is 21036B smaller. 64-bit targets\nshould see the same savings for the field section but no\ndifference for the methods section.\n\nChange-Id: I3e03e7b94129025c8a1c117c27645a34dec516d2\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": "05b7226787f1470ad93f6f632fed60f70bc8631e", "tree": "0f73cb94559ad3eeec6b6bb863377f4cb7366af1", "parents": [ "5592c688070fa327f56036c4c67c790289db63d9" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jun 17 16:11:12 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Jun 17 16:55:40 2015 -0700" }, "message": "Fix some java_lang_Class related moving GC bugs\n\nThere was some missing handles around mirror::Class*.\n\nBug: 21898408\nChange-Id: Icb754074dfb469473101d20d6873a5bc3274abc5\n" }, { "commit": "281e5002fe818ce75e6ca7b7f9bdffcd2a43231a", "tree": "6648ffb3f15feb5d48e0ca87d8b8d2a9b73e06ad", "parents": [ "8d73a91d8f4c89ac152d18081b07093f68c56a11" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 11 16:21:42 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jun 11 17:12:32 2015 -0700" }, "message": "Fix compaction bug in Class_getDeclaredMethodsUnchecked\n\nAdded handle to fix the bug.\n\n(cherry picked from commit b89a8c66e360575d2dc1ec1229235735c56968ff)\n\nBug: 21638351\nChange-Id: I1c3abea33aa825d3a28c1fc5cb415508686ad93e\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": "4e067787fff6cb6ce027d003c5ffe5dee4cb0d92", "tree": "dac4e669b0419145506d9134c0a0fe02a9a47921", "parents": [ "c3912c8a2db109a15603554fd456f56cd0a69ad0" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 13 13:13:24 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed May 13 14:17:20 2015 -0700" }, "message": "Also print class for NoSuchFieldException\n\nExample message:\nCaused by: java.lang.NoSuchFieldException: No field value in class Ljava/lang/String;\n\nAdded test.\n\n(cherry picked from commit 3beb245da9392818e3154d47593f82cf0ef69aac)\n\nBug: 20881251\nBug: 21027454\n\nChange-Id: I4043cbf26c3077952b6c151da0d0edd980da26b1\n" }, { "commit": "69bdcb29fdbd8266374e3793cb4e28dcc5daf0f9", "tree": "60b122ece08b503b9f1d020306ab4151d4680ae6", "parents": [ "f0a3d3dad248ad0dcb7cfc6addcc66d8bd117e48" ], "author": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Apr 27 20:54:07 2015 -0700" }, "committer": { "name": "Jeff Hao", "email": "jeffhao@google.com", "time": "Mon Apr 27 20:55:25 2015 -0700" }, "message": "Fix java_lang_Class newInstance for strings; also quick entrypoints.\n\nChange-Id: I35fd23c5a9051e1ffda0ecc2cbafb5d318c7b5e6\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": "2cebb24bfc3247d3e9be138a3350106737455918", "tree": "d04d27d21b3c7733d784e303f01f873bb99e7770", "parents": [ "1f02f1a7b3073b8fef07770a67fbf94afad317f0" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Apr 21 16:50:40 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Apr 22 12:44:27 2015 -0700" }, "message": "Replace NULL with nullptr\n\nAlso fixed some lines that were too long, and a few other minor\ndetails.\n\nChange-Id: I6efba5fb6e03eb5d0a300fddb2a75bf8e2f175cb\n" }, { "commit": "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": "c136312832d4be25db2ecc5673967d71d0ad4b9c", "tree": "b9cdcccdd947632b2031ae2b4c3a9c3658ef3366", "parents": [ "1576be32be4a99a1cffdaaf209a3cd67e8b2f88a" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Apr 09 14:13:13 2015 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Fri Apr 10 18:46:03 2015 +0100" }, "message": "Avoid using dex cache array pointers in libart.\n\nIn preparation for making dex cache arrays native, avoid\nusing them in Java code.\n\nThis causes a performance regression for our reflection\nbenchmarks. Class_getDeclaredMethod and Class_getMethod\ntake an up to 30% hit, measured using the Quick compiler.\nWe accept this hit at this stage and we will tune the\nperformance after we\u0027re done with the larger effort.\n\nCompanion libcore/ change:\n https://android-review.googlesource.com/146069\n\nBug: 20134538\nChange-Id: Ibbef3b50043a1311cd40723ed42e1f1c609b8fc1\n" }, { "commit": "02d2f290380c6aeff5f5a089ee38fa5d55e46130", "tree": "7a1e2bd74768db29f25b087e2120fb4c61897da5", "parents": [ "1f940310658cd5a15e12305463fb6d2d508bbd26" ], "author": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Apr 03 13:35:16 2015 -0700" }, "committer": { "name": "Hiroshi Yamauchi", "email": "yamauchi@google.com", "time": "Fri Apr 03 13:35:16 2015 -0700" }, "message": "Fix the read barrier builds.\n\nA check failure (!UseJit() in ArtMethod::Invoke()) in forced interpret\nonly mode in the eng build with JIT enabled by default on target.\n\nA build failure (inl header file issues) in the USE_BAKER_READ_BARRIER\nbuild.\n\nChange-Id: Ib13632b10864ad8b4691b66971c0ab5d2ae1e675\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" } ], "next": "277ccbd200ea43590dfc06a93ae184a765327ad0" }