)]}' { "log": [ { "commit": "6624d58c11b43a10c802037bf1c5754ca276156c", "tree": "11e6ae686e70c091d50893f1169131c4c31081ad", "parents": [ "f9dbb97a1625b61a395406351e042921f9cfb455" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Sep 01 15:02:00 2020 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Sep 02 13:10:42 2020 +0000" }, "message": "Fix bug in StackVisitor::GetVReg.\n\nFloats can be stored in core registers within compiled code, so use the\nrepresentation returned by the stack maps to know which register to read\na value.\n\nBug: 147572335\nTest: 457-regs\nChange-Id: Ibe6642f2fae8206f2c230006ae85d73b47501c3b\n" }, { "commit": "6fc471e510d6a4e9c31fcab6c0542e2efdf50099", "tree": "81e3a310a73380e798a6b26583cb32d642e69468", "parents": [ "9f92d9b512c41c17b19f90cc3afa495952e17474" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Mar 03 16:51:33 2020 -0800" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Mar 03 16:53:24 2020 -0800" }, "message": "Check vreg count on debuggable\n\nPreviously we were only performing a check that we got the correct\nnumber of vregs on deoptimization when kIsDebugBuild. Extend this\ncheck to also occur when the runtime is JavaDebuggable.\n\nBug: 144947842\nTest: none\nChange-Id: I7d240008391a1499c159e478b59a509e2362e99f\n" }, { "commit": "51ad7fe2ecfb731d5d23e08ec3c58909825d0beb", "tree": "3382516c2f1cd52ae3dbb5ecec2b48149de5a630", "parents": [ "89b85ebb4a0daf390701f1b395d1cd01239a75a1" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Feb 04 12:46:47 2020 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Feb 05 17:11:49 2020 +0000" }, "message": "Code cleanup around exception handling and stack walking.\n\n- Remove unused code setting the method and dex pc below an upcall.\n- Clear cur_oat_quick_method_header when stack walking to be consistent\n in having a runtime method and a null oat method header.\n\nTest: test.py\nChange-Id: I87672d193eb2e62add3ae7b8a42f2202e8eb927c\n" }, { "commit": "e91e795a77b96d58276f75b1b244a5509ef8c215", "tree": "d34f8164bb7adb8ed46e3c989877012555e194c3", "parents": [ "aa6f6f1c86705fe3cedf929bd30d1e6f51524594" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Jan 23 10:15:56 2020 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 31 08:20:57 2020 +0000" }, "message": "Replace instrumention id with stack pointers.\n\nReplace instrumentation ids for instrumentation frames, which are\ninherently broken (see b/72608560), and use stack addresses instead\nto properly identify which frames to pop / unwind.\n\nBug: 72608560\nBug: 148166031\n\nTest: ./art/test/testrunner/testrunner.py --trace --debuggable --ndebuggable --optimizing --interpreter --jit --debug --ndebug -j32\nTest: run-libjdwp-tests.sh\nTest: 2011-stack-walk-concurrent-instrument\nTest: ./art/test/run-test --host --dev --runtime-option -verbose:deopt,plugin --prebuild --compact-dex-level fast --jit --no-relocate --create-runner --runtime-option -Xcheck:jni 1965-get-set-local-primitive-no-tables\n art/tools/parallel_run.py -j80 /tmp/path/to/runit.sh --out failure.txt\n\nChange-Id: I71f6e55b9da608796cd3142b147f7b50bbd292ec\n" }, { "commit": "00391824f4ee89f9fbed178a1ee32bc29fa77b3b", "tree": "aea6bc5e49801c5b4816257ab16a97181ef0d911", "parents": [ "001e5b33ba7065dde0b85450830b605733ae1685" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 10 10:17:23 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 18 06:50:39 2019 +0000" }, "message": "Add an implementation of Nterp for x64.\n\nAnd enable it on x64 when runtime and ArtMethod requirements are met\n(see nterp.cc).\n\nTest: test.py\nBug: 112676029\nChange-Id: I772cd20a20fdc0ff99529df7495801d773091584\n" }, { "commit": "013d1ee96b928f3bda9031e94d4a69f827133ce6", "tree": "4e374c043330f123dc3888922b554fd1291349b4", "parents": [ "9ca8b2bf46978e3a5698f8a27b48aa7eff3514df" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Dec 04 16:18:15 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 17 09:43:08 2019 +0000" }, "message": "Introduce the notion of an nterp frame.\n\nSee comments in nterp_helpers.cc. An nterp frame follows the\ncalling conventions and exception handling of the compiler. There are\nno ManagedStack transitions, and the compiler and interpreter can\njust call each other directly.\n\nFor the stack walker, an nterp frame looks like a compiled frame.\n\nThis CL introduces an nterp frame, another CL will contain an\nimplementation for x64.\n\nBug: 119800099\nTest: test.py\nChange-Id: Ie9b691f58908b7f283b4cd63b84b651526155d27\n" }, { "commit": "cffa254b3b7c307b557ba2250fc822db352d3293", "tree": "ef49730b09bd0f4ebb26387311ed4501db3e29bf", "parents": [ "9a8634a7bc4f69e6c89933f906ddd0a45dd05aa7" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Jul 01 15:31:41 2019 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Jul 22 13:11:49 2019 +0000" }, "message": "Add fast path for exception vreg copying.\n\nIt is not necessary to decode CodeInfo in GetVReg\nsince the caller already did it.\n\nTest: ./art/test.py -b --host --64 --interpreter\nChange-Id: I0f8941f43acdc0f2c43b78ef87d3e796e320c959\n" }, { "commit": "2808be84a8f86542450670acdb5e6b55498ff5f0", "tree": "f41b63e3109a6a6a9bd534f32b8d92f20eb9ef35", "parents": [ "460f05480379d5c75555d06c4938bc990ce6ff83" ], "author": { "name": "Artem Serov", "email": "artem.serov@linaro.org", "time": "Thu Dec 20 19:15:11 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jul 15 09:30:36 2019 +0000" }, "message": "ART: Introduce stackmap with no DexRegInfo.\n\nSome of safepoints don\u0027t need to have DexRegisterMap info;\nthis will decrease the stackmap size.\n\n.oat file size reduction:\n - boot.oat: -233 kb (-5.4%)\n - boot-framework.oat: -704 kb (-4.9%)\n\nTest: 461-get-reference-vreg, 466-get-live-vreg.\nTest: 543-env-long-ref, 616-cha*.\nTest: test-art-target, +gc-stress.\n\nChange-Id: Idbad355770e30a30dcf14127642e03ee666878b8\n" }, { "commit": "62e7c097988a52add5f51640ff8cfd79229a9dbb", "tree": "bbdc0ab64f8f6dffd8fb38029b8fab7becd72144", "parents": [ "e7bd7aa01c05ed8ed7d2227f5fc7c4e8ea0861cf" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 08 09:43:01 2019 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Jan 08 14:15:24 2019 +0000" }, "message": "Put the deoptimized method in the trace tag.\n\nUseful when debugging with a trace.\n\nTest: m\nChange-Id: I59cb5d6810acdeb7cdd40604f8b123cb42aa8be7\n" }, { "commit": "d5ae80c223a2aa51b6a54faa0f014acdd013103f", "tree": "25b44b63625d83f36dc405d9ef9ff00b8dfc8fe3", "parents": [ "f355ec703f519c3091853d85f61904be6dfb68aa", "813a86307e1f3437ed9d17aeea2c5c6ffcda5c67" ], "author": { "name": "Treehugger Robot", "email": "treehugger-gerrit@google.com", "time": "Thu Nov 29 23:36:04 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Nov 29 23:36:04 2018 +0000" }, "message": "Merge \"Use ArtMethod::GetDex{File,Cache}() more.\"" }, { "commit": "813a86307e1f3437ed9d17aeea2c5c6ffcda5c67", "tree": "6ab0118464a3f721dc967d52a9607a96ce6e4188", "parents": [ "7e09737125ddb8c6f243ac1882d2a23da3f109c4" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 29 16:17:01 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Thu Nov 29 17:54:54 2018 +0000" }, "message": "Use ArtMethod::GetDex{File,Cache}() more.\n\nDo not go through the declaring class when it can cause\na DexFile or DexCache mismatch for obsolete methods.\n\nAlso fix similar potential mismatch in hiddenapi.\n\nThis is a follow-up to\n https://android-review.googlesource.com/834082 ,\n https://android-review.googlesource.com/836008 .\n\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nBug: 119830111\nChange-Id: I3fdf1aa1bc7bab816d5d8034b107506a32438b77\n" }, { "commit": "c7d878d4be3f0971d4b86266308c7540a26c2856", "tree": "041a6d897df81ae64f25c211f16e8640959a35f5", "parents": [ "d295a75d952742604eaae0530daea19dd6fd782c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 19 18:42:06 2018 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Nov 19 17:09:07 2018 -0800" }, "message": "Revert^4 \"ART: Add StackVisitor accepting a lambda\"\n\nThis reverts commit ec43a01e0ac948c59d5b1f9c3812f2901b48942a.\n\nReason for revert: Added missing code.\n\nBug: 115837065\nTest: m test-art-host\nChange-Id: Id30ccdf9aa9a6e56a914254793e399f2712c882d\n" }, { "commit": "ec43a01e0ac948c59d5b1f9c3812f2901b48942a", "tree": "57e1a23dcc0beba8e98841b8cf063f6153b08c9b", "parents": [ "3d477f3a3eea757a49ca621cc579f711f22fccdd" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sat Nov 17 13:10:40 2018 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Sat Nov 17 13:10:40 2018 +0000" }, "message": "Revert \"Revert^2 \"ART: Add StackVisitor accepting a lambda\"\"\n\nThis reverts commit 3d477f3a3eea757a49ca621cc579f711f22fccdd.\n\nBug: 115837065\n\nReason for revert: Breaks jdwp tests.\n\nChange-Id: I09249331798970751a20c6b41675c3efef72adfb\n" }, { "commit": "3d477f3a3eea757a49ca621cc579f711f22fccdd", "tree": "b4b402d0181610c62062c8c597ef30ee2840b6b1", "parents": [ "54c7da9c50ee85ade636605cd6ea18b4c2bc69fa" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Nov 16 16:40:45 2018 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Nov 16 10:13:38 2018 -0800" }, "message": "Revert^2 \"ART: Add StackVisitor accepting a lambda\"\n\nThis reverts commit 8248490f24e8582ce2ead8cd878d8a2c38310a48.\n\nReason for revert: Fixed instrumentation.cc\n\nBug: 115837065\nTest: m test-art-host\nChange-Id: I484833f4712c835fcaf3452dca4cae5b031d5a7d\n" }, { "commit": "8248490f24e8582ce2ead8cd878d8a2c38310a48", "tree": "f542af6560561526fc413e361e3323a615b5429c", "parents": [ "36f8d22c672498753b9edc66ba11acc9816b2a17" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Nov 16 00:47:49 2018 +0000" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Nov 16 00:47:49 2018 +0000" }, "message": "Revert \"ART: Add StackVisitor accepting a lambda\"\n\nThis reverts commit 36f8d22c672498753b9edc66ba11acc9816b2a17.\n\nReason for revert: Seems to be breaking test 687\n\nBug: 115837065\nChange-Id: I83bb1a9d76cc701c3d582778e5047ebd5dab5d29\nTest: TreeHugger\n" }, { "commit": "36f8d22c672498753b9edc66ba11acc9816b2a17", "tree": "7a7195bf16db14d7f56f94a4d6421f6837b2f686", "parents": [ "a53da8f8170ddaff3e4160641938ba1b7806202c" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Oct 08 08:13:42 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Nov 02 16:01:30 2018 -0700" }, "message": "ART: Add StackVisitor accepting a lambda\n\nTo simplify implementation of visitors. Rewrite some\nvisitors.\n\nBug: 115837065\nTest: mmma art\nTest: m test-art-host\nChange-Id: I04172d7626b702c1d69c63eba65526378935a25d\n" }, { "commit": "98ea9d9d82ab078ca10fa7f8e02eddda94cf1d98", "tree": "a848b7e41ff227a2d3d4d6795ec11089f39cb6ca", "parents": [ "02338775e33b553be51d44ff60bb1ef8e527bd94" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Oct 19 14:06:15 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Oct 23 15:19:55 2018 -0700" }, "message": "ART: Refactor for bugprone-argument-comment\n\nHandles runtime.\n\nBug: 116054210\nTest: WITH_TIDY\u003d1 mmma art\nChange-Id: Ibc0d5086809d647f0ce4df5452eb84442d27ecf0\n" }, { "commit": "0aa7a5a6a7bc350b79351f52e26c97747e927acf", "tree": "a583889668d9703d16d37586d56421bd27c62661", "parents": [ "e12575640dca5118bf96245f373acda276c22178" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Wed Oct 10 15:58:14 2018 +0000" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Thu Oct 11 16:41:54 2018 -0700" }, "message": "Revert^4 \"JVMTI PopFrame support\"\n\nThis reverts commit 202b617acf477e8e8e11915f467120a0bd518e74.\nThis unreverts commit 202b617acf.\nThis unreverts commit 88a2a9d7a1.\n\nThere were several bugs with the implementation of pop-frame related\nto interactions between the jit, exception handling, class-loading,\nand deoptimization.\n\n- We were instrumenting the target thread stack in cases where it was\n unnecessary which caused the exception handler to incorrectly\n determine that a method was not deoptimizable. This caused the\n pop-frame to be ignored.\n\n- We were incorrectly sending ExceptionCatch events if an exception\n suppressed by pop-frame would have been caught in the current frame.\n\n- We were allowing pop-frame to be used on threads suspended in the\n ClassLoad or ClassPrepare events despite having surprising semantics\n in that situation (see b/117615146).\n\nNeeded to modify test 1953 slightly for inclusion in CTS. I needed to\nmake the CTS entrypoint not run the class-load tests (since the cts\nconfiguration means the classes are loaded by the verifier and not the\ninterpreter). I updated the expected.txt and check script to reflect\nthis.\n\nReason for revert: Fixed issue causing Exception events to sometimes\n eat PopFrame and other issues.\n\nTest: ./test.py --host\nTest: ./art/tools/run-libjdwp-tests.sh --mode\u003dhost\n\nBug: 73255278\nBug: 111357976\nBug: 117533193\nBug: 117615146\n\nChange-Id: I655c4fe769938cf41d7589f931d6710cf2001506\n" }, { "commit": "fa6a1b0a1d9d186a0ad9bb413a4e49f1ffd187d8", "tree": "6d3bef945030868d7e78162dc3f4f35a07ce410a", "parents": [ "a0878aa9f4f4102e56a6381b28a8d7c38dec80b9" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Sep 07 08:11:55 2018 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Sep 07 09:13:50 2018 -0700" }, "message": "ART: Continue adding override annotations\n\nUse Clang-tidy\u0027s modernize-use-override to add more annotations. Ignore\ninferred annotations on destructors.\n\nBug: 32619234\nTest: mmma art\nChange-Id: Ic432c928e398d44df9171e42db04ee19946e6887\n" }, { "commit": "bbc6e7edb5fca4a61ac53dd9bce79cb4f0bb3403", "tree": "0fbce767bc383358cf4cd65aafc74140e1850982", "parents": [ "19379b58bd433da91230e4fe6cd96e7416d16adc" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Fri Aug 24 16:58:47 2018 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Tue Aug 28 11:06:07 2018 +0100" }, "message": "Use \u0027final\u0027 and \u0027override\u0027 specifiers directly in ART.\n\nRemove all uses of macros \u0027FINAL\u0027 and \u0027OVERRIDE\u0027 and replace them with\n\u0027final\u0027 and \u0027override\u0027 specifiers. Remove all definitions of these\nmacros as well, which were located in these files:\n- libartbase/base/macros.h\n- test/913-heaps/heaps.cc\n- test/ti-agent/ti_macros.h\n\nART is now using C++14; the \u0027final\u0027 and \u0027override\u0027 specifiers have\nbeen introduced in C++11.\n\nTest: mmma art\nChange-Id: I256c7758155a71a2940ef2574925a44076feeebf\n" }, { "commit": "2c8206ffeb07cebe2f612b8c66c0f4f4132e2be2", "tree": "9de26eb2e85f2ab46421caf0f1ca773dfcfca7f6", "parents": [ "2d441b1253cca39bf3078422e8f0b2f4ec1bb408" ], "author": { "name": "Alex Light", "email": "allight@google.com", "time": "Fri Jun 08 14:51:09 2018 -0700" }, "committer": { "name": "Alex Light", "email": "allight@google.com", "time": "Tue Jul 10 08:44:17 2018 -0700" }, "message": "Correctly send MethodExit events from exception handlers\n\nDue to the way that exceptions are handled it was possible for a\nMethodExit event to be sent multiple times for the same exception.\nThis fixes that issue by correctly keeping track of which exception\nevents have already been sent and correctly tracking the current\nexception being thrown.\n\nTest: ./test/testrunner/testrunner.py --host --runtime-option\u003d-Xplugin:libtracefast-trampolined.so\nTest: ./test.py --host\n\nChange-Id: I7fa09f0f3f82181430b18805da5ba8daf68d4b89\n" }, { "commit": "a296f908e186398ff413c78f2460cb3138e531ab", "tree": "04592d0bcf8afac399dc562efdcac491072366eb", "parents": [ "b275072ddfb1bfcacc914aff08cb0f5774000258", "93bd3619de4616040d672a4ed3825976bed4bfce" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Jul 05 00:03:21 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Jul 05 00:03:21 2018 +0000" }, "message": "Merge \"Use iterators to access stack map data.\"" }, { "commit": "93bd3619de4616040d672a4ed3825976bed4bfce", "tree": "2ab3424374ed3d316053ed8e9ec2d4b95ea2ef4d", "parents": [ "f07d5617770c37d87447c8bddf105eb0469ab093" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon Jul 02 19:30:18 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Jul 04 14:23:12 2018 +0100" }, "message": "Use iterators to access stack map data.\n\nTry to simplify the code using the recently added iterators.\n\nTest: test-art-host-gtest-stack_map_test\nChange-Id: I0b9f54df01749ee6ec3a67cfb07ba636a2489c89\n" }, { "commit": "35d5b8a2c5d2fce03be59aa003c3bf3c1b481be0", "tree": "7665f6d1527be61af13c8ef53f10833dd4200cfd", "parents": [ "b28683f43231e65860ecf91c96a8c0234542c019" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jul 03 09:18:32 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Jul 03 09:18:32 2018 +0100" }, "message": "ART: Do not use std::\u003ccontainer\u003e::at().\n\nThese functions are specified as throwing std::out_of_range\nand we do not use exceptions.\n\nTest: m\nTest: m test-art-host-gtest\nTest: testrunner.py --host --optimizing\nChange-Id: I67c365ed6d779c101a18b9f386c751c48ca76e16\n" }, { "commit": "e1402125e8363b49e176c6072893d1c110a05d2f", "tree": "44ec055be78cdcb8086a598cbf791f8c4627e157", "parents": [ "86decb6a3e3ebba8c3c67bfd25c12d9a85794f65" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed Jun 13 18:20:45 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Jun 14 12:31:44 2018 +0100" }, "message": "Move some helper methods to DexRegisterLocation.\n\nTest: test-art-host-gtest-stack_map_test\nChange-Id: I0abab008159db023d531df69214cd3bb8c0639bd\n" }, { "commit": "fd89b0739d4000ea70f28bf53dea531027024f5a", "tree": "1c9f3e1b046d60df405fb63f04cf83d3be4ff0f5", "parents": [ "6de8833fb64e59301eada4005ed04da995796170" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Sun Jun 03 12:00:22 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Jun 07 12:00:18 2018 +0100" }, "message": "Remove unused GetDexRegisterMap arguments.\n\nThey are no longer needed in the new encoding.\n\nI reuse the local variables in most places to DCHECK the size\nof the decoded register map. This has one catch though:\nWe sometimes omit all dex registers, so the DCHECK should be\ndone only after checking if the map is empty (if applicable).\n\nTest: test-art-host-gtest-stack_map_test\nChange-Id: I94b67029842374bc8eb7c9e5eac76fc93a651f24\n" }, { "commit": "6e69e52a12883386f91d014324bebee867ca7877", "tree": "7d1850f0a59e4ea2d013d6d479cd616b66922970", "parents": [ "6eb4d5e4bc2ce068004c1d7c85dbfff0c5efd11d" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Sun Jun 03 12:00:14 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue Jun 05 16:50:38 2018 +0100" }, "message": "Remove depth argument from InlineInfo accessors in stack maps.\n\nThe InlineInfo class actually represented a list of inlining\ninformation for a given stack map, and the depth argument was\nused everywhere to select to desired element from the list.\nThis was verbose and inconsistent with the other classes.\n\nChange the InlineInfo class to represent a single inlining,\nand select the desired depth when getting it from CodeInfo.\n\nTest: test-art-host-gtest-stack_map_test\nChange-Id: I35b73e6704854f0203f51d4dbdbed5b1d1cd5a3b\n" }, { "commit": "71ec1cc0665cdb9d39f4fd284d68962020417a53", "tree": "8d19b11ab3e04213c46b1b12ae94dbf54b305bfb", "parents": [ "03f16e4de1e3289414888fe0a5104205a781bd2f" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri May 18 15:57:25 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Jun 01 13:26:46 2018 +0100" }, "message": "Rewrite dex register map encoding in stackmaps.\n\nSimplify code by encoding dex register maps using BitTables.\nThe overall design is unchanged (bitmask+indices+catalogue).\n\nThis CL saves ~0.4% of .oat file size.\n\nThe dex register map decoding is factor of 3 faster now\n(based on the time to verify the register maps on Arm).\nThis is not too surprising as the old version was O(n^2).\n\nIt also reduces compiler arena memory usage by 11% since the\nBitTableBuilder is more memory efficient, we store less\nintermediate data, and we deduplicate most data on the fly.\n\nTest: test-art-host-gtest-stack_map_test\nChange-Id: Ib703a5ddf7f581280522d589e4a2bfebe53c26a9\n" }, { "commit": "21d45b4f4ef3661ac33d622aaac5dd6d2ce7deb8", "tree": "121e046a949025bca4d766fdfdc5ea58e4ea4a10", "parents": [ "d02b23f7ee9664213216a82bfdcb0ee83824de04" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed May 30 06:35:05 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed May 30 17:59:24 2018 +0100" }, "message": "Simplify DexRegisterMap API.\n\nStore some of the needed decoding state explicitly to avoid passing it\naround all the time. The DexRegisterMap class is rewritten in next CL.\n\nTest: test-art-host-gtest-stack_map_test\nChange-Id: Ie268dff2a1c1da2e08f0e6799ae51c30e11f350b\n" }, { "commit": "4b59d107f91601c4e0095d7a9db40970d4ed6956", "tree": "b120add4870b86cefc308ec3a0100851cfc1a4c0", "parents": [ "da4ff8bfcf03d6186b7d21c7bd8b90248f7b2f96" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Tue May 29 21:46:10 2018 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed May 30 14:32:44 2018 +0100" }, "message": "Revert^2 \"Optimize register mask and stack mask in stack maps.\"\n\nThis reverts commit 8b20b5c1f5b454b2f8b8bff492c88724b5002600.\n\nReason for revert: Retry submit unmodified after fixing the test.\n\nUse BitTable to store the masks as well and move the\ndeduplication responsibility to the BitTable builders.\n\nDon\u0027t generate entries for masks which are all zeros.\nThis saves 0.2% of .oat file size on both Arm64 and Arm.\n\nEncode registers as (value+shift) due to tailing zeros.\nThis saves 1.0% of .oat file size on Arm64 and 0.2% on Arm.\n\nTest: test-art-target-gtest-exception_test\nTest: test-art-host-gtest-bit_table_test\nTest: test-art-host-gtest-stack_map_test\nChange-Id: Ib643776dbec3f051cc29cd13ff39e453fab5fae9\n" }, { "commit": "8b20b5c1f5b454b2f8b8bff492c88724b5002600", "tree": "51bea8ddfff23b1f6b0323eaeacf42e6f6199015", "parents": [ "ffaf87a429766ed80e6afee5bebea93db539620b" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 29 15:32:55 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 29 15:32:55 2018 +0000" }, "message": "Revert \"Optimize register mask and stack mask in stack maps.\"\n\nThis reverts commit ffaf87a429766ed80e6afee5bebea93db539620b.\n\nReason for revert: Breaks exception_test32 on target\nfor CMS and heap poisoning configs.\n\nChange-Id: I127c17f693e28211a799f73a50e73105edee7e4c\n" }, { "commit": "ffaf87a429766ed80e6afee5bebea93db539620b", "tree": "d79637f4b6a564facf4b837c3ff125bb3755594e", "parents": [ "5513c2b68a08109a5bfd811c7b2c8bbc66244e8e" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Wed May 23 14:44:39 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon May 28 08:45:37 2018 +0100" }, "message": "Optimize register mask and stack mask in stack maps.\n\nUse BitTable to store the masks as well and move the\ndeduplication responsibility to the BitTable builders.\n\nDon\u0027t generate entries for masks which are all zeros.\nThis saves 0.2% of .oat file size on both Arm64 and Arm.\n\nEncode registers as (value+shift) due to tailing zeros.\nThis saves 1.0% of .oat file size on Arm64 and 0.2% on Arm.\n\nTest: test-art-host-gtest\nChange-Id: I636b7edd49e10e8afc9f2aa385b5980f7ee0e1f1\n" }, { "commit": "aaebb1468cc95838aa58ed537d03de000c9cfe28", "tree": "80f325dab976dccd846e3bd5d1b86266be24e3dc", "parents": [ "79866c1f089672eca1fb7776c669c2cef3d6ad3e", "6ec2a1bf1cbecf17546df780dd0ad769042e1874" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 23 14:58:19 2018 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Wed May 23 14:58:19 2018 +0000" }, "message": "Merge \"ObjPtr\u003c\u003e-ify UnstartedRuntime, fix 2 stale reference uses.\"" }, { "commit": "6ec2a1bf1cbecf17546df780dd0ad769042e1874", "tree": "09f7430f020e04dc892ff2c2152bb773cd45c0dd", "parents": [ "2d3065e6ca0bd707bc998b7d260bb8e8ec07cf87" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue May 22 15:33:48 2018 +0100" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed May 23 13:48:31 2018 +0100" }, "message": "ObjPtr\u003c\u003e-ify UnstartedRuntime, fix 2 stale reference uses.\n\nTest: Rely on TreeHugger.\nBug: 31113334\nChange-Id: I35f76c3e3b94dfca18dbe67aba065a1270f4e5ee\n" }, { "commit": "052f8ca1776ed7deb4f036498edd69eb6a1b942f", "tree": "ae993f1a634b258e124df12663a1d24859917b05", "parents": [ "1c515f0b9d621869a2e67ce5e0f5532d0f2f323a" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Apr 26 15:42:54 2018 +0100" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Mon May 21 20:28:41 2018 +0100" }, "message": "Rewrite stackmap encoding code.\n\nRemove most of the code related to handling of bit encodings.\nThe design is still same; the encodings are just more implicit.\nMost of the complexity is replaced with a single BitTable class,\nwhich is a generic purpose table of tightly bit-packed integers.\nIt has its own header which stores the bit-encoding of columns,\nand that removes the need to handle the encodings explicitly.\n\nOther classes, like StackMap, are accessors into the BitTable,\nwith named getter methods for the individual columns.\n\nThis CL saves ~1% of .oat file size (~4% of stackmap size).\n\nTest: test-art-host-gtest\nChange-Id: I7e92683753b0cc376300e3b23d892feac3670890\n" }, { "commit": "fac21782ed87ce800813cfa0f9b4ee4bb4aac462", "tree": "f14147d03574751cc2c153cf925354c722111177", "parents": [ "68f0680e83179cfe0127fda54a8e02a8552bf619" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Tue Mar 13 17:01:09 2018 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Wed Mar 14 11:06:34 2018 +0000" }, "message": "Fix delivering async exception while in compiled code.\n\nChange artInstrumentationMethodExitFromCode() to check\nfor async exceptions.\n\nAlso fix kDebugExceptionDelivery \u003d true causing stale\nObjPtr\u003c\u003e use.\n\nTest: run-test --jit -runtime-option -Xjitthreshold:0 \\\n 1934-jvmti-signal-thread\nTest: Repeat with kDebugExceptionDelivery \u003d true.\nBug: 74583459\nChange-Id: Ia2e777cc3ccef4eba549aa290f8bc12608eafe44\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": "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": "808c7a57bb913b13c22884f57cdacd59bf1fdb3f", "tree": "d7f0d7cabaac5a7646c25bae584a82a9aa279cc0", "parents": [ "64bae9fb677aa0e2406d13ea9f8ebaa92e16f978" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 15 11:19:33 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Dec 22 09:41:17 2017 -0800" }, "message": "Make CodeItem fields private\n\nMake code item fields private and use accessors. Added a hand full of\nfriend classes to reduce the size of the change.\n\nChanged default to be nullable and removed CreateNullable.\nCreateNullable was a bad API since it defaulted to the unsafe, may\nadd a CreateNonNullable if it\u0027s important for performance.\n\nMotivation:\nHave a different layout for code items in cdex.\n\nBug: 63756964\nTest: test-art-host-gtest\nTest: test/testrunner/testrunner.py --host\nTest: art/tools/run-jdwp-tests.sh \u0027--mode\u003dhost\u0027 \u0027--variant\u003dX32\u0027 --debug\n\nChange-Id: I42bc7435e20358682075cb6de52713b595f95bf9\n" }, { "commit": "170331f0e44a0e07fcfe0b5932517e0500f5cd1f", "tree": "4bb837aa3b6e19c898df23894beb61957885e985", "parents": [ "39b378ca3b4b6dc6da1651b84ee4289cd9bff0f8" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 07 18:41:03 2017 -0800" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Thu Dec 07 18:41:03 2017 -0800" }, "message": "ART: Remove base/logging from heap-inl.h\n\nHide the single uncommon VLOG and remove the include. Fix up\ntransitive includes.\n\nTest: m\nChange-Id: I917df597cb62c57040c1fb0e0079df4d95e5a658\n" }, { "commit": "b77b698f4550458725b72d3a451adca765f369cc", "tree": "edfc6a5b96325a6b4cdaa1a4e4428f1b677c7745", "parents": [ "e5b35ed787fbfb388d162361310bae5b0e7682a7" ], "author": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Jun 08 18:03:48 2017 +0100" }, "committer": { "name": "Roland Levillain", "email": "rpl@google.com", "time": "Thu Sep 28 18:05:39 2017 +0100" }, "message": "Check the to-space invariant on a delivered exception.\n\nTest: art/test/testrunner/testrunner.py\nBug: 62339341\nBug: 12687968\nChange-Id: Ida5e91031cd94429a72fcc9d4d1333d21dd07421\n" }, { "commit": "e2abbc604ce003c776c00ecf1293796bb4c4ac5a", "tree": "f7d124d1861cad2162c30dfe932bb4e1beaf41ef", "parents": [ "7090dfe84f78b1928fcbdfd664d0dd9ea52633ff" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Sep 15 11:59:26 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 18 11:36:24 2017 -0700" }, "message": "ART: Move kDexNoIndex to dex_file_types.h\n\nDefine the constant with the types to allow lowering the dependency\non DexFile.\n\nTest: m\nChange-Id: I3c61421db45be96d2057e01b1a7825883d8bd178\n" }, { "commit": "646d6388758db566eb1302baf274e538e5bdb8b9", "tree": "72d192b2120450589c40d1462b5ff373efd2cb69", "parents": [ "18e91619f38349e29367567a70f2f974a8cdaf67" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Aug 09 10:50:00 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Aug 09 13:23:41 2017 +0100" }, "message": "Move deopt and jit logs from LOG(INFO) to VLOG(jit).\n\nThey served their purposes. Now stop spamming logcat.\n\nTest: test.py\nChange-Id: I53c961a6ad95ea891f2f09b7e11e7535da4fdaff\n" }, { "commit": "4e92c3ce7ef354620a785553bbada554fca83a67", "tree": "42029deff4d3ba7f89b5fdbf79ff410da575f431", "parents": [ "549844e9ccf432d1396b19af890eedb602b8ba04" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 08 09:34:26 2017 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed May 10 09:27:33 2017 +0100" }, "message": "Add runtime reasons for deopt.\n\nCurrently to help investigate. Also:\n1) Log when deoptimization happens (which method and what reason)\n2) Trace when deoptimization happens (to make it visible in systrace)\n\nbug:37655083\nTest: test-art-host test-art-target\nChange-Id: I0c2d87b40db09e8e475cf97a7c784a034c585e97\n" }, { "commit": "6d7abbd2324f544c6b6da42bb6b9b531df0ce3cd", "tree": "669705a042bbc77c7963c117a9bf6f7a2308f9c8", "parents": [ "0dfc315a096bf5678cb5182771e3fdcca8ea4433" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 24 13:19:09 2017 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Apr 24 14:18:45 2017 -0700" }, "message": "ART: More header cleanup - method_verifier.h\n\nMove enumerations to own header. Move the compiler interface (of what\nthe compiler can tolerate) into its own header. Replace or remove\nmethod_verifier.h where possible.\n\nTest: mmma art\nChange-Id: I075fcb10b02b6c1c760daad31cb18eaa42067b6d\n" }, { "commit": "575d3e60c68b5cf481b615dde4a16283507b19ed", "tree": "4718f2747233e74268de21d804df9bfe0b7e4362", "parents": [ "857acf51fbc5a931939c20e9e299c69676baf654" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Mon Feb 06 11:00:40 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 08 09:51:27 2017 -0800" }, "message": "Clean up code info table layout\n\nPreviously:\nTable layout was computed multiple places like stack_map_stream,\nand getters. This made it difficult to add new stack map tables and\nmade the code hard to understand.\n\nThis change makes the table layout specified all inside of the code\ninfo. Updating the layout only requires changing ComputeTableOffsets.\n\nChanged the stack map inline info offset to be an index, so that it is\nnot require the inline infos are directly after the dex register table.\n\nOat file size for a large app: 94459576 -\u003e 93882040 (-0.61%)\n\nUpdated oatdump and fixed a bug that was incorrectly computing the\nregister mask bytes.\n\nBug: 34621054\n\nTest: test-art-host\n\nChange-Id: I3a7f141e09d5a18bce2bc6c9439835244a22016e\n" }, { "commit": "1a20b6801f2432a42b906f0de01e7e9586526aec", "tree": "ae3faaf42a2b560601fc024ae16898a5dfa42261", "parents": [ "357dcb73934356239292c46d6fbedba734da5e00" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 31 14:25:16 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Wed Feb 01 12:46:43 2017 -0800" }, "message": "Deduplicate register masks\n\nData is commonly shared between different stack maps. The register\nmasks are stored after the stack masks.\n\nOat size for a large app:\n96722288 -\u003e 94485872 (-2.31%)\n\nAverage oat size reduction according to golem -3.193%.\n\nBug: 34621054\n\nTest: test-art-host\n\nChange-Id: I5eacf668992e866d11ddba0c01675038a16cdfb4\n" }, { "commit": "45aa598cd1773f5eb1705dec13bea059238e054d", "tree": "f5a6fd7445a74218547cd6c4e28dc835821c355e", "parents": [ "f2042db1b41cc21cc540c5ad7d353cbe1e3a32df" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Mar 18 02:15:09 2016 +0000" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 31 10:15:26 2017 -0800" }, "message": "Deduplicate stack masks\n\nThe stack masks repeat often enough so that it is worth deduplicating\nthem.\n\nOat size for a large app:\n98143600 -\u003e 96722288 (-1.44%)\n\nBug: 34621054\n\nTest: test-art-host\nChange-Id: If73d51e46066357049d5be2e406ae9a32b7ff1f4\n" }, { "commit": "500d4918e5fb6eafb76b36c4ec736f974835b94c", "tree": "5c6be88b473e60296ae9df22bfc55cfca85a3d28", "parents": [ "fdd869faca8dff54a456ccb5f1a68ecb44143721", "433b79a9130df5f2375b9d3d7518c1f3fb84a467" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 20:56:11 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jan 30 20:56:11 2017 +0000" }, "message": "Merge \"Revert \"Revert \"Make --debuggable rely on JIT code.\"\"\"" }, { "commit": "433b79a9130df5f2375b9d3d7518c1f3fb84a467", "tree": "dddc053886464b99be29b9ca69039d74d49605ca", "parents": [ "81356645157af44152c7b7db383596b5cf3479b5" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 20:54:45 2017 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 20:54:45 2017 +0000" }, "message": "Revert \"Revert \"Make --debuggable rely on JIT code.\"\"\n\nFixed by https://android-review.googlesource.com/#/c/330165/\n\nThis reverts commit 81356645157af44152c7b7db383596b5cf3479b5.\n\nChange-Id: Ifb74e1cc90ab6dea621f7f54a00b540d6ccd0cf6\n" }, { "commit": "d7c4ad030c2c316f5e1b2049ff8fa925b56bee2b", "tree": "fee68e3431cd62d6f5d6da89c1e105993bfb5528", "parents": [ "f22bf9f809def57c96a19d932467a175bbd9c08c", "81356645157af44152c7b7db383596b5cf3479b5" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 20:20:19 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jan 30 20:20:20 2017 +0000" }, "message": "Merge \"Revert \"Make --debuggable rely on JIT code.\"\"" }, { "commit": "81356645157af44152c7b7db383596b5cf3479b5", "tree": "59aca3792abfa47c7424e4dac13248651e4662b2", "parents": [ "a0619e25aacf8b8074132f4951f75fdbcfd42925" ], "author": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 20:18:02 2017 +0000" }, "committer": { "name": "Vladimir Marko", "email": "vmarko@google.com", "time": "Mon Jan 30 20:19:28 2017 +0000" }, "message": "Revert \"Make --debuggable rely on JIT code.\"\n\nBreaks 137-cfi with ART_TEST_RUN_TEST_DEBUGGABLE\u003dtrue\n\nThis reverts commit a0619e25aacf8b8074132f4951f75fdbcfd42925.\n\nbug: 28769520\nChange-Id: Ifd7b8fc7c9b72c0a523fd57c4b1b80edd3547caa\n" }, { "commit": "8f28bd479af445293474867191a0bf1cf0f08a4f", "tree": "2bf92d368a6ad5b766bef70a3ee9c35d7af6db71", "parents": [ "ab5afeedfcb496026d8fe0774de99ee895888119", "a0619e25aacf8b8074132f4951f75fdbcfd42925" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 15:09:44 2017 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Mon Jan 30 15:09:45 2017 +0000" }, "message": "Merge \"Make --debuggable rely on JIT code.\"" }, { "commit": "a0619e25aacf8b8074132f4951f75fdbcfd42925", "tree": "dddc053886464b99be29b9ca69039d74d49605ca", "parents": [ "ca21dc47adeed92a15a9d3fd090bdd0e6654679c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Dec 20 13:57:43 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Jan 30 12:41:52 2017 +0000" }, "message": "Make --debuggable rely on JIT code.\n\nRemoves -Xfully-deoptable in the process, which was added as a\ntemporary workaround until this CL.\nPartial revert of https://android-review.googlesource.com/#/c/302232/\n\nMakes things consistent with existing infrastructure:\n\n- Parse the --debuggable from the compiler options, just like\n --compiler-filter.\n\n- Add DEBUG_JAVA_DEBUGGABLE, passed by the zygote, for debuggable apps.\n\n- Java debuggable now solely relies on JIT for simplicity.\n\n- Debugging under userdebug for non-java-debuggable apps is still\n best effort.\n\nTest: test-art-host, jdwp\n\nbug: 28769520\nChange-Id: Id0593aacd85b9780da97f20914a50943957c858f\n" }, { "commit": "12f1b99775bbf7dd82d0a897587ab6ed0e75ee22", "tree": "f8cb3a7334652e59577b7ff97a1805d1bf4ddd94", "parents": [ "d8f6e6430b26bf199f4a52f0624becb7c29f3c19" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Thu Jan 19 18:00:45 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri Jan 27 10:04:05 2017 -0800" }, "message": "Remove alignment bits in between stack maps\n\nSaves 0.65% of boot.oat size, probably similar on apps. Added\nBitMemoryRegion to avoid requiring adding state to StackMap. Added\ntest to memory_region_test.\n\nTest: clean-oat-host \u0026\u0026 test-art-host\n\nBug: 34621054\n\nChange-Id: I40279c59e262bd5e3c6a9135f83e22b5b6900d68\n" }, { "commit": "f5769e15aa9f058a5af987f0d1354f16f4464a7d", "tree": "8a55c514c70a701a06102d3cbda6e4fe8a84242c", "parents": [ "2e17788df086284845108c6b39954a5c74e93a69" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 10 15:54:41 2017 -0800" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Tue Jan 10 15:57:11 2017 -0800" }, "message": "Move most of art::Thread to ObjPtr\n\nBug: 31113334\n\nTest: test-art-host\n\nChange-Id: I73d0de3c875d3ec3d732034fdc961dae79ef4070\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": "3fec9ac0d5af1358d216eb2fdc2000ec0205f3f0", "tree": "f38d8d8aae51f53e7ee6b474f47597b784fc2316", "parents": [ "0cfe19af3b7395658210ea6044a65c9811962a7a" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Tue Sep 13 10:47:28 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 26 10:59:22 2016 -0700" }, "message": "ART: Use libbase logging\n\nMove most of our logging infrastructure over to system/core/base.\nRetain VLOG.\n\nUsing unified Android infrastructure has two main advantages. First,\nit reduces the complexity/maintenance burden in ART. Second, it\nallows to detach logging for the cases where we do not want or need\na runtime, e.g., dexdump, the disassembler, etc. As a part of the\nlatter, libbase is also supported for all hosts (including Windows).\n\nFrom a developer viewpoint, there are minor behavior changes for the\nLOG statements (see above), but otherwise usage is the same. Explicit\nseverity enum items are in the android::base namespace now.\n\nBug: 31338270\nTest: m test-art-host\nChange-Id: I5abcb2f45f5b03d49951874c48544f72a283a91b\n" }, { "commit": "9323e6e44060545d6c4f925139572868f65b7fe3", "tree": "65f3f380605d75f34d49aec49fca494bec26ef5b", "parents": [ "58735fbfbb2da0c0a19b3239da07c2353c40767d" ], "author": { "name": "David Sehr", "email": "sehr@google.com", "time": "Tue Sep 13 08:58:35 2016 -0700" }, "committer": { "name": "David Sehr", "email": "sehr@google.com", "time": "Wed Sep 14 08:01:39 2016 -0700" }, "message": "Separate annotations from dexfile reading.\n\nBug: 22322814\nChange-Id: I867d66da407dd80394a10d19903ebbc1ec3986ff\nTest: test-art\n" }, { "commit": "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": "f711f2cf3c28dfe865e36f17419a16f06a0ebb5a", "tree": "3657d68b2a2fb67f4919cb97ca279cf9310224e9", "parents": [ "bbfa42aa4c4b5cbc2ed2d8bdea6ff76da615a8aa" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon May 23 12:29:39 2016 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Thu May 26 13:24:54 2016 -0700" }, "message": "Partial fragment deoptimization\n\nWe used to do either single frame deoptimization, or full fragment\ndeoptimization which deoptimizes all the frames in a fragment.\nThis change allows some methods to be not deoptimizeable, likely due\nto some kind of optimization. So we need another deoptimization mode\nthat unwinds partial fragment. Deoptimizations are now generalized into\neither full or partial fragment. A full fragment deoptimization will\ndeopt all frames in the fragment, and then returns from the invoke stub\nto enter interpreter. A partial fragment deoptimization will deopt a\nsingle frame, or all frames up to the method that\u0027s not deoptimizeable,\nand then jumps to the interpreter bridge.\n\nCurrently code not deoptimizeable is the code in boot image since the\ncode may not be compiled with debuggable flag.\n\nBug: 28769520\nChange-Id: I875c694791cc8ebd5121abcd92ce7b0db95aca38\n" }, { "commit": "bf80a0678e9c0cfda87deffac5012cb69956e6dc", "tree": "e4f76c105b347a55caf1d32bb1e2542826525afd", "parents": [ "e07a0b0c6b694ddd2d101cff6bc04ec93a6c8125", "471118ef120cb075ccba9e307e2c1b9b5a584840" ], "author": { "name": "Chih-hung Hsieh", "email": "chh@google.com", "time": "Fri Apr 29 22:57:26 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Apr 29 22:57:26 2016 +0000" }, "message": "Merge \"Fix google-explicit-constructor warnings.\"" }, { "commit": "471118ef120cb075ccba9e307e2c1b9b5a584840", "tree": "584184f923f2f7f9931893f6a87d21239eab0451", "parents": [ "3f8bcb5e9e15f5a93ff0416d7b32aefd3761e146" ], "author": { "name": "Chih-Hung Hsieh", "email": "chh@google.com", "time": "Fri Apr 29 14:27:41 2016 -0700" }, "committer": { "name": "Chih-Hung Hsieh", "email": "chh@google.com", "time": "Fri Apr 29 14:27:41 2016 -0700" }, "message": "Fix google-explicit-constructor warnings.\n\nBug: 28341362\nChange-Id: Ic7e236a7f58c4021a3f8fb4536a6457d262fe62e\n" }, { "commit": "ffc87076dda9878cb2cc098149bae441d38b9268", "tree": "e587208d6a8e62532792add3e1ace6b4e6d73e0f", "parents": [ "97cbc9206e9adc473a90650ebdb5d620f517ff04" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Wed Apr 20 14:22:09 2016 +0100" }, "committer": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Thu Apr 28 16:15:04 2016 +0100" }, "message": "Split profile recording from jit compilation\n\nWe still use ProfileInfo objects to record profile information. That\ngives us the flexibility to add the inline caches in the future and the\nconvenience of the already implemented GC.\n\nIf UseJIT is false and SaveProfilingInfo true, we will only record the\nProfileInfo and never launch compilation tasks.\n\nBug: 27916886\n\n(cherry picked from commit e5de54cfab5f14ba0b8ff25d8d60901c7021943f)\n\nChange-Id: I68afc181d71447895fb12346c1806e99bcab1de2\n" }, { "commit": "fe2b13717879abae993603534a145372e3110aae", "tree": "5b499eb28af5e91e00ff86683f51df029a2b86b4", "parents": [ "5c26e430fbeaf56e3a5da4e3f167ff35fbb71ef9", "09ed09866da6d8c7448ef297c148bfa577a247c2" ], "author": { "name": "Calin Juravle", "email": "calin@google.com", "time": "Fri Apr 01 12:28:51 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Fri Apr 01 12:28:51 2016 +0000" }, "message": "Merge \"Pack stack map entries on bit level to save space.\"" }, { "commit": "09ed09866da6d8c7448ef297c148bfa577a247c2", "tree": "dad6a5dae6ca6131f1eba201eaa371edc6d9929d", "parents": [ "e28ad4b91591c226ed404a2b01104bb99bfeb28f" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Fri Feb 12 21:58:43 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Mar 31 22:12:07 2016 +0100" }, "message": "Pack stack map entries on bit level to save space.\n\nUse only the minimum number of bits required to store stack map data.\nFor example, if native_pc needs 5 bits and dex_pc needs 3 bits, they\nwill share the first byte of the stack map entry.\n\nThe header is changed to store bit offsets of the fields rather than\nbyte sizes. Offsets also make it easier to access later fields without\ncalculating sum of all previous sizes.\n\nAll of the header fields are byte sized or encoded as ULEB128 instead\nof the previous fixed size encoding. This shrinks it by about half.\n\nIt saves 3.6 MB from non-debuggable boot.oat (AOSP).\nIt saves 3.1 MB from debuggable boot.oat (AOSP).\n\nIt saves 2.8 MB (of 99.4 MB) from /system/framework/arm/ (GOOG).\nIt saves 1.0 MB (of 27.8 MB) from /system/framework/oat/arm/ (GOOG).\n\nField loads from stackmaps seem to get around 10% faster.\n(based on the time it takes to load all stackmap entries from boot.oat)\n\nBug: 27640410\nChange-Id: I8bf0996b4eb24300c1b0dfc6e9d99fe85d04a1b7\n" }, { "commit": "bf9611f821697b14bf9e170f503c3f47613b046b", "tree": "c5d6de40e988c1d92f51639243443b225e45808f", "parents": [ "782d0464ddc92fd5d55a330068d2d255336c16ba" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Fri Mar 25 16:58:00 2016 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Mar 28 15:51:07 2016 -0700" }, "message": "ART: Clean up verifier\n\nClean up verifier post-Quick.\n\nChange-Id: I0b05e10dd06edd228fe2068c8afffc4b7d7fdffa\n" }, { "commit": "dcedaaa4dc6d05b6044ecef50f2d9bb95575d49c", "tree": "bb225ab5b2ceae62765e424b8ece66ae66ae9f71", "parents": [ "7d66ac5038a81554d89f79b419e3a80e63ca1f92", "7dc11782ff0a5dffcd8108f256f8975f0b3e8076" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Feb 25 15:15:58 2016 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Thu Feb 25 15:15:59 2016 +0000" }, "message": "Merge \"Implement \u003c\u003c operator for DexRegisterLocation::Kind.\"" }, { "commit": "7dc11782ff0a5dffcd8108f256f8975f0b3e8076", "tree": "c3f6463fb378775d9b348f8fcfccfcdb410cfcc2", "parents": [ "99da8cd8818e015b21d1c2af73ac40749698c930" ], "author": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Feb 25 13:23:56 2016 +0000" }, "committer": { "name": "David Srbecky", "email": "dsrbecky@google.com", "time": "Thu Feb 25 14:07:27 2016 +0000" }, "message": "Implement \u003c\u003c operator for DexRegisterLocation::Kind.\n\nThis makes it comparable in DCHECK_EQ and similar methods.\n\nChange-Id: I6b5b237be89325850ae6860d011fd6741189ab01\n" }, { "commit": "b52de2427700ebdecc904aa2f3f59a079319abd5", "tree": "1dda0a3061f62bfd2dd13a9dad0fac75d41c2d51", "parents": [ "52d9c2f0c584877e9c7f9e859d8068b3a2ccb12f" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 19 12:43:12 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Feb 19 12:43:12 2016 +0000" }, "message": "Fix braino in single frame deopt.\n\nhandler_method_header_ is actually the OatQuickMethodHeader\nof the caller of the method that is deoptimizing (as we will\nresume at its call site). We should use the OatQuickMethodHeader\nof the method that is deoptimizing and pass it to the code cache\nto invalidate the method.\n\nChange-Id: I2b61a96b8415e9477e3ac902c54a314aeb0811a0\n" }, { "commit": "b88d59ef4fe611fe47e50a6a19785e03bbd5f93b", "tree": "7db030759bf4eb20df3bfef823e75b5a39a313a7", "parents": [ "b93c21e83c8fbf0191093c01a8951adb5be9010b" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Feb 17 11:31:49 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed Feb 17 11:32:49 2016 +0000" }, "message": "Be a bit smarter with JIT code triggering deoptimization.\n\nDo not re-use an OSR method that triggered deoptimization.\nAlso add a stack overflow check before doing OSR.\n\nbug:27094810\nChange-Id: I6ff6a7fb9b3df9b7c0ff37e3610595efa70ad067\n" }, { "commit": "012fc4e9d9b66b3ffb7838b0e29dadbb4863ee69", "tree": "abba308e56a3d81e5c3fceebc95fb37a0ffe9c7c", "parents": [ "5ee288c9dd99614e3a238f5efceeec6456e3499d" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 08 15:58:19 2016 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Jan 08 17:14:17 2016 +0000" }, "message": "Don\u0027t encode a DexRegisterMap if there is no live register.\n\nChange-Id: I76a291e6a0ac37f0590d16c7f5b866115588bc55\n" }, { "commit": "73be1e8f8609708f6624bb297c9628de44fd8b6f", "tree": "47a22745797a345dea8dd846aad00334da0d5bf1", "parents": [ "47229aa5848df7d45578dbdd9285f57dfa9399a8" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Sep 17 15:22:56 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Dec 04 12:08:02 2015 +0000" }, "message": "Inline monomorphic calls.\n\nChange-Id: If38171c2dc7d4a4378df5d050afc4fff4499c98f\n" }, { "commit": "efc3f02504e8e3768cc18ab5347c5a6d19d6a935", "tree": "43dcd255401c6270b83ef83a95a5ab00416ad535", "parents": [ "5f2e628b280dfc5f8385e489d44cc0960fcea484" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Oct 28 12:19:06 2015 -0500" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Wed Oct 28 20:16:06 2015 -0500" }, "message": "ART: Fix deopt from optimized code under inlining\n\nDeoptimization of inlined frames would use the outer method\u0027s\nvreg map, thus starting interpreter in bogus state.\n\nBug: 25331616\nChange-Id: I1d4aefc731bb1386f3e6186a89b59981836480b3\n" }, { "commit": "338565069cc221cb73ba55dc586c8ab8dc0d592c", "tree": "2b1a2b9df87c58635fda4fd1e63b173b1bb85f0c", "parents": [ "620021e643518dc93dc618d3e9df1c33ff8a491c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 20 15:52:58 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Oct 22 13:25:39 2015 +0100" }, "message": "Do not run the verifier when deopting optimized frames.\n\nThe stack maps contain all the required information.\n\nChange-Id: I709e72b3de0d76a4203a3a004f093071e7865642\n" }, { "commit": "524e7ea8cd17bad17bd9f3e0ccbb19ad0d4d9c02", "tree": "ad07cc96f633bdae839ff2f1553d84b9c864a930", "parents": [ "b697c447eb61c2e14315166ec3b0d16375ae403c" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Oct 16 17:13:34 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 20 11:52:11 2015 +0100" }, "message": "Remove ArtCode.\n\n- Instead use OatQuickMethodHeader.\n- Various cleanups now that we don\u0027t have all those\n ArtMethod -\u003e ArtCode -\u003e OatQuickMethodHeader indirections.\n\nAs a consequence of this cleanup, exception handling got a bit\nfaster.\n\nParserCombinators benchmark (exception intensive) on x64: (lower is better)\nBefore:\nParserCombinators(RunTime): 1062500.0 us.\nAfter:\nParserCombinators(RunTime): 833000.0 us.\n\nChange-Id: Idac917b6f1b0dc254ad68fb3781cd61bccadb0f3\n" }, { "commit": "6bc4374e3fa00e3ee5e832e1761c43e0b8a71558", "tree": "38118523aa6b9a92fbdcd7bf1d32ca6185a3d3f0", "parents": [ "16065ce56394c73c87dcb78ead4164ddc80fddb3" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon Oct 12 18:11:10 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Tue Oct 13 12:05:27 2015 +0100" }, "message": "Add an abstraction over a compiled code.\n\nThat\u0027s just step 1, moving code-related functions of ArtMethod to\nanother class. That class is only a wrapper on an ArtMethod, but will\nbe changed to be a wrapper around compiled code.\n\nChange-Id: I6f35fc06d37220558dff61691e51ae20066b0dd6\n" }, { "commit": "639bdd13993644a267f177f8f5936496bda65e2b", "tree": "fc68e344355ab1c0cfef4d47dd8f2cbc9fc7e022", "parents": [ "b175a45be78152606a1db1eac0b79ef8777644f5" ], "author": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Wed Jun 03 11:22:45 2015 -0700" }, "committer": { "name": "Andreas Gampe", "email": "agampe@google.com", "time": "Mon Sep 28 15:37:28 2015 -0700" }, "message": "ART: Single-frame deopt\n\nAdd deoptimization of a single frame. Works by removing the managed\ncode frame and jumping into the quick-to-interpreter bridge, and\nthe bridge understanding a stored ShadowFrame.\n\nWe need a separate fixup pass. For x86, we leave the return address\non the stack so we don\u0027t need to push it there.\n\nBug: 21611912\nChange-Id: I06625685ced8b054244f8685ab50b238a705b9d2\n" }, { "commit": "99170c636dfae4908b102347cfe9f92bad1881cc", "tree": "1ecff5dd69ea20b09e622cb88e7835a55dfa4be8", "parents": [ "5836f8465ddd0a7f5cbeec9c30d739bd0619867f" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Jul 06 11:10:37 2015 -0700" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Sep 17 17:07:22 2015 +0200" }, "message": "Deoptimization support in optimizing compiler for setting local values\n\nDue to compiler optimizations, we may not always be able to update\nthe value of a local variable in a compiled frame (like a variable\nseen as constant by the compiler). To avoid that situation, we simply\ndeoptimize compiled frames updated by the debugger so they are\nexecuted by the interpreter with the updated value.\n\nWhen the debugger attempts to set a local variable (actually a DEX\nregister or a pair of registers) in a compiled frame, we allocate a\nShadowFrame associated to that frame (using its frame id) and set the\nnew value in that ShadowFrame. When we know we are about to continue\nthe execution of the compiled frame, we deoptimize the stack using\nthe preallocated ShadowFrame (instead of creating a new one). We\ninitialize it with the current value of all DEX registers except\nthe ones that have been set by the debugger. Therefore, the\nShadowFrame represent the runtime context modified by the debugger.\n\nBumps oat version to force recompilation.\n\nBug: 19944235\nChange-Id: I0ebe6241264f7a3be0f14ee4516c1f7436e04da6\n" }, { "commit": "5836f8465ddd0a7f5cbeec9c30d739bd0619867f", "tree": "363e49b9b7c866186c45ad508bf1a2ed82892d21", "parents": [ "5559f9decadf23e887030fa35ad9d0fcc922046b", "26f728661a08062a373a3203b72dc2555c2aed2d" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 15 16:54:02 2015 +0000" }, "committer": { "name": "Gerrit Code Review", "email": "noreply-gerritcodereview@google.com", "time": "Tue Sep 15 16:54:02 2015 +0000" }, "message": "Merge \"Cleanup thread access in StackVisitor\"" }, { "commit": "72f7b880d5d0057b9fac3a51ef3a0f22909bc633", "tree": "5a5a92f0c52bd334f662ba2f51885540874973bb", "parents": [ "b505997b2176bd29a108cb6c33d06d4ef29ba001" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Sep 15 17:00:52 2015 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Sep 15 17:50:43 2015 +0100" }, "message": "ART: Fix 004-ReferenceMap run test\n\nThis patch adds a new option to ArtMethod::ToNativeQuickPc to select\nthe order of iteration over stack maps. The method is only used by\nthe runtime to find native_pc of catch blocks, but also by the\n004-ReferenceMap test which uses it to find native_pc of a safepoint.\n\nChange-Id: Idb2b34aabf1ac7249c30a00806af7d63d7e682dd\n" }, { "commit": "77a48ae01bbc5b05ca009cf09e2fcb53e4c8ff23", "tree": "780c7d6bdee784c2f8248979de348491cfb63b34", "parents": [ "659562aaf133c41b8d90ec9216c07646f0f14362" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Sep 15 12:34:04 2015 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Tue Sep 15 12:34:04 2015 +0000" }, "message": "Revert \"Revert \"ART: Register allocation and runtime support for try/catch\"\"\n\nThe original CL triggered b/24084144 which has been fixed\nby Ib72e12a018437c404e82f7ad414554c66a4c6f8c.\n\nThis reverts commit 659562aaf133c41b8d90ec9216c07646f0f14362.\n\nChange-Id: Id8980436172457d0fcb276349c4405f7c4110a55\n" }, { "commit": "26f728661a08062a373a3203b72dc2555c2aed2d", "tree": "4f6e8168ac7ca422c7de3c7cdda6db21958358fd", "parents": [ "b6f7dd330f115fc977d1d5a10122c41c9dd7c210" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 15 09:52:07 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 15 11:15:12 2015 +0200" }, "message": "Cleanup thread access in StackVisitor\n\nAdds method StackVisitor::GetThread to give access to the visited\nThread* so we no longer need to copy that pointer in subclasses.\n\nAlso adds a few missing const and DISALLOW_COPY_AND_ASSIGN.\n\nChange-Id: I57649ee7742ef4ef1e01447ac2fbb66f977b22eb\n" }, { "commit": "659562aaf133c41b8d90ec9216c07646f0f14362", "tree": "be1beae390262bf2f5a17bfa44de93081a849d07", "parents": [ "b022fa1300e6d78639b3b910af0cf85c43df44bb" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Sep 14 21:26:33 2015 +0000" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Sep 14 21:26:33 2015 +0000" }, "message": "Revert \"ART: Register allocation and runtime support for try/catch\"\n\nBreaks libcore test org.apache.harmony.security.tests.java.security.KeyStorePrivateKeyEntryTest#testGetCertificateChain. Need to investigate.\n\nThis reverts commit b022fa1300e6d78639b3b910af0cf85c43df44bb.\n\nChange-Id: Ib24d3a80064d963d273e557a93469c95f37b1f6f\n" }, { "commit": "b022fa1300e6d78639b3b910af0cf85c43df44bb", "tree": "780c7d6bdee784c2f8248979de348491cfb63b34", "parents": [ "e481c006e8b055a31d9c7cff27f4145e57e3c113" ], "author": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Thu Aug 20 17:47:48 2015 +0100" }, "committer": { "name": "David Brazdil", "email": "dbrazdil@google.com", "time": "Mon Sep 14 20:42:58 2015 +0100" }, "message": "ART: Register allocation and runtime support for try/catch\n\nThis patch completes a series of CLs that add support for try/catch\nin the Optimizing compiler. With it, Optimizing can compile all\nmethods containing try/catch, provided they don\u0027t contain catch loops.\nFuture work will focus on improving performance of the generated code.\n\nSsaLivenessAnalysis was updated to propagate liveness information of\ninstructions live at catch blocks, and to keep location information on\ninstructions which may be caught by catch phis.\n\nRegisterAllocator was extended to spill values used after catch, and\nto allocate spill slots for catch phis. Catch phis generated for the\nsame vreg share a spill slot as the raw value must be the same.\n\nLocation builders and slow paths were updated to reflect the fact that\nthrowing an exception may not lead to escaping the method.\n\nInstruction code generators are forbidden from using of implicit null\nchecks in try blocks as live registers need to be saved before handing\nover to the runtime.\n\nCodeGenerator emits a stack map for each catch block, storing locations\nof catch phis. CodeInfo and StackMapStream recognize this new type of\nstack map and store them separate from other stack maps to avoid dex_pc\nconflicts.\n\nAfter having found the target catch block to deliver an exception to,\nQuickExceptionHandler looks up the dex register maps at the throwing\ninstruction and the catch block and copies the values over to their\nrespective locations.\n\nThe runtime-support approach was selected because it allows for the\nbest performance in the normal control-flow path, since no propagation\nof catch phi values is necessary until the exception is thrown. In\naddition, it also greatly simplifies the register allocation phase.\n\nConstantHoisting was removed from LICMTest because it instantiated\n(now abstract) HConstant and was bogus anyway (constants are always in\nthe entry block).\n\nChange-Id: Ie31038ad8e3ee0c13a5bbbbaf5f0b3e532310e4e\n" }, { "commit": "520633bebd2bf4d70884d30f179dbde9f275aac6", "tree": "0006fb9db71545d9aaa88df3952a7488b30b2ae6", "parents": [ "e0a49e3a93aa54b9e603d797d9e925a98742753a" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Sep 08 17:03:36 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Thu Sep 10 17:39:56 2015 +0200" }, "message": "Support deoptimization on exception\n\nAllows to deoptimize when an exception is being thrown. We only\ndeoptimize if an executable frame (starting from the catch handler)\nneeds to be executed with the interpreter.\n\nBefore executing deoptimized frames, the exception is restored. The\ninterpreter starts by handling this exception at the point of the\nthrowing instruction.\n\nBug: 23714835\nChange-Id: I0c5f7d4b257644acf12210aae8e5b6bb0f4af1f7\n" }, { "commit": "b995c0ab7d6313f32e704e79c1e7dd21f5f160d4", "tree": "9055d85d7cabb23dd2129e0ea9164011f5238b22", "parents": [ "009c34cba875885d9540696f33255a9b355d6e15" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Aug 24 14:27:01 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Mon Aug 24 15:15:00 2015 +0200" }, "message": "Update instrumentation stack after exception event\n\nIf instrumentation is updated on an exception event, we must update\nthe instrumentation stack (to remove all frames that will be unwound\nby the exception) before doing the long jump.\n\nBug: 23471864\nChange-Id: Ic91552bb7280c54bcc58b7ba03a17040b0b0f5ef\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": "f795869da0a1fa006fdcdacd8afb6149a63fc1a7", "tree": "b4cd23012e62b80788870f2c85c30fc6032299fa", "parents": [ "430afefe1cced35cf5eb12ce7a6dcbfc5007c97e" ], "author": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jun 09 14:09:14 2015 +0200" }, "committer": { "name": "Sebastien Hertz", "email": "shertz@google.com", "time": "Tue Jun 09 17:17:09 2015 +0200" }, "message": "Follow up on CL 151605\n\n- Fixes return type of StackedShadowFrameRecord::GetType\n- Makes StackedShadowFrameType an enum class (scoped enum)\n- Moves DeoptimizationReturnValueRecord and StackedShadowFrameRecord\n to thread.cc file and use forward declaration in thread.h header\n- Fixes tools/generate-operator-out.py for scoped enum classes.\n\nBug: 20845490\nChange-Id: I6b67e288b1db563699161e58ec2e2330d42dd8f5\n" }, { "commit": "1f2d3ba6af52cf6f566deb38b7e07735c9a08fb6", "tree": "af6ae612d43c50703292d39a8a0b37dda3f919a6", "parents": [ "ccd6337f31d13706c602f3d9436e9b4025075b63" ], "author": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon May 18 12:12:50 2015 -0700" }, "committer": { "name": "Mingyao Yang", "email": "mingyao@google.com", "time": "Mon Jun 08 00:53:55 2015 -0700" }, "message": "Fix nested deoptimization.\n\nHandle nested deoptimization cases. Create a stacked shadow frame\nrecords to keep track of deoptimization shadow frames. Shadow frames\nunder construction can be tracked in the same stack.\n\nBug: 20845490\nChange-Id: I768285792c29e7c3cfcd21e7a2600802506024d8\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": "8e5bd18fc665d7ec5461ea068e98740a65da754c", "tree": "83441cdfdab06709b573aad2ab731cc65c10b9f1", "parents": [ "cdeb0b5fede4c06488f43a212591e661d946bc78" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Wed May 06 11:34:34 2015 +0100" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Mon May 11 15:23:25 2015 +0100" }, "message": "Add a flag to StackVisitor for inlining.\n\nThe flag tells whether the stack walk needs to include inlined\nJava frames.\n\nThis does not do anything just yet, as we\u0027re not inlining anyways.\n\nChange-Id: I716e25094fe56fa335ca1f9a398c1bcdba478e73\n" }, { "commit": "50030ef998be09789da4a9a56738362852068f12", "tree": "02df1bd0954f9b970434d1363466b0cfe182bc12", "parents": [ "6727a48193db2a0cf01af971cccffe1a6518c247" ], "author": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 08 14:19:26 2015 -0700" }, "committer": { "name": "Mathieu Chartier", "email": "mathieuc@google.com", "time": "Fri May 08 15:09:17 2015 -0700" }, "message": "Check IsReferenceVReg during deopt\n\nRequired since the quick GC maps may not agree with the verifier\nones. Without this check we may copy stale object references into\nthe shadow frame.\n\n(cherry picked from commit f00baf56ef647684888a407dbb6adadd704a2039)\n\nBug: 20736048\nChange-Id: I7783c8a8ee45cf601b08b4c38f1dec7f7d11380c\n" }, { "commit": "241a9588c6d7d0fcb4c92da40c7141863930083a", "tree": "4a6868893dc742f6322775d198a2305f645f2f3f", "parents": [ "a93b104b47da67eaa50f8a6b7280cc6c438ef37b" ], "author": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Mon Apr 27 15:19:41 2015 -0700" }, "committer": { "name": "Christopher Ferris", "email": "cferris@google.com", "time": "Mon May 04 12:10:07 2015 -0700" }, "message": "Fix mismatched new[]/delete.\n\nAnother two cases where a new[] is used but only a delete occurs.\n\nBug: 18202869\nChange-Id: If68264807150f3a9783e44ef8823cc366bff8df2\n" }, { "commit": "15b9d5274399736ac09705f0507df24fac4f00c1", "tree": "04564a9265f5dccefdd32ea7bdd25adc0267f80b", "parents": [ "8bc616d09f93523f4bc982cc60c377b00161522a" ], "author": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Thu Mar 12 15:05:13 2015 +0000" }, "committer": { "name": "Nicolas Geoffray", "email": "ngeoffray@google.com", "time": "Fri Mar 13 10:26:47 2015 +0000" }, "message": "API change in StackVisitor::GetVReg*.\n\n- Remove GetVReg() and SetVReg() that were expecting to always succeed.\n- Change Quick-only methods to take a FromQuickCode suffix.\n- Change deopt to use dead values when GetVReg does not succeed:\n the optimizing compiler will not have a location for uninitialized\n Dex registers and potentially dead registers.\n\nChange-Id: Ida05773a97aff8aa69e0caf42ea961f80f854b77\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" } ], "next": "7642cfc90fc9c3ebfd8e3b5041915705c93b5cf0" }